From f67e7ce67885dea4a8ed308fbffd84108a08ec22 Mon Sep 17 00:00:00 2001 From: Micah Lee Date: Mon, 9 Nov 2020 19:47:57 -0800 Subject: End web thread successfully when cleaning up after a tab, which prevents crashes after running the tests; and fix msi_filename in Windows build script --- desktop/package/windows/build.py | 4 +++- desktop/src/onionshare/tab/tab.py | 2 ++ desktop/tests/run.sh | 6 +++--- 3 files changed, 8 insertions(+), 4 deletions(-) diff --git a/desktop/package/windows/build.py b/desktop/package/windows/build.py index 9dd467f2..de2f921c 100644 --- a/desktop/package/windows/build.py +++ b/desktop/package/windows/build.py @@ -39,7 +39,9 @@ def main(): print("○ Create the binary") run(["briefcase", "create"], desktop_dir) run(["briefcase", "package"], desktop_dir) - msi_filename = glob.glob(os.path.join(cli_dir, "windows", "OnionShare-*.msi"))[0] + msi_filename = glob.glob(os.path.join(desktop_dir, "windows", "OnionShare-*.msi"))[ + 0 + ] print(f"○ Created unsigned installer: {msi_filename}") print(f"○ Signing installer") diff --git a/desktop/src/onionshare/tab/tab.py b/desktop/src/onionshare/tab/tab.py index 6a7624b9..b5e4e6a8 100644 --- a/desktop/src/onionshare/tab/tab.py +++ b/desktop/src/onionshare/tab/tab.py @@ -663,4 +663,6 @@ class Tab(QtWidgets.QWidget): return False def cleanup(self): + self.get_mode().web_thread.quit() + self.get_mode().web_thread.wait() self.app.cleanup() diff --git a/desktop/tests/run.sh b/desktop/tests/run.sh index 4a01dfb8..833c1516 100755 --- a/desktop/tests/run.sh +++ b/desktop/tests/run.sh @@ -1,5 +1,5 @@ #!/bin/bash -pytest -v tests/test_gui_tabs.py -pytest -v tests/test_gui_share.py -pytest -v tests/test_gui_receive.py +pytest -v tests/test_gui_tabs.py && \ +pytest -v tests/test_gui_share.py && \ +pytest -v tests/test_gui_receive.py && \ pytest -v tests/test_gui_website.py -- cgit v1.2.3-54-g00ecf