fix(api-ci): preserve Redis URL in real DB E2E #1202

Merged

andrei merged 2 commits from agent/devops/pix-15306 into master 2026-08-02 06:20:20 +00:00

Conversation 1 Commits 2 Files changed 7 +2 -7

andrei commented 2026-08-02 05:19:26 +00:00

Owner

Copy link

PIX-15306

Что изменено:

• real-DB Vitest E2E больше не перетирают REDIS_URL на пустую строку; CI API job сохраняет service Redis URL (redis://redis:6379) вместо fallback на localhost.

Проверка:

• pnpm --filter api exec vitest run src/__tests__/rate-limit-ip-key.test.ts --reporter=verbose — passed.
• E2E_DATABASE_URL=postgresql://postgres:postgres@127.0.0.1:5432/pix15306_e2e REDIS_URL=redis://127.0.0.1:6380 NODE_OPTIONS=--max-old-space-size=4096 pnpm --filter api exec vitest run --no-file-parallelism src/services/p2pEscrow/__tests__/revert.e2e.scenarios.int.test.ts --reporter=verbose — passed, grep по ECONNREFUSED|NOAUTH|EnvironmentTeardownError|Unhandled Rejection пустой.
• Полный локальный .int.test.ts дополнительно запускался на чистой БД и Redis; исходных Redis/teardown сигнатур нет, но локально остаются 2 независимых fail в payout.e2e.int.test.ts по balance/daily-limit, не относящиеся к этому fix (в красном master evidence все 38 тестов проходили, падение было только teardown).

PIX-15306 Что изменено: - real-DB Vitest E2E больше не перетирают REDIS_URL на пустую строку; CI API job сохраняет service Redis URL (`redis://redis:6379`) вместо fallback на localhost. Проверка: - `pnpm --filter api exec vitest run src/__tests__/rate-limit-ip-key.test.ts --reporter=verbose` — passed. - `E2E_DATABASE_URL=postgresql://postgres:postgres@127.0.0.1:5432/pix15306_e2e REDIS_URL=redis://127.0.0.1:6380 NODE_OPTIONS=--max-old-space-size=4096 pnpm --filter api exec vitest run --no-file-parallelism src/services/p2pEscrow/__tests__/revert.e2e.scenarios.int.test.ts --reporter=verbose` — passed, grep по `ECONNREFUSED|NOAUTH|EnvironmentTeardownError|Unhandled Rejection` пустой. - Полный локальный `.int.test.ts` дополнительно запускался на чистой БД и Redis; исходных Redis/teardown сигнатур нет, но локально остаются 2 независимых fail в `payout.e2e.int.test.ts` по balance/daily-limit, не относящиеся к этому fix (в красном master evidence все 38 тестов проходили, падение было только teardown).

andrei added 1 commit 2026-08-02 05:19:27 +00:00

fix(api-ci): preserve redis url in real db e2e ef723764eb

andrei commented 2026-08-02 05:28:38 +00:00

Author

Owner

Copy link

CTO review: замечаний к патчу нет. Изменение минимально убирает перетирание REDIS_URL в real-DB int-тестах и сохраняет CI service Redis URL. Не мержить до зелёного Forgejo Actions run 19370, особенно job API.

CTO review: замечаний к патчу нет. Изменение минимально убирает перетирание REDIS_URL в real-DB int-тестах и сохраняет CI service Redis URL. Не мержить до зелёного Forgejo Actions run 19370, особенно job API.

andrei added 1 commit 2026-08-02 05:56:01 +00:00

fix(api-ci): clean risk scores in webhook e2e reset c9086124e1

andrei merged commit 9a3ae65836 into master 2026-08-02 06:20:20 +00:00

andrei referenced this pull request from a commit 2026-08-02 06:20:21 +00:00

Merge pull request 'PIX-15306: fix API E2E Redis/cleanup' (#1202)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

ci-failing

  

duplicate

  

needs-rebase

No labels

ci-failing

duplicate

needs-rebase

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

europa-tech-srl/europatech!1202

No description provided.