Added a function to MockFileDialog along with the change in original …

…files to make sure multiple directories are added when directories are chosen

src/vorta/views/archive_tab.py

@@ -642,8 +642,8 @@ def receive():
                     self.app.jobs_manager.add_job(job)
 
         dialog = choose_file_dialog(self, self.tr("Choose Mount Point"), want_folder=True)
-        dialog.filesSelected.connect(receive)
-        dialog.open()
+        if dialog.exec():
+            receive()
 
     def mount_result(self, result):
         if result['returncode'] == 0:
@@ -789,8 +789,8 @@ def receive():
                         self._set_status(params['message'])
 
             dialog = choose_file_dialog(self, self.tr("Choose Extraction Point"), want_folder=True)
-            dialog.filesSelected.connect(receive)
-            dialog.open()
+            if dialog.exec():
+                receive()
 
         window = ExtractDialog(archive, model)
         self._toggle_all_buttons(True)

src/vorta/views/repo_add_dialog.py

@@ -72,8 +72,8 @@ def receive():
                 self.is_remote_repo = False
 
         dialog = choose_file_dialog(self, self.tr("Choose Location of Borg Repository"))
-        dialog.filesSelected.connect(receive)
-        dialog.open()
+        if dialog.exec():
+            receive()
 
     def use_remote_repo_action(self):
         self.repoURL.setText('')

src/vorta/views/source_tab.py

@@ -302,8 +302,8 @@ def receive():
 
         msg = self.tr("Choose directory to back up") if want_folder else self.tr("Choose file(s) to back up")
         dialog = choose_file_dialog(self, msg, want_folder=want_folder)
-        dialog.filesSelected.connect(receive)
-        dialog.open()
+        if dialog.exec():
+            receive()
 
     def source_copy(self, index=None):
         """

tests/integration/conftest.py

@@ -189,6 +189,9 @@ def __init__(self, *args, **kwargs):
 
         def open(self, func):
             func()
+
+        def exec(self):
+            return 1
 
         def selectedFiles(self):
             if self.subdirectory:

tests/unit/conftest.py

@@ -85,6 +85,9 @@ def __init__(self, *args, **kwargs):
 
         def open(self, func):
             func()
+
+        def exec(self):
+            return 1
 
         def selectedFiles(self):
             return ['/tmp']

tests/unit/test_archives.py

@@ -14,6 +14,9 @@ class MockFileDialog:
     def open(self, func):
         func()
 
+    def exec(self):
+        return 1
+
     def selectedFiles(self):
         return ['/tmp']