ci(ui-tests): enforce a sensible timeout

The `ui-tests` job currently takes around half a minute to a minute.
Therefore it is quite alarming when the job times out after the default
of 6h. Let's let it time out after 10 minutes (we can always increase it
when more tests are added).

Unfortunately, this changes the failure mode and `cancelled()` will no
longer return `true`. Which means that the existing `if:` clauses have
to be adapted for showing the logs and taking & uploading a screenshot.
While thinking of the logs, I realized that it would be good also to
have them in the case that the run succeeded, so let's always show them.

Signed-off-by: Johannes Schindelin <[email protected]>

Author: dscho

.github/workflows/ui-tests.yml

@@ -83,14 +83,15 @@ jobs:
             ui-tests
       - name: Run UI tests
         id: ui-tests
+        timeout-minutes: 10
         working-directory: ui-tests
         run: |
           "${env:RUNNER_TEMP}\ahk\AutoHotKey64.exe" /ErrorStdOut /force ui-tests/ctrl-c.ahk background-hook.ahk "$PWD\bg-hook" 2>&1 | Out-Default
           if (!$?) { echo "::error::Test failed!" } else { echo "::notice::Test log" }
           type bg-hook.log
           if ($p.ExitCode -ne 0) { exit 1 }
-      - name: Show logs, if canceled
-        if: cancelled()
+      - name: Show logs
+        if: always()
         working-directory: ui-tests
         run: type bg-hook.log
       - name: Take screenshot, if canceled