Revert removal of TWD's slots, add emit_repaint in mock, re #24475

qt/python/mantidqt/widgets/common/test_mocks/mock_mantid.py

@@ -110,3 +110,5 @@ class MockWorkspace:
         self.rowCount = Mock(return_value=self.row_count)
 
         self.column = Mock(return_value=[1] * self.row_count)
+
+        self.emit_repaint = Mock()

qt/python/mantidqt/widgets/tableworkspacedisplay/presenter.py

@@ -85,7 +85,7 @@ class TableWorkspaceDisplay(ObservingPresenter):
         if self.model.workspace_equals(workspace_name):
             self.model = TableWorkspaceDisplayModel(workspace)
             self.load_data(self.view)
-            self.view.repaint_later()
+            self.view.emit_repaint()
 
     def handleItemChanged(self, item):
         """

qt/python/mantidqt/widgets/tableworkspacedisplay/test/test_tableworkspacedisplay_presenter.py

@@ -584,7 +584,7 @@ class TableWorkspaceDisplayPresenterTest(unittest.TestCase):
                 'mantidqt.widgets.tableworkspacedisplay.presenter.TableWorkspaceDisplay.load_data') as mock_load_data:
             presenter.replace_workspace(ws.TEST_NAME + "123", ws)
             self.assertNotCalled(mock_load_data)
-            self.assertNotCalled(view.repaint_later)
+            self.assertNotCalled(view.emit_repaint)
 
     @with_mock_presenter()
     def test_replace(self, ws, view, presenter):
@@ -594,7 +594,7 @@ class TableWorkspaceDisplayPresenterTest(unittest.TestCase):
                 'mantidqt.widgets.tableworkspacedisplay.presenter.TableWorkspaceDisplay.load_data') as mock_load_data:
             presenter.replace_workspace(ws.TEST_NAME, ws)
             mock_load_data.assert_called_once_with(view)
-            view.repaint_later.assert_called_once_with()
+            view.emit_repaint.assert_called_once_with()
 
 
 if __name__ == '__main__':

qt/python/mantidqt/widgets/tableworkspacedisplay/view.py

@@ -13,7 +13,7 @@ import sys
 from functools import partial
 
 from qtpy import QtGui
-from qtpy.QtCore import QVariant, Qt, Signal
+from qtpy.QtCore import QVariant, Qt, Signal, Slot
 from qtpy.QtGui import QKeySequence
 from qtpy.QtWidgets import (QAction, QHeaderView, QItemEditorFactory, QMenu, QMessageBox,
                             QStyledItemDelegate, QTableWidget)
@@ -64,17 +64,32 @@ class TableWorkspaceDisplayView(QTableWidget, ObservingView):
         self.rename_signal.connect(self._run_rename)
         self.repaint_signal.connect(self._run_repaint)
 
-        header = self.horizontalHeader()
-        header.sectionDoubleClicked.connect(self.handle_double_click)
-
         self.resize(600, 400)
         self.show()
 
+        header = self.horizontalHeader()
+        header.sectionDoubleClicked.connect(self.handle_double_click)
+
     def resizeEvent(self, event):
         QTableWidget.resizeEvent(self, event)
         header = self.horizontalHeader()
         header.setSectionResizeMode(QHeaderView.Interactive)
 
+    def emit_repaint(self):
+        self.repaint_signal.emit()
+
+    @Slot()
+    def _run_repaint(self):
+        self.viewport().update()
+
+    @Slot()
+    def _run_close(self):
+        self.close()
+
+    @Slot(str)
+    def _run_rename(self, new_name):
+        self._rename(new_name)
+
     def handle_double_click(self, section):
         header = self.horizontalHeader()
         header.resizeSection(section, header.defaultSectionSize())