]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
test: remove a now-erroneous "mon add" CLI test from ceph_test_argparse.py 35906/head
authorGreg Farnum <gfarnum@redhat.com>
Thu, 17 Sep 2020 00:37:23 +0000 (00:37 +0000)
committerGreg Farnum <gfarnum@redhat.com>
Thu, 17 Sep 2020 00:37:23 +0000 (00:37 +0000)
The "mon add" command now lets you pass in arbitrary numbers of strings,
so that you can include locations, so this test is invalid.

I considered updating it to only allow a single non-spaced string, but

datacenter=site1 rack=abc host=host1

is accepted elsewhere, so let's keep that consistent and just remove
this test instead.

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
src/test/pybind/test_ceph_argparse.py

index 3682d9bbf4e553d074f1ddd4fc6fd62589117de3..e9596a177c593650f3ee7857b98d11a698a1bc9c 100755 (executable)
@@ -468,9 +468,6 @@ class TestMon(TestArgparse):
         assert_equal({}, validate_command(sigdict, ['mon', 'add',
                                                     'name',
                                                     '400.500.600.700']))
-        assert_equal({}, validate_command(sigdict, ['mon', 'add', 'name',
-                                                    '1.2.3.4:1234',
-                                                    'toomany']))
 
     def test_remove(self):
         self.assert_valid_command(['mon', 'remove', 'name'])