Skip to content

test(listener): add e2e local notification smoke test suite (#723) - #727

Open
ravendevhub wants to merge 1 commit into
Core-Foundry:mainfrom
ravendevhub:test/e2e-local-notification-smoke-test-723
Open

test(listener): add e2e local notification smoke test suite (#723)#727
ravendevhub wants to merge 1 commit into
Core-Foundry:mainfrom
ravendevhub:test/e2e-local-notification-smoke-test-723

Conversation

@ravendevhub

Copy link
Copy Markdown

Summary

Resolves #723 by introducing an isolated, end-to-end local notification smoke test that verifies event ingestion through notification model generation without sending traffic to real external webhook destinations.

Changes

  1. Smoke Test Suite (listener/src/__tests__/local-notification-smoke.test.ts):
    • Ingests synthetic Soroban contract events into the local EventRegistry.
    • Verifies pipeline normalization and notification payload generation.
    • Strictly mocks external webhook channels guaranteeing no live network calls.
  2. Runner Script & Package Target:
    • Added npm run test:smoke script to listener/package.json.
    • Created scripts/run-smoke-test.sh runner script.
  3. Documentation (docs/LOCAL_NOTIFICATION_SMOKE_TEST.md):
    • Outlines test scope, execution commands, and CI integration.

Acceptance Criteria

  • A representative event can enter the local pipeline.
  • Event processing completes successfully.
  • A notification payload is generated.
  • External notification delivery is mocked or disabled.
  • The test can run reliably in CI.
  • Setup and execution are documented.

…ndry#723)

- Implement isolated Jest test exercising event ingestion to notification model generation
- Mock external webhook and transport layers to prevent live network dependencies
- Add test:smoke npm script and run-smoke-test.sh runner
- Document smoke test architecture and verification in docs/LOCAL_NOTIFICATION_SMOKE_TEST.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add End-to-End Local Notification Smoke Tes

1 participant