Inspect exit code when testing if running

Author: elibroftw

.github/workflows/build.yml

@@ -47,10 +47,19 @@ jobs:
           python build.py --ci --no-deps --no-tests --no-install
 
       - name: Test build
+        id: test
         run: |
           .venv\Scripts\Activate.ps1
           python build.py --ci --no-deps --no-build --no-install
 
+      - name: Save
+        if: failure() && steps.test.outcome == 'failure'
+        uses: actions/upload-artifact@v4
+        with:
+          name: failed-build
+          path: |
+            dist/Music Caster Setup.exe
+
       # upload the build continuously to avoid duplicate build on a tag
       # script will exit with zero if release for VERSION exists since --ci is used
       - name: Upload build

build.py

@@ -609,6 +609,13 @@ def upgrade_yt_dlp():
             f'"{DIST_DIR}/Music Caster OneDir/Music Caster" -m --debug',
             shell=True)
         time.sleep(5)
+        if p.returncode is not None:
+            print('got return code', p.returncode)
+        test(
+            'No return code',
+            lambda: p.returncode is None,
+            True,
+        )
         test(
             'Music Caster Should Be Running',
             lambda: is_already_running(threshold=1),

src/music_caster.py

@@ -297,6 +297,11 @@ def activate_instance(port=2001, default_timeout=0.5, to_port=2004):
                 print('not moving database because file already exists')
             else:
                 os.rename(DatabaseConnection.OLD_DATABASE_FILE, DatabaseConnection.DATABASE_FILE)
+        try:
+            with DatabaseConnection() as conn:
+                pass
+        except Exception:
+            sys.exit(67)
 
     # 0.5 seconds gone to 3rd party imports
     from flask import Flask, jsonify, render_template, request, redirect, send_file, Response, make_response