diff --git a/scripts/Engineering/gui/engineering_diffraction/tabs/focus/presenter.py b/scripts/Engineering/gui/engineering_diffraction/tabs/focus/presenter.py index 5b2cb8de3ca09b8e8ca65d2ce2d2144d9b1ed664..c28e614d8a8e0796edeba7500c403bf7caf2a12d 100644 --- a/scripts/Engineering/gui/engineering_diffraction/tabs/focus/presenter.py +++ b/scripts/Engineering/gui/engineering_diffraction/tabs/focus/presenter.py @@ -13,7 +13,6 @@ from Engineering.gui.engineering_diffraction.tabs.common.vanadium_corrections im from Engineering.gui.engineering_diffraction.tabs.common.cropping.cropping_widget import CroppingWidget from mantidqt.utils.asynchronous import AsyncTask from mantidqt.utils.observer_pattern import Observer -from mantid.simpleapi import logger class FocusPresenter(object): @@ -95,8 +94,7 @@ class FocusPresenter(object): return False return True - def _on_worker_error(self, failure_info): - logger.warning(str(failure_info)) + def _on_worker_error(self, _): self.emit_enable_button_signal() def set_focus_controls_enabled(self, enabled): diff --git a/scripts/Engineering/gui/engineering_diffraction/tabs/focus/test/test_focus_presenter.py b/scripts/Engineering/gui/engineering_diffraction/tabs/focus/test/test_focus_presenter.py index 581d8b6cb2ee2e563a34f774e167253aa5acaf76..9efb66904b0470b6ddde1e23e08ea0fec827632d 100644 --- a/scripts/Engineering/gui/engineering_diffraction/tabs/focus/test/test_focus_presenter.py +++ b/scripts/Engineering/gui/engineering_diffraction/tabs/focus/test/test_focus_presenter.py @@ -76,14 +76,12 @@ class FocusPresenterTest(unittest.TestCase): self.view.set_plot_output_enabled.assert_called_with(True) @patch(tab_path + ".presenter.FocusPresenter.emit_enable_button_signal") - @patch(tab_path + ".presenter.logger.warning") - def test_on_worker_error_posts_to_logger_and_enables_controls(self, logger, emit): + def test_on_worker_error_posts_to_logger_and_enables_controls(self, emit): fail_info = 2024278 self.presenter._on_worker_error(fail_info) - logger.assert_called_with(str(fail_info)) - self.assertEqual(emit.call_count, 1) + self.assertEqual(1, emit.call_count) def test_get_both_banks(self): self.view.get_crop_checked.return_value = False