Skip to content

Fix notifier test payload API surface#127

Open
sorinc03 wants to merge 1 commit into
Tx-wat:mainfrom
sorinc03:codex/notifier-test-payload-122
Open

Fix notifier test payload API surface#127
sorinc03 wants to merge 1 commit into
Tx-wat:mainfrom
sorinc03:codex/notifier-test-payload-122

Conversation

@sorinc03
Copy link
Copy Markdown

Fixes #122.

Summary

  • move the synthetic test_payload helper into the CLI, where test-webhook is the only caller
  • remove test_payload from the txwatch-notifier public API
  • add a focused CLI unit test for the generated test-webhook payload context
  • drop the now-unused notifier chrono dependency and add the direct CLI dependencies needed by the helper

Validation

  • git diff --check
  • Not run: cargo fmt --check / cargo test because this environment does not have cargo, rustc, or rustfmt installed.

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.

Fix: txwatch-notifier::test_payload is public but only intended for CLI use

1 participant