From 970fce73c235a17c2519fa53945aa3942fa9efe5 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 13 Nov 2023 13:08:54 +0100
Subject: [PATCH 01/24] DOC: comments on planned refactoring of actions

---
 dtool_lookup_gui/main.py               |  2 ++
 dtool_lookup_gui/utils/copy_manager.py |  6 ++++++
 dtool_lookup_gui/views/main_window.py  | 20 ++++++++++++++++----
 3 files changed, 24 insertions(+), 4 deletions(-)

diff --git a/dtool_lookup_gui/main.py b/dtool_lookup_gui/main.py
index dd85f0bd..27cee0fb 100644
--- a/dtool_lookup_gui/main.py
+++ b/dtool_lookup_gui/main.py
@@ -88,6 +88,8 @@ def __init__(self, *args, loop=None, **kwargs):
         self.loop = loop
         self.args = None
 
+        # TODO: instantiate CopyManager here and assign as a property of this Application class
+
     def do_activate(self):
         logger.debug("do_activate")
 
diff --git a/dtool_lookup_gui/utils/copy_manager.py b/dtool_lookup_gui/utils/copy_manager.py
index 059fb4d7..882ed6ef 100644
--- a/dtool_lookup_gui/utils/copy_manager.py
+++ b/dtool_lookup_gui/utils/copy_manager.py
@@ -36,11 +36,16 @@ class CopyManager:
 
     def __init__(self, progress_revealer, progress_popover):
         # Note: This is not particularly abstract, as it interacts directly with the Gtk widgets
+
+        # TODO: instead of a single progress_revealer and progress_popover assigned at instantiation,
+        # the copy manager would have an arbitrary amount of those that can be dynamically attached and detached
+
         self._progress_revealer = progress_revealer
         self._progress_chart = progress_revealer.get_child().get_child()
         self._progress_popover = progress_popover
 
     async def copy(self, dataset, destination):
+        # TODO: here the copy manager would loop over all attached status widgets
         self._progress_revealer.set_reveal_child(True)
         tracker = self._progress_popover.add_status_box(
             self.progress_update, f'Copying dataset »{dataset}« to »{destination}«')
@@ -61,6 +66,7 @@ async def copy(self, dataset, destination):
             self._progress_revealer.set_reveal_child(False)
 
     def progress_update(self):
+        # TODO: here the copy manager would loop over all attached status widgets
         # Refresh pie chart
         total_length = sum([len(tracker) for tracker in self._progress_popover.status_boxes])
         total_step = sum([tracker.step for tracker in self._progress_popover.status_boxes])
diff --git a/dtool_lookup_gui/views/main_window.py b/dtool_lookup_gui/views/main_window.py
index e09d9d00..cc653c0c 100644
--- a/dtool_lookup_gui/views/main_window.py
+++ b/dtool_lookup_gui/views/main_window.py
@@ -24,6 +24,11 @@
 # SOFTWARE.
 #
 
+# TODO: any atomic operations that reflect core dtool behavior as documented
+# on https://dtoolcore.readthedocs.io/en/latest/api/dtoolcore.html#dtoolcore.DataSetCreator
+# or https://dtoolcore.readthedocs.io/en/latest/api/dtoolcore.html#dtoolcore.ProtoDataSet
+# should move to atomic actions on the main app level
+
 import asyncio
 import logging
 import os
@@ -178,8 +183,6 @@ class MainWindow(Gtk.ApplicationWindow):
     contents_per_page = Gtk.Template.Child()
     linting_errors_button = Gtk.Template.Child()
 
-
-
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
 
@@ -269,6 +272,12 @@ def __init__(self, *args, **kwargs):
 
         self.dependency_graph_widget.search_by_uuid = self._search_by_uuid
 
+        # TODO: is it possible to attach the CopyManager to the main app instead of this main window?
+        # One possibility would be to refactor the CopyManager in a way that it can have any
+        # number of arbitrary progress_revealer and progress_popover components attached.
+        # The copy manager lives at the Application scope.
+        # The main window here attaches its own progress_revealer and progress_popover to the
+        # CopyManager of the application instead of instantiating the copy manager here.
         self._copy_manager = CopyManager(self.progress_revealer, self.progress_popover)
 
         _logger.debug(f"Constructed main window for app '{self.application.get_application_id()}'")
@@ -420,7 +429,6 @@ async def retry():
         # If a widget was passed in, re-enable it
         self.enable_pagination_buttons()
 
-
     def _search_by_uuid(self, uuid):
         search_text = dump_single_line_query_text({"uuid": uuid})
         self._search_by_search_text(search_text)
@@ -775,6 +783,7 @@ def on_add_items_clicked(self, widget):
             fpaths = dialog.get_filenames()
             for fpath in fpaths:
                 # uri = urllib.parse.unquote(uri, encoding='utf-8', errors='replace')
+                # TODO: _add_item should turn into an action on the app level as well
                 self._add_item(fpath)
         elif response == Gtk.ResponseType.CANCEL:
             pass
@@ -844,6 +853,7 @@ def on_linting_errors_button_clicked(self, widget):
         else:
             pass
 
+    # TODO: try to refactor into a do_freeze action in the main app as well
     @Gtk.Template.Callback()
     def on_freeze_clicked(self, widget):
         row = self.dataset_list_box.get_selected_row()
@@ -984,7 +994,9 @@ def enable_pagination_buttons(self):
         self.page_advancer_button.set_sensitive(True)
         self.next_page_advancer_button.set_sensitive(True)
 
-    # @Gtk.Template.Callback(), not in .ui
+    # TODO: this should be an action do_copy
+    # if it is possible to hand to strings, e.g. source and destination to an action, then this action should
+    # go to the main app.
     def on_copy_clicked(self, widget):
         async def _copy():
             try:

From 5b809fdc5855c301cc32c9e1aec09c0bd2216bf9 Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Mon, 20 Nov 2023 01:30:35 +0100
Subject: [PATCH 02/24] WIP:TST

---
 dtool_lookup_gui/views/main_window.py | 13 +++--
 test/test_main_window_actions.py      | 76 +++++++++++++++++++++++++--
 2 files changed, 83 insertions(+), 6 deletions(-)

diff --git a/dtool_lookup_gui/views/main_window.py b/dtool_lookup_gui/views/main_window.py
index cc653c0c..f3183985 100644
--- a/dtool_lookup_gui/views/main_window.py
+++ b/dtool_lookup_gui/views/main_window.py
@@ -201,8 +201,7 @@ def __init__(self, *args, **kwargs):
 
         self.readme_buffer.connect("changed", self.on_readme_buffer_changed)
 
-        self.error_bar.set_revealed(False)
-        self.progress_revealer.set_reveal_child(False)
+
 
         # connect log handler to error bar
         root_logger = logging.getLogger()
@@ -221,6 +220,13 @@ def __init__(self, *args, **kwargs):
         self.server_versions_dialog = ServerVersionsDialog(application=self.application)
         self.error_linting_dialog = LintingErrorsDialog(application=self.application)
 
+        # Initialize the progress revealer and popover
+        self.progress_revealer = None  # Replace this with actual initialization
+        self.progress_popover = None  # Replace this with actual initialization
+
+        # Initialize the CopyManager
+        self._copy_manager = CopyManager(self.progress_revealer, self.progress_popover)
+
         # window-scoped actions
 
         # search action
@@ -278,7 +284,8 @@ def __init__(self, *args, **kwargs):
         # The copy manager lives at the Application scope.
         # The main window here attaches its own progress_revealer and progress_popover to the
         # CopyManager of the application instead of instantiating the copy manager here.
-        self._copy_manager = CopyManager(self.progress_revealer, self.progress_popover)
+        self.get_action_group("app").activate_action("start-copy-manager", None)
+
 
         _logger.debug(f"Constructed main window for app '{self.application.get_application_id()}'")
 
diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index a4b5514c..bdb1392a 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,3 +1,73 @@
-import asyncio
-import logging
-import pytest
\ No newline at end of file
+import pytest
+from unittest.mock import Mock, patch, ANY
+from gi.repository import Gtk
+from dtool_lookup_gui.views.main_window import MainWindow
+
+@pytest.fixture
+def main_window():
+    # Assuming MainWindow can be instantiated without parameters
+    window = MainWindow()
+    window.dataset_list_box = Mock()
+    window.dataset_list_box.get_selected_row.return_value = Mock(dataset=Mock())
+    return window
+
+@pytest.mark.asyncio
+async def test_app_id(app):
+    assert app.get_application_id() == 'de.uni-freiburg.dtool-lookup-gui'
+
+@pytest.mark.asyncio
+async def test_do_refresh_view(main_window):
+    with patch.object(main_window, 'refresh', new_callable=Mock) as mock_refresh:
+        mock_action = Mock()
+        mock_value = Mock()
+        await main_window.do_refresh_view(mock_action, mock_value)
+        mock_refresh.assert_called_once()
+
+@pytest.mark.asyncio
+async def test_do_get_item(main_window):
+    """Test do_get_item action."""
+    with patch('shutil.copyfile') as mock_copyfile, \
+            patch('dtool_lookup_gui.views.main_window.launch_default_app_for_uri') as mock_launch_app, \
+            patch('dtool_lookup_gui.views.main_window.settings') as mock_settings, \
+            patch.object(main_window, '_get_selected_items',
+                         return_value=[('item', 'uuid')]) as mock_get_selected_items:
+        # Set settings value
+        mock_settings.open_downloaded_item = False
+
+        # Mock value to mimic the GVariant returned by the value parameter
+        value = Mock()
+        value.get_string.return_value = "/path/to/destination"
+
+        # Mock action to mimic the Gio.SimpleAction
+        action = Mock()
+
+        # Perform the get item action
+        await main_window.do_get_item(action, value)
+
+        # Assert that copyfile was called
+        mock_copyfile.assert_called_once_with(ANY, "/path/to/destination")
+
+        # Assert that _get_selected_items was called
+        mock_get_selected_items.assert_called_once()
+
+        # Optionally assert launch_default_app_for_uri call based on settings
+        if mock_settings.open_downloaded_item:
+            mock_launch_app.assert_called_once_with("/path/to/destination")
+@pytest.mark.asyncio
+async def test_do_search_select_and_show(main_window):
+    """Test the do_search_select_and_show action."""
+
+    # Mock the '_search_select_and_show' method on the MainWindow instance
+    with patch.object(main_window, '_search_select_and_show', new_callable=Mock) as mock_search_select_and_show:
+        # Create a mock object for the value parameter
+        mock_value = Mock()
+        mock_value.get_string.return_value = "test search text"
+
+        # Mock action to mimic the Gio.SimpleAction
+        mock_action = Mock()
+
+        # Call the do_search_select_and_show method
+        await main_window.do_search_select_and_show(mock_action, mock_value)
+
+        # Assert that _search_select_and_show was called with the correct search text
+        mock_search_select_and_show.assert_called_once_with("test search text")

From 49214e8243e32b18f6209337667917a18d59bf4b Mon Sep 17 00:00:00 2001
From: ASHWIN VAZHAPPILLY <42440470+ashdriod@users.noreply.github.com>
Date: Mon, 20 Nov 2023 01:34:59 +0100
Subject: [PATCH 03/24] Update main.py

---
 dtool_lookup_gui/main.py | 54 +++++++++++++++++++++++++++++++++++-----
 1 file changed, 48 insertions(+), 6 deletions(-)

diff --git a/dtool_lookup_gui/main.py b/dtool_lookup_gui/main.py
index 27cee0fb..bb183f74 100644
--- a/dtool_lookup_gui/main.py
+++ b/dtool_lookup_gui/main.py
@@ -1,6 +1,5 @@
 #
-# Copyright 2021-2023 Johannes Laurin Hörmann
-#           2023 Ashwin Vazhappilly
+# Copyright 2021-2022 Johannes Laurin Hörmann
 #           2021 Lars Pastewka
 #
 # ### MIT license
@@ -34,7 +33,6 @@
 
 import dtoolcore
 import dtool_lookup_api.core.config
-
 from dtool_lookup_api.core.LookupClient import authenticate
 
 import gi
@@ -87,8 +85,9 @@ def __init__(self, *args, loop=None, **kwargs):
                          flags=Gio.ApplicationFlags.HANDLES_COMMAND_LINE, **kwargs)
         self.loop = loop
         self.args = None
+        self._progress_revealer = None
+        self._progress_popover = None
 
-        # TODO: instantiate CopyManager here and assign as a property of this Application class
 
     def do_activate(self):
         logger.debug("do_activate")
@@ -106,7 +105,7 @@ def do_activate(self):
             # self.window = AppWindow(application=self, title="Main Window")
             logger.debug("Build GUI.")
 
-            win = MainWindow(application=self)
+            win = LoginWindow(application=self)
             glob_pattern = os.path.join(os.path.dirname(__file__), os.pardir, 'data','icons','*','dtool_logo.xpm')
             icon_file_list = glob.glob(glob_pattern)
             if len(icon_file_list) > 0:
@@ -117,7 +116,7 @@ def do_activate(self):
             else:
                 logger.warning("Could not load app icons.")
             win.connect('destroy', lambda _: self.loop.stop())
-            self.loop.call_soon(win.refresh)  # Populate widgets after event loop starts
+            #self.loop.call_soon(win.refresh)  # Populate widgets after event loop starts
 
         logger.debug("Present main window.")
         win.present()
@@ -236,6 +235,12 @@ def do_startup(self):
         renew_token_action.connect("activate", self.do_renew_token)
         self.add_action(renew_token_action)
 
+        # copy-manager action
+        copy_action = Gio.SimpleAction.new("start-copy", GLib.VariantType.new("(ss)"))
+        copy_action.connect("activate", self.do_copy)
+        self.add_action(copy_action)
+
+
         Gtk.Application.do_startup(self)
 
     # custom application-scoped actions
@@ -340,6 +345,43 @@ async def retrieve_token(auth_url, username, password):
             username,
             password))
 
+    async def do_copy(self, action, value):
+        dataset, destination = value.unpack()
+
+        # Ensure that _progress_revealer and _progress_popover are initialized
+        if not self._progress_revealer or not self._progress_popover:
+            logger.error("Progress revealer or popover not initialized")
+            return
+
+        self._progress_revealer.set_reveal_child(True)
+        tracker = self._progress_popover.add_status_box(
+            self.progress_update, f'Copying dataset "{dataset}" to "{destination}"')
+
+        try:
+            await dataset.copy(destination, progressbar=tracker)
+        except ChildProcessError as exc:
+            logger.error(str(exc))
+            # Handle specific exceptions here
+        except Exception as e:
+            logger.error(f"Unexpected error: {e}")
+            # Handle other unexpected errors
+
+        tracker.set_done()
+        self.check_all_operations_done()
+
+    def progress_update(self):
+        if not self._progress_popover or not self._progress_chart:
+            logger.error("Progress popover or chart not initialized")
+            return
+
+        total_length = sum([len(tracker) for tracker in self._progress_popover.status_boxes])
+        total_step = sum([tracker.step for tracker in self._progress_popover.status_boxes])
+
+        if total_length > 0:
+            self._progress_chart.set_fraction(total_step / total_length)
+        else:
+            self._progress_chart.set_fraction(1.0)
+
 
 def run_gui():
     GObject.type_register(GtkSource.View)

From 1d0c21cdcf78ce5acefeda1035910714b511f311 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 20 Nov 2023 13:28:02 +0100
Subject: [PATCH 04/24] ENH: add main_window property to app

---
 dtool_lookup_gui/main.py | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/dtool_lookup_gui/main.py b/dtool_lookup_gui/main.py
index bb183f74..3fca5283 100644
--- a/dtool_lookup_gui/main.py
+++ b/dtool_lookup_gui/main.py
@@ -87,7 +87,11 @@ def __init__(self, *args, loop=None, **kwargs):
         self.args = None
         self._progress_revealer = None
         self._progress_popover = None
+        self._main_window = None
 
+    @property
+    def main_window(self):
+        return self._main_window
 
     def do_activate(self):
         logger.debug("do_activate")
@@ -105,7 +109,10 @@ def do_activate(self):
             # self.window = AppWindow(application=self, title="Main Window")
             logger.debug("Build GUI.")
 
-            win = LoginWindow(application=self)
+            win = MainWindow(application=self)
+            self._main_window = win
+            logger.debug("New main window ID: %s", win.get_id())
+
             glob_pattern = os.path.join(os.path.dirname(__file__), os.pardir, 'data','icons','*','dtool_logo.xpm')
             icon_file_list = glob.glob(glob_pattern)
             if len(icon_file_list) > 0:
@@ -240,7 +247,6 @@ def do_startup(self):
         copy_action.connect("activate", self.do_copy)
         self.add_action(copy_action)
 
-
         Gtk.Application.do_startup(self)
 
     # custom application-scoped actions

From 59939d0929e2f9efbe2a7d9c5b678f8c3e5b1a83 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 20 Nov 2023 13:28:21 +0100
Subject: [PATCH 05/24] MAINT: disable copy manager for now

---
 dtool_lookup_gui/views/main_window.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dtool_lookup_gui/views/main_window.py b/dtool_lookup_gui/views/main_window.py
index f3183985..40da0f47 100644
--- a/dtool_lookup_gui/views/main_window.py
+++ b/dtool_lookup_gui/views/main_window.py
@@ -225,7 +225,7 @@ def __init__(self, *args, **kwargs):
         self.progress_popover = None  # Replace this with actual initialization
 
         # Initialize the CopyManager
-        self._copy_manager = CopyManager(self.progress_revealer, self.progress_popover)
+        # self._copy_manager = CopyManager(self.progress_revealer, self.progress_popover)
 
         # window-scoped actions
 

From 47e6d6187ba1a05a2011de7811ade554980dbf34 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 20 Nov 2023 13:29:01 +0100
Subject: [PATCH 06/24] TST: added start-copy action to actions list

---
 test/test_app.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/test_app.py b/test/test_app.py
index 6e02da0a..fe11e3c0 100644
--- a/test/test_app.py
+++ b/test/test_app.py
@@ -6,7 +6,6 @@
 from dtool_lookup_gui.views.config_details import ConfigDialog
 from dtool_lookup_gui.views.settings_dialog import SettingsDialog
 from dtool_lookup_gui.views.log_window import LogWindow
-from dtool_lookup_gui.views.login_window import LoginWindow
 from dtool_lookup_gui.views.main_window import MainWindow
 from dtool_lookup_gui.views.server_versions_dialog import ServerVersionsDialog
 from dtool_lookup_gui.views.error_linting_dialog import LintingErrorsDialog
@@ -50,6 +49,7 @@ async def test_app_window_types(app):
 @pytest.mark.asyncio
 async def test_app_list_actions(app):
     assert set(app.list_actions()) == set([
+        'start-copy',
         'toggle-logging',
         'reset-config',
         'renew-token',

From f72f54fe3259fe17910c90742e8618c141bf3c52 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 20 Nov 2023 13:30:01 +0100
Subject: [PATCH 07/24] TST: moved main window fixture to conftest.py

---
 test/conftest.py                 | 16 ++++++++++++++++
 test/test_main_window_actions.py | 10 +++-------
 2 files changed, 19 insertions(+), 7 deletions(-)

diff --git a/test/conftest.py b/test/conftest.py
index e36fc10e..8350b625 100644
--- a/test/conftest.py
+++ b/test/conftest.py
@@ -6,10 +6,12 @@
 gi.require_version('GtkSource', '4')
 from gi.repository import GLib, GObject, Gio, Gtk, GtkSource, GdkPixbuf
 
+import asyncio
 import gbulb
 gbulb.install(gtk=True)
 
 from dtool_lookup_gui.main import Application
+from dtool_lookup_gui.views.main_window import MainWindow
 
 
 logger = logging.getLogger(__name__)
@@ -77,6 +79,20 @@ def app(gtk_loop):
     yield app
 
 
+@pytest.fixture(scope="function")
+def main_window(app):
+    # Assuming MainWindow can be instantiated without parameters
+    # app.do_startup() # cannot directly call startup action, segmentation fault
+    window = MainWindow(application=app)
+    # issue: app level actions not available at this point, window init fails with
+    #       set_loglevel_action = self.get_action_group("app").lookup_action('set-loglevel')
+    #       E       AttributeError: 'NoneType' object has no attribute 'lookup_action'
+
+    # window.dataset_list_box = Mock()
+    # window.dataset_list_box.get_selected_row.return_value = Mock(dataset=Mock())
+    return window
+
+
 # ================================================================
 # fixtures related to https://github.com/pytest-dev/pytest-asyncio
 # ================================================================
diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index bdb1392a..ccb0ee45 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -3,18 +3,12 @@
 from gi.repository import Gtk
 from dtool_lookup_gui.views.main_window import MainWindow
 
-@pytest.fixture
-def main_window():
-    # Assuming MainWindow can be instantiated without parameters
-    window = MainWindow()
-    window.dataset_list_box = Mock()
-    window.dataset_list_box.get_selected_row.return_value = Mock(dataset=Mock())
-    return window
 
 @pytest.mark.asyncio
 async def test_app_id(app):
     assert app.get_application_id() == 'de.uni-freiburg.dtool-lookup-gui'
 
+
 @pytest.mark.asyncio
 async def test_do_refresh_view(main_window):
     with patch.object(main_window, 'refresh', new_callable=Mock) as mock_refresh:
@@ -23,6 +17,7 @@ async def test_do_refresh_view(main_window):
         await main_window.do_refresh_view(mock_action, mock_value)
         mock_refresh.assert_called_once()
 
+
 @pytest.mark.asyncio
 async def test_do_get_item(main_window):
     """Test do_get_item action."""
@@ -53,6 +48,7 @@ async def test_do_get_item(main_window):
         # Optionally assert launch_default_app_for_uri call based on settings
         if mock_settings.open_downloaded_item:
             mock_launch_app.assert_called_once_with("/path/to/destination")
+
 @pytest.mark.asyncio
 async def test_do_search_select_and_show(main_window):
     """Test the do_search_select_and_show action."""

From 485ac54ad9fb4b18c0d23e5bf0e13f5f1e2cf9bc Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 20 Nov 2023 13:44:00 +0100
Subject: [PATCH 08/24] WIP: await sleep stub

---
 test/test_main_window_actions.py | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index ccb0ee45..53081499 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,3 +1,5 @@
+import asyncio
+
 import pytest
 from unittest.mock import Mock, patch, ANY
 from gi.repository import Gtk
@@ -10,11 +12,12 @@ async def test_app_id(app):
 
 
 @pytest.mark.asyncio
-async def test_do_refresh_view(main_window):
-    with patch.object(main_window, 'refresh', new_callable=Mock) as mock_refresh:
+async def test_do_refresh_view(app):
+    await asyncio.sleep(3)  # we will need a functionality to actively await the availability of the app and main window
+    with patch.object(app.main_window, 'refresh', new_callable=Mock) as mock_refresh:
         mock_action = Mock()
         mock_value = Mock()
-        await main_window.do_refresh_view(mock_action, mock_value)
+        await app.main_window.do_refresh_view(mock_action, mock_value)
         mock_refresh.assert_called_once()
 
 

From 2410587f73ad955f07662657569f707dcb9bc509 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 20 Nov 2023 14:05:58 +0100
Subject: [PATCH 09/24] TST: removed main_window fixture

---
 test/conftest.py | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/test/conftest.py b/test/conftest.py
index 8350b625..eca18aac 100644
--- a/test/conftest.py
+++ b/test/conftest.py
@@ -79,20 +79,6 @@ def app(gtk_loop):
     yield app
 
 
-@pytest.fixture(scope="function")
-def main_window(app):
-    # Assuming MainWindow can be instantiated without parameters
-    # app.do_startup() # cannot directly call startup action, segmentation fault
-    window = MainWindow(application=app)
-    # issue: app level actions not available at this point, window init fails with
-    #       set_loglevel_action = self.get_action_group("app").lookup_action('set-loglevel')
-    #       E       AttributeError: 'NoneType' object has no attribute 'lookup_action'
-
-    # window.dataset_list_box = Mock()
-    # window.dataset_list_box.get_selected_row.return_value = Mock(dataset=Mock())
-    return window
-
-
 # ================================================================
 # fixtures related to https://github.com/pytest-dev/pytest-asyncio
 # ================================================================

From 9f7c88eae81dfb09213545946f396f8a248740f1 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 20 Nov 2023 14:06:22 +0100
Subject: [PATCH 10/24] TST: refresh-view test working

---
 test/test_main_window_actions.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 53081499..f648a3a1 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -13,11 +13,8 @@ async def test_app_id(app):
 
 @pytest.mark.asyncio
 async def test_do_refresh_view(app):
-    await asyncio.sleep(3)  # we will need a functionality to actively await the availability of the app and main window
     with patch.object(app.main_window, 'refresh', new_callable=Mock) as mock_refresh:
-        mock_action = Mock()
-        mock_value = Mock()
-        await app.main_window.do_refresh_view(mock_action, mock_value)
+        app.main_window.activate_action('refresh-view')
         mock_refresh.assert_called_once()
 
 

From 45622d3548c6ff2e15b87ec62465df0b76271083 Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Sun, 26 Nov 2023 19:47:04 +0100
Subject: [PATCH 11/24] Working GTK Main Window

---
 test/test_main_window_actions.py | 83 ++++++++------------------------
 1 file changed, 21 insertions(+), 62 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index f648a3a1..94600180 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,69 +1,28 @@
-import asyncio
-
+from unittest.mock import patch, MagicMock, create_autospec
 import pytest
-from unittest.mock import Mock, patch, ANY
 from gi.repository import Gtk
 from dtool_lookup_gui.views.main_window import MainWindow
 
+@pytest.fixture
+def mocked_gtk_app():
+    """Create a mocked GTK application."""
+    mock_app = create_autospec(Gtk.Application, instance=True)
+    # Set up any specific properties or return values you need for the mock here
+    return mock_app
 
-@pytest.mark.asyncio
-async def test_app_id(app):
-    assert app.get_application_id() == 'de.uni-freiburg.dtool-lookup-gui'
-
-
-@pytest.mark.asyncio
-async def test_do_refresh_view(app):
-    with patch.object(app.main_window, 'refresh', new_callable=Mock) as mock_refresh:
-        app.main_window.activate_action('refresh-view')
-        mock_refresh.assert_called_once()
-
-
-@pytest.mark.asyncio
-async def test_do_get_item(main_window):
-    """Test do_get_item action."""
-    with patch('shutil.copyfile') as mock_copyfile, \
-            patch('dtool_lookup_gui.views.main_window.launch_default_app_for_uri') as mock_launch_app, \
-            patch('dtool_lookup_gui.views.main_window.settings') as mock_settings, \
-            patch.object(main_window, '_get_selected_items',
-                         return_value=[('item', 'uuid')]) as mock_get_selected_items:
-        # Set settings value
-        mock_settings.open_downloaded_item = False
-
-        # Mock value to mimic the GVariant returned by the value parameter
-        value = Mock()
-        value.get_string.return_value = "/path/to/destination"
-
-        # Mock action to mimic the Gio.SimpleAction
-        action = Mock()
-
-        # Perform the get item action
-        await main_window.do_get_item(action, value)
-
-        # Assert that copyfile was called
-        mock_copyfile.assert_called_once_with(ANY, "/path/to/destination")
-
-        # Assert that _get_selected_items was called
-        mock_get_selected_items.assert_called_once()
-
-        # Optionally assert launch_default_app_for_uri call based on settings
-        if mock_settings.open_downloaded_item:
-            mock_launch_app.assert_called_once_with("/path/to/destination")
+@pytest.fixture
+def main_window_instance(mocked_gtk_app):
+    """Create an instance of MainWindow for testing with a mocked GTK application."""
+    with patch('dtool_lookup_gui.views.main_window.MainWindow') as mock_main_window:
+        instance = mock_main_window(application=mocked_gtk_app)
+        yield instance
 
 @pytest.mark.asyncio
-async def test_do_search_select_and_show(main_window):
-    """Test the do_search_select_and_show action."""
-
-    # Mock the '_search_select_and_show' method on the MainWindow instance
-    with patch.object(main_window, '_search_select_and_show', new_callable=Mock) as mock_search_select_and_show:
-        # Create a mock object for the value parameter
-        mock_value = Mock()
-        mock_value.get_string.return_value = "test search text"
-
-        # Mock action to mimic the Gio.SimpleAction
-        mock_action = Mock()
-
-        # Call the do_search_select_and_show method
-        await main_window.do_search_select_and_show(mock_action, mock_value)
-
-        # Assert that _search_select_and_show was called with the correct search text
-        mock_search_select_and_show.assert_called_once_with("test search text")
+async def test_main_window_creation(main_window_instance):
+    """Simple test to check if MainWindow is created with the mocked GTK application."""
+    assert main_window_instance is not None
+    assert isinstance(main_window_instance.application, MagicMock), "The application should be a MagicMock."
+
+    # Optionally, test if certain methods or properties are called or set
+    # main_window_instance.application.some_method.assert_called_once()
+    # assert main_window_instance.application.some_property == 'expected_value'

From 2b305131ec30f6a07519d8bd4cfb5ba782ef770f Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Mon, 27 Nov 2023 00:47:28 +0100
Subject: [PATCH 12/24] Working GTK Main Window

---
 test/test_main_window_actions.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 94600180..900fc08d 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -22,7 +22,9 @@ async def test_main_window_creation(main_window_instance):
     """Simple test to check if MainWindow is created with the mocked GTK application."""
     assert main_window_instance is not None
     assert isinstance(main_window_instance.application, MagicMock), "The application should be a MagicMock."
+    assert isinstance(main_window_instance.builder, MagicMock), "The builder should be a MagicMock."
+    assert isinstance(main_window_instance.window, MagicMock), "The window should be a MagicMock."
+    assert isinstance(main_window_instance.search_button, MagicMock), "The search_button should be a MagicMock."
+    assert isinstance(main_window_instance.search_entry, MagicMock), "The search_entry should be a MagicMock."
+    assert isinstance(main_window_instance.search_results_treeview, MagicMock), "The search_results_treeview should be a MagicMock."
 
-    # Optionally, test if certain methods or properties are called or set
-    # main_window_instance.application.some_method.assert_called_once()
-    # assert main_window_instance.application.some_property == 'expected_value'

From 027ade74f61b3fa457b448bbf7db3691b97e91eb Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Mon, 27 Nov 2023 02:16:10 +0100
Subject: [PATCH 13/24] Working GTK Main Window

---
 test/test_main_window_actions.py | 21 ++++++++++++++++++++-
 1 file changed, 20 insertions(+), 1 deletion(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 900fc08d..6191bf9f 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,4 +1,4 @@
-from unittest.mock import patch, MagicMock, create_autospec
+from unittest.mock import patch, MagicMock, create_autospec, Mock
 import pytest
 from gi.repository import Gtk
 from dtool_lookup_gui.views.main_window import MainWindow
@@ -28,3 +28,22 @@ async def test_main_window_creation(main_window_instance):
     assert isinstance(main_window_instance.search_entry, MagicMock), "The search_entry should be a MagicMock."
     assert isinstance(main_window_instance.search_results_treeview, MagicMock), "The search_results_treeview should be a MagicMock."
 
+@pytest.mark.asyncio
+async def test_do_refresh_view(main_window_instance):
+    """Test the do_refresh_view method triggers the refresh method."""
+
+    # 3. Validate Method Existence
+    assert hasattr(main_window_instance, 'refresh'), "MainWindow instance does not have a refresh method"
+
+    with patch.object(main_window_instance, 'refresh', new_callable=Mock) as mock_refresh:
+        # Trigger the do_refresh_view action
+        main_window_instance.do_refresh_view(None, None)
+
+        # Check if the refresh method was called
+        #mock_refresh.assert_called_once()
+
+
+    # Temporarily replacing the refresh method
+    main_window_instance.refresh = Mock()
+    main_window_instance.refresh()
+    main_window_instance.refresh.assert_called_once()

From a1c89fb6e57c5432d164622f069b25d06e6485a8 Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Mon, 27 Nov 2023 10:53:33 +0100
Subject: [PATCH 14/24]  Action Testing

---
 test/test_main_window_actions.py | 25 +++++++++++--------------
 1 file changed, 11 insertions(+), 14 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 6191bf9f..9f331688 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,6 +1,6 @@
 from unittest.mock import patch, MagicMock, create_autospec, Mock
 import pytest
-from gi.repository import Gtk
+from gi.repository import Gio, GLib, Gtk, GtkSource, Gdk
 from dtool_lookup_gui.views.main_window import MainWindow
 
 @pytest.fixture
@@ -28,22 +28,19 @@ async def test_main_window_creation(main_window_instance):
     assert isinstance(main_window_instance.search_entry, MagicMock), "The search_entry should be a MagicMock."
     assert isinstance(main_window_instance.search_results_treeview, MagicMock), "The search_results_treeview should be a MagicMock."
 
+
 @pytest.mark.asyncio
 async def test_do_refresh_view(main_window_instance):
-    """Test the do_refresh_view method triggers the refresh method."""
-
-    # 3. Validate Method Existence
-    assert hasattr(main_window_instance, 'refresh'), "MainWindow instance does not have a refresh method"
+    """Test the do_refresh_view action."""
 
+    # Mock the necessary method
     with patch.object(main_window_instance, 'refresh', new_callable=Mock) as mock_refresh:
-        # Trigger the do_refresh_view action
-        main_window_instance.do_refresh_view(None, None)
-
-        # Check if the refresh method was called
-        #mock_refresh.assert_called_once()
+        # Mock the action and value as needed, adjust based on your actual method signature
+        action = MagicMock()
+        value = MagicMock()  # Adjust this if your action requires a specific value
 
+        # Perform the refresh view action
+        main_window_instance.do_refresh_view(action, value)
 
-    # Temporarily replacing the refresh method
-    main_window_instance.refresh = Mock()
-    main_window_instance.refresh()
-    main_window_instance.refresh.assert_called_once()
+        # Assert that the refresh method was called
+        mock_refresh.assert_called_once()

From 6543b9145a043b53227f6d778a1fdab5e8056f5a Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 27 Nov 2023 12:55:57 +0100
Subject: [PATCH 15/24] TST: test action two ways

---
 test/test_main_window_actions.py | 58 ++++----------------------------
 1 file changed, 7 insertions(+), 51 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index f648a3a1..4916a679 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -13,57 +13,13 @@ async def test_app_id(app):
 
 @pytest.mark.asyncio
 async def test_do_refresh_view(app):
+    """Test the do_refresh_view method triggers the refresh method."""
+    
+    # Actually activate the untempered method with all side effects
+    app.main_window.do_refresh_view(None, None)
+
+    # Patch the main window's refresh method and make sure it's called when
+    # action activated via the Gtk framework
     with patch.object(app.main_window, 'refresh', new_callable=Mock) as mock_refresh:
         app.main_window.activate_action('refresh-view')
         mock_refresh.assert_called_once()
-
-
-@pytest.mark.asyncio
-async def test_do_get_item(main_window):
-    """Test do_get_item action."""
-    with patch('shutil.copyfile') as mock_copyfile, \
-            patch('dtool_lookup_gui.views.main_window.launch_default_app_for_uri') as mock_launch_app, \
-            patch('dtool_lookup_gui.views.main_window.settings') as mock_settings, \
-            patch.object(main_window, '_get_selected_items',
-                         return_value=[('item', 'uuid')]) as mock_get_selected_items:
-        # Set settings value
-        mock_settings.open_downloaded_item = False
-
-        # Mock value to mimic the GVariant returned by the value parameter
-        value = Mock()
-        value.get_string.return_value = "/path/to/destination"
-
-        # Mock action to mimic the Gio.SimpleAction
-        action = Mock()
-
-        # Perform the get item action
-        await main_window.do_get_item(action, value)
-
-        # Assert that copyfile was called
-        mock_copyfile.assert_called_once_with(ANY, "/path/to/destination")
-
-        # Assert that _get_selected_items was called
-        mock_get_selected_items.assert_called_once()
-
-        # Optionally assert launch_default_app_for_uri call based on settings
-        if mock_settings.open_downloaded_item:
-            mock_launch_app.assert_called_once_with("/path/to/destination")
-
-@pytest.mark.asyncio
-async def test_do_search_select_and_show(main_window):
-    """Test the do_search_select_and_show action."""
-
-    # Mock the '_search_select_and_show' method on the MainWindow instance
-    with patch.object(main_window, '_search_select_and_show', new_callable=Mock) as mock_search_select_and_show:
-        # Create a mock object for the value parameter
-        mock_value = Mock()
-        mock_value.get_string.return_value = "test search text"
-
-        # Mock action to mimic the Gio.SimpleAction
-        mock_action = Mock()
-
-        # Call the do_search_select_and_show method
-        await main_window.do_search_select_and_show(mock_action, mock_value)
-
-        # Assert that _search_select_and_show was called with the correct search text
-        mock_search_select_and_show.assert_called_once_with("test search text")

From 509f9cf2e72184ea29595a3169f3698328d2a0a6 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 27 Nov 2023 13:06:34 +0100
Subject: [PATCH 16/24] TST: some comments on how unit tests could look like

---
 test/test_main_window_actions.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index d4c7c32c..5a0c2fbc 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -11,8 +11,15 @@ async def test_do_refresh_view(app):
     # Actually activate the untempered method with all side effects
     app.main_window.do_refresh_view(None, None)
 
+    # We could check for any effects of the action here, e.g.
+    # have Gtk widgets been  populated with teh right data?
+
     # Patch the main window's refresh method and make sure it's called when
     # action activated via the Gtk framework
     with patch.object(app.main_window, 'refresh', new_callable=Mock) as mock_refresh:
         app.main_window.activate_action('refresh-view')
         mock_refresh.assert_called_once()
+
+        # We could replace the refresh mock with mocks of other calls
+        # embedded deeper in the hierarchy and assert that the call
+        # hierarchy is preserved as we expect it

From 2bf5656031f356be0458101a4c93bf3986416072 Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Thu, 30 Nov 2023 02:28:58 +0100
Subject: [PATCH 17/24] Actual call working

---
 test/test_main_window_actions.py | 58 ++++++++++++++++++++++++++------
 1 file changed, 47 insertions(+), 11 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 5a0c2fbc..767a6b03 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,25 +1,61 @@
-from unittest.mock import patch, MagicMock, create_autospec, Mock
 import pytest
-from gi.repository import Gtk
+from unittest.mock import patch, Mock, MagicMock
+import asyncio
+from gi.repository import Gtk, Gio, GLib
 from dtool_lookup_gui.views.main_window import MainWindow
+from unittest.mock import patch, MagicMock, AsyncMock
+
+
 
 
 @pytest.mark.asyncio
 async def test_do_refresh_view(app):
     """Test the do_refresh_view method triggers the refresh method."""
-    
-    # Actually activate the untempered method with all side effects
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+
     app.main_window.do_refresh_view(None, None)
+    # Assertions for side effects (if applicable)
 
-    # We could check for any effects of the action here, e.g.
-    # have Gtk widgets been  populated with teh right data?
 
-    # Patch the main window's refresh method and make sure it's called when
-    # action activated via the Gtk framework
+    # Patch the main window's refresh method
     with patch.object(app.main_window, 'refresh', new_callable=Mock) as mock_refresh:
         app.main_window.activate_action('refresh-view')
         mock_refresh.assert_called_once()
 
-        # We could replace the refresh mock with mocks of other calls
-        # embedded deeper in the hierarchy and assert that the call
-        # hierarchy is preserved as we expect it
+        # Check if get_selected_row was called on dataset_list_box
+        mock_dataset_list_box.get_selected_row.assert_called_once()
+
+
+from unittest.mock import AsyncMock
+
+@pytest.mark.asyncio
+async def test_do_get_item(app):
+    """Test the do_get_item method for copying a selected item."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+    mock_settings = MagicMock()
+    app.main_window.settings = mock_settings
+
+    # Mock _get_selected_items to return one item
+    app.main_window._get_selected_items = MagicMock(return_value=[('item_name', 'item_uuid')])
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_string("dummy_path")
+
+    # Mock async call in do_get_item
+    mock_dataset = MagicMock()
+    mock_dataset.get_item = AsyncMock()  # Use AsyncMock for async calls
+    mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
+
+    # Call the method with mock objects
+    app.main_window.do_get_item(mock_action, mock_variant)
+    # Use patch.object to mock do_get_item method
+
+

From 16c696354730c23bc45b795811a209eb417da3b9 Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Thu, 30 Nov 2023 17:29:23 +0100
Subject: [PATCH 18/24] TST:WIP

---
 test/test_main_window_actions.py | 66 ++++++++++++++++++++++----------
 1 file changed, 46 insertions(+), 20 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 767a6b03..183c6d9a 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,40 +1,36 @@
 import pytest
-from unittest.mock import patch, Mock, MagicMock
-import asyncio
-from gi.repository import Gtk, Gio, GLib
-from dtool_lookup_gui.views.main_window import MainWindow
 from unittest.mock import patch, MagicMock, AsyncMock
-
+from gi.repository import Gtk, Gio, GLib
 
 
 
 @pytest.mark.asyncio
-async def test_do_refresh_view(app):
-    """Test the do_refresh_view method triggers the refresh method."""
+async def test_do_refresh_view_direct_call(app):
+    """Test the direct call of the do_refresh_view method."""
 
     # Mock dependencies
     mock_dataset_list_box = MagicMock()
     app.main_window.dataset_list_box = mock_dataset_list_box
 
-
+    # Directly call the do_refresh_view method
     app.main_window.do_refresh_view(None, None)
-    # Assertions for side effects (if applicable)
 
+@pytest.mark.asyncio
+async def test_refresh_method_triggered_by_action(app):
+    """Test if the 'refresh-view' action triggers the refresh method."""
 
     # Patch the main window's refresh method
-    with patch.object(app.main_window, 'refresh', new_callable=Mock) as mock_refresh:
+    with patch.object(app.main_window, 'refresh', new_callable=MagicMock) as mock_refresh:
+        # Trigger the 'refresh-view' action
         app.main_window.activate_action('refresh-view')
-        mock_refresh.assert_called_once()
-
-        # Check if get_selected_row was called on dataset_list_box
-        mock_dataset_list_box.get_selected_row.assert_called_once()
 
+        # Assert that the refresh method was called once
+        mock_refresh.assert_called_once()
 
-from unittest.mock import AsyncMock
 
 @pytest.mark.asyncio
-async def test_do_get_item(app):
-    """Test the do_get_item method for copying a selected item."""
+async def test_do_get_item_direct_call(app):
+    """Test the do_get_item method for copying a selected item directly."""
 
     # Mock dependencies
     mock_dataset_list_box = MagicMock()
@@ -51,11 +47,41 @@ async def test_do_get_item(app):
 
     # Mock async call in do_get_item
     mock_dataset = MagicMock()
-    mock_dataset.get_item = AsyncMock()  # Use AsyncMock for async calls
+    mock_dataset.get_item = AsyncMock()
     mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
 
-    # Call the method with mock objects
+    # Directly call the method with mock objects
     app.main_window.do_get_item(mock_action, mock_variant)
-    # Use patch.object to mock do_get_item method
 
 
+@pytest.mark.asyncio
+async def test_do_get_item_action_trigger(app):
+    """Test if 'get-item' action triggers do_get_item method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+    mock_settings = MagicMock()
+    app.main_window.settings = mock_settings
+
+    # Setup necessary mocks for the action trigger
+    mock_dataset = MagicMock()
+    mock_dataset.get_item = AsyncMock()
+    mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
+    app.main_window._get_selected_items = MagicMock(return_value=[('item_name', 'item_uuid')])
+
+    # Create and add the action
+    dest_file_variant = GLib.Variant.new_string("dummy_path")
+    get_item_action = Gio.SimpleAction.new("get-item", dest_file_variant.get_type())
+    app.main_window.add_action(get_item_action)
+
+    # Patch do_get_item method after action is added
+    with patch.object(app.main_window, 'do_get_item', new_callable=MagicMock) as mock_do_get_item:
+        # Connect the action
+        get_item_action.connect("activate", app.main_window.do_get_item)
+
+        # Trigger the action
+        get_item_action.activate(dest_file_variant)
+
+        # Assert that do_get_item was called once
+        mock_do_get_item.assert_called_once_with(get_item_action, dest_file_variant)
\ No newline at end of file

From ca90ce2f394f68bd55887c5f89d22c3ae81e4f46 Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Thu, 30 Nov 2023 17:37:35 +0100
Subject: [PATCH 19/24] TST:Comments Added

---
 test/test_main_window_actions.py | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 183c6d9a..69a6077a 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,3 +1,11 @@
+# 1. Direct calls in tests isolate specific functionalities for focused, independent testing of components.
+# 2. Triggering actions and using mocks are crucial for integration testing, ensuring components interact correctly.
+# 3. Mocking and action triggering simulate real-world user interactions and application responses.
+# 4. Separating tests for direct calls and action triggers aids in maintaining clear, organized test structures.
+# 5. This approach enhances test suite readability and makes it easier to understand and update.
+
+
+
 import pytest
 from unittest.mock import patch, MagicMock, AsyncMock
 from gi.repository import Gtk, Gio, GLib
@@ -84,4 +92,6 @@ async def test_do_get_item_action_trigger(app):
         get_item_action.activate(dest_file_variant)
 
         # Assert that do_get_item was called once
-        mock_do_get_item.assert_called_once_with(get_item_action, dest_file_variant)
\ No newline at end of file
+        mock_do_get_item.assert_called_once_with(get_item_action, dest_file_variant)
+
+

From 5ad23f04fab8d3c1ebbe957c35e43713076c5d6d Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Sun, 3 Dec 2023 17:41:13 +0100
Subject: [PATCH 20/24]  Action Testing

---
 test/test_main_window_actions.py | 97 ++++++++++++++++++++++++++++++++
 1 file changed, 97 insertions(+)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 69a6077a..1304d044 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -95,3 +95,100 @@ async def test_do_get_item_action_trigger(app):
         mock_do_get_item.assert_called_once_with(get_item_action, dest_file_variant)
 
 
+@pytest.mark.asyncio
+async def test_do_search_select_and_show_direct_call(app):
+    """Test the do_search_select_and_show method for processing a search directly."""
+
+    # Mock dependencies
+    mock_base_uri_list_box = MagicMock()
+    app.main_window.base_uri_list_box = mock_base_uri_list_box
+
+    # Mock _search_select_and_show to simulate search behavior
+    app.main_window._search_select_and_show = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_string("test_search_query")
+
+    # Directly call the method with mock objects
+    app.main_window.do_search_select_and_show(mock_action, mock_variant)
+
+    # Assert that _search_select_and_show was called with the correct query
+    app.main_window._search_select_and_show.assert_called_once_with("test_search_query")
+
+
+@pytest.mark.asyncio
+async def test_do_search_select_and_show_action_trigger(app):
+    """Test if 'search-select-show' action triggers do_search_select_and_show method."""
+
+    # Mock dependencies
+    mock_base_uri_list_box = MagicMock()
+    app.main_window.base_uri_list_box = mock_base_uri_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._search_select_and_show = MagicMock()
+
+    # Create and add the action
+    search_variant = GLib.Variant.new_string("test_search_query")
+    search_select_show_action = Gio.SimpleAction.new("search-select-show", search_variant.get_type())
+    app.main_window.add_action(search_select_show_action)
+
+    # Patch do_search_select_and_show method after action is added
+    with patch.object(app.main_window, 'do_search_select_and_show', new_callable=MagicMock) as mock_do_search_select_and_show:
+        # Connect the action
+        search_select_show_action.connect("activate", app.main_window.do_search_select_and_show)
+
+        # Trigger the action
+        search_select_show_action.activate(search_variant)
+
+        # Assert that do_search_select_and_show was called once
+        mock_do_search_select_and_show.assert_called_once_with(search_select_show_action, search_variant)
+
+@pytest.mark.asyncio
+async def test_do_show_dataset_details_by_uri_direct_call(app):
+    """Test the do_show_dataset_details_by_uri method for showing dataset details directly."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Mock _show_dataset_details_by_uri to simulate behavior
+    app.main_window._show_dataset_details_by_uri = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_string("dummy_uri")
+
+    # Directly call the method with mock objects
+    app.main_window.do_show_dataset_details_by_uri(mock_action, mock_variant)
+
+    # Assert that _show_dataset_details_by_uri was called with the correct uri
+    app.main_window._show_dataset_details_by_uri.assert_called_once_with("dummy_uri")
+
+
+@pytest.mark.asyncio
+async def test_do_show_dataset_details_by_uri_action_trigger(app):
+    """Test if 'show-dataset-by-uri' action triggers do_show_dataset_details_by_uri method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._show_dataset_details_by_uri = MagicMock()
+
+    # Create and add the action
+    uri_variant = GLib.Variant.new_string("dummy_uri")
+    show_dataset_by_uri_action = Gio.SimpleAction.new("show-dataset-by-uri", uri_variant.get_type())
+    app.main_window.add_action(show_dataset_by_uri_action)
+
+    # Patch do_show_dataset_details_by_uri method after action is added
+    with patch.object(app.main_window, 'do_show_dataset_details_by_uri', new_callable=MagicMock) as mock_do_show_dataset_details_by_uri:
+        # Connect the action
+        show_dataset_by_uri_action.connect("activate", app.main_window.do_show_dataset_details_by_uri)
+
+        # Trigger the action
+        show_dataset_by_uri_action.activate(uri_variant)
+
+        # Assert that do_show_dataset_details_by_uri was called once
+        mock_do_show_dataset_details_by_uri.assert_called_once_with(show_dataset_by_uri_action, uri_variant)
\ No newline at end of file

From 8e1bd13b2b3299355401454af85ad945a6a53d46 Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Sun, 3 Dec 2023 18:14:19 +0100
Subject: [PATCH 21/24]  Action Testing

---
 test/test_main_window_actions.py | 150 ++++++++++++++++++++++++++++++-
 1 file changed, 149 insertions(+), 1 deletion(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 1304d044..cf9178f0 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -191,4 +191,152 @@ async def test_do_show_dataset_details_by_uri_action_trigger(app):
         show_dataset_by_uri_action.activate(uri_variant)
 
         # Assert that do_show_dataset_details_by_uri was called once
-        mock_do_show_dataset_details_by_uri.assert_called_once_with(show_dataset_by_uri_action, uri_variant)
\ No newline at end of file
+        mock_do_show_dataset_details_by_uri.assert_called_once_with(show_dataset_by_uri_action, uri_variant)
+
+@pytest.mark.asyncio
+async def test_do_show_dataset_details_by_row_index_direct_call(app):
+    """Test the do_show_dataset_details_by_row_index method for showing dataset details by index directly."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Mock _show_dataset_details_by_row_index to simulate behavior
+    app.main_window._show_dataset_details_by_row_index = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_uint32(1)  # Example index
+
+    # Directly call the method with mock objects
+    app.main_window.do_show_dataset_details_by_row_index(mock_action, mock_variant)
+
+    # Assert that _show_dataset_details_by_row_index was called with the correct index
+    app.main_window._show_dataset_details_by_row_index.assert_called_once_with(1)
+
+
+@pytest.mark.asyncio
+async def test_do_show_dataset_details_by_row_index_action_trigger(app):
+    """Test if 'show-dataset' action triggers do_show_dataset_details_by_row_index method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._show_dataset_details_by_row_index = MagicMock()
+
+    # Create and add the action
+    row_index_variant = GLib.Variant.new_uint32(1)  # Example index
+    show_dataset_action = Gio.SimpleAction.new("show-dataset", row_index_variant.get_type())
+    app.main_window.add_action(show_dataset_action)
+
+    # Patch do_show_dataset_details_by_row_index method after action is added
+    with patch.object(app.main_window, 'do_show_dataset_details_by_row_index', new_callable=MagicMock) as mock_do_show_dataset_details_by_row_index:
+        # Connect the action
+        show_dataset_action.connect("activate", app.main_window.do_show_dataset_details_by_row_index)
+
+        # Trigger the action
+        show_dataset_action.activate(row_index_variant)
+
+        # Assert that do_show_dataset_details_by_row_index was called once
+        mock_do_show_dataset_details_by_row_index.assert_called_once_with(show_dataset_action, row_index_variant)
+
+
+@pytest.mark.asyncio
+async def test_do_select_dataset_row_by_row_index_direct_call(app):
+    """Test the do_select_dataset_row_by_row_index method for selecting a dataset row directly."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Mock _select_dataset_row_by_row_index to simulate behavior
+    app.main_window._select_dataset_row_by_row_index = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_uint32(1)  # Example index
+
+    # Directly call the method with mock objects
+    app.main_window.do_select_dataset_row_by_row_index(mock_action, mock_variant)
+
+    # Assert that _select_dataset_row_by_row_index was called with the correct index
+    app.main_window._select_dataset_row_by_row_index.assert_called_once_with(1)
+
+
+@pytest.mark.asyncio
+async def test_do_select_dataset_row_by_row_index_action_trigger(app):
+    """Test if 'select-dataset' action triggers do_select_dataset_row_by_row_index method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._select_dataset_row_by_row_index = MagicMock()
+
+    # Create and add the action
+    row_index_variant = GLib.Variant.new_uint32(1)  # Example index
+    select_dataset_action = Gio.SimpleAction.new("select-dataset", row_index_variant.get_type())
+    app.main_window.add_action(select_dataset_action)
+
+    # Patch do_select_dataset_row_by_row_index method after action is added
+    with patch.object(app.main_window, 'do_select_dataset_row_by_row_index', new_callable=MagicMock) as mock_do_select_dataset_row_by_row_index:
+        # Connect the action
+        select_dataset_action.connect("activate", app.main_window.do_select_dataset_row_by_row_index)
+
+        # Trigger the action
+        select_dataset_action.activate(row_index_variant)
+
+        # Assert that do_select_dataset_row_by_row_index was called once
+        mock_do_select_dataset_row_by_row_index.assert_called_once_with(select_dataset_action, row_index_variant)
+
+@pytest.mark.asyncio
+async def test_do_search_direct_call(app):
+    """Test the do_search method for initiating a search directly."""
+
+    # Mock dependencies
+    mock_base_uri_list_box = MagicMock()
+    app.main_window.base_uri_list_box = mock_base_uri_list_box
+
+    # Mock _search to simulate behavior
+    app.main_window._search = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_string("test_search_query")
+
+    # Directly call the method with mock objects
+    app.main_window.do_search(mock_action, mock_variant)
+
+    # Assert that _search was called with the correct search text
+    app.main_window._search.assert_called_once_with("test_search_query")
+
+
+@pytest.mark.asyncio
+async def test_do_search_action_trigger(app):
+    """Test if 'search' action triggers do_search method."""
+
+    # Mock dependencies
+    mock_base_uri_list_box = MagicMock()
+    app.main_window.base_uri_list_box = mock_base_uri_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._search = MagicMock()
+
+    # Create and add the action
+    search_text_variant = GLib.Variant.new_string("test_search_query")
+    search_action = Gio.SimpleAction.new("search", search_text_variant.get_type())
+    app.main_window.add_action(search_action)
+
+    # Patch do_search method after action is added
+    with patch.object(app.main_window, 'do_search', new_callable=MagicMock) as mock_do_search:
+        # Connect the action
+        search_action.connect("activate", app.main_window.do_search)
+
+        # Trigger the action
+        search_action.activate(search_text_variant)
+
+        # Assert that do_search was called once
+        mock_do_search.assert_called_once_with(search_action, search_text_variant)

From ebdd23ac3b4474765e1458f960ef2960bc398a9b Mon Sep 17 00:00:00 2001
From: ashdriod <ashwinvazhappilly@gmail.com>
Date: Mon, 4 Dec 2023 01:03:13 +0100
Subject: [PATCH 22/24] ENH:TST

---
 test/test_main_window_actions.py | 690 ++++++++++++++++---------------
 1 file changed, 348 insertions(+), 342 deletions(-)

diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index cf9178f0..03a6f5c0 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -1,342 +1,348 @@
-# 1. Direct calls in tests isolate specific functionalities for focused, independent testing of components.
-# 2. Triggering actions and using mocks are crucial for integration testing, ensuring components interact correctly.
-# 3. Mocking and action triggering simulate real-world user interactions and application responses.
-# 4. Separating tests for direct calls and action triggers aids in maintaining clear, organized test structures.
-# 5. This approach enhances test suite readability and makes it easier to understand and update.
-
-
-
-import pytest
-from unittest.mock import patch, MagicMock, AsyncMock
-from gi.repository import Gtk, Gio, GLib
-
-
-
-@pytest.mark.asyncio
-async def test_do_refresh_view_direct_call(app):
-    """Test the direct call of the do_refresh_view method."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-
-    # Directly call the do_refresh_view method
-    app.main_window.do_refresh_view(None, None)
-
-@pytest.mark.asyncio
-async def test_refresh_method_triggered_by_action(app):
-    """Test if the 'refresh-view' action triggers the refresh method."""
-
-    # Patch the main window's refresh method
-    with patch.object(app.main_window, 'refresh', new_callable=MagicMock) as mock_refresh:
-        # Trigger the 'refresh-view' action
-        app.main_window.activate_action('refresh-view')
-
-        # Assert that the refresh method was called once
-        mock_refresh.assert_called_once()
-
-
-@pytest.mark.asyncio
-async def test_do_get_item_direct_call(app):
-    """Test the do_get_item method for copying a selected item directly."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-    mock_settings = MagicMock()
-    app.main_window.settings = mock_settings
-
-    # Mock _get_selected_items to return one item
-    app.main_window._get_selected_items = MagicMock(return_value=[('item_name', 'item_uuid')])
-
-    # Create a mock action and variant
-    mock_action = MagicMock(spec=Gio.SimpleAction)
-    mock_variant = GLib.Variant.new_string("dummy_path")
-
-    # Mock async call in do_get_item
-    mock_dataset = MagicMock()
-    mock_dataset.get_item = AsyncMock()
-    mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
-
-    # Directly call the method with mock objects
-    app.main_window.do_get_item(mock_action, mock_variant)
-
-
-@pytest.mark.asyncio
-async def test_do_get_item_action_trigger(app):
-    """Test if 'get-item' action triggers do_get_item method."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-    mock_settings = MagicMock()
-    app.main_window.settings = mock_settings
-
-    # Setup necessary mocks for the action trigger
-    mock_dataset = MagicMock()
-    mock_dataset.get_item = AsyncMock()
-    mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
-    app.main_window._get_selected_items = MagicMock(return_value=[('item_name', 'item_uuid')])
-
-    # Create and add the action
-    dest_file_variant = GLib.Variant.new_string("dummy_path")
-    get_item_action = Gio.SimpleAction.new("get-item", dest_file_variant.get_type())
-    app.main_window.add_action(get_item_action)
-
-    # Patch do_get_item method after action is added
-    with patch.object(app.main_window, 'do_get_item', new_callable=MagicMock) as mock_do_get_item:
-        # Connect the action
-        get_item_action.connect("activate", app.main_window.do_get_item)
-
-        # Trigger the action
-        get_item_action.activate(dest_file_variant)
-
-        # Assert that do_get_item was called once
-        mock_do_get_item.assert_called_once_with(get_item_action, dest_file_variant)
-
-
-@pytest.mark.asyncio
-async def test_do_search_select_and_show_direct_call(app):
-    """Test the do_search_select_and_show method for processing a search directly."""
-
-    # Mock dependencies
-    mock_base_uri_list_box = MagicMock()
-    app.main_window.base_uri_list_box = mock_base_uri_list_box
-
-    # Mock _search_select_and_show to simulate search behavior
-    app.main_window._search_select_and_show = MagicMock()
-
-    # Create a mock action and variant
-    mock_action = MagicMock(spec=Gio.SimpleAction)
-    mock_variant = GLib.Variant.new_string("test_search_query")
-
-    # Directly call the method with mock objects
-    app.main_window.do_search_select_and_show(mock_action, mock_variant)
-
-    # Assert that _search_select_and_show was called with the correct query
-    app.main_window._search_select_and_show.assert_called_once_with("test_search_query")
-
-
-@pytest.mark.asyncio
-async def test_do_search_select_and_show_action_trigger(app):
-    """Test if 'search-select-show' action triggers do_search_select_and_show method."""
-
-    # Mock dependencies
-    mock_base_uri_list_box = MagicMock()
-    app.main_window.base_uri_list_box = mock_base_uri_list_box
-
-    # Setup necessary mocks for the action trigger
-    app.main_window._search_select_and_show = MagicMock()
-
-    # Create and add the action
-    search_variant = GLib.Variant.new_string("test_search_query")
-    search_select_show_action = Gio.SimpleAction.new("search-select-show", search_variant.get_type())
-    app.main_window.add_action(search_select_show_action)
-
-    # Patch do_search_select_and_show method after action is added
-    with patch.object(app.main_window, 'do_search_select_and_show', new_callable=MagicMock) as mock_do_search_select_and_show:
-        # Connect the action
-        search_select_show_action.connect("activate", app.main_window.do_search_select_and_show)
-
-        # Trigger the action
-        search_select_show_action.activate(search_variant)
-
-        # Assert that do_search_select_and_show was called once
-        mock_do_search_select_and_show.assert_called_once_with(search_select_show_action, search_variant)
-
-@pytest.mark.asyncio
-async def test_do_show_dataset_details_by_uri_direct_call(app):
-    """Test the do_show_dataset_details_by_uri method for showing dataset details directly."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-
-    # Mock _show_dataset_details_by_uri to simulate behavior
-    app.main_window._show_dataset_details_by_uri = MagicMock()
-
-    # Create a mock action and variant
-    mock_action = MagicMock(spec=Gio.SimpleAction)
-    mock_variant = GLib.Variant.new_string("dummy_uri")
-
-    # Directly call the method with mock objects
-    app.main_window.do_show_dataset_details_by_uri(mock_action, mock_variant)
-
-    # Assert that _show_dataset_details_by_uri was called with the correct uri
-    app.main_window._show_dataset_details_by_uri.assert_called_once_with("dummy_uri")
-
-
-@pytest.mark.asyncio
-async def test_do_show_dataset_details_by_uri_action_trigger(app):
-    """Test if 'show-dataset-by-uri' action triggers do_show_dataset_details_by_uri method."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-
-    # Setup necessary mocks for the action trigger
-    app.main_window._show_dataset_details_by_uri = MagicMock()
-
-    # Create and add the action
-    uri_variant = GLib.Variant.new_string("dummy_uri")
-    show_dataset_by_uri_action = Gio.SimpleAction.new("show-dataset-by-uri", uri_variant.get_type())
-    app.main_window.add_action(show_dataset_by_uri_action)
-
-    # Patch do_show_dataset_details_by_uri method after action is added
-    with patch.object(app.main_window, 'do_show_dataset_details_by_uri', new_callable=MagicMock) as mock_do_show_dataset_details_by_uri:
-        # Connect the action
-        show_dataset_by_uri_action.connect("activate", app.main_window.do_show_dataset_details_by_uri)
-
-        # Trigger the action
-        show_dataset_by_uri_action.activate(uri_variant)
-
-        # Assert that do_show_dataset_details_by_uri was called once
-        mock_do_show_dataset_details_by_uri.assert_called_once_with(show_dataset_by_uri_action, uri_variant)
-
-@pytest.mark.asyncio
-async def test_do_show_dataset_details_by_row_index_direct_call(app):
-    """Test the do_show_dataset_details_by_row_index method for showing dataset details by index directly."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-
-    # Mock _show_dataset_details_by_row_index to simulate behavior
-    app.main_window._show_dataset_details_by_row_index = MagicMock()
-
-    # Create a mock action and variant
-    mock_action = MagicMock(spec=Gio.SimpleAction)
-    mock_variant = GLib.Variant.new_uint32(1)  # Example index
-
-    # Directly call the method with mock objects
-    app.main_window.do_show_dataset_details_by_row_index(mock_action, mock_variant)
-
-    # Assert that _show_dataset_details_by_row_index was called with the correct index
-    app.main_window._show_dataset_details_by_row_index.assert_called_once_with(1)
-
-
-@pytest.mark.asyncio
-async def test_do_show_dataset_details_by_row_index_action_trigger(app):
-    """Test if 'show-dataset' action triggers do_show_dataset_details_by_row_index method."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-
-    # Setup necessary mocks for the action trigger
-    app.main_window._show_dataset_details_by_row_index = MagicMock()
-
-    # Create and add the action
-    row_index_variant = GLib.Variant.new_uint32(1)  # Example index
-    show_dataset_action = Gio.SimpleAction.new("show-dataset", row_index_variant.get_type())
-    app.main_window.add_action(show_dataset_action)
-
-    # Patch do_show_dataset_details_by_row_index method after action is added
-    with patch.object(app.main_window, 'do_show_dataset_details_by_row_index', new_callable=MagicMock) as mock_do_show_dataset_details_by_row_index:
-        # Connect the action
-        show_dataset_action.connect("activate", app.main_window.do_show_dataset_details_by_row_index)
-
-        # Trigger the action
-        show_dataset_action.activate(row_index_variant)
-
-        # Assert that do_show_dataset_details_by_row_index was called once
-        mock_do_show_dataset_details_by_row_index.assert_called_once_with(show_dataset_action, row_index_variant)
-
-
-@pytest.mark.asyncio
-async def test_do_select_dataset_row_by_row_index_direct_call(app):
-    """Test the do_select_dataset_row_by_row_index method for selecting a dataset row directly."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-
-    # Mock _select_dataset_row_by_row_index to simulate behavior
-    app.main_window._select_dataset_row_by_row_index = MagicMock()
-
-    # Create a mock action and variant
-    mock_action = MagicMock(spec=Gio.SimpleAction)
-    mock_variant = GLib.Variant.new_uint32(1)  # Example index
-
-    # Directly call the method with mock objects
-    app.main_window.do_select_dataset_row_by_row_index(mock_action, mock_variant)
-
-    # Assert that _select_dataset_row_by_row_index was called with the correct index
-    app.main_window._select_dataset_row_by_row_index.assert_called_once_with(1)
-
-
-@pytest.mark.asyncio
-async def test_do_select_dataset_row_by_row_index_action_trigger(app):
-    """Test if 'select-dataset' action triggers do_select_dataset_row_by_row_index method."""
-
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-
-    # Setup necessary mocks for the action trigger
-    app.main_window._select_dataset_row_by_row_index = MagicMock()
-
-    # Create and add the action
-    row_index_variant = GLib.Variant.new_uint32(1)  # Example index
-    select_dataset_action = Gio.SimpleAction.new("select-dataset", row_index_variant.get_type())
-    app.main_window.add_action(select_dataset_action)
-
-    # Patch do_select_dataset_row_by_row_index method after action is added
-    with patch.object(app.main_window, 'do_select_dataset_row_by_row_index', new_callable=MagicMock) as mock_do_select_dataset_row_by_row_index:
-        # Connect the action
-        select_dataset_action.connect("activate", app.main_window.do_select_dataset_row_by_row_index)
-
-        # Trigger the action
-        select_dataset_action.activate(row_index_variant)
-
-        # Assert that do_select_dataset_row_by_row_index was called once
-        mock_do_select_dataset_row_by_row_index.assert_called_once_with(select_dataset_action, row_index_variant)
-
-@pytest.mark.asyncio
-async def test_do_search_direct_call(app):
-    """Test the do_search method for initiating a search directly."""
-
-    # Mock dependencies
-    mock_base_uri_list_box = MagicMock()
-    app.main_window.base_uri_list_box = mock_base_uri_list_box
-
-    # Mock _search to simulate behavior
-    app.main_window._search = MagicMock()
-
-    # Create a mock action and variant
-    mock_action = MagicMock(spec=Gio.SimpleAction)
-    mock_variant = GLib.Variant.new_string("test_search_query")
-
-    # Directly call the method with mock objects
-    app.main_window.do_search(mock_action, mock_variant)
-
-    # Assert that _search was called with the correct search text
-    app.main_window._search.assert_called_once_with("test_search_query")
-
-
-@pytest.mark.asyncio
-async def test_do_search_action_trigger(app):
-    """Test if 'search' action triggers do_search method."""
-
-    # Mock dependencies
-    mock_base_uri_list_box = MagicMock()
-    app.main_window.base_uri_list_box = mock_base_uri_list_box
-
-    # Setup necessary mocks for the action trigger
-    app.main_window._search = MagicMock()
-
-    # Create and add the action
-    search_text_variant = GLib.Variant.new_string("test_search_query")
-    search_action = Gio.SimpleAction.new("search", search_text_variant.get_type())
-    app.main_window.add_action(search_action)
-
-    # Patch do_search method after action is added
-    with patch.object(app.main_window, 'do_search', new_callable=MagicMock) as mock_do_search:
-        # Connect the action
-        search_action.connect("activate", app.main_window.do_search)
-
-        # Trigger the action
-        search_action.activate(search_text_variant)
-
-        # Assert that do_search was called once
-        mock_do_search.assert_called_once_with(search_action, search_text_variant)
+# 1. Direct calls in tests isolate specific functionalities for focused, independent testing of components.
+# 2. Triggering actions and using mocks are crucial for integration testing, ensuring components interact correctly.
+# 3. Mocking and action triggering simulate real-world user interactions and application responses.
+# 4. Separating tests for direct calls and action triggers aids in maintaining clear, organized test structures.
+# 5. This approach enhances test suite readability and makes it easier to understand and update.
+
+
+import pytest
+from unittest.mock import patch, MagicMock, AsyncMock
+from gi.repository import Gtk, Gio, GLib
+
+
+@pytest.mark.asyncio
+async def test_do_refresh_view_direct_call(app):
+    """Test the direct call of the do_refresh_view method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Directly call the do_refresh_view method
+    app.main_window.do_refresh_view(None, None)
+
+
+@pytest.mark.asyncio
+async def test_refresh_method_triggered_by_action(app):
+    """Test if the 'refresh-view' action triggers the refresh method."""
+
+    # Patch the main window's refresh method
+    with patch.object(app.main_window, 'refresh', new_callable=MagicMock) as mock_refresh:
+        # Trigger the 'refresh-view' action
+        app.main_window.activate_action('refresh-view')
+
+        # Assert that the refresh method was called once
+        mock_refresh.assert_called_once()
+
+
+@pytest.mark.asyncio
+async def test_do_get_item_direct_call(app):
+    """Test the do_get_item method for copying a selected item directly."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+    mock_settings = MagicMock()
+    app.main_window.settings = mock_settings
+
+    # Mock _get_selected_items to return one item
+    app.main_window._get_selected_items = MagicMock(return_value=[('item_name', 'item_uuid')])
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_string("dummy_path")
+
+    # Mock async call in do_get_item
+    mock_dataset = MagicMock()
+    mock_dataset.get_item = AsyncMock()
+    mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
+
+    # Directly call the method with mock objects
+    app.main_window.do_get_item(mock_action, mock_variant)
+
+
+@pytest.mark.asyncio
+async def test_do_get_item_action_trigger(app):
+    """Test if 'get-item' action triggers do_get_item method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+    mock_settings = MagicMock()
+    app.main_window.settings = mock_settings
+
+    # Setup necessary mocks for the action trigger
+    mock_dataset = MagicMock()
+    mock_dataset.get_item = AsyncMock()
+    mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
+    app.main_window._get_selected_items = MagicMock(return_value=[('item_name', 'item_uuid')])
+
+    # Create and add the action
+    dest_file_variant = GLib.Variant.new_string("dummy_path")
+    get_item_action = Gio.SimpleAction.new("get-item", dest_file_variant.get_type())
+    app.main_window.add_action(get_item_action)
+
+    # Patch do_get_item method after action is added
+    with patch.object(app.main_window, 'do_get_item', new_callable=MagicMock) as mock_do_get_item:
+        # Connect the action
+        get_item_action.connect("activate", app.main_window.do_get_item)
+
+        # Trigger the action
+        get_item_action.activate(dest_file_variant)
+
+        # Assert that do_get_item was called once
+        mock_do_get_item.assert_called_once_with(get_item_action, dest_file_variant)
+
+
+@pytest.mark.asyncio
+async def test_do_search_select_and_show_direct_call(app):
+    """Test the do_search_select_and_show method for processing a search directly."""
+
+    # Mock dependencies
+    mock_base_uri_list_box = MagicMock()
+    app.main_window.base_uri_list_box = mock_base_uri_list_box
+
+    # Mock _search_select_and_show to simulate search behavior
+    app.main_window._search_select_and_show = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_string("test_search_query")
+
+    # Directly call the method with mock objects
+    app.main_window.do_search_select_and_show(mock_action, mock_variant)
+
+    # Assert that _search_select_and_show was called with the correct query
+    app.main_window._search_select_and_show.assert_called_once_with("test_search_query")
+
+
+@pytest.mark.asyncio
+async def test_do_search_select_and_show_action_trigger(app):
+    """Test if 'search-select-show' action triggers do_search_select_and_show method."""
+
+    # Mock dependencies
+    mock_base_uri_list_box = MagicMock()
+    app.main_window.base_uri_list_box = mock_base_uri_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._search_select_and_show = MagicMock()
+
+    # Create and add the action
+    search_variant = GLib.Variant.new_string("test_search_query")
+    search_select_show_action = Gio.SimpleAction.new("search-select-show", search_variant.get_type())
+    app.main_window.add_action(search_select_show_action)
+
+    # Patch do_search_select_and_show method after action is added
+    with patch.object(app.main_window, 'do_search_select_and_show',
+                      new_callable=MagicMock) as mock_do_search_select_and_show:
+        # Connect the action
+        search_select_show_action.connect("activate", app.main_window.do_search_select_and_show)
+
+        # Trigger the action
+        search_select_show_action.activate(search_variant)
+
+        # Assert that do_search_select_and_show was called once
+        mock_do_search_select_and_show.assert_called_once_with(search_select_show_action, search_variant)
+
+
+@pytest.mark.asyncio
+async def test_do_show_dataset_details_by_uri_direct_call(app):
+    """Test the do_show_dataset_details_by_uri method for showing dataset details directly."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Mock _show_dataset_details_by_uri to simulate behavior
+    app.main_window._show_dataset_details_by_uri = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_string("dummy_uri")
+
+    # Directly call the method with mock objects
+    app.main_window.do_show_dataset_details_by_uri(mock_action, mock_variant)
+
+    # Assert that _show_dataset_details_by_uri was called with the correct uri
+    app.main_window._show_dataset_details_by_uri.assert_called_once_with("dummy_uri")
+
+
+@pytest.mark.asyncio
+async def test_do_show_dataset_details_by_uri_action_trigger(app):
+    """Test if 'show-dataset-by-uri' action triggers do_show_dataset_details_by_uri method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._show_dataset_details_by_uri = MagicMock()
+
+    # Create and add the action
+    uri_variant = GLib.Variant.new_string("dummy_uri")
+    show_dataset_by_uri_action = Gio.SimpleAction.new("show-dataset-by-uri", uri_variant.get_type())
+    app.main_window.add_action(show_dataset_by_uri_action)
+
+    # Patch do_show_dataset_details_by_uri method after action is added
+    with patch.object(app.main_window, 'do_show_dataset_details_by_uri',
+                      new_callable=MagicMock) as mock_do_show_dataset_details_by_uri:
+        # Connect the action
+        show_dataset_by_uri_action.connect("activate", app.main_window.do_show_dataset_details_by_uri)
+
+        # Trigger the action
+        show_dataset_by_uri_action.activate(uri_variant)
+
+        # Assert that do_show_dataset_details_by_uri was called once
+        mock_do_show_dataset_details_by_uri.assert_called_once_with(show_dataset_by_uri_action, uri_variant)
+
+
+@pytest.mark.asyncio
+async def test_do_show_dataset_details_by_row_index_direct_call(app):
+    """Test the do_show_dataset_details_by_row_index method for showing dataset details by index directly."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Mock _show_dataset_details_by_row_index to simulate behavior
+    app.main_window._show_dataset_details_by_row_index = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_uint32(1)  # Example index
+
+    # Directly call the method with mock objects
+    app.main_window.do_show_dataset_details_by_row_index(mock_action, mock_variant)
+
+    # Assert that _show_dataset_details_by_row_index was called with the correct index
+    app.main_window._show_dataset_details_by_row_index.assert_called_once_with(1)
+
+
+@pytest.mark.asyncio
+async def test_do_show_dataset_details_by_row_index_action_trigger(app):
+    """Test if 'show-dataset' action triggers do_show_dataset_details_by_row_index method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._show_dataset_details_by_row_index = MagicMock()
+
+    # Create and add the action
+    row_index_variant = GLib.Variant.new_uint32(1)  # Example index
+    show_dataset_action = Gio.SimpleAction.new("show-dataset", row_index_variant.get_type())
+    app.main_window.add_action(show_dataset_action)
+
+    # Patch do_show_dataset_details_by_row_index method after action is added
+    with patch.object(app.main_window, 'do_show_dataset_details_by_row_index',
+                      new_callable=MagicMock) as mock_do_show_dataset_details_by_row_index:
+        # Connect the action
+        show_dataset_action.connect("activate", app.main_window.do_show_dataset_details_by_row_index)
+
+        # Trigger the action
+        show_dataset_action.activate(row_index_variant)
+
+        # Assert that do_show_dataset_details_by_row_index was called once
+        mock_do_show_dataset_details_by_row_index.assert_called_once_with(show_dataset_action, row_index_variant)
+
+
+@pytest.mark.asyncio
+async def test_do_select_dataset_row_by_row_index_direct_call(app):
+    """Test the do_select_dataset_row_by_row_index method for selecting a dataset row directly."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Mock _select_dataset_row_by_row_index to simulate behavior
+    app.main_window._select_dataset_row_by_row_index = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_uint32(1)  # Example index
+
+    # Directly call the method with mock objects
+    app.main_window.do_select_dataset_row_by_row_index(mock_action, mock_variant)
+
+    # Assert that _select_dataset_row_by_row_index was called with the correct index
+    app.main_window._select_dataset_row_by_row_index.assert_called_once_with(1)
+
+
+@pytest.mark.asyncio
+async def test_do_select_dataset_row_by_row_index_action_trigger(app):
+    """Test if 'select-dataset' action triggers do_select_dataset_row_by_row_index method."""
+
+    # Mock dependencies
+    mock_dataset_list_box = MagicMock()
+    app.main_window.dataset_list_box = mock_dataset_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._select_dataset_row_by_row_index = MagicMock()
+
+    # Create and add the action
+    row_index_variant = GLib.Variant.new_uint32(1)  # Example index
+    select_dataset_action = Gio.SimpleAction.new("select-dataset", row_index_variant.get_type())
+    app.main_window.add_action(select_dataset_action)
+
+    # Patch do_select_dataset_row_by_row_index method after action is added
+    with patch.object(app.main_window, 'do_select_dataset_row_by_row_index',
+                      new_callable=MagicMock) as mock_do_select_dataset_row_by_row_index:
+        # Connect the action
+        select_dataset_action.connect("activate", app.main_window.do_select_dataset_row_by_row_index)
+
+        # Trigger the action
+        select_dataset_action.activate(row_index_variant)
+
+        # Assert that do_select_dataset_row_by_row_index was called once
+        mock_do_select_dataset_row_by_row_index.assert_called_once_with(select_dataset_action, row_index_variant)
+
+
+@pytest.mark.asyncio
+async def test_do_search_direct_call(app):
+    """Test the do_search method for initiating a search directly."""
+
+    # Mock dependencies
+    mock_base_uri_list_box = MagicMock()
+    app.main_window.base_uri_list_box = mock_base_uri_list_box
+
+    # Mock _search to simulate behavior
+    app.main_window._search = MagicMock()
+
+    # Create a mock action and variant
+    mock_action = MagicMock(spec=Gio.SimpleAction)
+    mock_variant = GLib.Variant.new_string("test_search_query")
+
+    # Directly call the method with mock objects
+    app.main_window.do_search(mock_action, mock_variant)
+
+    # Assert that _search was called with the correct search text
+    app.main_window._search.assert_called_once_with("test_search_query")
+
+
+@pytest.mark.asyncio
+async def test_do_search_action_trigger(app):
+    """Test if 'search' action triggers do_search method."""
+
+    # Mock dependencies
+    mock_base_uri_list_box = MagicMock()
+    app.main_window.base_uri_list_box = mock_base_uri_list_box
+
+    # Setup necessary mocks for the action trigger
+    app.main_window._search = MagicMock()
+
+    # Create and add the action
+    search_text_variant = GLib.Variant.new_string("test_search_query")
+    search_action = Gio.SimpleAction.new("search", search_text_variant.get_type())
+    app.main_window.add_action(search_action)
+
+    # Patch do_search method after action is added
+    with patch.object(app.main_window, 'do_search', new_callable=MagicMock) as mock_do_search:
+        # Connect the action
+        search_action.connect("activate", app.main_window.do_search)
+
+        # Trigger the action
+        search_action.activate(search_text_variant)
+
+        # Assert that do_search was called once
+        mock_do_search.assert_called_once_with(search_action, search_text_variant)

From 68c037bb5e0482b438cf973406ccd569581aff87 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 4 Dec 2023 14:00:11 +0100
Subject: [PATCH 23/24] MAINT: pagination information display more resilient

---
 dtool_lookup_gui/views/main_window.py       |  20 ++--
 test/data/mock_config_info_response.json    |   0
 test/data/mock_dataset_search_response.json | 112 ++++++++++++++++++++
 3 files changed, 126 insertions(+), 6 deletions(-)
 create mode 100644 test/data/mock_config_info_response.json
 create mode 100644 test/data/mock_dataset_search_response.json

diff --git a/dtool_lookup_gui/views/main_window.py b/dtool_lookup_gui/views/main_window.py
index 40da0f47..0484b744 100644
--- a/dtool_lookup_gui/views/main_window.py
+++ b/dtool_lookup_gui/views/main_window.py
@@ -352,15 +352,23 @@ async def _refresh_base_uri_list_box(self):
     def _update_search_summary(self, datasets):
         row = self.base_uri_list_box.search_results_row
         total_size = sum([0 if dataset.size_int is None else dataset.size_int for dataset in datasets])
-        total_value = self.pagination['total']
+        if hasattr(self.pagination, 'total'):
+            total_value = self.pagination['total']
+        else:
+            _logger.warning("No pagination information available.")
+            total_value = len(datasets)
+
         row.info_label.set_text(f'{total_value} datasets, {sizeof_fmt(total_size).strip()}')
 
     def _update_main_statusbar(self):
-        total_number = self.pagination['total']
-        current_page = self.pagination['page']
-        last_page = self.pagination['last_page']
-        self.main_statusbar.push(0,
-                                 f"Total Number of Datasets: {total_number}, Current Page: {current_page} of {last_page}")
+        if hasattr(self.pagination, 'total') and hasattr(self.pagination, 'page') and hasattr(self.pagination, 'last_page'):
+            total_number = self.pagination['total']
+            current_page = self.pagination['page']
+            last_page = self.pagination['last_page']
+            self.main_statusbar.push(0,
+                                     f"Total Number of Datasets: {total_number}, Current Page: {current_page} of {last_page}")
+        else:
+            _logger.warning("No pagination information available to display in status bar.")
 
     def contents_per_page_changed(self, widget):
         self.contents_per_page_value = widget.get_active_text()
diff --git a/test/data/mock_config_info_response.json b/test/data/mock_config_info_response.json
new file mode 100644
index 00000000..e69de29b
diff --git a/test/data/mock_dataset_search_response.json b/test/data/mock_dataset_search_response.json
new file mode 100644
index 00000000..01058946
--- /dev/null
+++ b/test/data/mock_dataset_search_response.json
@@ -0,0 +1,112 @@
+[
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1604860720.736269,
+    "creator_username": "jotelha",
+    "frozen_at": 1604864525.691079,
+    "name": "simple_test_dataset",
+    "number_of_items": 1,
+    "size_in_bytes": 17,
+    "uri": "smb://test-share/1a1f9fad-8589-413e-9602-5bbd66bfe675",
+    "uuid": "1a1f9fad-8589-413e-9602-5bbd66bfe675"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.587261,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.768132,
+    "name": "2021-02-25-03-53-32-174712-probeonsubst--econversionminimizationandequilibration",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/51e67a6e-3f38-43e9-bb79-01700dc09c61",
+    "uuid": "51e67a6e-3f38-43e9-bb79-01700dc09c61"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.769833,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.780084,
+    "name": "2021-02-25-03-53-23-833298-probeonsubst--econversionminimizationandequilibration",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/01211ad2-45ee-42f3-bc82-b24725462605",
+    "uuid": "01211ad2-45ee-42f3-bc82-b24725462605"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.781018,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.791484,
+    "name": "2020-12-15-12-53-46-528135-c-075-n-169-m-169-s-monolayer-substratepassivation",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/8aace7f4-e3c5-4030-a3c4-5c6cd085ee8d",
+    "uuid": "8aace7f4-e3c5-4030-a3c4-5c6cd085ee8d"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.791886,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.802498,
+    "name": "2021-02-25-03-53-23-833991-probeonsubst--econversionminimizationandequilibration",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/77bcca75-e5b0-419e-85cc-ebd5a206870e",
+    "uuid": "77bcca75-e5b0-419e-85cc-ebd5a206870e"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.802969,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.813227,
+    "name": "2021-02-25-03-53-19-751578-probeonsubst--econversionminimizationandequilibration",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/10516e27-9655-44ec-b1af-c03d69478fb6",
+    "uuid": "10516e27-9655-44ec-b1af-c03d69478fb6"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.813718,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.824184,
+    "name": "2021-02-25-02-41-42-359804-probeonsubst--econversionminimizationandequilibration",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/d99ce5bc-5b0a-45b1-9a41-477b332928ce",
+    "uuid": "d99ce5bc-5b0a-45b1-9a41-477b332928ce"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.824581,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.834963,
+    "name": "2020-12-15-12-53-46-629591-c-15-n-338-m-338-s-hemicylinders-substratepassivation",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/fde8992d-464d-4260-9425-c918c4ed1291",
+    "uuid": "fde8992d-464d-4260-9425-c918c4ed1291"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.835389,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.845345,
+    "name": "2021-02-25-03-52-54-623082-probeonsubst--econversionminimizationandequilibration",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/910cf24a-9a48-43e2-8094-6c5db8045df4",
+    "uuid": "910cf24a-9a48-43e2-8094-6c5db8045df4"
+  },
+  {
+    "base_uri": "smb://test-share",
+    "created_at": 1681394343.845761,
+    "creator_username": "jotelha",
+    "frozen_at": 1681394343.856255,
+    "name": "2021-02-25-02-41-42-396760-probeonsubst--econversionminimizationandequilibration",
+    "number_of_items": 3,
+    "size_in_bytes": 12,
+    "uri": "smb://test-share/617abde9-54c7-40e2-b09f-d207dd00098a",
+    "uuid": "617abde9-54c7-40e2-b09f-d207dd00098a"
+  }
+]
\ No newline at end of file

From 161a8cac5ee5b6a33a38b7702d67944c54647226 Mon Sep 17 00:00:00 2001
From: Johannes Laurin Hoermann <johannes.hoermann@imtek.uni-freiburg.de>
Date: Mon, 4 Dec 2023 14:04:52 +0100
Subject: [PATCH 24/24] TST: on the way to proper testing against a populated
 app

---
 test/conftest.py                         |  65 +++
 test/data/mock_config_info_response.json | 569 +++++++++++++++++++++++
 test/test_main_window_actions.py         |  69 ++-
 3 files changed, 680 insertions(+), 23 deletions(-)

diff --git a/test/conftest.py b/test/conftest.py
index eca18aac..0428fea2 100644
--- a/test/conftest.py
+++ b/test/conftest.py
@@ -1,6 +1,10 @@
+import json
 import logging
+import os
 import pytest
 
+from unittest.mock import patch
+
 import gi
 gi.require_version('Gtk', '3.0')
 gi.require_version('GtkSource', '4')
@@ -10,6 +14,8 @@
 import gbulb
 gbulb.install(gtk=True)
 
+import dtool_lookup_api.core.LookupClient
+
 from dtool_lookup_gui.main import Application
 from dtool_lookup_gui.views.main_window import MainWindow
 
@@ -17,6 +23,9 @@
 logger = logging.getLogger(__name__)
 
 
+_HERE = os.path.dirname(os.path.abspath(__file__))
+# _ROOT = os.path.join(_HERE, "..")
+
 # ==========================================================================
 # fixtures from https://github.com/beeware/gbulb/blob/main/tests/conftest.py
 # ==========================================================================
@@ -78,6 +87,62 @@ def app(gtk_loop):
     app = Application(loop=gtk_loop)
     yield app
 
+@pytest.fixture(scope="function")
+def mock_token(scope="function"):
+    """Provide a mock authentication token"""
+    yield "eyJ0eXAiOiJKV1QiLCJhbGciOiJSUzI1NiJ9.eyJmcmVzaCI6ZmFsc2UsImlhdCI6MTcwMTY4OTU5MywianRpIjoiYTdkN2Y5ZWItZGI3MS00YjExLWFhMzktZGQ2YzgzOTJmOWE4IiwidHlwZSI6ImFjY2VzcyIsInN1YiI6InRlc3R1c2VyIiwibmJmIjoxNzAxNjg5NTkzLCJleHAiOjE3MDE2OTMxOTN9.t9SQ00ecZRc-pspz-Du7321xfWIzgTTFKobkNed1CuQYHvtNrc3vdHYbqCWaYCqZpEVF8RlltldT4Lookx6vNgnW4olpiS2KTZ-X2asMhn7SShDtUJuU54CGeViWzYX_V_Pzckoe_cgjFkOutRvnwy_072Whnmc0TwYojwNqUScAIJRu0pzym84JngloXfdI7r25GcRVNtzsGUl7DDfrIz4aSOeVDAVEXhPjgEatKsvNdVZl1DIJsTZpuI7Jh7ZW1WsyjonqHR0J0kIVQn9imQyLyS9_CtmURBQ3kabx6cxhpx5LADrzLutSu24eA4FyECOdzjJ3SPGb9nIVTEDxQg"
+
+
+@pytest.fixture(scope="function")
+def mock_dataset_list():
+    """Provide a mock dataset list"""
+    with open(os.path.join(_HERE, 'data', 'mock_dataset_search_response.json'), 'r') as f:
+        dataset_list = json.load(f)
+
+    yield dataset_list
+
+
+@pytest.fixture(scope="function")
+def mock_config_info():
+    """Provide a mock server config info"""
+    with open(os.path.join(_HERE, 'data', 'mock_config_info_response.json'), 'r') as f:
+        config_info = json.load(f)
+
+    yield config_info
+
+
+@pytest.fixture(scope="function")
+def app_without_authentication(app, mock_token):
+    """Removes authentication from app."""
+    with patch("dtool_lookup_api.core.LookupClient.authenticate", return_value=mock_token):
+        yield app
+
+
+@pytest.fixture(scope="function")
+def app_with_mock_dtool_lookup_api_calls(
+        app_without_authentication, mock_dataset_list, mock_config_info):
+    """Replaces lookup api calls with mock methods that return fake lists of datasets."""
+
+    # TODO: figure out whether mocked methods work as they should
+    with (
+            patch("dtool_lookup_api.core.LookupClient.TokenBasedLookupClient.all", return_value=mock_dataset_list),
+            patch("dtool_lookup_api.core.LookupClient.TokenBasedLookupClient.search", return_value=mock_dataset_list),
+            patch("dtool_lookup_api.core.LookupClient.TokenBasedLookupClient.config", return_value=mock_config_info),
+            patch("dtool_lookup_api.core.LookupClient.ConfigurationBasedLookupClient.has_valid_token", return_value=True)
+        ):
+
+        yield app_without_authentication
+
+
+@pytest.fixture(scope="function")
+def populated_app(app_with_mock_dtool_lookup_api_calls):
+    """Provides app populated with mock datasets."""
+
+    # TODO: figure out how to populate and refresh app with data here before app launched
+    # app.main_window.activate_action('refresh-view')
+
+    yield app_with_mock_dtool_lookup_api_calls
+
 
 # ================================================================
 # fixtures related to https://github.com/pytest-dev/pytest-asyncio
diff --git a/test/data/mock_config_info_response.json b/test/data/mock_config_info_response.json
index e69de29b..d9da01d2 100644
--- a/test/data/mock_config_info_response.json
+++ b/test/data/mock_config_info_response.json
@@ -0,0 +1,569 @@
+{
+  "allow_access_from": "0.0.0.0/0",
+  "allow_direct_aggregation": true,
+  "allow_direct_query": true,
+  "api_spec_options": {
+    "components": {
+      "headers": {
+        "PAGINATION": {
+          "description": "Pagination metadata",
+          "schema": {
+            "$ref": "#/components/schemas/PaginationMetadata"
+          }
+        }
+      },
+      "responses": {
+        "DEFAULT_ERROR": {
+          "content": {
+            "application/json": {
+              "schema": {
+                "$ref": "#/components/schemas/Error"
+              }
+            }
+          },
+          "description": "Default error response"
+        },
+        "UNPROCESSABLE_ENTITY": {
+          "content": {
+            "application/json": {
+              "schema": {
+                "$ref": "#/components/schemas/Error"
+              }
+            }
+          },
+          "description": "Unprocessable Entity"
+        }
+      },
+      "schemas": {
+        "BaseURI": {
+          "properties": {
+            "base_uri": {
+              "maxLength": 255,
+              "nullable": true,
+              "type": "string"
+            }
+          },
+          "type": "object"
+        },
+        "Dataset": {
+          "properties": {
+            "base_uri": {},
+            "created_at": {
+              "format": "date-time",
+              "type": "string"
+            },
+            "creator_username": {
+              "maxLength": 255,
+              "type": "string"
+            },
+            "frozen_at": {
+              "format": "date-time",
+              "type": "string"
+            },
+            "name": {
+              "maxLength": 80,
+              "type": "string"
+            },
+            "uri": {
+              "maxLength": 255,
+              "type": "string"
+            },
+            "uuid": {
+              "maxLength": 36,
+              "type": "string"
+            }
+          },
+          "required": [
+            "created_at",
+            "creator_username",
+            "frozen_at",
+            "name",
+            "uri",
+            "uuid"
+          ],
+          "type": "object"
+        },
+        "DependencyKeys": {
+          "properties": {
+            "dependency_keys": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            }
+          },
+          "type": "object"
+        },
+        "Error": {
+          "properties": {
+            "code": {
+              "description": "Error code",
+              "type": "integer"
+            },
+            "errors": {
+              "description": "Errors",
+              "type": "object"
+            },
+            "message": {
+              "description": "Error message",
+              "type": "string"
+            },
+            "status": {
+              "description": "Error name",
+              "type": "string"
+            }
+          },
+          "type": "object"
+        },
+        "Item": {
+          "properties": {
+            "hash": {
+              "type": "string"
+            },
+            "relpath": {
+              "type": "string"
+            },
+            "size_in_bytes": {
+              "type": "integer"
+            },
+            "utc_timestamp": {
+              "type": "number"
+            }
+          },
+          "type": "object"
+        },
+        "Manifest": {
+          "properties": {
+            "dtoolcore_version": {
+              "type": "string"
+            },
+            "hash_function": {
+              "type": "string"
+            },
+            "items": {
+              "additionalProperties": {
+                "$ref": "#/components/schemas/Item"
+              },
+              "type": "object"
+            }
+          },
+          "type": "object"
+        },
+        "PaginationMetadata": {
+          "properties": {
+            "first_page": {
+              "type": "integer"
+            },
+            "last_page": {
+              "type": "integer"
+            },
+            "next_page": {
+              "type": "integer"
+            },
+            "page": {
+              "type": "integer"
+            },
+            "previous_page": {
+              "type": "integer"
+            },
+            "total": {
+              "type": "integer"
+            },
+            "total_pages": {
+              "type": "integer"
+            }
+          },
+          "type": "object"
+        },
+        "QueryDataset": {
+          "properties": {
+            "aggregation": {
+              "items": {
+                "type": "object"
+              },
+              "type": "array"
+            },
+            "base_uris": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "creator_usernames": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "query": {
+              "type": "object"
+            },
+            "tags": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "uuids": {
+              "items": {
+                "format": "uuid",
+                "type": "string"
+              },
+              "type": "array"
+            }
+          },
+          "type": "object"
+        },
+        "RegisterDataset": {
+          "properties": {
+            "annotations": {
+              "type": "object"
+            },
+            "base_uri": {
+              "type": "string"
+            },
+            "created_at": {
+              "type": "string"
+            },
+            "creator_username": {
+              "type": "string"
+            },
+            "frozen_at": {
+              "type": "string"
+            },
+            "manifest": {
+              "$ref": "#/components/schemas/Manifest"
+            },
+            "name": {
+              "type": "string"
+            },
+            "number_of_items": {
+              "type": "integer"
+            },
+            "readme": {
+              "type": "string"
+            },
+            "size_in_bytes": {
+              "type": "integer"
+            },
+            "tags": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "type": {
+              "type": "string"
+            },
+            "uri": {
+              "type": "string"
+            },
+            "uuid": {
+              "format": "uuid",
+              "type": "string"
+            }
+          },
+          "type": "object"
+        },
+        "RegisterUser": {
+          "properties": {
+            "is_admin": {
+              "type": "boolean"
+            },
+            "username": {
+              "type": "string"
+            }
+          },
+          "type": "object"
+        },
+        "SearchDataset": {
+          "properties": {
+            "base_uris": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "creator_usernames": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "free_text": {
+              "type": "string"
+            },
+            "tags": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "uuids": {
+              "items": {
+                "format": "uuid",
+                "type": "string"
+              },
+              "type": "array"
+            }
+          },
+          "type": "object"
+        },
+        "Summary": {
+          "properties": {
+            "base_uris": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "creator_usernames": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "datasets_per_base_uri": {
+              "additionalProperties": {
+                "type": "integer"
+              },
+              "type": "object"
+            },
+            "datasets_per_creator": {
+              "additionalProperties": {
+                "type": "integer"
+              },
+              "type": "object"
+            },
+            "datasets_per_tag": {
+              "additionalProperties": {
+                "type": "integer"
+              },
+              "type": "object"
+            },
+            "number_of_datasets": {
+              "type": "integer"
+            },
+            "tags": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            }
+          },
+          "type": "object"
+        },
+        "URI": {
+          "properties": {
+            "uri": {
+              "type": "string"
+            }
+          },
+          "type": "object"
+        },
+        "URIPermission": {
+          "properties": {
+            "base_uri": {
+              "type": "string"
+            },
+            "users_with_register_permissions": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "users_with_search_permissions": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            }
+          },
+          "type": "object"
+        },
+        "User": {
+          "properties": {
+            "id": {
+              "type": "integer"
+            },
+            "is_admin": {
+              "type": "boolean"
+            },
+            "username": {
+              "maxLength": 64,
+              "nullable": true,
+              "type": "string"
+            }
+          },
+          "type": "object"
+        },
+        "UserResponse": {
+          "properties": {
+            "is_admin": {
+              "type": "boolean"
+            },
+            "register_permissions_on_base_uris": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "search_permissions_on_base_uris": {
+              "items": {
+                "type": "string"
+              },
+              "type": "array"
+            },
+            "username": {
+              "type": "string"
+            }
+          },
+          "type": "object"
+        }
+      },
+      "securitySchemes": {
+        "bearerAuth": {
+          "bearerFormat": "JWT",
+          "scheme": "bearer",
+          "type": "http"
+        }
+      }
+    },
+    "security": [
+      {
+        "bearerAuth": []
+      }
+    ],
+    "servers": [
+      {
+        "description": "dtool-lookup-server demo instance",
+        "url": "https://demo.dtool.dev/lookup"
+      }
+    ],
+    "x-internal-id": "2"
+  },
+  "api_title": "dtool-lookup-server API",
+  "api_version": "v1",
+  "application_root": "/",
+  "bucket_to_base_uri": {
+    "test-bucket": "s3://test-bucket"
+  },
+  "config_secrets_to_obfuscate": [
+    "JWT_PRIVATE_KEY",
+    "SECRET_KEY",
+    "SQLALCHEMY_DATABASE_URI",
+    "SEARCH_MONGO_URI",
+    "SEARCH_MONGO_DB",
+    "SEARCH_MONGO_COLLECTION",
+    "RETRIEVE_MONGO_URI",
+    "RETRIEVE_MONGO_DB",
+    "RETRIEVE_MONGO_COLLECTION",
+    "MONGO_URI",
+    "MONGO_DB",
+    "MONGO_COLLECTION"
+  ],
+  "cors_expose_headers": [
+    "X-Pagination"
+  ],
+  "debug": false,
+  "dependency_keys": [
+    "readme.derived_from.uuid",
+    "annotations.source_dataset_uuid"
+  ],
+  "dynamic_dependency_keys": true,
+  "enable_dependency_view": true,
+  "explain_template_loading": false,
+  "force_rebuild_dependency_view": false,
+  "jsonify_prettyprint_regular": true,
+  "jwt_access_cookie_name": "access_token_cookie",
+  "jwt_access_cookie_path": "/",
+  "jwt_access_csrf_cookie_name": "csrf_access_token",
+  "jwt_access_csrf_cookie_path": "/",
+  "jwt_access_csrf_field_name": "csrf_token",
+  "jwt_access_csrf_header_name": "X-CSRF-TOKEN",
+  "jwt_access_token_expires": "0:15:00",
+  "jwt_algorithm": "RS256",
+  "jwt_cookie_csrf_protect": true,
+  "jwt_cookie_domain": null,
+  "jwt_cookie_samesite": null,
+  "jwt_cookie_secure": false,
+  "jwt_csrf_check_form": false,
+  "jwt_csrf_in_cookies": true,
+  "jwt_csrf_methods": [
+    "POST",
+    "PUT",
+    "PATCH",
+    "DELETE"
+  ],
+  "jwt_decode_algorithms": null,
+  "jwt_decode_audience": null,
+  "jwt_decode_issuer": null,
+  "jwt_decode_leeway": 0,
+  "jwt_encode_audience": null,
+  "jwt_encode_issuer": null,
+  "jwt_encode_nbf": true,
+  "jwt_error_message_key": "msg",
+  "jwt_header_name": "Authorization",
+  "jwt_header_type": "Bearer",
+  "jwt_identity_claim": "sub",
+  "jwt_json_key": "access_token",
+  "jwt_private_key": "***",
+  "jwt_public_key": "-----BEGIN PUBLIC KEY-----\nMIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA2QZiOhF3IlUPmFeiNscN\nVhJNBEBBYpwK9qIuw8eEr0TGHTUiskp/MptbRB35OY1JRCw258o0OBGJO6seBuii\nZ7VOug+MM+KhUBql3WaboSWVGGevlCTh7SvKU9IVUh7BeRvsKAOYo7zJw29mxax8\nm6JnYtP3VdbLWh26RRzG1b7DimxSD7/GUvhal3ORI6oAJortsUFkMxM9cwxGV3ra\nQ5g/ZO55us+tFxZiVUBSE8rny8U7e/48d4kPgf9oRKkiptCe0xEk8g63WlRgNg5i\nORIveW7AQfxgnHI8jYYID+mpea/QllAEs8ydum0JtZKrtBolmBAk1Ua+zvGJC/Ay\ndQIDAQAB\n-----END PUBLIC KEY-----\n",
+  "jwt_query_string_name": "jwt",
+  "jwt_query_string_value_prefix": "",
+  "jwt_refresh_cookie_name": "refresh_token_cookie",
+  "jwt_refresh_cookie_path": "/",
+  "jwt_refresh_csrf_cookie_name": "csrf_refresh_token",
+  "jwt_refresh_csrf_cookie_path": "/",
+  "jwt_refresh_csrf_field_name": "csrf_token",
+  "jwt_refresh_csrf_header_name": "X-CSRF-TOKEN",
+  "jwt_refresh_json_key": "refresh_token",
+  "jwt_refresh_token_expires": "30 days, 0:00:00",
+  "jwt_secret_key": null,
+  "jwt_session_cookie": true,
+  "jwt_token_location": "headers",
+  "max_content_length": null,
+  "max_cookie_size": 4093,
+  "mongo_collection": "***",
+  "mongo_db": "***",
+  "mongo_dependency_view_bookkeeping": "dep_views",
+  "mongo_dependency_view_cache_size": 10,
+  "mongo_dependency_view_prefix": "dep:",
+  "mongo_uri": "***",
+  "openapi_redoc_path": "/redoc",
+  "openapi_redoc_url": "https://cdn.jsdelivr.net/npm/redoc@next/bundles/redoc.standalone.js",
+  "openapi_swagger_ui_path": "/swagger",
+  "openapi_swagger_ui_url": "https://cdn.jsdelivr.net/npm/swagger-ui-dist/",
+  "openapi_url_prefix": "/doc",
+  "openapi_version": "3.0.2",
+  "permanent_session_lifetime": "31 days, 0:00:00",
+  "preferred_url_scheme": "http",
+  "propagate_exceptions": null,
+  "retrieve_mongo_collection": "***",
+  "retrieve_mongo_db": "***",
+  "retrieve_mongo_uri": "***",
+  "search_mongo_collection": "***",
+  "search_mongo_db": "***",
+  "search_mongo_uri": "***",
+  "secret_key": "***",
+  "send_file_max_age_default": null,
+  "server_name": null,
+  "session_cookie_domain": null,
+  "session_cookie_httponly": true,
+  "session_cookie_name": "session",
+  "session_cookie_path": null,
+  "session_cookie_samesite": null,
+  "session_cookie_secure": false,
+  "session_refresh_each_request": true,
+  "sqlalchemy_binds": {},
+  "sqlalchemy_database_uri": "***",
+  "sqlalchemy_echo": false,
+  "sqlalchemy_engine_options": {},
+  "sqlalchemy_record_queries": false,
+  "sqlalchemy_track_modifications": false,
+  "templates_auto_reload": null,
+  "testing": false,
+  "trap_bad_request_errors": null,
+  "trap_http_exceptions": false,
+  "use_x_sendfile": false
+}
\ No newline at end of file
diff --git a/test/test_main_window_actions.py b/test/test_main_window_actions.py
index 03a6f5c0..62b94d33 100644
--- a/test/test_main_window_actions.py
+++ b/test/test_main_window_actions.py
@@ -3,7 +3,7 @@
 # 3. Mocking and action triggering simulate real-world user interactions and application responses.
 # 4. Separating tests for direct calls and action triggers aids in maintaining clear, organized test structures.
 # 5. This approach enhances test suite readability and makes it easier to understand and update.
-
+import asyncio
 
 import pytest
 from unittest.mock import patch, MagicMock, AsyncMock
@@ -14,10 +14,6 @@
 async def test_do_refresh_view_direct_call(app):
     """Test the direct call of the do_refresh_view method."""
 
-    # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-
     # Directly call the do_refresh_view method
     app.main_window.do_refresh_view(None, None)
 
@@ -36,29 +32,49 @@ async def test_refresh_method_triggered_by_action(app):
 
 
 @pytest.mark.asyncio
+async def test_do_get_item_direct_call_fails_due_to_no_selected_item(app):
+    """Test that the do_get_item method for copying a selected item fails when not item is selected."""
+
+    mock_variant = GLib.Variant.new_string("dummy_path")
+
+    # Directly call the method with mock objects
+    with pytest.raises(AttributeError, match="'NoneType' object has no attribute 'dataset'"):
+        app.main_window.do_get_item(None, mock_variant)
+
+@pytest.mark.asyncio
+@pytest.mark.skip(reason="no way of currently testing this")
+async def test_populate_dataset_list(populated_app):
+
+    populated_app.main_window.activate_action('refresh-view')
+
+    await asyncio.sleep(3600)
+
+@pytest.mark.asyncio
+@pytest.mark.skip(reason="no way of currently testing this")
 async def test_do_get_item_direct_call(app):
     """Test the do_get_item method for copying a selected item directly."""
 
     # Mock dependencies
-    mock_dataset_list_box = MagicMock()
-    app.main_window.dataset_list_box = mock_dataset_list_box
-    mock_settings = MagicMock()
-    app.main_window.settings = mock_settings
+    # mock_dataset_list_box = MagicMock()
+    # app.main_window.dataset_list_box = mock_dataset_list_box
+    # mock_settings = MagicMock()
+    # app.main_window.settings = mock_settings
 
     # Mock _get_selected_items to return one item
-    app.main_window._get_selected_items = MagicMock(return_value=[('item_name', 'item_uuid')])
+    # app.main_window._get_selected_items = MagicMock(return_value=[('item_name', 'item_uuid')])
 
     # Create a mock action and variant
-    mock_action = MagicMock(spec=Gio.SimpleAction)
+    # mock_action = MagicMock(spec=Gio.SimpleAction)
     mock_variant = GLib.Variant.new_string("dummy_path")
 
     # Mock async call in do_get_item
-    mock_dataset = MagicMock()
-    mock_dataset.get_item = AsyncMock()
-    mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
+    # mock_dataset = MagicMock()
+    # mock_dataset.get_item = AsyncMock()
+    # mock_dataset_list_box.get_selected_row.return_value = MagicMock(dataset=mock_dataset)
 
     # Directly call the method with mock objects
-    app.main_window.do_get_item(mock_action, mock_variant)
+    with pytest.raises(AttributeError, match="'NoneType' object has no attribute 'dataset'"):
+        app.main_window.do_get_item(None, mock_variant)
 
 
 @pytest.mark.asyncio
@@ -93,27 +109,34 @@ async def test_do_get_item_action_trigger(app):
         # Assert that do_get_item was called once
         mock_do_get_item.assert_called_once_with(get_item_action, dest_file_variant)
 
-
+# TODO: let's start with this unit est and transform it into a proper test on GUI behavior
 @pytest.mark.asyncio
-async def test_do_search_select_and_show_direct_call(app):
+@pytest.mark.skip(reason="no way of currently testing this")
+async def test_do_search_select_and_show_direct_call(populated_app, mock_dataset_list):
     """Test the do_search_select_and_show method for processing a search directly."""
 
     # Mock dependencies
-    mock_base_uri_list_box = MagicMock()
-    app.main_window.base_uri_list_box = mock_base_uri_list_box
+    #mock_base_uri_list_box = MagicMock()
+    #populated_app.main_window.base_uri_list_box = mock_base_uri_list_box
 
     # Mock _search_select_and_show to simulate search behavior
-    app.main_window._search_select_and_show = MagicMock()
+    #populated_app.main_window._search_select_and_show = MagicMock()
 
     # Create a mock action and variant
-    mock_action = MagicMock(spec=Gio.SimpleAction)
+    #mock_action = MagicMock(spec=Gio.SimpleAction)
     mock_variant = GLib.Variant.new_string("test_search_query")
 
     # Directly call the method with mock objects
-    app.main_window.do_search_select_and_show(mock_action, mock_variant)
+    populated_app.main_window.do_search_select_and_show(None, mock_variant)
+
+    # TODO: assert that
+    # * dataset list is populated with information returned by dtool_lookup_api.search
+    # * first dataset in list is selected in the GUI
+    # * content of dataset is displayed on the right hand side
+    # --> need to figure out how to inspect the GUI elements
 
     # Assert that _search_select_and_show was called with the correct query
-    app.main_window._search_select_and_show.assert_called_once_with("test_search_query")
+    # populated_app.main_window._search_select_and_show.assert_called_once_with("test_search_query")
 
 
 @pytest.mark.asyncio