/brz/remove-bazaar

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

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_options.py

  • Committer: Vincent Ladeuil
  • Date: 2008-08-31 19:39:01 UTC
  • mto: (3669.1.1 trunk)
  • mto: This revision was merged to the branch mainline in revision 3670.
  • Revision ID: v.ladeuil+lp@free.fr-20080831193901-q64gd8ekc3lyrjdl
Fix bug #263249 by setting valid default _param_name.

* bzrlib/tests/test_options.py:
(TestListOptions.test_list_option_with_dash): Test that long list
options got valid _param_name.

* bzrlib/option.py:
(Option.__init__): Ensure that _param_name got a legal python
attribute name (as optparse do).

* bzrlib/commands.py:
(Command.get_help_text): Drive-by typo correction.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_options.py
256
256
        opts, args = self.parse(options, ['--hello=world', '--hello=sailor'])
257
257
        self.assertEqual(['world', 'sailor'], opts.hello)
258
258
 
 
259
    def test_list_option_with_dash(self):
 
260
        options = [option.ListOption('with-dash', type=str)]
 
261
        opts, args = self.parse(options, ['--with-dash=world',
 
262
                                          '--with-dash=sailor'])
 
263
        self.assertEqual(['world', 'sailor'], opts.with_dash)
 
264
 
259
265
    def test_list_option_no_arguments(self):
260
266
        options = [option.ListOption('hello', type=str)]
261
267
        opts, args = self.parse(options, [])