~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_plugins.py

  • Committer: Martin Pool
  • Date: 2005-05-09 04:38:31 UTC
  • Revision ID: mbp@sourcefrog.net-20050509043831-d45f7832b7d4d5b0
- better message when refusing to add symlinks (from mpe)

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
# Copyright (C) 2005, 2007 Canonical Ltd
2
 
#
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.
7
 
#
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.
12
 
#
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
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
16
 
 
17
 
"""Tests for plugins"""
18
 
 
19
 
# XXX: There are no plugin tests at the moment because the plugin module
20
 
# affects the global state of the process.  See bzrlib/plugins.py for more
21
 
# comments.
22
 
 
23
 
import os
24
 
from StringIO import StringIO
25
 
import sys
26
 
import zipfile
27
 
 
28
 
from bzrlib import plugin, tests
29
 
import bzrlib.plugin
30
 
import bzrlib.plugins
31
 
import bzrlib.commands
32
 
import bzrlib.help
33
 
from bzrlib.symbol_versioning import zero_ninetyone
34
 
from bzrlib.tests import TestCase, TestCaseInTempDir
35
 
from bzrlib.osutils import pathjoin, abspath, normpath
36
 
 
37
 
 
38
 
PLUGIN_TEXT = """\
39
 
import bzrlib.commands
40
 
class cmd_myplug(bzrlib.commands.Command):
41
 
    '''Just a simple test plugin.'''
42
 
    aliases = ['mplg']
43
 
    def run(self):
44
 
        print 'Hello from my plugin'
45
 
"""
46
 
 
47
 
# TODO: Write a test for plugin decoration of commands.
48
 
 
49
 
class TestLoadingPlugins(TestCaseInTempDir):
50
 
 
51
 
    activeattributes = {}
52
 
 
53
 
    def test_plugins_with_the_same_name_are_not_loaded(self):
54
 
        # This test tests that having two plugins in different directories does
55
 
        # not result in both being loaded when they have the same name.  get a
56
 
        # file name we can use which is also a valid attribute for accessing in
57
 
        # activeattributes. - we cannot give import parameters.
58
 
        tempattribute = "0"
59
 
        self.failIf(tempattribute in self.activeattributes)
60
 
        # set a place for the plugins to record their loading, and at the same
61
 
        # time validate that the location the plugins should record to is
62
 
        # valid and correct.
63
 
        bzrlib.tests.test_plugins.TestLoadingPlugins.activeattributes \
64
 
            [tempattribute] = []
65
 
        self.failUnless(tempattribute in self.activeattributes)
66
 
        # create two plugin directories
67
 
        os.mkdir('first')
68
 
        os.mkdir('second')
69
 
        # write a plugin that will record when its loaded in the 
70
 
        # tempattribute list.
71
 
        template = ("from bzrlib.tests.test_plugins import TestLoadingPlugins\n"
72
 
                    "TestLoadingPlugins.activeattributes[%r].append('%s')\n")
73
 
 
74
 
        outfile = open(os.path.join('first', 'plugin.py'), 'w')
75
 
        try:
76
 
            outfile.write(template % (tempattribute, 'first'))
77
 
            outfile.write('\n')
78
 
        finally:
79
 
            outfile.close()
80
 
 
81
 
        outfile = open(os.path.join('second', 'plugin.py'), 'w')
82
 
        try:
83
 
            outfile.write(template % (tempattribute, 'second'))
84
 
            outfile.write('\n')
85
 
        finally:
86
 
            outfile.close()
87
 
 
88
 
        try:
89
 
            bzrlib.plugin.load_from_path(['first', 'second'])
90
 
            self.assertEqual(['first'], self.activeattributes[tempattribute])
91
 
        finally:
92
 
            # remove the plugin 'plugin'
93
 
            del self.activeattributes[tempattribute]
94
 
            if 'bzrlib.plugins.plugin' in sys.modules:
95
 
                del sys.modules['bzrlib.plugins.plugin']
96
 
            if getattr(bzrlib.plugins, 'plugin', None):
97
 
                del bzrlib.plugins.plugin
98
 
        self.failIf(getattr(bzrlib.plugins, 'plugin', None))
99
 
 
100
 
    def test_plugins_from_different_dirs_can_demand_load(self):
101
 
        # This test tests that having two plugins in different
102
 
        # directories with different names allows them both to be loaded, when
103
 
        # we do a direct import statement.
104
 
        # Determine a file name we can use which is also a valid attribute
105
 
        # for accessing in activeattributes. - we cannot give import parameters.
106
 
        tempattribute = "different-dirs"
107
 
        self.failIf(tempattribute in self.activeattributes)
108
 
        # set a place for the plugins to record their loading, and at the same
109
 
        # time validate that the location the plugins should record to is
110
 
        # valid and correct.
111
 
        bzrlib.tests.test_plugins.TestLoadingPlugins.activeattributes \
112
 
            [tempattribute] = []
113
 
        self.failUnless(tempattribute in self.activeattributes)
114
 
        # create two plugin directories
115
 
        os.mkdir('first')
116
 
        os.mkdir('second')
117
 
        # write plugins that will record when they are loaded in the 
118
 
        # tempattribute list.
119
 
        template = ("from bzrlib.tests.test_plugins import TestLoadingPlugins\n"
120
 
                    "TestLoadingPlugins.activeattributes[%r].append('%s')\n")
121
 
 
122
 
        outfile = open(os.path.join('first', 'pluginone.py'), 'w')
123
 
        try:
124
 
            outfile.write(template % (tempattribute, 'first'))
125
 
            outfile.write('\n')
126
 
        finally:
127
 
            outfile.close()
128
 
 
129
 
        outfile = open(os.path.join('second', 'plugintwo.py'), 'w')
130
 
        try:
131
 
            outfile.write(template % (tempattribute, 'second'))
132
 
            outfile.write('\n')
133
 
        finally:
134
 
            outfile.close()
135
 
 
136
 
        oldpath = bzrlib.plugins.__path__
137
 
        try:
138
 
            bzrlib.plugins.__path__ = ['first', 'second']
139
 
            exec "import bzrlib.plugins.pluginone"
140
 
            self.assertEqual(['first'], self.activeattributes[tempattribute])
141
 
            exec "import bzrlib.plugins.plugintwo"
142
 
            self.assertEqual(['first', 'second'],
143
 
                self.activeattributes[tempattribute])
144
 
        finally:
145
 
            # remove the plugin 'plugin'
146
 
            del self.activeattributes[tempattribute]
147
 
            if getattr(bzrlib.plugins, 'pluginone', None):
148
 
                del bzrlib.plugins.pluginone
149
 
            if getattr(bzrlib.plugins, 'plugintwo', None):
150
 
                del bzrlib.plugins.plugintwo
151
 
        self.failIf(getattr(bzrlib.plugins, 'pluginone', None))
152
 
        self.failIf(getattr(bzrlib.plugins, 'plugintwo', None))
153
 
 
154
 
    def test_plugins_can_load_from_directory_with_trailing_slash(self):
155
 
        # This test tests that a plugin can load from a directory when the
156
 
        # directory in the path has a trailing slash.
157
 
        # check the plugin is not loaded already
158
 
        self.failIf(getattr(bzrlib.plugins, 'ts_plugin', None))
159
 
        tempattribute = "trailing-slash"
160
 
        self.failIf(tempattribute in self.activeattributes)
161
 
        # set a place for the plugin to record its loading, and at the same
162
 
        # time validate that the location the plugin should record to is
163
 
        # valid and correct.
164
 
        bzrlib.tests.test_plugins.TestLoadingPlugins.activeattributes \
165
 
            [tempattribute] = []
166
 
        self.failUnless(tempattribute in self.activeattributes)
167
 
        # create a directory for the plugin
168
 
        os.mkdir('plugin_test')
169
 
        # write a plugin that will record when its loaded in the 
170
 
        # tempattribute list.
171
 
        template = ("from bzrlib.tests.test_plugins import TestLoadingPlugins\n"
172
 
                    "TestLoadingPlugins.activeattributes[%r].append('%s')\n")
173
 
 
174
 
        outfile = open(os.path.join('plugin_test', 'ts_plugin.py'), 'w')
175
 
        try:
176
 
            outfile.write(template % (tempattribute, 'plugin'))
177
 
            ourfile.write('\n')
178
 
        finally:
179
 
            outfile.close()
180
 
 
181
 
        try:
182
 
            bzrlib.plugin.load_from_path(['plugin_test'+os.sep])
183
 
            self.assertEqual(['plugin'], self.activeattributes[tempattribute])
184
 
        finally:
185
 
            # remove the plugin 'plugin'
186
 
            del self.activeattributes[tempattribute]
187
 
            if getattr(bzrlib.plugins, 'ts_plugin', None):
188
 
                del bzrlib.plugins.ts_plugin
189
 
        self.failIf(getattr(bzrlib.plugins, 'ts_plugin', None))
190
 
 
191
 
 
192
 
class TestAllPlugins(TestCaseInTempDir):
193
 
 
194
 
    def test_plugin_appears_in_all_plugins(self):
195
 
        # This test tests a new plugin appears in bzrlib.plugin.all_plugins().
196
 
        # check the plugin is not loaded already
197
 
        self.failIf(getattr(bzrlib.plugins, 'plugin', None))
198
 
        # write a plugin that _cannot_ fail to load.
199
 
        file('plugin.py', 'w').write("\n")
200
 
        try:
201
 
            bzrlib.plugin.load_from_path(['.'])
202
 
            all_plugins = self.applyDeprecated(zero_ninetyone,
203
 
                bzrlib.plugin.all_plugins)
204
 
            self.failUnless('plugin' in all_plugins)
205
 
            self.failUnless(getattr(bzrlib.plugins, 'plugin', None))
206
 
            self.assertEqual(all_plugins['plugin'], bzrlib.plugins.plugin)
207
 
        finally:
208
 
            # remove the plugin 'plugin'
209
 
            if 'bzrlib.plugins.plugin' in sys.modules:
210
 
                del sys.modules['bzrlib.plugins.plugin']
211
 
            if getattr(bzrlib.plugins, 'plugin', None):
212
 
                del bzrlib.plugins.plugin
213
 
        self.failIf(getattr(bzrlib.plugins, 'plugin', None))
214
 
 
215
 
 
216
 
class TestPlugins(TestCaseInTempDir):
217
 
 
218
 
    def setup_plugin(self, source=""):
219
 
        # This test tests a new plugin appears in bzrlib.plugin.plugins().
220
 
        # check the plugin is not loaded already
221
 
        self.failIf(getattr(bzrlib.plugins, 'plugin', None))
222
 
        # write a plugin that _cannot_ fail to load.
223
 
        file('plugin.py', 'w').write(source + '\n')
224
 
        self.addCleanup(self.teardown_plugin)
225
 
        bzrlib.plugin.load_from_path(['.'])
226
 
    
227
 
    def teardown_plugin(self):
228
 
        # remove the plugin 'plugin'
229
 
        if 'bzrlib.plugins.plugin' in sys.modules:
230
 
            del sys.modules['bzrlib.plugins.plugin']
231
 
        if getattr(bzrlib.plugins, 'plugin', None):
232
 
            del bzrlib.plugins.plugin
233
 
        self.failIf(getattr(bzrlib.plugins, 'plugin', None))
234
 
 
235
 
    def test_plugin_appears_in_plugins(self):
236
 
        self.setup_plugin()
237
 
        self.failUnless('plugin' in bzrlib.plugin.plugins())
238
 
        self.failUnless(getattr(bzrlib.plugins, 'plugin', None))
239
 
        plugins = bzrlib.plugin.plugins()
240
 
        plugin = plugins['plugin']
241
 
        self.assertIsInstance(plugin, bzrlib.plugin.PlugIn)
242
 
        self.assertEqual(bzrlib.plugins.plugin, plugin.module)
243
 
 
244
 
    def test_trivial_plugin_get_path(self):
245
 
        self.setup_plugin()
246
 
        plugins = bzrlib.plugin.plugins()
247
 
        plugin = plugins['plugin']
248
 
        plugin_path = self.test_dir + '/plugin.py'
249
 
        self.assertIsSameRealPath(plugin_path, normpath(plugin.path()))
250
 
 
251
 
    def test_no_test_suite_gives_None_for_test_suite(self):
252
 
        self.setup_plugin()
253
 
        plugin = bzrlib.plugin.plugins()['plugin']
254
 
        self.assertEqual(None, plugin.test_suite())
255
 
 
256
 
    def test_test_suite_gives_test_suite_result(self):
257
 
        source = """def test_suite(): return 'foo'"""
258
 
        self.setup_plugin(source)
259
 
        plugin = bzrlib.plugin.plugins()['plugin']
260
 
        self.assertEqual('foo', plugin.test_suite())
261
 
 
262
 
    def test_no_version_info(self):
263
 
        self.setup_plugin()
264
 
        plugin = bzrlib.plugin.plugins()['plugin']
265
 
        self.assertEqual(None, plugin.version_info())
266
 
 
267
 
    def test_with_version_info(self):
268
 
        self.setup_plugin("version_info = (1, 2, 3, 'dev', 4)")
269
 
        plugin = bzrlib.plugin.plugins()['plugin']
270
 
        self.assertEqual((1, 2, 3, 'dev', 4), plugin.version_info())
271
 
 
272
 
    def test_short_version_info_gets_padded(self):
273
 
        # the gtk plugin has version_info = (1,2,3) rather than the 5-tuple.
274
 
        # so we adapt it
275
 
        self.setup_plugin("version_info = (1, 2, 3)")
276
 
        plugin = bzrlib.plugin.plugins()['plugin']
277
 
        self.assertEqual((1, 2, 3, 'final', 0), plugin.version_info())
278
 
 
279
 
    def test_no_version_info___version__(self):
280
 
        self.setup_plugin()
281
 
        plugin = bzrlib.plugin.plugins()['plugin']
282
 
        self.assertEqual("unknown", plugin.__version__)
283
 
 
284
 
    def test___version__with_version_info(self):
285
 
        self.setup_plugin("version_info = (1, 2, 3, 'dev', 4)")
286
 
        plugin = bzrlib.plugin.plugins()['plugin']
287
 
        self.assertEqual("1.2.3dev4", plugin.__version__)
288
 
 
289
 
    def test_final__version__with_version_info(self):
290
 
        self.setup_plugin("version_info = (1, 2, 3, 'final', 4)")
291
 
        plugin = bzrlib.plugin.plugins()['plugin']
292
 
        self.assertEqual("1.2.3", plugin.__version__)
293
 
 
294
 
 
295
 
class TestPluginHelp(TestCaseInTempDir):
296
 
 
297
 
    def split_help_commands(self):
298
 
        help = {}
299
 
        current = None
300
 
        for line in self.run_bzr('help commands')[0].splitlines():
301
 
            if not line.startswith(' '):
302
 
                current = line.split()[0]
303
 
            help[current] = help.get(current, '') + line
304
 
 
305
 
        return help
306
 
 
307
 
    def test_plugin_help_builtins_unaffected(self):
308
 
        # Check we don't get false positives
309
 
        help_commands = self.split_help_commands()
310
 
        for cmd_name in bzrlib.commands.builtin_command_names():
311
 
            if cmd_name in bzrlib.commands.plugin_command_names():
312
 
                continue
313
 
            try:
314
 
                help = bzrlib.commands.get_cmd_object(cmd_name).get_help_text()
315
 
            except NotImplementedError:
316
 
                # some commands have no help
317
 
                pass
318
 
            else:
319
 
                self.assertNotContainsRe(help, 'plugin "[^"]*"')
320
 
 
321
 
            if cmd_name in help_commands.keys():
322
 
                # some commands are hidden
323
 
                help = help_commands[cmd_name]
324
 
                self.assertNotContainsRe(help, 'plugin "[^"]*"')
325
 
 
326
 
    def test_plugin_help_shows_plugin(self):
327
 
        # Create a test plugin
328
 
        os.mkdir('plugin_test')
329
 
        f = open(pathjoin('plugin_test', 'myplug.py'), 'w')
330
 
        f.write(PLUGIN_TEXT)
331
 
        f.close()
332
 
 
333
 
        try:
334
 
            # Check its help
335
 
            bzrlib.plugin.load_from_path(['plugin_test'])
336
 
            bzrlib.commands.register_command( bzrlib.plugins.myplug.cmd_myplug)
337
 
            help = self.run_bzr('help myplug')[0]
338
 
            self.assertContainsRe(help, 'plugin "myplug"')
339
 
            help = self.split_help_commands()['myplug']
340
 
            self.assertContainsRe(help, '\[myplug\]')
341
 
        finally:
342
 
            # unregister command
343
 
            if bzrlib.commands.plugin_cmds.get('myplug', None):
344
 
                del bzrlib.commands.plugin_cmds['myplug']
345
 
            # remove the plugin 'myplug'
346
 
            if getattr(bzrlib.plugins, 'myplug', None):
347
 
                delattr(bzrlib.plugins, 'myplug')
348
 
 
349
 
 
350
 
class TestPluginFromZip(TestCaseInTempDir):
351
 
 
352
 
    def make_zipped_plugin(self, zip_name, filename):
353
 
        z = zipfile.ZipFile(zip_name, 'w')
354
 
        z.writestr(filename, PLUGIN_TEXT)
355
 
        z.close()
356
 
 
357
 
    def check_plugin_load(self, zip_name, plugin_name):
358
 
        self.assertFalse(plugin_name in dir(bzrlib.plugins),
359
 
                         'Plugin already loaded')
360
 
        old_path = bzrlib.plugins.__path__
361
 
        try:
362
 
            # this is normally done by load_plugins -> set_plugins_path
363
 
            bzrlib.plugins.__path__ = [zip_name]
364
 
            bzrlib.plugin.load_from_zip(zip_name)
365
 
            self.assertTrue(plugin_name in dir(bzrlib.plugins),
366
 
                            'Plugin is not loaded')
367
 
        finally:
368
 
            # unregister plugin
369
 
            if getattr(bzrlib.plugins, plugin_name, None):
370
 
                delattr(bzrlib.plugins, plugin_name)
371
 
                del sys.modules['bzrlib.plugins.' + plugin_name]
372
 
            bzrlib.plugins.__path__ = old_path
373
 
 
374
 
    def test_load_module(self):
375
 
        self.make_zipped_plugin('./test.zip', 'ziplug.py')
376
 
        self.check_plugin_load('./test.zip', 'ziplug')
377
 
 
378
 
    def test_load_package(self):
379
 
        self.make_zipped_plugin('./test.zip', 'ziplug/__init__.py')
380
 
        self.check_plugin_load('./test.zip', 'ziplug')
381
 
 
382
 
 
383
 
class TestSetPluginsPath(TestCase):
384
 
    
385
 
    def test_set_plugins_path(self):
386
 
        """set_plugins_path should set the module __path__ correctly."""
387
 
        old_path = bzrlib.plugins.__path__
388
 
        try:
389
 
            bzrlib.plugins.__path__ = []
390
 
            expected_path = bzrlib.plugin.set_plugins_path()
391
 
            self.assertEqual(expected_path, bzrlib.plugins.__path__)
392
 
        finally:
393
 
            bzrlib.plugins.__path__ = old_path
394
 
 
395
 
    def test_set_plugins_path_with_trailing_slashes(self):
396
 
        """set_plugins_path should set the module __path__ based on
397
 
        BZR_PLUGIN_PATH."""
398
 
        old_path = bzrlib.plugins.__path__
399
 
        old_env = os.environ.get('BZR_PLUGIN_PATH')
400
 
        try:
401
 
            bzrlib.plugins.__path__ = []
402
 
            os.environ['BZR_PLUGIN_PATH'] = "first\\//\\" + os.pathsep + \
403
 
                "second/\\/\\/"
404
 
            bzrlib.plugin.set_plugins_path()
405
 
            expected_path = ['first', 'second',
406
 
                os.path.dirname(bzrlib.plugins.__file__)]
407
 
            self.assertEqual(expected_path, bzrlib.plugins.__path__)
408
 
        finally:
409
 
            bzrlib.plugins.__path__ = old_path
410
 
            if old_env != None:
411
 
                os.environ['BZR_PLUGIN_PATH'] = old_env
412
 
            else:
413
 
                del os.environ['BZR_PLUGIN_PATH']
414
 
 
415
 
class TestHelpIndex(tests.TestCase):
416
 
    """Tests for the PluginsHelpIndex class."""
417
 
 
418
 
    def test_default_constructable(self):
419
 
        index = plugin.PluginsHelpIndex()
420
 
 
421
 
    def test_get_topics_None(self):
422
 
        """Searching for None returns an empty list."""
423
 
        index = plugin.PluginsHelpIndex()
424
 
        self.assertEqual([], index.get_topics(None))
425
 
 
426
 
    def test_get_topics_for_plugin(self):
427
 
        """Searching for plugin name gets its docstring."""
428
 
        index = plugin.PluginsHelpIndex()
429
 
        # make a new plugin here for this test, even if we're run with
430
 
        # --no-plugins
431
 
        self.assertFalse(sys.modules.has_key('bzrlib.plugins.demo_module'))
432
 
        demo_module = FakeModule('', 'bzrlib.plugins.demo_module')
433
 
        sys.modules['bzrlib.plugins.demo_module'] = demo_module
434
 
        try:
435
 
            topics = index.get_topics('demo_module')
436
 
            self.assertEqual(1, len(topics))
437
 
            self.assertIsInstance(topics[0], plugin.ModuleHelpTopic)
438
 
            self.assertEqual(demo_module, topics[0].module)
439
 
        finally:
440
 
            del sys.modules['bzrlib.plugins.demo_module']
441
 
 
442
 
    def test_get_topics_no_topic(self):
443
 
        """Searching for something that is not a plugin returns []."""
444
 
        # test this by using a name that cannot be a plugin - its not
445
 
        # a valid python identifier.
446
 
        index = plugin.PluginsHelpIndex()
447
 
        self.assertEqual([], index.get_topics('nothing by this name'))
448
 
 
449
 
    def test_prefix(self):
450
 
        """PluginsHelpIndex has a prefix of 'plugins/'."""
451
 
        index = plugin.PluginsHelpIndex()
452
 
        self.assertEqual('plugins/', index.prefix)
453
 
 
454
 
    def test_get_plugin_topic_with_prefix(self):
455
 
        """Searching for plugins/demo_module returns help."""
456
 
        index = plugin.PluginsHelpIndex()
457
 
        self.assertFalse(sys.modules.has_key('bzrlib.plugins.demo_module'))
458
 
        demo_module = FakeModule('', 'bzrlib.plugins.demo_module')
459
 
        sys.modules['bzrlib.plugins.demo_module'] = demo_module
460
 
        try:
461
 
            topics = index.get_topics('plugins/demo_module')
462
 
            self.assertEqual(1, len(topics))
463
 
            self.assertIsInstance(topics[0], plugin.ModuleHelpTopic)
464
 
            self.assertEqual(demo_module, topics[0].module)
465
 
        finally:
466
 
            del sys.modules['bzrlib.plugins.demo_module']
467
 
 
468
 
 
469
 
class FakeModule(object):
470
 
    """A fake module to test with."""
471
 
 
472
 
    def __init__(self, doc, name):
473
 
        self.__doc__ = doc
474
 
        self.__name__ = name
475
 
 
476
 
 
477
 
class TestModuleHelpTopic(tests.TestCase):
478
 
    """Tests for the ModuleHelpTopic class."""
479
 
 
480
 
    def test_contruct(self):
481
 
        """Construction takes the module to document."""
482
 
        mod = FakeModule('foo', 'foo')
483
 
        topic = plugin.ModuleHelpTopic(mod)
484
 
        self.assertEqual(mod, topic.module)
485
 
 
486
 
    def test_get_help_text_None(self):
487
 
        """A ModuleHelpTopic returns the docstring for get_help_text."""
488
 
        mod = FakeModule(None, 'demo')
489
 
        topic = plugin.ModuleHelpTopic(mod)
490
 
        self.assertEqual("Plugin 'demo' has no docstring.\n",
491
 
            topic.get_help_text())
492
 
 
493
 
    def test_get_help_text_no_carriage_return(self):
494
 
        """ModuleHelpTopic.get_help_text adds a \n if needed."""
495
 
        mod = FakeModule('one line of help', 'demo')
496
 
        topic = plugin.ModuleHelpTopic(mod)
497
 
        self.assertEqual("one line of help\n",
498
 
            topic.get_help_text())
499
 
 
500
 
    def test_get_help_text_carriage_return(self):
501
 
        """ModuleHelpTopic.get_help_text adds a \n if needed."""
502
 
        mod = FakeModule('two lines of help\nand more\n', 'demo')
503
 
        topic = plugin.ModuleHelpTopic(mod)
504
 
        self.assertEqual("two lines of help\nand more\n",
505
 
            topic.get_help_text())
506
 
 
507
 
    def test_get_help_text_with_additional_see_also(self):
508
 
        mod = FakeModule('two lines of help\nand more', 'demo')
509
 
        topic = plugin.ModuleHelpTopic(mod)
510
 
        self.assertEqual("two lines of help\nand more\nSee also: bar, foo\n",
511
 
            topic.get_help_text(['foo', 'bar']))
512
 
 
513
 
    def test_get_help_topic(self):
514
 
        """The help topic for a plugin is its module name."""
515
 
        mod = FakeModule('two lines of help\nand more', 'bzrlib.plugins.demo')
516
 
        topic = plugin.ModuleHelpTopic(mod)
517
 
        self.assertEqual('demo', topic.get_help_topic())
518
 
        mod = FakeModule('two lines of help\nand more', 'bzrlib.plugins.foo_bar')
519
 
        topic = plugin.ModuleHelpTopic(mod)
520
 
        self.assertEqual('foo_bar', topic.get_help_topic())