fix(ci): test failures should fail the build

The 'Check if tests exist' step was actually running tests with
continue-on-error: true. If tests failed, it would set has-tests=false
and skip the actual test step, making CI appear green even with failing tests.

Changed to check if a test script exists in package.json without running it,
and removed continue-on-error so test failures now properly fail the build.

Fixes the issue where PR #3014 had failing tests but CI was green.

Author: domdomegg

.github/workflows/typescript.yml

@@ -45,12 +45,15 @@ jobs:
         id: check-tests
         working-directory: src/${{ matrix.package }}
         run: |
-          if npm run test --silent 2>/dev/null; then
-            echo "has-tests=true" >> $GITHUB_OUTPUT
+          if grep -q '"test":' package.json; then
+            if grep -q '"test": "echo \\"Error: no test specified\\" && exit 1"' package.json; then
+              echo "has-tests=false" >> $GITHUB_OUTPUT
+            else
+              echo "has-tests=true" >> $GITHUB_OUTPUT
+            fi
           else
             echo "has-tests=false" >> $GITHUB_OUTPUT
           fi
-        continue-on-error: true
 
       - name: Run tests
         if: steps.check-tests.outputs.has-tests == 'true'