From c16c0c904150c924f3920eb67ae7d242f9775277 Mon Sep 17 00:00:00 2001 From: Craig Silverstein Date: Fri, 2 Mar 2018 21:01:47 -0800 Subject: [PATCH] Do better testing that we actually call run_serial(). --- tests/unit/test_checker_manager.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/unit/test_checker_manager.py b/tests/unit/test_checker_manager.py index e3d2379..02397f0 100644 --- a/tests/unit/test_checker_manager.py +++ b/tests/unit/test_checker_manager.py @@ -22,7 +22,9 @@ def test_oserrors_cause_serial_fall_back(): style_guide = style_guide_mock() with mock.patch('_multiprocessing.SemLock', side_effect=err): manager = checker.Manager(style_guide, [], []) - manager.run() + with mock.patch.object(manager, 'run_serial') as serial: + manager.run() + assert serial.call_count == 1 assert manager.using_multiprocessing is False @@ -34,7 +36,9 @@ def test_oserrors_are_reraised(is_windows): with mock.patch('_multiprocessing.SemLock', side_effect=err): with pytest.raises(OSError): manager = checker.Manager(style_guide, [], []) - manager.run() + with mock.patch.object(manager, 'run_serial') as serial: + manager.run() + assert serial.call_count == 0 def test_multiprocessing_is_disabled():