From c1becbe43cb878295d50f7b17e9fdf944b0e9b5d Mon Sep 17 00:00:00 2001 From: Kostas Karmas Date: Fri, 6 Nov 2020 09:19:21 +0100 Subject: [PATCH] Fix failing tests in TestMachineManager When the machine manager calls the _onGlobalContainerChanged(), it calls updateNumberExtruders Enabled, which triggers the signal numberExtrudersEnabledChanged. This, in turn, triggers the need to check the MachineManager's pyqtProperty numberExtrudersEnabled. Now, since this property has no setter, it cannot be patched. Instead, to work properly, patch the updateNumberExtrudersEnabled. CURA-7827 --- tests/TestMachineManager.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/TestMachineManager.py b/tests/TestMachineManager.py index 4f15a0670c..ae681f927d 100644 --- a/tests/TestMachineManager.py +++ b/tests/TestMachineManager.py @@ -21,6 +21,7 @@ def machine_manager(application, extruder_manager, container_registry, global_st application.getGlobalContainerStack = MagicMock(return_value = global_stack) with patch("cura.Settings.CuraContainerRegistry.CuraContainerRegistry.getInstance", MagicMock(return_value=container_registry)): manager = MachineManager(application) + manager.updateNumberExtrudersEnabled = MagicMock() manager._onGlobalContainerChanged() return manager @@ -253,4 +254,4 @@ def test_isActiveQualityNotSupported(machine_manager): def test_isActiveQualityNotSupported_noQualityGroup(machine_manager): machine_manager.activeQualityGroup = MagicMock(return_value=None) - assert not machine_manager.isActiveQualitySupported \ No newline at end of file + assert not machine_manager.isActiveQualitySupported