/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_commands.py

  • Committer: John Arbash Meinel
  • Date: 2009-06-12 18:05:15 UTC
  • mto: (4371.4.5 vila-better-heads)
  • mto: This revision was merged to the branch mainline in revision 4449.
  • Revision ID: john@arbash-meinel.com-20090612180515-t0cwbjsnve094oik
Add a failing test for handling nodes that are in the same linear chain.

It fails because the ancestry skipping causes us to miss the fact that the two nodes
are actually directly related. We could check at the beginning, as the 
code used to do, but I think that will be incomplete for the more-than-two
heads cases.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_commands.py
1
 
# Copyright (C) 2005-2010 Canonical Ltd
 
1
# Copyright (C) 2004, 2005 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
24
24
    config,
25
25
    errors,
26
26
    option,
27
 
    symbol_versioning,
28
27
    tests,
29
28
    )
30
29
from bzrlib.commands import display_command
65
64
    @staticmethod
66
65
    def get_command(options):
67
66
        class cmd_foo(commands.Command):
68
 
            __doc__ = 'Bar'
 
67
            'Bar'
69
68
 
70
69
            takes_options = options
71
70
 
112
111
 
113
112
    def test_unicode(self):
114
113
        my_config = self._get_config("[ALIASES]\n"
115
 
            u'iam=whoami "Erik B\u00e5gfors <erik@bagfors.nu>"\n')
 
114
            u"iam=whoami 'Erik B\u00e5gfors <erik@bagfors.nu>'\n")
116
115
        self.assertEqual([u'whoami', u'Erik B\u00e5gfors <erik@bagfors.nu>'],
117
116
                          commands.get_alias("iam", config=my_config))
118
117
 
120
119
class TestSeeAlso(tests.TestCase):
121
120
    """Tests for the see also functional of Command."""
122
121
 
123
 
    @staticmethod
124
 
    def _get_command_with_see_also(see_also):
125
 
        class ACommand(commands.Command):
126
 
            __doc__ = """A sample command."""
127
 
            _see_also = see_also
128
 
        return ACommand()
129
 
 
130
122
    def test_default_subclass_no_see_also(self):
131
 
        command = self._get_command_with_see_also([])
 
123
        class ACommand(commands.Command):
 
124
            """A sample command."""
 
125
        command = ACommand()
132
126
        self.assertEqual([], command.get_see_also())
133
127
 
134
128
    def test__see_also(self):
135
129
        """When _see_also is defined, it sets the result of get_see_also()."""
136
 
        command = self._get_command_with_see_also(['bar', 'foo'])
 
130
        class ACommand(commands.Command):
 
131
            _see_also = ['bar', 'foo']
 
132
        command = ACommand()
137
133
        self.assertEqual(['bar', 'foo'], command.get_see_also())
138
134
 
139
135
    def test_deduplication(self):
140
136
        """Duplicates in _see_also are stripped out."""
141
 
        command = self._get_command_with_see_also(['foo', 'foo'])
 
137
        class ACommand(commands.Command):
 
138
            _see_also = ['foo', 'foo']
 
139
        command = ACommand()
142
140
        self.assertEqual(['foo'], command.get_see_also())
143
141
 
144
142
    def test_sorted(self):
145
143
        """_see_also is sorted by get_see_also."""
146
 
        command = self._get_command_with_see_also(['foo', 'bar'])
 
144
        class ACommand(commands.Command):
 
145
            _see_also = ['foo', 'bar']
 
146
        command = ACommand()
147
147
        self.assertEqual(['bar', 'foo'], command.get_see_also())
148
148
 
149
149
    def test_additional_terms(self):
150
150
        """Additional terms can be supplied and are deduped and sorted."""
151
 
        command = self._get_command_with_see_also(['foo', 'bar'])
 
151
        class ACommand(commands.Command):
 
152
            _see_also = ['foo', 'bar']
 
153
        command = ACommand()
152
154
        self.assertEqual(['bar', 'foo', 'gam'],
153
155
            command.get_see_also(['gam', 'bar', 'gam']))
154
156
 
161
163
        del sys.modules['bzrlib.tests.fake_command']
162
164
        global lazy_command_imported
163
165
        lazy_command_imported = False
164
 
        commands.install_bzr_command_hooks()
165
166
 
166
167
    @staticmethod
167
168
    def remove_fake():
204
205
        # commands are registered).
205
206
        # when they are simply created.
206
207
        hook_calls = []
207
 
        commands.install_bzr_command_hooks()
208
208
        commands.Command.hooks.install_named_hook(
209
209
            "extend_command", hook_calls.append, None)
210
210
        # create a command, should not fire
211
 
        class cmd_test_extend_command_hook(commands.Command):
212
 
            __doc__ = """A sample command."""
 
211
        class ACommand(commands.Command):
 
212
            """A sample command."""
 
213
        cmd = ACommand()
213
214
        self.assertEqual([], hook_calls)
214
215
        # -- as a builtin
215
216
        # register the command class, should not fire
216
217
        try:
217
 
            commands.builtin_command_registry.register(cmd_test_extend_command_hook)
 
218
            builtins.cmd_test_extend_command_hook = ACommand
218
219
            self.assertEqual([], hook_calls)
219
220
            # and ask for the object, should fire
220
221
            cmd = commands.get_cmd_object('test-extend-command-hook')
224
225
            self.assertSubset([cmd], hook_calls)
225
226
            del hook_calls[:]
226
227
        finally:
227
 
            commands.builtin_command_registry.remove('test-extend-command-hook')
 
228
            del builtins.cmd_test_extend_command_hook
228
229
        # -- as a plugin lazy registration
229
230
        try:
230
231
            # register the command class, should not fire
236
237
            self.assertEqual([cmd], hook_calls)
237
238
        finally:
238
239
            commands.plugin_cmds.remove('fake')
239
 
 
240
 
 
241
 
class TestGetCommandHook(tests.TestCase):
242
 
 
243
 
    def test_fires_on_get_cmd_object(self):
244
 
        # The get_command(cmd) hook fires when commands are delivered to the
245
 
        # ui.
246
 
        commands.install_bzr_command_hooks()
247
 
        hook_calls = []
248
 
        class ACommand(commands.Command):
249
 
            __doc__ = """A sample command."""
250
 
        def get_cmd(cmd_or_None, cmd_name):
251
 
            hook_calls.append(('called', cmd_or_None, cmd_name))
252
 
            if cmd_name in ('foo', 'info'):
253
 
                return ACommand()
254
 
        commands.Command.hooks.install_named_hook(
255
 
            "get_command", get_cmd, None)
256
 
        # create a command directly, should not fire
257
 
        cmd = ACommand()
258
 
        self.assertEqual([], hook_calls)
259
 
        # ask by name, should fire and give us our command
260
 
        cmd = commands.get_cmd_object('foo')
261
 
        self.assertEqual([('called', None, 'foo')], hook_calls)
262
 
        self.assertIsInstance(cmd, ACommand)
263
 
        del hook_calls[:]
264
 
        # ask by a name that is supplied by a builtin - the hook should still
265
 
        # fire and we still get our object, but we should see the builtin
266
 
        # passed to the hook.
267
 
        cmd = commands.get_cmd_object('info')
268
 
        self.assertIsInstance(cmd, ACommand)
269
 
        self.assertEqual(1, len(hook_calls))
270
 
        self.assertEqual('info', hook_calls[0][2])
271
 
        self.assertIsInstance(hook_calls[0][1], builtins.cmd_info)
272
 
 
273
 
 
274
 
class TestGetMissingCommandHook(tests.TestCase):
275
 
 
276
 
    def hook_missing(self):
277
 
        """Hook get_missing_command for testing."""
278
 
        self.hook_calls = []
279
 
        class ACommand(commands.Command):
280
 
            __doc__ = """A sample command."""
281
 
        def get_missing_cmd(cmd_name):
282
 
            self.hook_calls.append(('called', cmd_name))
283
 
            if cmd_name in ('foo', 'info'):
284
 
                return ACommand()
285
 
        commands.Command.hooks.install_named_hook(
286
 
            "get_missing_command", get_missing_cmd, None)
287
 
        self.ACommand = ACommand
288
 
 
289
 
    def test_fires_on_get_cmd_object(self):
290
 
        # The get_missing_command(cmd) hook fires when commands are delivered to the
291
 
        # ui.
292
 
        self.hook_missing()
293
 
        # create a command directly, should not fire
294
 
        self.cmd = self.ACommand()
295
 
        self.assertEqual([], self.hook_calls)
296
 
        # ask by name, should fire and give us our command
297
 
        cmd = commands.get_cmd_object('foo')
298
 
        self.assertEqual([('called', 'foo')], self.hook_calls)
299
 
        self.assertIsInstance(cmd, self.ACommand)
300
 
        del self.hook_calls[:]
301
 
        # ask by a name that is supplied by a builtin - the hook should not
302
 
        # fire and we still get our object.
303
 
        commands.install_bzr_command_hooks()
304
 
        cmd = commands.get_cmd_object('info')
305
 
        self.assertNotEqual(None, cmd)
306
 
        self.assertEqual(0, len(self.hook_calls))
307
 
 
308
 
    def test_skipped_on_HelpCommandIndex_get_topics(self):
309
 
        # The get_missing_command(cmd_name) hook is not fired when
310
 
        # looking up help topics.
311
 
        self.hook_missing()
312
 
        topic = commands.HelpCommandIndex()
313
 
        topics = topic.get_topics('foo')
314
 
        self.assertEqual([], self.hook_calls)
315
 
 
316
 
 
317
 
class TestListCommandHook(tests.TestCase):
318
 
 
319
 
    def test_fires_on_all_command_names(self):
320
 
        # The list_commands() hook fires when all_command_names() is invoked.
321
 
        hook_calls = []
322
 
        commands.install_bzr_command_hooks()
323
 
        def list_my_commands(cmd_names):
324
 
            hook_calls.append('called')
325
 
            cmd_names.update(['foo', 'bar'])
326
 
            return cmd_names
327
 
        commands.Command.hooks.install_named_hook(
328
 
            "list_commands", list_my_commands, None)
329
 
        # Get a command, which should not trigger the hook.
330
 
        cmd = commands.get_cmd_object('info')
331
 
        self.assertEqual([], hook_calls)
332
 
        # Get all command classes (for docs and shell completion).
333
 
        cmds = list(commands.all_command_names())
334
 
        self.assertEqual(['called'], hook_calls)
335
 
        self.assertSubset(['foo', 'bar'], cmds)
336
 
 
337
 
class TestDeprecations(tests.TestCase):
338
 
 
339
 
    def test_shlex_split_unicode_deprecation(self):
340
 
        res = self.applyDeprecated(
341
 
                symbol_versioning.deprecated_in((2, 2, 0)),
342
 
                commands.shlex_split_unicode, 'whatever')