bzr branch
http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar
|
7015.1.3
by Martin
Tweak copyright headers |
1 |
# Copyright (C) 2005-2012, 2016 Canonical Ltd
|
2 |
# Copyright (C) 2017-2018 Breezy developers
|
|
|
1887.1.1
by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines, |
3 |
#
|
|
750
by Martin Pool
- stubbed-out tests for python plugins |
4 |
# This program is free software; you can redistribute it and/or modify
|
5 |
# it under the terms of the GNU General Public License as published by
|
|
6 |
# the Free Software Foundation; either version 2 of the License, or
|
|
7 |
# (at your option) any later version.
|
|
|
1887.1.1
by Adeodato Simó
Do not separate paragraphs in the copyright statement with blank lines, |
8 |
#
|
|
750
by Martin Pool
- stubbed-out tests for python plugins |
9 |
# This program is distributed in the hope that it will be useful,
|
10 |
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
11 |
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
12 |
# GNU General Public License for more details.
|
|
|
2052.3.1
by John Arbash Meinel
Add tests to cleanup the copyright of all source files |
13 |
#
|
|
750
by Martin Pool
- stubbed-out tests for python plugins |
14 |
# You should have received a copy of the GNU General Public License
|
15 |
# along with this program; if not, write to the Free Software
|
|
|
4183.7.1
by Sabin Iacob
update FSF mailing address |
16 |
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
|
|
750
by Martin Pool
- stubbed-out tests for python plugins |
17 |
|
18 |
"""Tests for plugins"""
|
|
19 |
||
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
20 |
import importlib |
|
2967.4.5
by Daniel Watkins
Added test for badly-named plugins. |
21 |
import logging |
|
1185.16.83
by mbp at sourcefrog
- notes on testability of plugins |
22 |
import os |
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
23 |
import sys |
|
7015.1.1
by Martin
Switch to compatible means of creating a module |
24 |
import types |
|
750
by Martin Pool
- stubbed-out tests for python plugins |
25 |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
26 |
import breezy |
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
27 |
from .. import ( |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
28 |
osutils, |
29 |
plugin, |
|
30 |
tests, |
|
31 |
)
|
|
|
7236.3.4
by Jelmer Vernooij
Add tests. |
32 |
from ..tests.features import pkg_resources_feature |
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
33 |
from ..sixish import ( |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
34 |
StringIO, |
35 |
viewkeys, |
|
|
6621.22.2
by Martin
Use BytesIO or StringIO from bzrlib.sixish |
36 |
)
|
|
1141
by Martin Pool
- rename FunctionalTest to TestCaseInTempDir |
37 |
|
|
1185.16.83
by mbp at sourcefrog
- notes on testability of plugins |
38 |
|
|
1492
by Robert Collins
Support decoration of commands. |
39 |
# TODO: Write a test for plugin decoration of commands.
|
|
1515
by Robert Collins
* Plugins with the same name in different directories in the bzr plugin |
40 |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
41 |
invalidate_caches = getattr(importlib, "invalidate_caches", lambda: None) |
42 |
||
43 |
||
|
5616.7.10
by Martin Pool
Clean up describe_plugins to sort loaded and unloaded plugins together. |
44 |
class BaseTestPlugins(tests.TestCaseInTempDir): |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
45 |
"""TestCase that isolates plugin imports and cleans up on completion.""" |
46 |
||
47 |
def setUp(self): |
|
48 |
super(BaseTestPlugins, self).setUp() |
|
49 |
self.module_name = "breezy.testingplugins" |
|
50 |
self.module_prefix = self.module_name + "." |
|
|
7015.1.1
by Martin
Switch to compatible means of creating a module |
51 |
self.module = types.ModuleType(self.module_name) |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
52 |
|
53 |
self.overrideAttr(plugin, "_MODULE_PREFIX", self.module_prefix) |
|
54 |
self.overrideAttr(breezy, "testingplugins", self.module) |
|
55 |
||
56 |
sys.modules[self.module_name] = self.module |
|
57 |
self.addCleanup(self._unregister_all) |
|
58 |
self.addCleanup(self._unregister_finder) |
|
59 |
||
60 |
invalidate_caches() |
|
61 |
||
62 |
def reset(self): |
|
63 |
"""Remove all global testing state and clean up module.""" |
|
64 |
# GZ 2017-06-02: Ideally don't do this, write new test or generate
|
|
65 |
# bytecode by other mechanism.
|
|
66 |
self.log("resetting plugin testing context") |
|
67 |
self._unregister_all() |
|
68 |
self._unregister_finder() |
|
69 |
sys.modules[self.module_name] = self.module |
|
70 |
for name in list(self.module.__dict__): |
|
71 |
if name[:2] != '__': |
|
72 |
delattr(self.module, name) |
|
73 |
invalidate_caches() |
|
74 |
self.plugins = None |
|
75 |
||
76 |
def update_module_paths(self, paths): |
|
77 |
paths = plugin.extend_path(paths, self.module_name) |
|
78 |
self.module.__path__ = paths |
|
79 |
self.log("using %r", paths) |
|
80 |
return paths |
|
81 |
||
|
7490.98.2
by Jelmer Vernooij
Don't hide problems from plugins by default. |
82 |
def load_with_paths(self, paths, warn_load_problems=True): |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
83 |
self.log("loading plugins!") |
|
7490.98.2
by Jelmer Vernooij
Don't hide problems from plugins by default. |
84 |
plugin.load_plugins( |
85 |
self.update_module_paths(paths), state=self, |
|
86 |
warn_load_problems=warn_load_problems) |
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
87 |
|
|
5086.5.8
by Vincent Ladeuil
Make sure we can load from a non-standard directory name. |
88 |
def create_plugin(self, name, source=None, dir='.', file_name=None): |
89 |
if source is None: |
|
90 |
source = '''\ |
|
91 |
"""This is the doc for %s""" |
|
92 |
''' % (name) |
|
|
5086.1.6
by Vincent Ladeuil
Crude fix for bug #411413. |
93 |
if file_name is None: |
94 |
file_name = name + '.py' |
|
95 |
# 'source' must not fail to load
|
|
|
5086.1.7
by Vincent Ladeuil
Cleaner fix for bug #411413. |
96 |
path = osutils.pathjoin(dir, file_name) |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
97 |
with open(path, 'w') as f: |
|
5086.1.6
by Vincent Ladeuil
Crude fix for bug #411413. |
98 |
f.write(source + '\n') |
99 |
||
|
5086.5.8
by Vincent Ladeuil
Make sure we can load from a non-standard directory name. |
100 |
def create_plugin_package(self, name, dir=None, source=None): |
101 |
if dir is None: |
|
102 |
dir = name |
|
103 |
if source is None: |
|
104 |
source = '''\ |
|
105 |
"""This is the doc for %s""" |
|
106 |
dir_source = '%s' |
|
107 |
''' % (name, dir) |
|
108 |
os.makedirs(dir) |
|
109 |
self.create_plugin(name, source, dir, |
|
|
5086.1.6
by Vincent Ladeuil
Crude fix for bug #411413. |
110 |
file_name='__init__.py') |
111 |
||
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
112 |
def promote_cache(self, directory): |
113 |
"""Move bytecode files out of __pycache__ in given directory.""" |
|
114 |
cache_dir = os.path.join(directory, '__pycache__') |
|
115 |
if os.path.isdir(cache_dir): |
|
116 |
for name in os.listdir(cache_dir): |
|
117 |
magicless_name = '.'.join(name.split('.')[0::name.count('.')]) |
|
118 |
rel = osutils.relpath(self.test_dir, cache_dir) |
|
119 |
self.log("moving %s in %s to %s", name, rel, magicless_name) |
|
120 |
os.rename(os.path.join(cache_dir, name), |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
121 |
os.path.join(directory, magicless_name)) |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
122 |
|
123 |
def _unregister_finder(self): |
|
124 |
"""Removes any test copies of _PluginsAtFinder from sys.meta_path.""" |
|
125 |
idx = len(sys.meta_path) |
|
126 |
while idx: |
|
127 |
idx -= 1 |
|
128 |
finder = sys.meta_path[idx] |
|
129 |
if getattr(finder, "prefix", "") == self.module_prefix: |
|
130 |
self.log("removed %r from sys.meta_path", finder) |
|
131 |
sys.meta_path.pop(idx) |
|
132 |
||
133 |
def _unregister_all(self): |
|
134 |
"""Remove all plugins in the test namespace from sys.modules.""" |
|
135 |
for name in list(sys.modules): |
|
136 |
if name.startswith(self.module_prefix) or name == self.module_name: |
|
137 |
self.log("removed %s from sys.modules", name) |
|
138 |
del sys.modules[name] |
|
139 |
||
140 |
def assertPluginModules(self, plugin_dict): |
|
141 |
self.assertEqual( |
|
142 |
dict((k[len(self.module_prefix):], sys.modules[k]) |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
143 |
for k in sys.modules if k.startswith(self.module_prefix)), |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
144 |
plugin_dict) |
|
5268.6.1
by Vincent Ladeuil
Drive-by fix of the submodule leak. |
145 |
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
146 |
def assertPluginUnknown(self, name): |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
147 |
self.assertTrue(getattr(self.module, name, None) is None) |
148 |
self.assertFalse(self.module_prefix + name in sys.modules) |
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
149 |
|
150 |
def assertPluginKnown(self, name): |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
151 |
self.assertTrue(getattr(self.module, name, None) is not None) |
152 |
self.assertTrue(self.module_prefix + name in sys.modules) |
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
153 |
|
154 |
||
|
5616.7.10
by Martin Pool
Clean up describe_plugins to sort loaded and unloaded plugins together. |
155 |
class TestLoadingPlugins(BaseTestPlugins): |
|
1515
by Robert Collins
* Plugins with the same name in different directories in the bzr plugin |
156 |
|
157 |
activeattributes = {} |
|
158 |
||
159 |
def test_plugins_with_the_same_name_are_not_loaded(self): |
|
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
160 |
# This test tests that having two plugins in different directories does
|
161 |
# not result in both being loaded when they have the same name. get a
|
|
162 |
# file name we can use which is also a valid attribute for accessing in
|
|
163 |
# activeattributes. - we cannot give import parameters.
|
|
|
1515
by Robert Collins
* Plugins with the same name in different directories in the bzr plugin |
164 |
tempattribute = "0" |
|
5784.1.1
by Martin Pool
Stop using failIf, failUnless, etc |
165 |
self.assertFalse(tempattribute in self.activeattributes) |
|
1515
by Robert Collins
* Plugins with the same name in different directories in the bzr plugin |
166 |
# set a place for the plugins to record their loading, and at the same
|
167 |
# time validate that the location the plugins should record to is
|
|
168 |
# valid and correct.
|
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
169 |
self.__class__.activeattributes[tempattribute] = [] |
|
5784.1.1
by Martin Pool
Stop using failIf, failUnless, etc |
170 |
self.assertTrue(tempattribute in self.activeattributes) |
|
1515
by Robert Collins
* Plugins with the same name in different directories in the bzr plugin |
171 |
# create two plugin directories
|
172 |
os.mkdir('first') |
|
173 |
os.mkdir('second') |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
174 |
# write a plugin that will record when its loaded in the
|
|
1515
by Robert Collins
* Plugins with the same name in different directories in the bzr plugin |
175 |
# tempattribute list.
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
176 |
template = ("from breezy.tests.test_plugins import TestLoadingPlugins\n" |
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
177 |
"TestLoadingPlugins.activeattributes[%r].append('%s')\n") |
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
178 |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
179 |
with open(os.path.join('first', 'plugin.py'), 'w') as outfile: |
|
2911.6.1
by Blake Winton
Change 'print >> f,'s to 'f.write('s. |
180 |
outfile.write(template % (tempattribute, 'first')) |
181 |
outfile.write('\n') |
|
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
182 |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
183 |
with open(os.path.join('second', 'plugin.py'), 'w') as outfile: |
|
2911.6.1
by Blake Winton
Change 'print >> f,'s to 'f.write('s. |
184 |
outfile.write(template % (tempattribute, 'second')) |
185 |
outfile.write('\n') |
|
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
186 |
|
|
1515
by Robert Collins
* Plugins with the same name in different directories in the bzr plugin |
187 |
try: |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
188 |
self.load_with_paths(['first', 'second']) |
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
189 |
self.assertEqual(['first'], self.activeattributes[tempattribute]) |
190 |
finally: |
|
191 |
del self.activeattributes[tempattribute] |
|
192 |
||
193 |
def test_plugins_from_different_dirs_can_demand_load(self): |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
194 |
self.assertFalse('breezy.plugins.pluginone' in sys.modules) |
195 |
self.assertFalse('breezy.plugins.plugintwo' in sys.modules) |
|
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
196 |
# This test tests that having two plugins in different
|
197 |
# directories with different names allows them both to be loaded, when
|
|
198 |
# we do a direct import statement.
|
|
199 |
# Determine a file name we can use which is also a valid attribute
|
|
200 |
# for accessing in activeattributes. - we cannot give import parameters.
|
|
201 |
tempattribute = "different-dirs" |
|
|
5784.1.1
by Martin Pool
Stop using failIf, failUnless, etc |
202 |
self.assertFalse(tempattribute in self.activeattributes) |
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
203 |
# set a place for the plugins to record their loading, and at the same
|
204 |
# time validate that the location the plugins should record to is
|
|
205 |
# valid and correct.
|
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
206 |
breezy.tests.test_plugins.TestLoadingPlugins.activeattributes[tempattribute] = [ |
207 |
]
|
|
|
5784.1.1
by Martin Pool
Stop using failIf, failUnless, etc |
208 |
self.assertTrue(tempattribute in self.activeattributes) |
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
209 |
# create two plugin directories
|
210 |
os.mkdir('first') |
|
211 |
os.mkdir('second') |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
212 |
# write plugins that will record when they are loaded in the
|
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
213 |
# tempattribute list.
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
214 |
template = ("from breezy.tests.test_plugins import TestLoadingPlugins\n" |
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
215 |
"TestLoadingPlugins.activeattributes[%r].append('%s')\n") |
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
216 |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
217 |
with open(os.path.join('first', 'pluginone.py'), 'w') as outfile: |
|
2911.6.1
by Blake Winton
Change 'print >> f,'s to 'f.write('s. |
218 |
outfile.write(template % (tempattribute, 'first')) |
219 |
outfile.write('\n') |
|
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
220 |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
221 |
with open(os.path.join('second', 'plugintwo.py'), 'w') as outfile: |
|
2911.6.1
by Blake Winton
Change 'print >> f,'s to 'f.write('s. |
222 |
outfile.write(template % (tempattribute, 'second')) |
223 |
outfile.write('\n') |
|
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
224 |
|
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
225 |
try: |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
226 |
self.assertPluginUnknown('pluginone') |
227 |
self.assertPluginUnknown('plugintwo') |
|
228 |
self.update_module_paths(['first', 'second']) |
|
229 |
exec("import %spluginone" % self.module_prefix) |
|
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
230 |
self.assertEqual(['first'], self.activeattributes[tempattribute]) |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
231 |
exec("import %splugintwo" % self.module_prefix) |
|
2256.2.2
by Robert Collins
Allow 'import bzrlib.plugins.NAME' to work when the plugin NAME has not |
232 |
self.assertEqual(['first', 'second'], |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
233 |
self.activeattributes[tempattribute]) |
|
1515
by Robert Collins
* Plugins with the same name in different directories in the bzr plugin |
234 |
finally: |
235 |
del self.activeattributes[tempattribute] |
|
|
1516
by Robert Collins
* bzrlib.plugin.all_plugins has been changed from an attribute to a |
236 |
|
|
2652.2.1
by Blake Winton
Add a test for BZR_PLUGIN_PATH, and code and another test to allow BZR_PLUGIN_PATH to contain trailing slashes. |
237 |
def test_plugins_can_load_from_directory_with_trailing_slash(self): |
238 |
# This test tests that a plugin can load from a directory when the
|
|
239 |
# directory in the path has a trailing slash.
|
|
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
240 |
# check the plugin is not loaded already
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
241 |
self.assertPluginUnknown('ts_plugin') |
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
242 |
tempattribute = "trailing-slash" |
|
5784.1.1
by Martin Pool
Stop using failIf, failUnless, etc |
243 |
self.assertFalse(tempattribute in self.activeattributes) |
|
2652.2.3
by Blake Winton
Understand the code and comments of the test, instead of just cargo-culting them. |
244 |
# set a place for the plugin to record its loading, and at the same
|
245 |
# time validate that the location the plugin should record to is
|
|
|
2652.2.1
by Blake Winton
Add a test for BZR_PLUGIN_PATH, and code and another test to allow BZR_PLUGIN_PATH to contain trailing slashes. |
246 |
# valid and correct.
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
247 |
breezy.tests.test_plugins.TestLoadingPlugins.activeattributes[tempattribute] = [ |
248 |
]
|
|
|
5784.1.1
by Martin Pool
Stop using failIf, failUnless, etc |
249 |
self.assertTrue(tempattribute in self.activeattributes) |
|
2652.2.3
by Blake Winton
Understand the code and comments of the test, instead of just cargo-culting them. |
250 |
# create a directory for the plugin
|
251 |
os.mkdir('plugin_test') |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
252 |
# write a plugin that will record when its loaded in the
|
|
2652.2.1
by Blake Winton
Add a test for BZR_PLUGIN_PATH, and code and another test to allow BZR_PLUGIN_PATH to contain trailing slashes. |
253 |
# tempattribute list.
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
254 |
template = ("from breezy.tests.test_plugins import TestLoadingPlugins\n" |
|
2652.2.1
by Blake Winton
Add a test for BZR_PLUGIN_PATH, and code and another test to allow BZR_PLUGIN_PATH to contain trailing slashes. |
255 |
"TestLoadingPlugins.activeattributes[%r].append('%s')\n") |
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
256 |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
257 |
with open(os.path.join('plugin_test', 'ts_plugin.py'), 'w') as outfile: |
|
2911.6.1
by Blake Winton
Change 'print >> f,'s to 'f.write('s. |
258 |
outfile.write(template % (tempattribute, 'plugin')) |
|
2911.6.4
by Blake Winton
Fix test failures |
259 |
outfile.write('\n') |
|
2652.2.7
by Blake Winton
fix lines which were wider than 79 chars. Also handle files a little more safely. |
260 |
|
|
2652.2.1
by Blake Winton
Add a test for BZR_PLUGIN_PATH, and code and another test to allow BZR_PLUGIN_PATH to contain trailing slashes. |
261 |
try: |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
262 |
self.load_with_paths(['plugin_test' + os.sep]) |
|
2652.2.3
by Blake Winton
Understand the code and comments of the test, instead of just cargo-culting them. |
263 |
self.assertEqual(['plugin'], self.activeattributes[tempattribute]) |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
264 |
self.assertPluginKnown('ts_plugin') |
|
2652.2.1
by Blake Winton
Add a test for BZR_PLUGIN_PATH, and code and another test to allow BZR_PLUGIN_PATH to contain trailing slashes. |
265 |
finally: |
266 |
del self.activeattributes[tempattribute] |
|
267 |
||
|
7490.98.2
by Jelmer Vernooij
Don't hide problems from plugins by default. |
268 |
def load_and_capture(self, name, warn_load_problems=True): |
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
269 |
"""Load plugins from '.' capturing the output. |
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
270 |
|
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
271 |
:param name: The name of the plugin.
|
272 |
:return: A string with the log from the plugin loading call.
|
|
273 |
"""
|
|
|
2967.4.5
by Daniel Watkins
Added test for badly-named plugins. |
274 |
# Capture output
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
275 |
stream = StringIO() |
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
276 |
try: |
277 |
handler = logging.StreamHandler(stream) |
|
|
6622.1.33
by Jelmer Vernooij
Fix more tests (all?) |
278 |
log = logging.getLogger('brz') |
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
279 |
log.addHandler(handler) |
280 |
try: |
|
|
7490.98.2
by Jelmer Vernooij
Don't hide problems from plugins by default. |
281 |
self.load_with_paths( |
282 |
['.'], warn_load_problems=warn_load_problems) |
|
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
283 |
finally: |
284 |
# Stop capturing output
|
|
285 |
handler.flush() |
|
286 |
handler.close() |
|
287 |
log.removeHandler(handler) |
|
288 |
return stream.getvalue() |
|
289 |
finally: |
|
290 |
stream.close() |
|
|
3943.8.1
by Marius Kruger
remove all trailing whitespace from bzr source |
291 |
|
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
292 |
def test_plugin_with_bad_api_version_reports(self): |
|
5616.7.3
by Martin Pool
Put plugin warnings into both the apport and plain crash report |
293 |
"""Try loading a plugin that requests an unsupported api. |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
294 |
|
|
7490.98.2
by Jelmer Vernooij
Don't hide problems from plugins by default. |
295 |
Observe that it records the problem but doesn't complain on stderr
|
296 |
when warn_load_problems=False
|
|
|
5616.7.3
by Martin Pool
Put plugin warnings into both the apport and plain crash report |
297 |
"""
|
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
298 |
name = 'wants100.py' |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
299 |
with open(name, 'w') as f: |
|
6672.1.2
by Jelmer Vernooij
Remove breezy.api. |
300 |
f.write("import breezy\n" |
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
301 |
"from breezy.errors import IncompatibleVersion\n" |
302 |
"raise IncompatibleVersion(breezy, [(1, 0, 0)], (0, 0, 5))\n") |
|
|
7490.98.2
by Jelmer Vernooij
Don't hide problems from plugins by default. |
303 |
log = self.load_and_capture(name, warn_load_problems=False) |
304 |
self.assertNotContainsRe(log, r"It supports breezy version") |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
305 |
self.assertEqual({'wants100'}, viewkeys(self.plugin_warnings)) |
|
5616.7.1
by Martin Pool
Record but don't show warnings about updated plugins |
306 |
self.assertContainsRe( |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
307 |
self.plugin_warnings['wants100'][0], |
|
6672.1.2
by Jelmer Vernooij
Remove breezy.api. |
308 |
r"It supports breezy version") |
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
309 |
|
310 |
def test_plugin_with_bad_name_does_not_load(self): |
|
311 |
# The file name here invalid for a python module.
|
|
|
6622.1.33
by Jelmer Vernooij
Fix more tests (all?) |
312 |
name = 'brz-bad plugin-name..py' |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
313 |
open(name, 'w').close() |
|
3766.3.2
by Robert Collins
Fix reporting of incompatible api plugin load errors, fixing bug 279451. |
314 |
log = self.load_and_capture(name) |
315 |
self.assertContainsRe(log, |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
316 |
r"Unable to load 'brz-bad plugin-name\.' in '\.' as a plugin " |
317 |
"because the file path isn't a valid module name; try renaming "
|
|
318 |
"it to 'bad_plugin_name_'\\.") |
|
|
2967.4.5
by Daniel Watkins
Added test for badly-named plugins. |
319 |
|
|
7490.98.2
by Jelmer Vernooij
Don't hide problems from plugins by default. |
320 |
def test_plugin_with_error_suppress(self): |
321 |
# The file name here invalid for a python module.
|
|
322 |
name = 'some_error.py' |
|
323 |
with open(name, 'w') as f: |
|
324 |
f.write('raise Exception("bad")\n') |
|
325 |
log = self.load_and_capture(name, warn_load_problems=False) |
|
326 |
self.assertEqual('', log) |
|
327 |
||
|
7490.98.1
by Jelmer Vernooij
Add test for plugin that fails to load. |
328 |
def test_plugin_with_error(self): |
329 |
# The file name here invalid for a python module.
|
|
330 |
name = 'some_error.py' |
|
331 |
with open(name, 'w') as f: |
|
332 |
f.write('raise Exception("bad")\n') |
|
|
7490.98.2
by Jelmer Vernooij
Don't hide problems from plugins by default. |
333 |
log = self.load_and_capture(name, warn_load_problems=True) |
334 |
self.assertEqual( |
|
335 |
'Unable to load plugin \'some_error\' from \'.\': bad\n', log) |
|
|
7490.98.1
by Jelmer Vernooij
Add test for plugin that fails to load. |
336 |
|
|
1516
by Robert Collins
* bzrlib.plugin.all_plugins has been changed from an attribute to a |
337 |
|
|
5616.7.10
by Martin Pool
Clean up describe_plugins to sort loaded and unloaded plugins together. |
338 |
class TestPlugins(BaseTestPlugins): |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
339 |
|
340 |
def setup_plugin(self, source=""): |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
341 |
# This test tests a new plugin appears in breezy.plugin.plugins().
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
342 |
# check the plugin is not loaded already
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
343 |
self.assertPluginUnknown('plugin') |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
344 |
# write a plugin that _cannot_ fail to load.
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
345 |
with open('plugin.py', 'w') as f: |
346 |
f.write(source + '\n') |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
347 |
self.load_with_paths(['.']) |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
348 |
|
|
6759.4.3
by Jelmer Vernooij
Avoid accessing global state. |
349 |
def test_plugin_loaded(self): |
350 |
self.assertPluginUnknown('plugin') |
|
351 |
self.assertIs(None, breezy.plugin.get_loaded_plugin('plugin')) |
|
352 |
self.setup_plugin() |
|
353 |
p = breezy.plugin.get_loaded_plugin('plugin') |
|
354 |
self.assertIsInstance(p, breezy.plugin.PlugIn) |
|
355 |
self.assertIs(p.module, sys.modules[self.module_prefix + 'plugin']) |
|
356 |
||
|
6780.1.1
by Jelmer Vernooij
Check for plugin existing in sys.modules but being None. |
357 |
def test_plugin_loaded_disabled(self): |
358 |
self.assertPluginUnknown('plugin') |
|
359 |
self.overrideEnv('BRZ_DISABLE_PLUGINS', 'plugin') |
|
360 |
self.setup_plugin() |
|
361 |
self.assertIs(None, breezy.plugin.get_loaded_plugin('plugin')) |
|
362 |
||
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
363 |
def test_plugin_appears_in_plugins(self): |
364 |
self.setup_plugin() |
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
365 |
self.assertPluginKnown('plugin') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
366 |
p = self.plugins['plugin'] |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
367 |
self.assertIsInstance(p, breezy.plugin.PlugIn) |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
368 |
self.assertIs(p.module, sys.modules[self.module_prefix + 'plugin']) |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
369 |
|
370 |
def test_trivial_plugin_get_path(self): |
|
371 |
self.setup_plugin() |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
372 |
p = self.plugins['plugin'] |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
373 |
plugin_path = self.test_dir + '/plugin.py' |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
374 |
self.assertIsSameRealPath(plugin_path, osutils.normpath(p.path())) |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
375 |
|
|
3193.2.1
by Alexander Belchenko
show path to plugin module as *.py instead of *.pyc if python source available |
376 |
def test_plugin_get_path_py_not_pyc(self): |
|
5086.1.3
by Vincent Ladeuil
Fix imports in test_plugins. |
377 |
# first import creates plugin.pyc
|
378 |
self.setup_plugin() |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
379 |
self.promote_cache(self.test_dir) |
380 |
self.reset() |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
381 |
self.load_with_paths(['.']) # import plugin.pyc |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
382 |
p = plugin.plugins()['plugin'] |
|
3193.2.1
by Alexander Belchenko
show path to plugin module as *.py instead of *.pyc if python source available |
383 |
plugin_path = self.test_dir + '/plugin.py' |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
384 |
self.assertIsSameRealPath(plugin_path, osutils.normpath(p.path())) |
|
3193.2.1
by Alexander Belchenko
show path to plugin module as *.py instead of *.pyc if python source available |
385 |
|
386 |
def test_plugin_get_path_pyc_only(self): |
|
|
5086.1.3
by Vincent Ladeuil
Fix imports in test_plugins. |
387 |
# first import creates plugin.pyc (or plugin.pyo depending on __debug__)
|
388 |
self.setup_plugin() |
|
|
3193.2.1
by Alexander Belchenko
show path to plugin module as *.py instead of *.pyc if python source available |
389 |
os.unlink(self.test_dir + '/plugin.py') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
390 |
self.promote_cache(self.test_dir) |
391 |
self.reset() |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
392 |
self.load_with_paths(['.']) # import plugin.pyc (or .pyo) |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
393 |
p = plugin.plugins()['plugin'] |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
394 |
plugin_path = self.test_dir + '/plugin' + plugin.COMPILED_EXT |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
395 |
self.assertIsSameRealPath(plugin_path, osutils.normpath(p.path())) |
|
3193.2.1
by Alexander Belchenko
show path to plugin module as *.py instead of *.pyc if python source available |
396 |
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
397 |
def test_no_test_suite_gives_None_for_test_suite(self): |
398 |
self.setup_plugin() |
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
399 |
p = plugin.plugins()['plugin'] |
400 |
self.assertEqual(None, p.test_suite()) |
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
401 |
|
402 |
def test_test_suite_gives_test_suite_result(self): |
|
403 |
source = """def test_suite(): return 'foo'""" |
|
404 |
self.setup_plugin(source) |
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
405 |
p = plugin.plugins()['plugin'] |
406 |
self.assertEqual('foo', p.test_suite()) |
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
407 |
|
|
3302.8.21
by Vincent Ladeuil
Fixed as per Robert's review. |
408 |
def test_no_load_plugin_tests_gives_None_for_load_plugin_tests(self): |
|
3302.8.10
by Vincent Ladeuil
Prepare bzrlib.plugin to use the new test loader. |
409 |
self.setup_plugin() |
|
5086.1.3
by Vincent Ladeuil
Fix imports in test_plugins. |
410 |
loader = tests.TestUtil.TestLoader() |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
411 |
p = plugin.plugins()['plugin'] |
412 |
self.assertEqual(None, p.load_plugin_tests(loader)) |
|
|
3302.8.10
by Vincent Ladeuil
Prepare bzrlib.plugin to use the new test loader. |
413 |
|
|
3302.8.21
by Vincent Ladeuil
Fixed as per Robert's review. |
414 |
def test_load_plugin_tests_gives_load_plugin_tests_result(self): |
|
3302.8.10
by Vincent Ladeuil
Prepare bzrlib.plugin to use the new test loader. |
415 |
source = """ |
|
6625.1.5
by Martin
Drop custom load_tests implementation and use unittest signature |
416 |
def load_tests(loader, standard_tests, pattern):
|
|
3302.8.10
by Vincent Ladeuil
Prepare bzrlib.plugin to use the new test loader. |
417 |
return 'foo'"""
|
418 |
self.setup_plugin(source) |
|
|
5086.1.3
by Vincent Ladeuil
Fix imports in test_plugins. |
419 |
loader = tests.TestUtil.TestLoader() |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
420 |
p = plugin.plugins()['plugin'] |
421 |
self.assertEqual('foo', p.load_plugin_tests(loader)) |
|
422 |
||
423 |
def check_version_info(self, expected, source='', name='plugin'): |
|
424 |
self.setup_plugin(source) |
|
425 |
self.assertEqual(expected, plugin.plugins()[name].version_info()) |
|
|
3302.8.10
by Vincent Ladeuil
Prepare bzrlib.plugin to use the new test loader. |
426 |
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
427 |
def test_no_version_info(self): |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
428 |
self.check_version_info(None) |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
429 |
|
430 |
def test_with_version_info(self): |
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
431 |
self.check_version_info((1, 2, 3, 'dev', 4), |
432 |
"version_info = (1, 2, 3, 'dev', 4)") |
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
433 |
|
434 |
def test_short_version_info_gets_padded(self): |
|
435 |
# the gtk plugin has version_info = (1,2,3) rather than the 5-tuple.
|
|
436 |
# so we adapt it
|
|
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
437 |
self.check_version_info((1, 2, 3, 'final', 0), |
438 |
"version_info = (1, 2, 3)") |
|
439 |
||
440 |
def check_version(self, expected, source=None, name='plugin'): |
|
441 |
self.setup_plugin(source) |
|
442 |
self.assertEqual(expected, plugins[name].__version__) |
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
443 |
|
444 |
def test_no_version_info___version__(self): |
|
445 |
self.setup_plugin() |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
446 |
plugin = breezy.plugin.plugins()['plugin'] |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
447 |
self.assertEqual("unknown", plugin.__version__) |
448 |
||
|
3777.6.7
by Marius Kruger
* Can now also handle non-iteratable and string plugin versions. |
449 |
def test_str__version__with_version_info(self): |
450 |
self.setup_plugin("version_info = '1.2.3'") |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
451 |
plugin = breezy.plugin.plugins()['plugin'] |
|
3777.6.7
by Marius Kruger
* Can now also handle non-iteratable and string plugin versions. |
452 |
self.assertEqual("1.2.3", plugin.__version__) |
453 |
||
454 |
def test_noniterable__version__with_version_info(self): |
|
455 |
self.setup_plugin("version_info = (1)") |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
456 |
plugin = breezy.plugin.plugins()['plugin'] |
|
3777.6.7
by Marius Kruger
* Can now also handle non-iteratable and string plugin versions. |
457 |
self.assertEqual("1", plugin.__version__) |
458 |
||
459 |
def test_1__version__with_version_info(self): |
|
460 |
self.setup_plugin("version_info = (1,)") |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
461 |
plugin = breezy.plugin.plugins()['plugin'] |
|
3777.6.7
by Marius Kruger
* Can now also handle non-iteratable and string plugin versions. |
462 |
self.assertEqual("1", plugin.__version__) |
463 |
||
464 |
def test_1_2__version__with_version_info(self): |
|
|
3777.6.5
by Marius Kruger
add 2 more tests for plugin version numbers |
465 |
self.setup_plugin("version_info = (1, 2)") |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
466 |
plugin = breezy.plugin.plugins()['plugin'] |
|
3777.6.5
by Marius Kruger
add 2 more tests for plugin version numbers |
467 |
self.assertEqual("1.2", plugin.__version__) |
468 |
||
|
3777.6.7
by Marius Kruger
* Can now also handle non-iteratable and string plugin versions. |
469 |
def test_1_2_3__version__with_version_info(self): |
|
3777.6.5
by Marius Kruger
add 2 more tests for plugin version numbers |
470 |
self.setup_plugin("version_info = (1, 2, 3)") |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
471 |
plugin = breezy.plugin.plugins()['plugin'] |
|
3777.6.5
by Marius Kruger
add 2 more tests for plugin version numbers |
472 |
self.assertEqual("1.2.3", plugin.__version__) |
473 |
||
474 |
def test_candidate__version__with_version_info(self): |
|
|
3777.6.4
by Marius Kruger
fix tests |
475 |
self.setup_plugin("version_info = (1, 2, 3, 'candidate', 1)") |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
476 |
plugin = breezy.plugin.plugins()['plugin'] |
|
3777.6.4
by Marius Kruger
fix tests |
477 |
self.assertEqual("1.2.3rc1", plugin.__version__) |
478 |
||
479 |
def test_dev__version__with_version_info(self): |
|
480 |
self.setup_plugin("version_info = (1, 2, 3, 'dev', 0)") |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
481 |
plugin = breezy.plugin.plugins()['plugin'] |
|
3777.6.4
by Marius Kruger
fix tests |
482 |
self.assertEqual("1.2.3dev", plugin.__version__) |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
483 |
|
|
3777.6.7
by Marius Kruger
* Can now also handle non-iteratable and string plugin versions. |
484 |
def test_dev_fallback__version__with_version_info(self): |
485 |
self.setup_plugin("version_info = (1, 2, 3, 'dev', 4)") |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
486 |
plugin = breezy.plugin.plugins()['plugin'] |
|
4634.50.6
by John Arbash Meinel
Handle a plugin fallback versioning issue. |
487 |
self.assertEqual("1.2.3dev4", plugin.__version__) |
|
3777.6.7
by Marius Kruger
* Can now also handle non-iteratable and string plugin versions. |
488 |
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
489 |
def test_final__version__with_version_info(self): |
|
3777.6.4
by Marius Kruger
fix tests |
490 |
self.setup_plugin("version_info = (1, 2, 3, 'final', 0)") |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
491 |
plugin = breezy.plugin.plugins()['plugin'] |
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
492 |
self.assertEqual("1.2.3", plugin.__version__) |
493 |
||
|
4634.50.6
by John Arbash Meinel
Handle a plugin fallback versioning issue. |
494 |
def test_final_fallback__version__with_version_info(self): |
495 |
self.setup_plugin("version_info = (1, 2, 3, 'final', 2)") |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
496 |
plugin = breezy.plugin.plugins()['plugin'] |
|
5851.2.2
by Martin Pool
Format plugin version as 1.2.3.2 not 1.2.3.final.2 |
497 |
self.assertEqual("1.2.3.2", plugin.__version__) |
|
4634.50.6
by John Arbash Meinel
Handle a plugin fallback versioning issue. |
498 |
|
|
2762.2.1
by Robert Collins
* ``bzr plugins`` now lists the version number for each plugin in square |
499 |
|
|
2432.1.25
by Robert Collins
Return plugin module docstrings for 'bzr help plugin'. |
500 |
class TestHelpIndex(tests.TestCase): |
501 |
"""Tests for the PluginsHelpIndex class.""" |
|
502 |
||
503 |
def test_default_constructable(self): |
|
504 |
index = plugin.PluginsHelpIndex() |
|
505 |
||
506 |
def test_get_topics_None(self): |
|
507 |
"""Searching for None returns an empty list.""" |
|
508 |
index = plugin.PluginsHelpIndex() |
|
509 |
self.assertEqual([], index.get_topics(None)) |
|
510 |
||
|
2475.1.1
by Martin Pool
Rename test_plugin tests and the example module used there. |
511 |
def test_get_topics_for_plugin(self): |
512 |
"""Searching for plugin name gets its docstring.""" |
|
|
2432.1.25
by Robert Collins
Return plugin module docstrings for 'bzr help plugin'. |
513 |
index = plugin.PluginsHelpIndex() |
|
2475.1.1
by Martin Pool
Rename test_plugin tests and the example module used there. |
514 |
# make a new plugin here for this test, even if we're run with
|
515 |
# --no-plugins
|
|
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
516 |
self.assertFalse('breezy.plugins.demo_module' in sys.modules) |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
517 |
demo_module = FakeModule('', 'breezy.plugins.demo_module') |
518 |
sys.modules['breezy.plugins.demo_module'] = demo_module |
|
|
2457.1.1
by Robert Collins
(robertc) Fix bzr --no-plugins selftest which was broken by the help indices patch. (Robert Collins, Martin Pool) |
519 |
try: |
|
2475.1.1
by Martin Pool
Rename test_plugin tests and the example module used there. |
520 |
topics = index.get_topics('demo_module') |
|
2457.1.1
by Robert Collins
(robertc) Fix bzr --no-plugins selftest which was broken by the help indices patch. (Robert Collins, Martin Pool) |
521 |
self.assertEqual(1, len(topics)) |
522 |
self.assertIsInstance(topics[0], plugin.ModuleHelpTopic) |
|
523 |
self.assertEqual(demo_module, topics[0].module) |
|
524 |
finally: |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
525 |
del sys.modules['breezy.plugins.demo_module'] |
|
2432.1.25
by Robert Collins
Return plugin module docstrings for 'bzr help plugin'. |
526 |
|
527 |
def test_get_topics_no_topic(self): |
|
528 |
"""Searching for something that is not a plugin returns [].""" |
|
529 |
# test this by using a name that cannot be a plugin - its not
|
|
530 |
# a valid python identifier.
|
|
531 |
index = plugin.PluginsHelpIndex() |
|
532 |
self.assertEqual([], index.get_topics('nothing by this name')) |
|
533 |
||
534 |
def test_prefix(self): |
|
535 |
"""PluginsHelpIndex has a prefix of 'plugins/'.""" |
|
536 |
index = plugin.PluginsHelpIndex() |
|
537 |
self.assertEqual('plugins/', index.prefix) |
|
538 |
||
|
2475.1.1
by Martin Pool
Rename test_plugin tests and the example module used there. |
539 |
def test_get_plugin_topic_with_prefix(self): |
540 |
"""Searching for plugins/demo_module returns help.""" |
|
|
2432.1.25
by Robert Collins
Return plugin module docstrings for 'bzr help plugin'. |
541 |
index = plugin.PluginsHelpIndex() |
|
6624
by Jelmer Vernooij
Merge Python3 porting work ('py3 pokes') |
542 |
self.assertFalse('breezy.plugins.demo_module' in sys.modules) |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
543 |
demo_module = FakeModule('', 'breezy.plugins.demo_module') |
544 |
sys.modules['breezy.plugins.demo_module'] = demo_module |
|
|
2457.1.1
by Robert Collins
(robertc) Fix bzr --no-plugins selftest which was broken by the help indices patch. (Robert Collins, Martin Pool) |
545 |
try: |
|
2475.1.1
by Martin Pool
Rename test_plugin tests and the example module used there. |
546 |
topics = index.get_topics('plugins/demo_module') |
|
2457.1.1
by Robert Collins
(robertc) Fix bzr --no-plugins selftest which was broken by the help indices patch. (Robert Collins, Martin Pool) |
547 |
self.assertEqual(1, len(topics)) |
548 |
self.assertIsInstance(topics[0], plugin.ModuleHelpTopic) |
|
549 |
self.assertEqual(demo_module, topics[0].module) |
|
550 |
finally: |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
551 |
del sys.modules['breezy.plugins.demo_module'] |
|
2432.1.25
by Robert Collins
Return plugin module docstrings for 'bzr help plugin'. |
552 |
|
553 |
||
554 |
class FakeModule(object): |
|
555 |
"""A fake module to test with.""" |
|
556 |
||
557 |
def __init__(self, doc, name): |
|
558 |
self.__doc__ = doc |
|
559 |
self.__name__ = name |
|
560 |
||
561 |
||
562 |
class TestModuleHelpTopic(tests.TestCase): |
|
563 |
"""Tests for the ModuleHelpTopic class.""" |
|
564 |
||
565 |
def test_contruct(self): |
|
566 |
"""Construction takes the module to document.""" |
|
567 |
mod = FakeModule('foo', 'foo') |
|
568 |
topic = plugin.ModuleHelpTopic(mod) |
|
569 |
self.assertEqual(mod, topic.module) |
|
570 |
||
571 |
def test_get_help_text_None(self): |
|
572 |
"""A ModuleHelpTopic returns the docstring for get_help_text.""" |
|
573 |
mod = FakeModule(None, 'demo') |
|
574 |
topic = plugin.ModuleHelpTopic(mod) |
|
575 |
self.assertEqual("Plugin 'demo' has no docstring.\n", |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
576 |
topic.get_help_text()) |
|
2432.1.25
by Robert Collins
Return plugin module docstrings for 'bzr help plugin'. |
577 |
|
578 |
def test_get_help_text_no_carriage_return(self): |
|
579 |
"""ModuleHelpTopic.get_help_text adds a \n if needed.""" |
|
580 |
mod = FakeModule('one line of help', 'demo') |
|
581 |
topic = plugin.ModuleHelpTopic(mod) |
|
582 |
self.assertEqual("one line of help\n", |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
583 |
topic.get_help_text()) |
|
2432.1.25
by Robert Collins
Return plugin module docstrings for 'bzr help plugin'. |
584 |
|
585 |
def test_get_help_text_carriage_return(self): |
|
586 |
"""ModuleHelpTopic.get_help_text adds a \n if needed.""" |
|
587 |
mod = FakeModule('two lines of help\nand more\n', 'demo') |
|
588 |
topic = plugin.ModuleHelpTopic(mod) |
|
589 |
self.assertEqual("two lines of help\nand more\n", |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
590 |
topic.get_help_text()) |
|
2432.1.25
by Robert Collins
Return plugin module docstrings for 'bzr help plugin'. |
591 |
|
592 |
def test_get_help_text_with_additional_see_also(self): |
|
593 |
mod = FakeModule('two lines of help\nand more', 'demo') |
|
594 |
topic = plugin.ModuleHelpTopic(mod) |
|
|
7143.15.5
by Jelmer Vernooij
More PEP8 fixes. |
595 |
self.assertEqual( |
596 |
"two lines of help\nand more\n\n:See also: bar, foo\n", |
|
597 |
topic.get_help_text(['foo', 'bar'])) |
|
|
2432.1.29
by Robert Collins
Add get_help_topic to ModuleHelpTopic. |
598 |
|
599 |
def test_get_help_topic(self): |
|
600 |
"""The help topic for a plugin is its module name.""" |
|
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
601 |
mod = FakeModule('two lines of help\nand more', 'breezy.plugins.demo') |
|
2432.1.29
by Robert Collins
Add get_help_topic to ModuleHelpTopic. |
602 |
topic = plugin.ModuleHelpTopic(mod) |
603 |
self.assertEqual('demo', topic.get_help_topic()) |
|
|
6059.3.6
by Vincent Ladeuil
Fix tests failing on pqm. |
604 |
mod = FakeModule('two lines of help\nand more', |
|
6622.1.34
by Jelmer Vernooij
Rename brzlib => breezy. |
605 |
'breezy.plugins.foo_bar') |
|
2432.1.29
by Robert Collins
Add get_help_topic to ModuleHelpTopic. |
606 |
topic = plugin.ModuleHelpTopic(mod) |
607 |
self.assertEqual('foo_bar', topic.get_help_topic()) |
|
|
3835.2.7
by Aaron Bentley
Add tests for plugins |
608 |
|
609 |
||
|
5086.1.2
by Vincent Ladeuil
Cosmetic changes. |
610 |
class TestEnvPluginPath(tests.TestCase): |
|
4628.2.1
by Vincent Ladeuil
Start introducing accessors for plugin paths. |
611 |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
612 |
user = "USER" |
613 |
core = "CORE" |
|
614 |
site = "SITE" |
|
|
7236.3.2
by Jelmer Vernooij
Fix tests. |
615 |
entrypoints = "ENTRYPOINTS" |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
616 |
|
617 |
def check_path(self, expected_dirs, setting_dirs): |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
618 |
if setting_dirs is None: |
619 |
del os.environ['BRZ_PLUGIN_PATH'] |
|
620 |
else: |
|
621 |
os.environ['BRZ_PLUGIN_PATH'] = os.pathsep.join(setting_dirs) |
|
622 |
actual = [(p if t == 'path' else t.upper()) |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
623 |
for p, t in plugin._env_plugin_path()] |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
624 |
self.assertEqual(expected_dirs, actual) |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
625 |
|
|
4628.2.1
by Vincent Ladeuil
Start introducing accessors for plugin paths. |
626 |
def test_default(self): |
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
627 |
self.check_path([self.user, self.core, self.site], None) |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
628 |
|
629 |
def test_adhoc_policy(self): |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
630 |
self.check_path([self.user, self.core, self.site], |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
631 |
['+user', '+core', '+site']) |
632 |
||
633 |
def test_fallback_policy(self): |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
634 |
self.check_path([self.core, self.site, self.user], |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
635 |
['+core', '+site', '+user']) |
636 |
||
637 |
def test_override_policy(self): |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
638 |
self.check_path([self.user, self.site, self.core], |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
639 |
['+user', '+site', '+core']) |
640 |
||
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
641 |
def test_enable_entrypoints(self): |
642 |
self.check_path([self.user, self.core, self.site, self.entrypoints], |
|
643 |
['+user', '+core', '+site', '+entrypoints']) |
|
644 |
||
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
645 |
def test_disable_user(self): |
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
646 |
self.check_path([self.core, self.site], ['-user']) |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
647 |
|
648 |
def test_disable_user_twice(self): |
|
649 |
# Ensures multiple removals don't left cruft
|
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
650 |
self.check_path([self.core, self.site], ['-user', '-user']) |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
651 |
|
|
4628.2.5
by Vincent Ladeuil
Fixes prompted by review. |
652 |
def test_duplicates_are_removed(self): |
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
653 |
self.check_path([self.user, self.core, self.site], |
|
4628.2.5
by Vincent Ladeuil
Fixes prompted by review. |
654 |
['+user', '+user']) |
655 |
# And only the first reference is kept (since the later references will
|
|
|
5086.1.2
by Vincent Ladeuil
Cosmetic changes. |
656 |
# only produce '<plugin> already loaded' mutters)
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
657 |
self.check_path([self.user, self.core, self.site], |
|
4628.2.5
by Vincent Ladeuil
Fixes prompted by review. |
658 |
['+user', '+user', '+core', |
659 |
'+user', '+site', '+site', |
|
660 |
'+core']) |
|
661 |
||
|
5086.1.5
by Vincent Ladeuil
Fix typo in test name. |
662 |
def test_disable_overrides_enable(self): |
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
663 |
self.check_path([self.core, self.site], ['-user', '+user']) |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
664 |
|
665 |
def test_disable_core(self): |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
666 |
self.check_path([self.site], ['-core']) |
667 |
self.check_path([self.user, self.site], ['+user', '-core']) |
|
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
668 |
|
669 |
def test_disable_site(self): |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
670 |
self.check_path([self.core], ['-site']) |
671 |
self.check_path([self.user, self.core], ['-site', '+user']) |
|
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
672 |
|
673 |
def test_override_site(self): |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
674 |
self.check_path(['mysite', self.user, self.core], |
|
4628.2.3
by Vincent Ladeuil
Update doc and add NEWS entry. |
675 |
['mysite', '-site', '+user']) |
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
676 |
self.check_path(['mysite', self.core], |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
677 |
['mysite', '-site']) |
678 |
||
679 |
def test_override_core(self): |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
680 |
self.check_path(['mycore', self.user, self.site], |
|
4628.2.3
by Vincent Ladeuil
Update doc and add NEWS entry. |
681 |
['mycore', '-core', '+user', '+site']) |
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
682 |
self.check_path(['mycore', self.site], |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
683 |
['mycore', '-core']) |
684 |
||
685 |
def test_my_plugin_only(self): |
|
|
7236.3.2
by Jelmer Vernooij
Fix tests. |
686 |
self.check_path( |
687 |
['myplugin'], |
|
688 |
['myplugin', '-user', '-core', '-site', '-entrypoints']) |
|
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
689 |
|
690 |
def test_my_plugin_first(self): |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
691 |
self.check_path(['myplugin', self.core, self.site, self.user], |
|
4628.2.2
by Vincent Ladeuil
Add [+-]{user|core|site} handling in BZR_PLUGIN_PATH. |
692 |
['myplugin', '+core', '+site', '+user']) |
|
4628.2.1
by Vincent Ladeuil
Start introducing accessors for plugin paths. |
693 |
|
|
4628.2.5
by Vincent Ladeuil
Fixes prompted by review. |
694 |
def test_bogus_references(self): |
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
695 |
self.check_path(['+foo', '-bar', self.core, self.site], |
|
4628.2.5
by Vincent Ladeuil
Fixes prompted by review. |
696 |
['+foo', '-bar']) |
|
5086.1.4
by Vincent Ladeuil
Slight plugin tests rewriting. |
697 |
|
|
5086.1.6
by Vincent Ladeuil
Crude fix for bug #411413. |
698 |
|
|
5616.7.10
by Martin Pool
Clean up describe_plugins to sort loaded and unloaded plugins together. |
699 |
class TestDisablePlugin(BaseTestPlugins): |
|
5086.1.6
by Vincent Ladeuil
Crude fix for bug #411413. |
700 |
|
|
5086.1.8
by Vincent Ladeuil
Fix warnings during autoload, add doc and a NEWS entry. |
701 |
def test_cannot_import(self): |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
702 |
self.create_plugin_package('works') |
703 |
self.create_plugin_package('fails') |
|
704 |
self.overrideEnv('BRZ_DISABLE_PLUGINS', 'fails') |
|
705 |
self.update_module_paths(["."]) |
|
706 |
import breezy.testingplugins.works as works |
|
|
5086.1.6
by Vincent Ladeuil
Crude fix for bug #411413. |
707 |
try: |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
708 |
import breezy.testingplugins.fails as fails |
|
5086.1.6
by Vincent Ladeuil
Crude fix for bug #411413. |
709 |
except ImportError: |
710 |
pass
|
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
711 |
else: |
712 |
self.fail("Loaded blocked plugin: " + repr(fails)) |
|
713 |
self.assertPluginModules({'fails': None, 'works': works}) |
|
714 |
||
715 |
def test_partial_imports(self): |
|
716 |
self.create_plugin('good') |
|
717 |
self.create_plugin('bad') |
|
718 |
self.create_plugin_package('ugly') |
|
719 |
self.overrideEnv('BRZ_DISABLE_PLUGINS', 'bad:ugly') |
|
720 |
self.load_with_paths(['.']) |
|
721 |
self.assertEqual({'good'}, viewkeys(self.plugins)) |
|
722 |
self.assertPluginModules({ |
|
723 |
'good': self.plugins['good'].module, |
|
724 |
'bad': None, |
|
725 |
'ugly': None, |
|
726 |
})
|
|
727 |
# Ensure there are no warnings about plugins not being imported as
|
|
728 |
# the user has explictly requested they be disabled.
|
|
729 |
self.assertNotContainsRe(self.get_log(), r"Unable to load plugin") |
|
730 |
||
731 |
||
732 |
class TestEnvDisablePlugins(tests.TestCase): |
|
733 |
||
734 |
def _get_names(self, env_value): |
|
735 |
os.environ['BRZ_DISABLE_PLUGINS'] = env_value |
|
736 |
return plugin._env_disable_plugins() |
|
737 |
||
738 |
def test_unset(self): |
|
739 |
self.assertEqual([], plugin._env_disable_plugins()) |
|
740 |
||
741 |
def test_empty(self): |
|
742 |
self.assertEqual([], self._get_names('')) |
|
743 |
||
744 |
def test_single(self): |
|
745 |
self.assertEqual(['single'], self._get_names('single')) |
|
746 |
||
747 |
def test_multi(self): |
|
748 |
expected = ['one', 'two'] |
|
749 |
self.assertEqual(expected, self._get_names(os.pathsep.join(expected))) |
|
750 |
||
751 |
def test_mixed(self): |
|
752 |
value = os.pathsep.join(['valid', 'in-valid']) |
|
753 |
self.assertEqual(['valid'], self._get_names(value)) |
|
|
7143.15.5
by Jelmer Vernooij
More PEP8 fixes. |
754 |
self.assertContainsRe( |
755 |
self.get_log(), |
|
756 |
r"Invalid name 'in-valid' in BRZ_DISABLE_PLUGINS=" + repr(value)) |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
757 |
|
758 |
||
759 |
class TestEnvPluginsAt(tests.TestCase): |
|
760 |
||
761 |
def _get_paths(self, env_value): |
|
762 |
os.environ['BRZ_PLUGINS_AT'] = env_value |
|
763 |
return plugin._env_plugins_at() |
|
764 |
||
765 |
def test_empty(self): |
|
766 |
self.assertEqual([], plugin._env_plugins_at()) |
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
767 |
self.assertEqual([], self._get_paths('')) |
|
5268.5.1
by Vincent Ladeuil
Reproduce bug #591215. |
768 |
|
769 |
def test_one_path(self): |
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
770 |
self.assertEqual([('b', 'man')], self._get_paths('b@man')) |
|
5268.5.1
by Vincent Ladeuil
Reproduce bug #591215. |
771 |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
772 |
def test_multiple(self): |
773 |
self.assertEqual( |
|
774 |
[('tools', 'bzr-tools'), ('p', 'play.py')], |
|
775 |
self._get_paths(os.pathsep.join(('tools@bzr-tools', 'p@play.py')))) |
|
776 |
||
777 |
def test_many_at(self): |
|
778 |
self.assertEqual( |
|
779 |
[('church', 'StMichael@Plea@Norwich')], |
|
780 |
self._get_paths('church@StMichael@Plea@Norwich')) |
|
781 |
||
782 |
def test_only_py(self): |
|
783 |
self.assertEqual([('test', './test.py')], self._get_paths('./test.py')) |
|
784 |
||
785 |
def test_only_package(self): |
|
786 |
self.assertEqual([('py', '/opt/b/py')], self._get_paths('/opt/b/py')) |
|
787 |
||
788 |
def test_bad_name(self): |
|
789 |
self.assertEqual([], self._get_paths('/usr/local/bzr-git')) |
|
|
7143.15.5
by Jelmer Vernooij
More PEP8 fixes. |
790 |
self.assertContainsRe( |
791 |
self.get_log(), |
|
792 |
r"Invalid name 'bzr-git' in BRZ_PLUGINS_AT='/usr/local/bzr-git'") |
|
|
5268.5.1
by Vincent Ladeuil
Reproduce bug #591215. |
793 |
|
794 |
||
|
5616.7.10
by Martin Pool
Clean up describe_plugins to sort loaded and unloaded plugins together. |
795 |
class TestLoadPluginAt(BaseTestPlugins): |
|
5086.5.3
by Vincent Ladeuil
First shot at loading plugins from a specific directory. |
796 |
|
797 |
def setUp(self): |
|
798 |
super(TestLoadPluginAt, self).setUp() |
|
799 |
# Create the same plugin in two directories
|
|
|
5086.5.8
by Vincent Ladeuil
Make sure we can load from a non-standard directory name. |
800 |
self.create_plugin_package('test_foo', dir='non-standard-dir') |
|
5086.5.13
by Vincent Ladeuil
Reproduce bug #552922. |
801 |
# The "normal" directory, we use 'standard' instead of 'plugins' to
|
802 |
# avoid depending on the precise naming.
|
|
803 |
self.create_plugin_package('test_foo', dir='standard/test_foo') |
|
|
5086.5.8
by Vincent Ladeuil
Make sure we can load from a non-standard directory name. |
804 |
|
|
5086.5.14
by Vincent Ladeuil
Fix bug #552922 by controlling which files can be used to load a plugin. |
805 |
def assertTestFooLoadedFrom(self, path): |
|
5086.5.8
by Vincent Ladeuil
Make sure we can load from a non-standard directory name. |
806 |
self.assertPluginKnown('test_foo') |
|
5131.2.1
by Martin
Permit bzrlib to run under python -OO by explictly assigning to __doc__ for user-visible docstrings |
807 |
self.assertDocstring('This is the doc for test_foo', |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
808 |
self.module.test_foo) |
809 |
self.assertEqual(path, self.module.test_foo.dir_source) |
|
|
5086.5.3
by Vincent Ladeuil
First shot at loading plugins from a specific directory. |
810 |
|
811 |
def test_regular_load(self): |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
812 |
self.load_with_paths(['standard']) |
|
5086.5.13
by Vincent Ladeuil
Reproduce bug #552922. |
813 |
self.assertTestFooLoadedFrom('standard/test_foo') |
|
5086.5.3
by Vincent Ladeuil
First shot at loading plugins from a specific directory. |
814 |
|
815 |
def test_import(self): |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
816 |
self.overrideEnv('BRZ_PLUGINS_AT', 'test_foo@non-standard-dir') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
817 |
self.update_module_paths(['standard']) |
818 |
import breezy.testingplugins.test_foo |
|
|
5086.5.8
by Vincent Ladeuil
Make sure we can load from a non-standard directory name. |
819 |
self.assertTestFooLoadedFrom('non-standard-dir') |
820 |
||
821 |
def test_loading(self): |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
822 |
self.overrideEnv('BRZ_PLUGINS_AT', 'test_foo@non-standard-dir') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
823 |
self.load_with_paths(['standard']) |
824 |
self.assertTestFooLoadedFrom('non-standard-dir') |
|
825 |
||
826 |
def test_loading_other_name(self): |
|
827 |
self.overrideEnv('BRZ_PLUGINS_AT', 'test_foo@non-standard-dir') |
|
828 |
os.rename('standard/test_foo', 'standard/test_bar') |
|
829 |
self.load_with_paths(['standard']) |
|
|
5086.5.9
by Vincent Ladeuil
More tests. |
830 |
self.assertTestFooLoadedFrom('non-standard-dir') |
831 |
||
832 |
def test_compiled_loaded(self): |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
833 |
self.overrideEnv('BRZ_PLUGINS_AT', 'test_foo@non-standard-dir') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
834 |
self.load_with_paths(['standard']) |
|
5086.5.9
by Vincent Ladeuil
More tests. |
835 |
self.assertTestFooLoadedFrom('non-standard-dir') |
|
5235.1.1
by Martin
Make BZR_PLUGINS_AT tests that check filenames use a path-based assertion method rather than just string comparison |
836 |
self.assertIsSameRealPath('non-standard-dir/__init__.py', |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
837 |
self.module.test_foo.__file__) |
|
5086.5.9
by Vincent Ladeuil
More tests. |
838 |
|
839 |
# Try importing again now that the source has been compiled
|
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
840 |
os.remove('non-standard-dir/__init__.py') |
841 |
self.promote_cache('non-standard-dir') |
|
842 |
self.reset() |
|
843 |
self.load_with_paths(['standard']) |
|
|
5086.5.9
by Vincent Ladeuil
More tests. |
844 |
self.assertTestFooLoadedFrom('non-standard-dir') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
845 |
suffix = plugin.COMPILED_EXT |
846 |
self.assertIsSameRealPath('non-standard-dir/__init__' + suffix, |
|
847 |
self.module.test_foo.__file__) |
|
|
5086.5.9
by Vincent Ladeuil
More tests. |
848 |
|
849 |
def test_submodule_loading(self): |
|
850 |
# We create an additional directory under the one for test_foo
|
|
851 |
self.create_plugin_package('test_bar', dir='non-standard-dir/test_bar') |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
852 |
self.overrideEnv('BRZ_PLUGINS_AT', 'test_foo@non-standard-dir') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
853 |
self.update_module_paths(['standard']) |
|
7143.15.5
by Jelmer Vernooij
More PEP8 fixes. |
854 |
import breezy.testingplugins.test_foo # noqa: F401 |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
855 |
self.assertEqual(self.module_prefix + 'test_foo', |
856 |
self.module.test_foo.__package__) |
|
|
7143.15.5
by Jelmer Vernooij
More PEP8 fixes. |
857 |
import breezy.testingplugins.test_foo.test_bar # noqa: F401 |
|
5235.1.1
by Martin
Make BZR_PLUGINS_AT tests that check filenames use a path-based assertion method rather than just string comparison |
858 |
self.assertIsSameRealPath('non-standard-dir/test_bar/__init__.py', |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
859 |
self.module.test_foo.test_bar.__file__) |
|
5086.5.13
by Vincent Ladeuil
Reproduce bug #552922. |
860 |
|
|
5268.6.2
by Vincent Ladeuil
Reproduce bug #588959. |
861 |
def test_relative_submodule_loading(self): |
862 |
self.create_plugin_package('test_foo', dir='another-dir', source=''' |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
863 |
from . import test_bar
|
|
5268.6.2
by Vincent Ladeuil
Reproduce bug #588959. |
864 |
''') |
865 |
# We create an additional directory under the one for test_foo
|
|
866 |
self.create_plugin_package('test_bar', dir='another-dir/test_bar') |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
867 |
self.overrideEnv('BRZ_PLUGINS_AT', 'test_foo@another-dir') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
868 |
self.update_module_paths(['standard']) |
|
7143.15.5
by Jelmer Vernooij
More PEP8 fixes. |
869 |
import breezy.testingplugins.test_foo # noqa: F401 |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
870 |
self.assertEqual(self.module_prefix + 'test_foo', |
871 |
self.module.test_foo.__package__) |
|
|
5268.6.2
by Vincent Ladeuil
Reproduce bug #588959. |
872 |
self.assertIsSameRealPath('another-dir/test_bar/__init__.py', |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
873 |
self.module.test_foo.test_bar.__file__) |
|
5268.6.2
by Vincent Ladeuil
Reproduce bug #588959. |
874 |
|
|
5086.5.15
by Vincent Ladeuil
Fixed as per Ian's review. |
875 |
def test_loading_from___init__only(self): |
|
5086.5.13
by Vincent Ladeuil
Reproduce bug #552922. |
876 |
# We rename the existing __init__.py file to ensure that we don't load
|
877 |
# a random file
|
|
878 |
init = 'non-standard-dir/__init__.py' |
|
879 |
random = 'non-standard-dir/setup.py' |
|
880 |
os.rename(init, random) |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
881 |
self.overrideEnv('BRZ_PLUGINS_AT', 'test_foo@non-standard-dir') |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
882 |
self.load_with_paths(['standard']) |
|
5086.5.13
by Vincent Ladeuil
Reproduce bug #552922. |
883 |
self.assertPluginUnknown('test_foo') |
|
5086.5.14
by Vincent Ladeuil
Fix bug #552922 by controlling which files can be used to load a plugin. |
884 |
|
885 |
def test_loading_from_specific_file(self): |
|
886 |
plugin_dir = 'non-standard-dir' |
|
887 |
plugin_file_name = 'iamtestfoo.py' |
|
888 |
plugin_path = osutils.pathjoin(plugin_dir, plugin_file_name) |
|
889 |
source = '''\ |
|
890 |
"""This is the doc for %s""" |
|
891 |
dir_source = '%s' |
|
892 |
''' % ('test_foo', plugin_path) |
|
893 |
self.create_plugin('test_foo', source=source, |
|
894 |
dir=plugin_dir, file_name=plugin_file_name) |
|
|
6622.1.28
by Jelmer Vernooij
More renames; commands in output, environment variables. |
895 |
self.overrideEnv('BRZ_PLUGINS_AT', 'test_foo@%s' % plugin_path) |
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
896 |
self.load_with_paths(['standard']) |
|
5086.5.14
by Vincent Ladeuil
Fix bug #552922 by controlling which files can be used to load a plugin. |
897 |
self.assertTestFooLoadedFrom(plugin_path) |
|
5616.7.10
by Martin Pool
Clean up describe_plugins to sort loaded and unloaded plugins together. |
898 |
|
899 |
||
900 |
class TestDescribePlugins(BaseTestPlugins): |
|
901 |
||
902 |
def test_describe_plugins(self): |
|
|
5616.7.11
by Martin Pool
Additional tests and fixes for refactored describe_plugins. |
903 |
class DummyModule(object): |
904 |
__doc__ = 'Hi there' |
|
|
7143.15.2
by Jelmer Vernooij
Run autopep8. |
905 |
|
|
5616.7.11
by Martin Pool
Additional tests and fixes for refactored describe_plugins. |
906 |
class DummyPlugin(object): |
907 |
__version__ = '0.1.0' |
|
908 |
module = DummyModule() |
|
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
909 |
self.plugin_warnings = {'bad': ['Failed to load (just testing)']} |
910 |
self.plugins = {'good': DummyPlugin()} |
|
|
6614.1.3
by Vincent Ladeuil
Fix assertEquals being deprecated by using assertEqual. |
911 |
self.assertEqual("""\ |
|
5616.7.10
by Martin Pool
Clean up describe_plugins to sort loaded and unloaded plugins together. |
912 |
bad (failed to load)
|
913 |
** Failed to load (just testing)
|
|
914 |
||
|
5616.7.11
by Martin Pool
Additional tests and fixes for refactored describe_plugins. |
915 |
good 0.1.0
|
916 |
Hi there
|
|
917 |
||
|
6651.4.1
by Martin
Rewrite of the plugin module for Python 3 compat and general sanity |
918 |
""", ''.join(plugin.describe_plugins(state=self))) |
|
7236.3.4
by Jelmer Vernooij
Add tests. |
919 |
|
920 |
||
921 |
class DummyPlugin(object): |
|
922 |
"""Plugin.""" |
|
923 |
||
924 |
||
925 |
class TestLoadEnvPlugin(BaseTestPlugins): |
|
926 |
||
927 |
_test_needs_features = [pkg_resources_feature] |
|
928 |
||
929 |
def setup_plugin(self, source=""): |
|
930 |
# This test tests a new plugin appears in breezy.plugin.plugins().
|
|
931 |
# check the plugin is not loaded already
|
|
932 |
self.assertPluginUnknown('plugin') |
|
933 |
# write a plugin that _cannot_ fail to load.
|
|
934 |
import pkg_resources |
|
935 |
d = pkg_resources.Distribution(__file__) |
|
936 |
ep = pkg_resources.EntryPoint.parse( |
|
937 |
'plugin = ' + __name__ + ':DummyPlugin', dist=d) |
|
938 |
d._ep_map = {'breezy.plugin': {'plugin': ep}} |
|
|
7236.3.7
by Jelmer Vernooij
Fix tests. |
939 |
pkg_resources.working_set.add(d, 'plugin', replace=True) |
940 |
eps = list(pkg_resources.iter_entry_points('breezy.plugin')) |
|
941 |
self.assertEqual(['plugin'], [ep.name for ep in eps]) |
|
|
7236.3.4
by Jelmer Vernooij
Add tests. |
942 |
self.load_with_paths(['.']) |
943 |
self.addCleanup(d._ep_map.clear) |
|
944 |
||
945 |
def test_plugin_loaded(self): |
|
946 |
self.assertPluginUnknown('plugin') |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
947 |
self.overrideEnv('BRZ_PLUGIN_PATH', '+entrypoints') |
|
7236.3.4
by Jelmer Vernooij
Add tests. |
948 |
self.setup_plugin() |
949 |
p = self.plugins['plugin'] |
|
950 |
self.assertIsInstance(p, breezy.plugin.PlugIn) |
|
951 |
self.assertIs(p.module, sys.modules[self.module_prefix + 'plugin']) |
|
952 |
||
953 |
def test_plugin_loaded_disabled(self): |
|
954 |
self.assertPluginUnknown('plugin') |
|
|
7290.33.3
by Jelmer Vernooij
Fix tests. |
955 |
self.overrideEnv('BRZ_PLUGIN_PATH', '+entrypoints') |
|
7236.3.4
by Jelmer Vernooij
Add tests. |
956 |
self.overrideEnv('BRZ_DISABLE_PLUGINS', 'plugin') |
957 |
self.setup_plugin() |
|
958 |
self.assertNotIn('plugin', self.plugins) |