Skip to content

Commit fac52cc

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Remove testing of cliff command line options"
2 parents 6b80efb + 8c7920d commit fac52cc

2 files changed

Lines changed: 4 additions & 31 deletions

File tree

openstackclient/tests/network/common.py

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -33,30 +33,3 @@ def setUp(self):
3333
service_type="network",
3434
)
3535
self.api = self.app.client_manager.network.api
36-
37-
given_show_options = [
38-
'-f',
39-
'shell',
40-
'-c',
41-
'id',
42-
'--prefix',
43-
'TST',
44-
]
45-
then_show_options = [
46-
('formatter', 'shell'),
47-
('columns', ['id']),
48-
('prefix', 'TST'),
49-
]
50-
given_list_options = [
51-
'-f',
52-
'csv',
53-
'-c',
54-
'id',
55-
'--quote',
56-
'all',
57-
]
58-
then_list_options = [
59-
('formatter', 'csv'),
60-
('columns', ['id']),
61-
('quote_mode', 'all'),
62-
]

openstackclient/tests/network/v2/test_network.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -92,14 +92,14 @@ def test_create_all_options(self):
9292
"--project", identity_fakes_v3.project_name,
9393
"--project-domain", identity_fakes_v3.domain_name,
9494
FAKE_NAME,
95-
] + self.given_show_options
95+
]
9696
verifylist = [
9797
('admin_state', False),
9898
('shared', True),
9999
('project', identity_fakes_v3.project_name),
100100
('project_domain', identity_fakes_v3.domain_name),
101101
('name', FAKE_NAME),
102-
] + self.then_show_options
102+
]
103103
mocker = mock.Mock(return_value=copy.deepcopy(RESPONSE))
104104
self.app.client_manager.network.create_network = mocker
105105
identity_client = identity_fakes_v3.FakeIdentityv3Client(
@@ -519,8 +519,8 @@ def test_show_no_options(self, n_mock):
519519
self.assertEqual(FILTERED, result)
520520

521521
def test_show_all_options(self, n_mock):
522-
arglist = [FAKE_NAME] + self.given_show_options
523-
verifylist = [('identifier', FAKE_NAME)] + self.then_show_options
522+
arglist = [FAKE_NAME]
523+
verifylist = [('identifier', FAKE_NAME)]
524524
n_mock.return_value = copy.deepcopy(RECORD)
525525
self.cmd = network.ShowNetwork(self.app, self.namespace)
526526

0 commit comments

Comments
 (0)