~bzr-pqm/bzr/bzr.dev

5557.1.15 by John Arbash Meinel
Merge bzr.dev 5597 to resolve NEWS, aka bzr-2.3.txt
1
# Copyright (C) 2005-2011 Canonical Ltd
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
2
#
1185.12.59 by Aaron Bentley
Added command-quieting test
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
7
#
1185.12.59 by Aaron Bentley
Added command-quieting test
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
1887.1.1 by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines,
12
#
1185.12.59 by Aaron Bentley
Added command-quieting test
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
4183.7.1 by Sabin Iacob
update FSF mailing address
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
1185.12.59 by Aaron Bentley
Added command-quieting test
16
1948.1.1 by John Arbash Meinel
Raise a reasonable error when a command is non-ascii
17
import errno
5264.1.2 by Robert Collins
Final fix for 'no help for command' issue. We now show a clean message
18
import inspect
3785.1.4 by Aaron Bentley
Enable lazy-loading of commands
19
import sys
1948.1.1 by John Arbash Meinel
Raise a reasonable error when a command is non-ascii
20
21
from bzrlib import (
4000.1.1 by Robert Collins
Add a new hook Commands['extend_command'] for plugins that want to alter commands without overriding the entire command.
22
    builtins,
1948.1.1 by John Arbash Meinel
Raise a reasonable error when a command is non-ascii
23
    commands,
2155.2.1 by Marius Kruger
* Get command aliases to respect quoted arguments.
24
    config,
1948.1.1 by John Arbash Meinel
Raise a reasonable error when a command is non-ascii
25
    errors,
4251.1.1 by Aaron Bentley
Support hidden options.
26
    option,
5050.1.4 by Vincent Ladeuil
Really deprecate commands.shlex_split_unicode.
27
    symbol_versioning,
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
28
    tests,
1948.1.1 by John Arbash Meinel
Raise a reasonable error when a command is non-ascii
29
    )
30
from bzrlib.commands import display_command
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
31
from bzrlib.tests import TestSkipped
32
33
34
class TestCommands(tests.TestCase):
1948.1.1 by John Arbash Meinel
Raise a reasonable error when a command is non-ascii
35
5264.1.2 by Robert Collins
Final fix for 'no help for command' issue. We now show a clean message
36
    def test_all_commands_have_help(self):
37
        commands._register_builtin_commands()
38
        commands_without_help = set()
39
        base_doc = inspect.getdoc(commands.Command)
40
        for cmd_name in commands.all_command_names():
41
            cmd = commands.get_cmd_object(cmd_name)
42
            cmd_help = cmd.help()
43
            if not cmd_help or cmd_help == base_doc:
44
                commands_without_help.append(cmd_name)
45
        self.assertLength(0, commands_without_help)
46
1185.12.59 by Aaron Bentley
Added command-quieting test
47
    def test_display_command(self):
1185.33.18 by Martin Pool
[patch] handle bad IOError subclass raised by urlopen
48
        """EPIPE message is selectively suppressed"""
1185.12.59 by Aaron Bentley
Added command-quieting test
49
        def pipe_thrower():
50
            raise IOError(errno.EPIPE, "Bogus pipe error")
51
        self.assertRaises(IOError, pipe_thrower)
52
        @display_command
53
        def non_thrower():
54
            pipe_thrower()
55
        non_thrower()
56
        @display_command
57
        def other_thrower():
58
            raise IOError(errno.ESPIPE, "Bogus pipe error")
59
        self.assertRaises(IOError, other_thrower)
60
1948.1.1 by John Arbash Meinel
Raise a reasonable error when a command is non-ascii
61
    def test_unicode_command(self):
1948.1.8 by John Arbash Meinel
Don't raise UnicodeCommand on request, instead just let it fall out when we get to NoSuchCommand
62
        # This error is thrown when we can't find the command in the
63
        # list of available commands
64
        self.assertRaises(errors.BzrCommandError,
1948.1.1 by John Arbash Meinel
Raise a reasonable error when a command is non-ascii
65
                          commands.run_bzr, [u'cmd\xb5'])
1948.1.8 by John Arbash Meinel
Don't raise UnicodeCommand on request, instead just let it fall out when we get to NoSuchCommand
66
67
    def test_unicode_option(self):
68
        # This error is actually thrown by optparse, when it
69
        # can't find the given option
1913.2.4 by Martin Pool
python2.5 apparently has trouble with unicode options
70
        import optparse
71
        if optparse.__version__ == "1.5.3":
72
            raise TestSkipped("optparse 1.5.3 can't handle unicode options")
1948.1.8 by John Arbash Meinel
Don't raise UnicodeCommand on request, instead just let it fall out when we get to NoSuchCommand
73
        self.assertRaises(errors.BzrCommandError,
74
                          commands.run_bzr, ['log', u'--option\xb5'])
75
4251.1.1 by Aaron Bentley
Support hidden options.
76
    @staticmethod
77
    def get_command(options):
4251.1.7 by Aaron Bentley
Use generated subclass instead of patching Command instance
78
        class cmd_foo(commands.Command):
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
79
            __doc__ = 'Bar'
4251.1.7 by Aaron Bentley
Use generated subclass instead of patching Command instance
80
81
            takes_options = options
82
83
        return cmd_foo()
4251.1.1 by Aaron Bentley
Support hidden options.
84
85
    def test_help_hidden(self):
86
        c = self.get_command([option.Option('foo', hidden=True)])
87
        self.assertNotContainsRe(c.get_help_text(), '--foo')
88
89
    def test_help_not_hidden(self):
90
        c = self.get_command([option.Option('foo', hidden=False)])
91
        self.assertContainsRe(c.get_help_text(), '--foo')
92
5255.5.1 by Ian Clatworthy
Ensure builtin_command_names() is initialized correctly
93
5741.3.1 by Martin Pool
Add Command.invoked_as
94
class TestInvokedAs(tests.TestCase):
95
96
    def test_invoked_as(self):
97
        """The command object knows the actual name used to invoke it."""
5741.3.8 by Martin Pool
Fix slightly broken test
98
        commands.install_bzr_command_hooks()
99
        commands._register_builtin_commands()
5741.3.1 by Martin Pool
Add Command.invoked_as
100
        # get one from the real get_cmd_object.
101
        c = commands.get_cmd_object('ci')
102
        self.assertIsInstance(c, builtins.cmd_commit)
103
        self.assertEquals(c.invoked_as, 'ci')
104
105
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
106
class TestGetAlias(tests.TestCase):
2155.2.3 by Marius Kruger
* commands.py
107
108
    def _get_config(self, config_text):
5345.2.9 by Vincent Ladeuil
Rename IniBaseConfig.from_bytes to from_string.
109
        my_config = config.GlobalConfig.from_string(config_text)
2155.2.1 by Marius Kruger
* Get command aliases to respect quoted arguments.
110
        return my_config
111
112
    def test_simple(self):
2155.2.3 by Marius Kruger
* commands.py
113
        my_config = self._get_config("[ALIASES]\n"
2155.2.1 by Marius Kruger
* Get command aliases to respect quoted arguments.
114
            "diff=diff -r -2..-1\n")
115
        self.assertEqual([u'diff', u'-r', u'-2..-1'],
116
            commands.get_alias("diff", config=my_config))
117
118
    def test_single_quotes(self):
2155.2.3 by Marius Kruger
* commands.py
119
        my_config = self._get_config("[ALIASES]\n"
2155.2.1 by Marius Kruger
* Get command aliases to respect quoted arguments.
120
            "diff=diff -r -2..-1 --diff-options "
121
            "'--strip-trailing-cr -wp'\n")
122
        self.assertEqual([u'diff', u'-r', u'-2..-1', u'--diff-options',
123
                          u'--strip-trailing-cr -wp'],
124
                          commands.get_alias("diff", config=my_config))
125
126
    def test_double_quotes(self):
2155.2.3 by Marius Kruger
* commands.py
127
        my_config = self._get_config("[ALIASES]\n"
2155.2.1 by Marius Kruger
* Get command aliases to respect quoted arguments.
128
            "diff=diff -r -2..-1 --diff-options "
129
            "\"--strip-trailing-cr -wp\"\n")
130
        self.assertEqual([u'diff', u'-r', u'-2..-1', u'--diff-options',
131
                          u'--strip-trailing-cr -wp'],
132
                          commands.get_alias("diff", config=my_config))
133
134
    def test_unicode(self):
2155.2.3 by Marius Kruger
* commands.py
135
        my_config = self._get_config("[ALIASES]\n"
4913.5.4 by Gordon Tyler
Changed test_single_quotes to skip on win32 and fixed test_unicode to not fail on win32.
136
            u'iam=whoami "Erik B\u00e5gfors <erik@bagfors.nu>"\n')
2155.2.1 by Marius Kruger
* Get command aliases to respect quoted arguments.
137
        self.assertEqual([u'whoami', u'Erik B\u00e5gfors <erik@bagfors.nu>'],
138
                          commands.get_alias("iam", config=my_config))
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
139
140
141
class TestSeeAlso(tests.TestCase):
142
    """Tests for the see also functional of Command."""
143
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
144
    @staticmethod
145
    def _get_command_with_see_also(see_also):
146
        class ACommand(commands.Command):
147
            __doc__ = """A sample command."""
148
            _see_also = see_also
149
        return ACommand()
150
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
151
    def test_default_subclass_no_see_also(self):
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
152
        command = self._get_command_with_see_also([])
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
153
        self.assertEqual([], command.get_see_also())
154
155
    def test__see_also(self):
156
        """When _see_also is defined, it sets the result of get_see_also()."""
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
157
        command = self._get_command_with_see_also(['bar', 'foo'])
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
158
        self.assertEqual(['bar', 'foo'], command.get_see_also())
159
160
    def test_deduplication(self):
161
        """Duplicates in _see_also are stripped out."""
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
162
        command = self._get_command_with_see_also(['foo', 'foo'])
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
163
        self.assertEqual(['foo'], command.get_see_also())
164
165
    def test_sorted(self):
166
        """_see_also is sorted by get_see_also."""
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
167
        command = self._get_command_with_see_also(['foo', 'bar'])
2425.2.1 by Robert Collins
Command objects can now declare related help topics by having _see_also
168
        self.assertEqual(['bar', 'foo'], command.get_see_also())
2432.1.21 by Robert Collins
Teach Command.get_help_text to show additional help cross references when supplied.
169
170
    def test_additional_terms(self):
171
        """Additional terms can be supplied and are deduped and sorted."""
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
172
        command = self._get_command_with_see_also(['foo', 'bar'])
2432.1.21 by Robert Collins
Teach Command.get_help_text to show additional help cross references when supplied.
173
        self.assertEqual(['bar', 'foo', 'gam'],
174
            command.get_see_also(['gam', 'bar', 'gam']))
175
3785.1.4 by Aaron Bentley
Enable lazy-loading of commands
176
177
class TestRegisterLazy(tests.TestCase):
178
179
    def setUp(self):
4153.1.2 by Andrew Bennetts
Add missing TestCase.setUp upcalls.
180
        tests.TestCase.setUp(self)
3785.1.4 by Aaron Bentley
Enable lazy-loading of commands
181
        import bzrlib.tests.fake_command
182
        del sys.modules['bzrlib.tests.fake_command']
183
        global lazy_command_imported
184
        lazy_command_imported = False
4119.3.8 by Robert Collins
Get missing command support sorted out.
185
        commands.install_bzr_command_hooks()
3785.1.4 by Aaron Bentley
Enable lazy-loading of commands
186
187
    @staticmethod
188
    def remove_fake():
189
        commands.plugin_cmds.remove('fake')
190
191
    def assertIsFakeCommand(self, cmd_obj):
192
        from bzrlib.tests.fake_command import cmd_fake
193
        self.assertIsInstance(cmd_obj, cmd_fake)
194
195
    def test_register_lazy(self):
196
        """Ensure lazy registration works"""
197
        commands.plugin_cmds.register_lazy('cmd_fake', [],
198
                                           'bzrlib.tests.fake_command')
199
        self.addCleanup(self.remove_fake)
200
        self.assertFalse(lazy_command_imported)
201
        fake_instance = commands.get_cmd_object('fake')
202
        self.assertTrue(lazy_command_imported)
203
        self.assertIsFakeCommand(fake_instance)
204
205
    def test_get_unrelated_does_not_import(self):
206
        commands.plugin_cmds.register_lazy('cmd_fake', [],
207
                                           'bzrlib.tests.fake_command')
208
        self.addCleanup(self.remove_fake)
209
        commands.get_cmd_object('status')
210
        self.assertFalse(lazy_command_imported)
211
212
    def test_aliases(self):
213
        commands.plugin_cmds.register_lazy('cmd_fake', ['fake_alias'],
214
                                           'bzrlib.tests.fake_command')
215
        self.addCleanup(self.remove_fake)
216
        fake_instance = commands.get_cmd_object('fake_alias')
217
        self.assertIsFakeCommand(fake_instance)
4000.1.1 by Robert Collins
Add a new hook Commands['extend_command'] for plugins that want to alter commands without overriding the entire command.
218
219
220
class TestExtendCommandHook(tests.TestCase):
221
222
    def test_fires_on_get_cmd_object(self):
223
        # The extend_command(cmd) hook fires when commands are delivered to the
224
        # ui, not simply at registration (because lazy registered plugin
225
        # commands are registered).
226
        # when they are simply created.
227
        hook_calls = []
4119.3.8 by Robert Collins
Get missing command support sorted out.
228
        commands.install_bzr_command_hooks()
4000.1.1 by Robert Collins
Add a new hook Commands['extend_command'] for plugins that want to alter commands without overriding the entire command.
229
        commands.Command.hooks.install_named_hook(
230
            "extend_command", hook_calls.append, None)
231
        # create a command, should not fire
5018.1.12 by Martin Pool
Tests for commands must now properly register them, not just poke them into bzrlib.builtins
232
        class cmd_test_extend_command_hook(commands.Command):
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
233
            __doc__ = """A sample command."""
4000.1.1 by Robert Collins
Add a new hook Commands['extend_command'] for plugins that want to alter commands without overriding the entire command.
234
        self.assertEqual([], hook_calls)
235
        # -- as a builtin
236
        # register the command class, should not fire
237
        try:
5018.1.12 by Martin Pool
Tests for commands must now properly register them, not just poke them into bzrlib.builtins
238
            commands.builtin_command_registry.register(cmd_test_extend_command_hook)
4000.1.1 by Robert Collins
Add a new hook Commands['extend_command'] for plugins that want to alter commands without overriding the entire command.
239
            self.assertEqual([], hook_calls)
240
            # and ask for the object, should fire
241
            cmd = commands.get_cmd_object('test-extend-command-hook')
242
            # For resilience - to ensure all code paths hit it - we
243
            # fire on everything returned in the 'cmd_dict', which is currently
244
            # all known commands, so assert that cmd is in hook_calls
245
            self.assertSubset([cmd], hook_calls)
246
            del hook_calls[:]
247
        finally:
5018.1.12 by Martin Pool
Tests for commands must now properly register them, not just poke them into bzrlib.builtins
248
            commands.builtin_command_registry.remove('test-extend-command-hook')
4000.1.1 by Robert Collins
Add a new hook Commands['extend_command'] for plugins that want to alter commands without overriding the entire command.
249
        # -- as a plugin lazy registration
250
        try:
251
            # register the command class, should not fire
252
            commands.plugin_cmds.register_lazy('cmd_fake', [],
253
                                               'bzrlib.tests.fake_command')
254
            self.assertEqual([], hook_calls)
255
            # and ask for the object, should fire
256
            cmd = commands.get_cmd_object('fake')
257
            self.assertEqual([cmd], hook_calls)
258
        finally:
259
            commands.plugin_cmds.remove('fake')
4119.3.4 by Robert Collins
Add Command lookup hooks: list_commands and get_command.
260
261
262
class TestGetCommandHook(tests.TestCase):
263
264
    def test_fires_on_get_cmd_object(self):
265
        # The get_command(cmd) hook fires when commands are delivered to the
266
        # ui.
4119.3.8 by Robert Collins
Get missing command support sorted out.
267
        commands.install_bzr_command_hooks()
4119.3.4 by Robert Collins
Add Command lookup hooks: list_commands and get_command.
268
        hook_calls = []
269
        class ACommand(commands.Command):
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
270
            __doc__ = """A sample command."""
4119.3.4 by Robert Collins
Add Command lookup hooks: list_commands and get_command.
271
        def get_cmd(cmd_or_None, cmd_name):
272
            hook_calls.append(('called', cmd_or_None, cmd_name))
273
            if cmd_name in ('foo', 'info'):
274
                return ACommand()
275
        commands.Command.hooks.install_named_hook(
276
            "get_command", get_cmd, None)
277
        # create a command directly, should not fire
278
        cmd = ACommand()
279
        self.assertEqual([], hook_calls)
280
        # ask by name, should fire and give us our command
281
        cmd = commands.get_cmd_object('foo')
282
        self.assertEqual([('called', None, 'foo')], hook_calls)
283
        self.assertIsInstance(cmd, ACommand)
284
        del hook_calls[:]
285
        # ask by a name that is supplied by a builtin - the hook should still
286
        # fire and we still get our object, but we should see the builtin
287
        # passed to the hook.
288
        cmd = commands.get_cmd_object('info')
289
        self.assertIsInstance(cmd, ACommand)
290
        self.assertEqual(1, len(hook_calls))
291
        self.assertEqual('info', hook_calls[0][2])
292
        self.assertIsInstance(hook_calls[0][1], builtins.cmd_info)
293
294
4119.3.8 by Robert Collins
Get missing command support sorted out.
295
class TestGetMissingCommandHook(tests.TestCase):
296
5058.2.1 by Robert Collins
* ``bzr help`` will no longer trigger the get_missing_command hook when
297
    def hook_missing(self):
298
        """Hook get_missing_command for testing."""
299
        self.hook_calls = []
4119.3.8 by Robert Collins
Get missing command support sorted out.
300
        class ACommand(commands.Command):
5131.2.6 by Martin
Fix more tests which were failing under -OO that had been missed earlier
301
            __doc__ = """A sample command."""
4119.3.8 by Robert Collins
Get missing command support sorted out.
302
        def get_missing_cmd(cmd_name):
5058.2.1 by Robert Collins
* ``bzr help`` will no longer trigger the get_missing_command hook when
303
            self.hook_calls.append(('called', cmd_name))
4119.3.8 by Robert Collins
Get missing command support sorted out.
304
            if cmd_name in ('foo', 'info'):
305
                return ACommand()
306
        commands.Command.hooks.install_named_hook(
307
            "get_missing_command", get_missing_cmd, None)
5058.2.1 by Robert Collins
* ``bzr help`` will no longer trigger the get_missing_command hook when
308
        self.ACommand = ACommand
309
310
    def test_fires_on_get_cmd_object(self):
311
        # The get_missing_command(cmd) hook fires when commands are delivered to the
312
        # ui.
313
        self.hook_missing()
4119.3.8 by Robert Collins
Get missing command support sorted out.
314
        # create a command directly, should not fire
5058.2.1 by Robert Collins
* ``bzr help`` will no longer trigger the get_missing_command hook when
315
        self.cmd = self.ACommand()
316
        self.assertEqual([], self.hook_calls)
4119.3.8 by Robert Collins
Get missing command support sorted out.
317
        # ask by name, should fire and give us our command
318
        cmd = commands.get_cmd_object('foo')
5058.2.1 by Robert Collins
* ``bzr help`` will no longer trigger the get_missing_command hook when
319
        self.assertEqual([('called', 'foo')], self.hook_calls)
320
        self.assertIsInstance(cmd, self.ACommand)
321
        del self.hook_calls[:]
4119.3.8 by Robert Collins
Get missing command support sorted out.
322
        # ask by a name that is supplied by a builtin - the hook should not
323
        # fire and we still get our object.
324
        commands.install_bzr_command_hooks()
325
        cmd = commands.get_cmd_object('info')
326
        self.assertNotEqual(None, cmd)
5058.2.1 by Robert Collins
* ``bzr help`` will no longer trigger the get_missing_command hook when
327
        self.assertEqual(0, len(self.hook_calls))
328
329
    def test_skipped_on_HelpCommandIndex_get_topics(self):
330
        # The get_missing_command(cmd_name) hook is not fired when
331
        # looking up help topics.
332
        self.hook_missing()
333
        topic = commands.HelpCommandIndex()
334
        topics = topic.get_topics('foo')
335
        self.assertEqual([], self.hook_calls)
4119.3.8 by Robert Collins
Get missing command support sorted out.
336
337
4119.3.4 by Robert Collins
Add Command lookup hooks: list_commands and get_command.
338
class TestListCommandHook(tests.TestCase):
339
340
    def test_fires_on_all_command_names(self):
341
        # The list_commands() hook fires when all_command_names() is invoked.
342
        hook_calls = []
4119.3.8 by Robert Collins
Get missing command support sorted out.
343
        commands.install_bzr_command_hooks()
4119.3.4 by Robert Collins
Add Command lookup hooks: list_commands and get_command.
344
        def list_my_commands(cmd_names):
345
            hook_calls.append('called')
346
            cmd_names.update(['foo', 'bar'])
347
            return cmd_names
348
        commands.Command.hooks.install_named_hook(
349
            "list_commands", list_my_commands, None)
350
        # Get a command, which should not trigger the hook.
351
        cmd = commands.get_cmd_object('info')
352
        self.assertEqual([], hook_calls)
353
        # Get all command classes (for docs and shell completion).
354
        cmds = list(commands.all_command_names())
355
        self.assertEqual(['called'], hook_calls)
356
        self.assertSubset(['foo', 'bar'], cmds)