Changeset 14ec91c5 in sasview for src/sas/qtgui/Plotting

Timestamp:

Feb 2, 2018 9:21:48 AM (6 years ago)

Author:

Piotr Rozyczko <rozyczko@…>

Branches:

ESS_GUI, ESS_GUI_Docs, ESS_GUI_batch_fitting, ESS_GUI_bumps_abstraction, ESS_GUI_iss1116, ESS_GUI_iss879, ESS_GUI_iss959, ESS_GUI_opencl, ESS_GUI_ordering, ESS_GUI_sync_sascalc

Children:

a90c9c5

Parents:

5e66738

Message:

More code review related fixes

File:

• src/sas/qtgui/Plotting/UnitTesting/SlicerParametersTest.py (modified) (3 diffs)

src/sas/qtgui/Plotting/UnitTesting/SlicerParametersTest.py

@@ -1 +1 @@
 import sys
 import unittest
+import webbrowser
 from unittest.mock import MagicMock
 
@@ -72 +73 @@
         self.widget.show()
 
-        #Mock the QWebView method
-        QtWebKit.QWebView.show = MagicMock()
-        QtWebKit.QWebView.load = MagicMock()
+        #Mock the webbrowser.open method
+        webbrowser.open = MagicMock()
 
         # Invoke the action
@@ -80 +80 @@
 
         # Check if show() got called
-        self.assertTrue(QtWebKit.QWebView.show.called)
+        self.assertTrue(webbrowser.open.called)
 
         # Assure the filename is correct
-        self.assertIn("graph_help.html", QtWebKit.QWebView.load.call_args[0][0].toString())
+        self.assertIn("graph_help.html", webbrowser.open.call_args[0][0])
         
     def testSetModel(self):