Skip to content

Commit 217654e

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Replace assertItemsEqual with assertCountEqual"
2 parents 46ead4a + 7e5b528 commit 217654e

3 files changed

Lines changed: 15 additions & 15 deletions

File tree

openstackclient/tests/unit/compute/v2/test_server.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5101,8 +5101,8 @@ def test_server_list_with_locked(self):
51015101
self.search_opts['locked'] = True
51025102
self.servers_mock.list.assert_called_with(**self.kwargs)
51035103

5104-
self.assertItemsEqual(self.columns, columns)
5105-
self.assertItemsEqual(self.data, tuple(data))
5104+
self.assertCountEqual(self.columns, columns)
5105+
self.assertCountEqual(self.data, tuple(data))
51065106

51075107
def test_server_list_with_unlocked_v273(self):
51085108

@@ -5121,8 +5121,8 @@ def test_server_list_with_unlocked_v273(self):
51215121
self.search_opts['locked'] = False
51225122
self.servers_mock.list.assert_called_with(**self.kwargs)
51235123

5124-
self.assertItemsEqual(self.columns, columns)
5125-
self.assertItemsEqual(self.data, tuple(data))
5124+
self.assertCountEqual(self.columns, columns)
5125+
self.assertCountEqual(self.data, tuple(data))
51265126

51275127
def test_server_list_with_locked_and_unlocked(self):
51285128

@@ -5162,8 +5162,8 @@ def test_server_list_with_changes_before(self):
51625162

51635163
self.servers_mock.list.assert_called_with(**self.kwargs)
51645164

5165-
self.assertItemsEqual(self.columns, columns)
5166-
self.assertItemsEqual(self.data, tuple(data))
5165+
self.assertCountEqual(self.columns, columns)
5166+
self.assertCountEqual(self.data, tuple(data))
51675167

51685168
@mock.patch.object(iso8601, 'parse_date', side_effect=iso8601.ParseError)
51695169
def test_server_list_with_invalid_changes_before(

openstackclient/tests/unit/network/v2/test_local_ip.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ def test_create_no_options(self):
9696

9797
self.network.create_local_ip.assert_called_once_with(**{})
9898
self.assertEqual(set(self.columns), set(columns))
99-
self.assertItemsEqual(self.data, data)
99+
self.assertCountEqual(self.data, data)
100100

101101
def test_create_all_options(self):
102102
arglist = [
@@ -130,7 +130,7 @@ def test_create_all_options(self):
130130
'ip_mode': self.new_local_ip.ip_mode,
131131
})
132132
self.assertEqual(set(self.columns), set(columns))
133-
self.assertItemsEqual(self.data, data)
133+
self.assertCountEqual(self.data, data)
134134

135135

136136
class TestDeleteLocalIP(TestLocalIP):
@@ -263,7 +263,7 @@ def test_local_ip_list(self):
263263

264264
self.network.local_ips.assert_called_once_with(**{})
265265
self.assertEqual(self.columns, columns)
266-
self.assertItemsEqual(self.data, list(data))
266+
self.assertCountEqual(self.data, list(data))
267267

268268
def test_local_ip_list_name(self):
269269
arglist = [
@@ -278,7 +278,7 @@ def test_local_ip_list_name(self):
278278
self.network.local_ips.assert_called_once_with(
279279
**{'name': self.local_ips[0].name})
280280
self.assertEqual(self.columns, columns)
281-
self.assertItemsEqual(self.data, list(data))
281+
self.assertCountEqual(self.data, list(data))
282282

283283
def test_local_ip_list_project(self):
284284
project = identity_fakes_v3.FakeProject.create_one_project()
@@ -295,7 +295,7 @@ def test_local_ip_list_project(self):
295295
self.network.local_ips.assert_called_once_with(
296296
**{'project_id': project.id})
297297
self.assertEqual(self.columns, columns)
298-
self.assertItemsEqual(self.data, list(data))
298+
self.assertCountEqual(self.data, list(data))
299299

300300
def test_local_ip_project_domain(self):
301301
project = identity_fakes_v3.FakeProject.create_one_project()
@@ -314,7 +314,7 @@ def test_local_ip_project_domain(self):
314314

315315
self.network.local_ips.assert_called_once_with(**filters)
316316
self.assertEqual(self.columns, columns)
317-
self.assertItemsEqual(self.data, list(data))
317+
self.assertCountEqual(self.data, list(data))
318318

319319
def test_local_ip_list_network(self):
320320
arglist = [
@@ -477,4 +477,4 @@ def test_show_all_options(self):
477477
self.network.find_local_ip.assert_called_once_with(
478478
self._local_ip.name, ignore_missing=False)
479479
self.assertEqual(set(self.columns), set(columns))
480-
self.assertItemsEqual(self.data, list(data))
480+
self.assertCountEqual(self.data, list(data))

openstackclient/tests/unit/network/v2/test_subnet.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -918,7 +918,7 @@ def test_subnet_list_subnetpool_by_name(self):
918918

919919
self.network.subnets.assert_called_once_with(**filters)
920920
self.assertEqual(self.columns, columns)
921-
self.assertItemsEqual(self.data, list(data))
921+
self.assertCountEqual(self.data, list(data))
922922

923923
def test_subnet_list_subnetpool_by_id(self):
924924
subnet_pool = network_fakes.FakeSubnetPool.create_one_subnet_pool()
@@ -939,7 +939,7 @@ def test_subnet_list_subnetpool_by_id(self):
939939

940940
self.network.subnets.assert_called_once_with(**filters)
941941
self.assertEqual(self.columns, columns)
942-
self.assertItemsEqual(self.data, list(data))
942+
self.assertCountEqual(self.data, list(data))
943943

944944
def test_list_with_tag_options(self):
945945
arglist = [

0 commit comments

Comments
 (0)