diff --git a/mslib/msui/msui_mainwindow.py b/mslib/msui/msui_mainwindow.py index f1ec9768b..c0511432d 100644 --- a/mslib/msui/msui_mainwindow.py +++ b/mslib/msui/msui_mainwindow.py @@ -671,14 +671,15 @@ def handle_import_local(self, extension, function, pickertype): self.last_save_directory, f"Flight Track (*.{extension});;All files (*.*)", pickertype=pickertype) + if filenames is None: + return if self.local_active: - if filenames is not None: - activate = True - if len(filenames) > 1: - activate = False - for name in filenames: - self.create_new_flight_track(filename=name, function=function, activate=activate) - self.last_save_directory = fs.path.dirname(name) + activate = True + if len(filenames) > 1: + activate = False + for name in filenames: + self.create_new_flight_track(filename=name, function=function, activate=activate) + self.last_save_directory = fs.path.dirname(name) else: for name in filenames: self.mscolab.handle_import_msc(name, extension, function, pickertype) diff --git a/tests/_test_msui/test_mscolab.py b/tests/_test_msui/test_mscolab.py index f24344187..8c8e2d920 100644 --- a/tests/_test_msui/test_mscolab.py +++ b/tests/_test_msui/test_mscolab.py @@ -596,6 +596,22 @@ def test_import_file(self, name, qtbot): imported_wp = self.window.mscolab.waypoints_model assert len(imported_wp.waypoints) == name[2] + def test_none_import_file(self, qtbot): + with mock.patch("mslib.msui.msui_mainwindow.get_open_filenames", return_value=None) as mockopen: + self._connect_to_mscolab(qtbot) + modify_config_file({"MSS_auth": {self.url: self.userdata[0]}}) + self._login(qtbot, emailid=self.userdata[0], password=self.userdata[2]) + self._activate_operation_at_index(0) + wp = self.window.mscolab.waypoints_model + assert len(wp.waypoints) == 2 + for action in self.window.menuImportFlightTrack.actions(): + if action.objectName() == "actionImportFlightTrackFTML": + action.trigger() + break + assert mockopen.call_count == 1 + imported_wp = self.window.mscolab.waypoints_model + assert len(imported_wp.waypoints) == 2 + def test_work_locally_toggle(self, qtbot): self._connect_to_mscolab(qtbot) modify_config_file({"MSS_auth": {self.url: self.userdata[0]}})