Skip to content

Commit bae496e

Browse files
committed
Add assumptions for test cases
1 parent 1d11a04 commit bae496e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

test/test_player.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,18 @@ def test_play_virtual(self):
4949
def test_play_virtual_verbose(self):
5050
sys.argv = self.baseargs + ["-v", self.logfile]
5151
can.player.main()
52-
# TODO: add test two messages sent
5352
# TODO: add test message was printed
53+
self.assertEqual(self.mock_virtual_bus.send.call_count, 2)
5454
self.assertEqual(self.MockSleep.call_count, 2)
5555
self.assertSuccessfullCleanup()
5656

5757
def test_play_virtual_exit(self):
58-
self.MockSleep.side_effect = KeyboardInterrupt
58+
self.MockSleep.side_effect = [None, KeyboardInterrupt]
5959

6060
sys.argv = self.baseargs + [self.logfile]
6161
can.player.main()
62-
# TODO: add test one message sent
63-
self.assertEqual(self.MockSleep.call_count, 1)
62+
self.assertEqual(self.mock_virtual_bus.send.call_count, 1)
63+
self.assertEqual(self.MockSleep.call_count, 2)
6464
self.assertSuccessfullCleanup()
6565

6666
def test_play_error_frame(self):

0 commit comments

Comments
 (0)