Skip to content

Commit 16e35f1

Browse files
committed
Change mock/init order to make tests run
1 parent 4addd5f commit 16e35f1

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

test/test_pcan.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,11 @@ def test_reset(self, name, status, expected_result) -> None:
120120
)
121121
def test_get_device_number(self, name, status, expected_result) -> None:
122122
with self.subTest(name):
123-
self.mock_pcan.GetValue = Mock(return_value=(status, 1))
124123
self.bus = can.Bus(bustype="pcan", fd=True)
124+
# Mock GetValue after creation of bus to use first mock of
125+
# GetValue in constructor
126+
self.mock_pcan.GetValue = Mock(return_value=(status, 1))
127+
125128
self.assertEqual(self.bus.get_device_number(), expected_result)
126129
self.mock_pcan.GetValue.assert_called_once_with(
127130
PCAN_USBBUS1, PCAN_DEVICE_NUMBER

0 commit comments

Comments
 (0)