File: py3.11_fix_unit_tests.patch

package info (click to toggle)
ironic 1%3A21.1.0-3
  • links: PTS, VCS
  • area: main
  • in suites: bookworm
  • size: 18,704 kB
  • sloc: python: 151,217; sh: 3,175; pascal: 709; xml: 369; makefile: 79
file content (70 lines) | stat: -rw-r--r-- 3,059 bytes parent folder | download
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
Description: Fix unit tests for Python 3.11
 Mocks can no longer be provided as the specs for other Mocks.
 See https://github.com/python/cpython/issues/87644 and
 https://docs.python.org/3.11/whatsnew/3.11.html for more info.
Author: Riccardo Pittau <elfosardo@gmail.com>
Date: Wed, 07 Dec 2022 15:11:09 +0100
Change-Id: If7c10d9bfd0bb410b3bc5180b737439c92e515da
Bug-Debian: https://bugs.debian.org/1024783
Origin: upstream, https://review.opendev.org/c/openstack/ironic/+/866861
Last-Update: 2022-12-09

diff --git a/ironic/tests/unit/drivers/modules/irmc/test_inspect.py b/ironic/tests/unit/drivers/modules/irmc/test_inspect.py
index 5c66cb9..da91ec6 100644
--- a/ironic/tests/unit/drivers/modules/irmc/test_inspect.py
+++ b/ironic/tests/unit/drivers/modules/irmc/test_inspect.py
@@ -204,8 +204,8 @@
         _inspect_hardware_mock.return_value = (inspected_props,
                                                inspected_macs,
                                                new_traits)
-        new_port_mock1 = mock.MagicMock(spec=objects.Port)
-        new_port_mock2 = mock.MagicMock(spec=objects.Port)
+        new_port_mock1 = objects.Port
+        new_port_mock2 = objects.Port
 
         port_mock.side_effect = [new_port_mock1, new_port_mock2]
 
@@ -220,11 +220,11 @@
             port_mock.assert_has_calls([
                 mock.call(task.context, address=inspected_macs[0],
                           node_id=node_id),
+                mock.call.create(),
                 mock.call(task.context, address=inspected_macs[1],
-                          node_id=node_id)
-            ])
-            new_port_mock1.create.assert_called_once_with()
-            new_port_mock2.create.assert_called_once_with()
+                          node_id=node_id),
+                mock.call.create()
+            ], any_order=False)
 
             self.assertTrue(info_mock.called)
             task.node.refresh()
@@ -259,8 +259,9 @@
         _inspect_hardware_mock.return_value = (inspected_props,
                                                inspected_macs,
                                                new_traits)
-        new_port_mock1 = mock.MagicMock(spec=objects.Port)
-        new_port_mock2 = mock.MagicMock(spec=objects.Port)
+
+        new_port_mock1 = objects.Port
+        new_port_mock2 = objects.Port
 
         port_mock.side_effect = [new_port_mock1, new_port_mock2]
 
@@ -276,11 +277,11 @@
             port_mock.assert_has_calls([
                 mock.call(task.context, address=inspected_macs[0],
                           node_id=node_id),
+                mock.call.create(),
                 mock.call(task.context, address=inspected_macs[1],
-                          node_id=node_id)
-            ])
-            new_port_mock1.create.assert_called_once_with()
-            new_port_mock2.create.assert_called_once_with()
+                          node_id=node_id),
+                mock.call.create()
+            ], any_order=False)
 
             self.assertTrue(info_mock.called)
             task.node.refresh()