Skip to content

test: add Redis-backed integration coverage#60

Merged
darthfork merged 2 commits intomainfrom
test/issue-54-redis-integration
May 2, 2026
Merged

test: add Redis-backed integration coverage#60
darthfork merged 2 commits intomainfrom
test/issue-54-redis-integration

Conversation

@darthfork
Copy link
Copy Markdown
Owner

@darthfork darthfork commented May 2, 2026

Summary

  • Adds a Redis-backed integration suite covering delivery dedupe, shared workflow/job state, duplicate run transition suppression, and clear Redis connection failures.
  • Wires a dedicated Redis integration CI job with a disposable redis:7-alpine service.
  • Keeps async webhook processing usable with Redis after the HTTP request returns by detaching queued event contexts from request cancellation.

Test Plan

  • make unit-test
  • make integration-test
  • make redis-integration-test
  • make lint
  • make build

Closes #54

darthfork added 2 commits May 1, 2026 19:14
@darthfork
test: add redis-backed integration coverage
65e88fa 
- Add Redis integration tests for delivery dedupe and shared run state
- Run Redis-backed suite in CI with a disposable Redis service
- Preserve request context values after async webhook responses

Closes #54
@darthfork
test: include redis integration in full suite
7f2acd5
@darthfork darthfork enabled auto-merge (squash) May 2, 2026 02:22
@darthfork darthfork merged commit b5bb2ac into main May 2, 2026
3 checks passed
@darthfork darthfork deleted the test/issue-54-redis-integration branch May 2, 2026 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add Redis-backed integration tests for deduplication and shared state

1 participant

@darthfork