Run GUI tests as a separate testsuite

Author: GuillaumeGomez

.github/workflows/main.yml

@@ -50,17 +50,6 @@ jobs:
     - uses: actions/checkout@v4
     - name: Install Rust
       run: bash ci/install-rust.sh ${{ matrix.rust }} ${{ matrix.target }}
-    - name: Install npm
-      if: matrix.os != 'windows-latest'
-      uses: actions/setup-node@v3
-      with:
-        node-version: 20
-    - name: Install browser-ui-test
-      if: matrix.os != 'windows-latest'
-      run: npm install browser-ui-test@"${BROWSER_UI_TEST_VERSION}"
-    - name: Build and run tests (+ GUI)
-      if: matrix.os != 'windows-latest'
-      run: cargo test --locked --target ${{ matrix.target }} --test gui
     - name: Build and run tests
       run: cargo test --locked --target ${{ matrix.target }}
     - name: Test no default
@@ -84,6 +73,22 @@ jobs:
       run: rustup update stable && rustup default stable && rustup component add rustfmt
     - run: cargo fmt --check
 
+  gui:
+    name: GUI tests
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Install Rust
+        run: bash ci/install-rust.sh stable x86_64-unknown-linux-gnu
+      - name: Install npm
+        uses: actions/setup-node@v3
+        with:
+          node-version: 20
+      - name: Install browser-ui-test
+        run: npm install browser-ui-test@"${BROWSER_UI_TEST_VERSION}"
+      - name: Build and run tests (+ GUI)
+        run: cargo test --locked --target x86_64-unknown-linux-gnu --test gui
+
   # The success job is here to consolidate the total success/failure state of
   # all other jobs. This job is then included in the GitHub branch protection
   # rule which prevents merges unless all other jobs are passing. This makes

tests/gui/runner.rs

@@ -83,5 +83,5 @@ fn main() {
 
     // Then we run the GUI tests on it.
     let status = command.status().expect("failed to get command output");
-    assert!(status.success());
+    assert!(status.success(), "{status:?}");
 }