From 25f4e99cbb7529edeb1c4539f112dcc79233d594 Mon Sep 17 00:00:00 2001
From: Saffron Worker <saffron@kubetix.local>
Date: Wed, 10 Jun 2026 05:19:55 -0600
Subject: [PATCH] fix(ci): add exit-code check for reservation tests step

The reservation tests step in CI was missing exit-code enforcement,
meaning test failures would silently pass. Add proper exit-code
checking consistent with all other test steps (script, E2E, layout).

Refs #176
---
 .github/workflows/test.yml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 38cbe5e..77dd89c 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -108,7 +108,12 @@ jobs:
         run: |
           set -euo pipefail
           ./.tools/Godot_v${{ steps.godot-config.outputs.version }}-${{ steps.godot-config.outputs.status }}_linux.x86_64 --headless --path . --script res://tests/test_reservations.gd > reservation-tests.log 2>&1
+          RES_EXIT=$?
           cat reservation-tests.log
+          if [ $RES_EXIT -ne 0 ]; then
+            echo "::error::Reservation tests failed (exit code $RES_EXIT)"
+            exit 1
+          fi
 
   macos-validation:
     name: macOS validation