Skip to content

Commit 124f21d

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Compute: Fix DisplayCommandBase comments for cliff Command subclass tests"
2 parents 3d04c78 + b9de23d commit 124f21d

2 files changed

Lines changed: 0 additions & 10 deletions

File tree

openstackclient/tests/compute/v2/test_server.py

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,6 @@ def run_method_with_servers(self, method_name, server_count):
7979
]
8080
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
8181

82-
# DisplayCommandBase.take_action() returns two tuples
8382
self.cmd.take_action(parsed_args)
8483

8584
for s in servers:
@@ -411,7 +410,6 @@ def test_server_delete_no_options(self):
411410
]
412411
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
413412

414-
# DisplayCommandBase.take_action() returns two tuples
415413
self.cmd.take_action(parsed_args)
416414

417415
self.servers_mock.delete.assert_called_with(
@@ -431,7 +429,6 @@ def test_server_delete_multi_servers(self):
431429
]
432430
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
433431

434-
# DisplayCommandBase.take_action() returns two tuples
435432
self.cmd.take_action(parsed_args)
436433

437434
calls = []
@@ -451,7 +448,6 @@ def test_server_delete_wait_ok(self, mock_wait_for_delete):
451448
]
452449
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
453450

454-
# DisplayCommandBase.take_action() returns two tuples
455451
self.cmd.take_action(parsed_args)
456452

457453
self.servers_mock.delete.assert_called_with(
@@ -476,7 +472,6 @@ def test_server_delete_wait_fails(self, mock_wait_for_delete):
476472
]
477473
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
478474

479-
# DisplayCommandBase.take_action() returns two tuples
480475
self.assertRaises(SystemExit, self.cmd.take_action, parsed_args)
481476

482477
self.servers_mock.delete.assert_called_with(
@@ -861,7 +856,6 @@ def test_server_resize_no_options(self):
861856
]
862857
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
863858

864-
# DisplayCommandBase.take_action() returns two tuples
865859
self.cmd.take_action(parsed_args)
866860

867861
self.servers_mock.get.assert_called_with(
@@ -885,7 +879,6 @@ def test_server_resize(self):
885879
]
886880
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
887881

888-
# DisplayCommandBase.take_action() returns two tuples
889882
self.cmd.take_action(parsed_args)
890883

891884
self.servers_mock.get.assert_called_with(
@@ -914,7 +907,6 @@ def test_server_resize_confirm(self):
914907
]
915908
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
916909

917-
# DisplayCommandBase.take_action() returns two tuples
918910
self.cmd.take_action(parsed_args)
919911

920912
self.servers_mock.get.assert_called_with(
@@ -939,7 +931,6 @@ def test_server_resize_revert(self):
939931
]
940932
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
941933

942-
# DisplayCommandBase.take_action() returns two tuples
943934
self.cmd.take_action(parsed_args)
944935

945936
self.servers_mock.get.assert_called_with(

openstackclient/tests/compute/v2/test_service.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ def test_service_delete_no_options(self):
4949
]
5050
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
5151

52-
# DisplayCommandBase.take_action() returns two tuples
5352
self.cmd.take_action(parsed_args)
5453

5554
self.service_mock.delete.assert_called_with(

0 commit comments

Comments
 (0)