43
12
"""Option parser"""
44
13
eq = self.assertEquals
45
14
eq(parse_args(cmd_commit(), ['--help']),
46
([], {'fixes': [], 'help': True}))
16
eq(parse_args(cmd_status(), ['--all']),
47
18
eq(parse_args(cmd_commit(), ['--message=biter']),
48
([], {'fixes': [], 'message': 'biter'}))
19
([], {'message': 'biter'}))
20
## eq(parse_args(cmd_log(), '-r 500'.split()),
21
## ([], {'revision': RevisionSpec_int(500)}))
50
23
def test_no_more_opts(self):
51
24
"""Terminated options"""
52
25
self.assertEquals(parse_args(cmd_commit(), ['--', '-file-with-dashes']),
53
(['-file-with-dashes'], {'fixes': []}))
26
(['-file-with-dashes'], {}))
55
28
def test_option_help(self):
56
29
"""Options have help strings."""
57
out, err = self.run_bzr('commit --help')
58
self.assertContainsRe(out,
59
r'--file(.|\n)*Take commit message from this file\.')
60
self.assertContainsRe(out, r'-h.*--help')
30
out, err = self.run_bzr_captured(['commit', '--help'])
31
self.assertContainsRe(out, r'--file.*file containing commit message')
32
self.assertContainsRe(out, r'--help.*-h')
62
34
def test_option_help_global(self):
63
35
"""Global options have help strings."""
64
out, err = self.run_bzr('help status')
65
self.assertContainsRe(out, r'--show-ids.*Show internal object.')
36
out, err = self.run_bzr_captured(['help', 'status'])
37
self.assertContainsRe(out, r'--show-ids.*show internal object')
67
39
def test_option_arg_help(self):
68
40
"""Help message shows option arguments."""
69
out, err = self.run_bzr('help commit')
41
out, err = self.run_bzr_captured(['help', 'commit'])
70
42
self.assertEquals(err, '')
71
43
self.assertContainsRe(out, r'--file[ =]MSGFILE')
73
45
def test_unknown_short_opt(self):
74
out, err = self.run_bzr('help -r', retcode=3)
75
self.assertContainsRe(err, r'no such option')
77
def test_set_short_name(self):
78
o = option.Option('wiggle')
80
self.assertEqual(o.short_name(), 'w')
82
def test_allow_dash(self):
83
"""Test that we can pass a plain '-' as an argument."""
85
(['-'], {'fixes': []}), parse_args(cmd_commit(), ['-']))
87
def parse(self, options, args):
88
parser = option.get_optparser(dict((o.name, o) for o in options))
89
return parser.parse_args(args)
91
def test_conversion(self):
92
options = [option.Option('hello')]
93
opts, args = self.parse(options, ['--no-hello', '--hello'])
94
self.assertEqual(True, opts.hello)
95
opts, args = self.parse(options, [])
96
self.assertEqual(option.OptionParser.DEFAULT_VALUE, opts.hello)
97
opts, args = self.parse(options, ['--hello', '--no-hello'])
98
self.assertEqual(False, opts.hello)
99
options = [option.Option('number', type=int)]
100
opts, args = self.parse(options, ['--number', '6'])
101
self.assertEqual(6, opts.number)
102
self.assertRaises(errors.BzrCommandError, self.parse, options,
104
self.assertRaises(errors.BzrCommandError, self.parse, options,
107
def test_registry_conversion(self):
108
registry = bzrdir.BzrDirFormatRegistry()
109
registry.register_metadir('one', 'RepositoryFormat7', 'one help')
110
registry.register_metadir('two', 'RepositoryFormatKnit1', 'two help')
111
registry.register_metadir('hidden', 'RepositoryFormatKnit1',
112
'two help', hidden=True)
113
registry.set_default('one')
114
options = [option.RegistryOption('format', '', registry, str)]
115
opts, args = self.parse(options, ['--format', 'one'])
116
self.assertEqual({'format':'one'}, opts)
117
opts, args = self.parse(options, ['--format', 'two'])
118
self.assertEqual({'format':'two'}, opts)
119
self.assertRaises(errors.BadOptionValue, self.parse, options,
120
['--format', 'three'])
121
self.assertRaises(errors.BzrCommandError, self.parse, options,
123
options = [option.RegistryOption('format', '', registry, str,
124
value_switches=True)]
125
opts, args = self.parse(options, ['--two'])
126
self.assertEqual({'format':'two'}, opts)
127
opts, args = self.parse(options, ['--two', '--one'])
128
self.assertEqual({'format':'one'}, opts)
129
opts, args = self.parse(options, ['--two', '--one',
131
self.assertEqual({'format':'two'}, opts)
132
options = [option.RegistryOption('format', '', registry, str,
134
self.assertRaises(errors.BzrCommandError, self.parse, options,
137
def test_registry_converter(self):
138
options = [option.RegistryOption('format', '',
139
bzrdir.format_registry, bzrdir.format_registry.make_bzrdir)]
140
opts, args = self.parse(options, ['--format', 'knit'])
141
self.assertIsInstance(opts.format.repository_format,
142
knitrepo.RepositoryFormatKnit1)
144
def test_from_kwargs(self):
145
my_option = option.RegistryOption.from_kwargs('my-option',
146
help='test option', short='be short', be_long='go long')
147
self.assertEqual(['my-option'],
148
[x[0] for x in my_option.iter_switches()])
149
my_option = option.RegistryOption.from_kwargs('my-option',
150
help='test option', title="My option", short='be short',
151
be_long='go long', value_switches=True)
152
self.assertEqual(['my-option', 'be-long', 'short'],
153
[x[0] for x in my_option.iter_switches()])
156
registry = bzrdir.BzrDirFormatRegistry()
157
registry.register_metadir('one', 'RepositoryFormat7', 'one help')
158
registry.register_metadir('two',
159
'bzrlib.repofmt.knitrepo.RepositoryFormatKnit1',
162
registry.register_metadir('hidden', 'RepositoryFormat7', 'hidden help',
164
registry.set_default('one')
165
options = [option.RegistryOption('format', 'format help', registry,
166
str, value_switches=True, title='Formats')]
167
parser = option.get_optparser(dict((o.name, o) for o in options))
168
value = parser.format_option_help()
169
self.assertContainsRe(value, 'format.*format help')
170
self.assertContainsRe(value, 'one.*one help')
171
self.assertContainsRe(value, 'Formats:\n *--format')
172
self.assertNotContainsRe(value, 'hidden help')
174
def test_iter_switches(self):
175
opt = option.Option('hello', help='fg')
176
self.assertEqual(list(opt.iter_switches()),
177
[('hello', None, None, 'fg')])
178
opt = option.Option('hello', help='fg', type=int)
179
self.assertEqual(list(opt.iter_switches()),
180
[('hello', None, 'ARG', 'fg')])
181
opt = option.Option('hello', help='fg', type=int, argname='gar')
182
self.assertEqual(list(opt.iter_switches()),
183
[('hello', None, 'GAR', 'fg')])
184
registry = bzrdir.BzrDirFormatRegistry()
185
registry.register_metadir('one', 'RepositoryFormat7', 'one help')
186
registry.register_metadir('two',
187
'bzrlib.repofmt.knitrepo.RepositoryFormatKnit1',
190
registry.set_default('one')
191
opt = option.RegistryOption('format', 'format help', registry,
192
value_switches=False)
193
self.assertEqual(list(opt.iter_switches()),
194
[('format', None, 'ARG', 'format help')])
195
opt = option.RegistryOption('format', 'format help', registry,
197
self.assertEqual(list(opt.iter_switches()),
198
[('format', None, 'ARG', 'format help'),
199
('default', None, None, 'one help'),
200
('one', None, None, 'one help'),
201
('two', None, None, 'two help'),
205
class TestListOptions(TestCase):
206
"""Tests for ListOption, used to specify lists on the command-line."""
208
def parse(self, options, args):
209
parser = option.get_optparser(dict((o.name, o) for o in options))
210
return parser.parse_args(args)
212
def test_list_option(self):
213
options = [option.ListOption('hello', type=str)]
214
opts, args = self.parse(options, ['--hello=world', '--hello=sailor'])
215
self.assertEqual(['world', 'sailor'], opts.hello)
217
def test_list_option_no_arguments(self):
218
options = [option.ListOption('hello', type=str)]
219
opts, args = self.parse(options, [])
220
self.assertEqual([], opts.hello)
222
def test_list_option_with_int_type(self):
223
options = [option.ListOption('hello', type=int)]
224
opts, args = self.parse(options, ['--hello=2', '--hello=3'])
225
self.assertEqual([2, 3], opts.hello)
227
def test_list_option_with_int_type_can_be_reset(self):
228
options = [option.ListOption('hello', type=int)]
229
opts, args = self.parse(options, ['--hello=2', '--hello=3',
230
'--hello=-', '--hello=5'])
231
self.assertEqual([5], opts.hello)
233
def test_list_option_can_be_reset(self):
234
"""Passing an option of '-' to a list option should reset the list."""
235
options = [option.ListOption('hello', type=str)]
236
opts, args = self.parse(
237
options, ['--hello=a', '--hello=b', '--hello=-', '--hello=c'])
238
self.assertEqual(['c'], opts.hello)
241
class TestOptionDefinitions(TestCase):
242
"""Tests for options in the Bazaar codebase."""
244
def get_builtin_command_options(self):
246
for cmd_name, cmd_class in sorted(commands.get_all_cmds()):
248
for opt_name, opt in sorted(cmd.options().items()):
249
g.append((cmd_name, opt))
252
def test_global_options_used(self):
253
# In the distant memory, options could only be declared globally. Now
254
# we prefer to declare them in the command, unless like -r they really
255
# are used very widely with the exact same meaning. So this checks
256
# for any that should be garbage collected.
257
g = dict(option.Option.OPTIONS.items())
260
for cmd_name, cmd_class in sorted(commands.get_all_cmds()):
261
for option_or_name in sorted(cmd_class.takes_options):
262
if not isinstance(option_or_name, basestring):
263
self.assertIsInstance(option_or_name, option.Option)
264
elif not option_or_name in g:
265
msgs.append("apparent reference to undefined "
266
"global option %r from %r"
267
% (option_or_name, cmd_class))
269
used_globals.setdefault(option_or_name, []).append(cmd_name)
270
unused_globals = set(g.keys()) - set(used_globals.keys())
271
# not enforced because there might be plugins that use these globals
272
## for option_name in sorted(unused_globals):
273
## msgs.append("unused global option %r" % option_name)
274
## for option_name, cmds in sorted(used_globals.items()):
275
## if len(cmds) <= 1:
276
## msgs.append("global option %r is only used by %r"
277
## % (option_name, cmds))
279
self.fail("problems with global option definitions:\n"
282
def test_option_grammar(self):
284
# Option help should be written in sentence form, and have a final
285
# period and be all on a single line, because the display code will
287
option_re = re.compile(r'^[A-Z][^\n]+\.$')
288
for scope, option in self.get_builtin_command_options():
290
msgs.append('%-16s %-16s %s' %
291
((scope or 'GLOBAL'), option.name, 'NO HELP'))
292
elif not option_re.match(option.help):
293
msgs.append('%-16s %-16s %s' %
294
((scope or 'GLOBAL'), option.name, option.help))
296
self.fail("The following options don't match the style guide:\n"
46
out, err = self.run_bzr_captured(['help', '-r'], retcode=3)
47
self.assertContainsRe(err, r'unknown short option')
50
# >>> parse_args('log -r 500'.split())
51
# (['log'], {'revision': [<RevisionSpec_int 500>]})
52
# >>> parse_args('log -r500..600'.split())
53
# (['log'], {'revision': [<RevisionSpec_int 500>, <RevisionSpec_int 600>]})
54
# >>> parse_args('log -vr500..600'.split())
55
# (['log'], {'verbose': True, 'revision': [<RevisionSpec_int 500>, <RevisionSpec_int 600>]})
56
# >>> parse_args('log -rrevno:500..600'.split()) #the r takes an argument
57
# (['log'], {'revision': [<RevisionSpec_revno revno:500>, <RevisionSpec_int 600>]})