From a8963ccd326224e372cfae044ad3b94a1ce38d34 Mon Sep 17 00:00:00 2001 From: Greg Farnum Date: Thu, 17 Sep 2020 00:37:23 +0000 Subject: [PATCH] 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 --- src/test/pybind/test_ceph_argparse.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/test/pybind/test_ceph_argparse.py b/src/test/pybind/test_ceph_argparse.py index 3682d9bbf4e55..e9596a177c593 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']) -- 2.39.5