From: Greg Farnum Date: Thu, 17 Sep 2020 00:37:23 +0000 (+0000) Subject: test: remove a now-erroneous "mon add" CLI test from ceph_test_argparse.py X-Git-Tag: v17.0.0~1085^2 X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=a8963ccd326224e372cfae044ad3b94a1ce38d34;p=ceph-ci.git test: remove a now-erroneous "mon add" CLI test from ceph_test_argparse.py 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 --- diff --git a/src/test/pybind/test_ceph_argparse.py b/src/test/pybind/test_ceph_argparse.py index 3682d9bbf4e..e9596a177c5 100755 --- a/src/test/pybind/test_ceph_argparse.py +++ b/src/test/pybind/test_ceph_argparse.py @@ -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'])