From 3f8f56ee355b6e5b88e34029f7b760dd8b0bb593 Mon Sep 17 00:00:00 2001 From: William Mantly Date: Thu, 23 Jul 2026 16:22:26 -0400 Subject: [PATCH] ci: add redis service (audit/metrics/session models need it on import) model-redis opens a client when models/index is required, so with no redis the integration test saw ECONNREFUSED as a post-test unhandled rejection. Provide redis:7 at 127.0.0.1:6379 in CI (mirrors how sso-manager-node tests run). Co-Authored-By: Claude Opus 4.8 --- .github/workflows/pr-tests.yml | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/.github/workflows/pr-tests.yml b/.github/workflows/pr-tests.yml index 34ca0cf..10b4c10 100644 --- a/.github/workflows/pr-tests.yml +++ b/.github/workflows/pr-tests.yml @@ -13,6 +13,19 @@ jobs: name: Run Tests runs-on: ubuntu-latest + # The audit/metrics/session models are redis-backed (model-redis opens a + # client on import), so the tests need a reachable redis at 127.0.0.1:6379. + services: + redis: + image: redis:7-alpine + ports: + - 6379:6379 + options: >- + --health-cmd "redis-cli ping" + --health-interval 5s + --health-timeout 3s + --health-retries 10 + strategy: matrix: node-version: [20.x, 22.x]