From 0639f96cdb7e85f937678afaec26fc8905d9a9d4 Mon Sep 17 00:00:00 2001 From: Balazs Gibizer Date: Thu, 2 Apr 2020 14:20:53 +0200 Subject: [PATCH] Fix assertEqual param order in Accelerator tests To have a correct error message in case of failure the proper order of parameters are: self.assertEqual(expected, actual) Change-Id: Ia0e909c6ed19f0fd49aa3aa5f5f7a9ea987d9080 --- nova/tests/functional/test_servers.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nova/tests/functional/test_servers.py b/nova/tests/functional/test_servers.py index fe433bfca7bb..c2651f59f00b 100644 --- a/nova/tests/functional/test_servers.py +++ b/nova/tests/functional/test_servers.py @@ -7941,7 +7941,7 @@ class AcceleratorServerBase(integrated_helpers.ProviderUsageBaseTestCase): def _check_no_allocs_usage(self, server_uuid): allocs = self._get_allocations_by_server_uuid(server_uuid) - self.assertEqual(allocs, {}) + self.assertEqual({}, allocs) for i in range(self.NUM_HOSTS): host_alloc = self._get_allocations_by_provider_uuid( @@ -7952,7 +7952,7 @@ class AcceleratorServerBase(integrated_helpers.ProviderUsageBaseTestCase): self.assertEqual({}, device_alloc) usage = self._get_provider_usages( self.device_rp_uuids[i]).get('FPGA') - self.assertEqual(usage, 0) + self.assertEqual(0, usage) class AcceleratorServerTest(AcceleratorServerBase):