Skip to content

Commit 20f8646

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Use assertIsNone() instead of assertEqual(None, xxx)"
2 parents 61d9604 + ef64a8b commit 20f8646

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

openstackclient/tests/network/v2/test_security_group.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ def test_security_group_delete(self):
6161

6262
self.network.delete_security_group.assert_called_with(
6363
self._security_group)
64-
self.assertEqual(None, result)
64+
self.assertIsNone(result)
6565

6666

6767
class TestDeleteSecurityGroupCompute(TestSecurityGroup):
@@ -96,4 +96,4 @@ def test_security_group_delete(self):
9696

9797
self.compute.security_groups.delete.assert_called_with(
9898
self._security_group.id)
99-
self.assertEqual(None, result)
99+
self.assertIsNone(result)

openstackclient/tests/network/v2/test_security_group_rule.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def test_security_group_rule_delete(self):
6767

6868
self.network.delete_security_group_rule.assert_called_with(
6969
self._security_group_rule)
70-
self.assertEqual(None, result)
70+
self.assertIsNone(result)
7171

7272

7373
class TestDeleteSecurityGroupRuleCompute(TestSecurityGroupRuleCompute):
@@ -97,4 +97,4 @@ def test_security_group_rule_delete(self):
9797

9898
self.compute.security_group_rules.delete.assert_called_with(
9999
self._security_group_rule.id)
100-
self.assertEqual(None, result)
100+
self.assertIsNone(result)

openstackclient/tests/object/v1/test_container.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ def test_container_delete(self, c_mock, o_list_mock, o_delete_mock):
6767
]
6868
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
6969

70-
self.assertEqual(None, self.cmd.take_action(parsed_args))
70+
self.assertIsNone(self.cmd.take_action(parsed_args))
7171

7272
kwargs = {}
7373
c_mock.assert_called_with(
@@ -92,7 +92,7 @@ def test_recursive_delete(self, c_mock, o_list_mock, o_delete_mock):
9292
]
9393
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
9494

95-
self.assertEqual(None, self.cmd.take_action(parsed_args))
95+
self.assertIsNone(self.cmd.take_action(parsed_args))
9696

9797
kwargs = {}
9898
c_mock.assert_called_with(
@@ -120,7 +120,7 @@ def test_r_delete(self, c_mock, o_list_mock, o_delete_mock):
120120
]
121121
parsed_args = self.check_parser(self.cmd, arglist, verifylist)
122122

123-
self.assertEqual(None, self.cmd.take_action(parsed_args))
123+
self.assertIsNone(self.cmd.take_action(parsed_args))
124124

125125
kwargs = {}
126126
c_mock.assert_called_with(

0 commit comments

Comments
 (0)