Add tests for NatsStore watch logic #1753

Merged
mfreeman451 merged 1 commit from refs/pull/1753/head into 800-live-armis-poc-integration-into-serviceradar 2025-06-19 03:11:25 +00:00
mfreeman451 commented 2025-06-19 03:10:21 +00:00 (Migrated from github.com)
Owner

Imported from GitHub pull request.

Original GitHub pull request: #977
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/977
Original created: 2025-06-19T03:10:21Z
Original updated: 2025-06-23T20:59:37Z
Original head: carverauto/serviceradar:codex/verify-nats-kv-watcher-code
Original base: 800-live-armis-poc-integration-into-serviceradar
Original merged: 2025-06-19T03:11:25Z by @mfreeman451

Summary

  • add a test verifying that NatsStore's Watch method forwards updates from jetstream and cleans up on context cancel
  • create fake implementations of jetstream.KeyValue and KeyWatcher for testing

Testing

  • go test ./...

https://chatgpt.com/codex/tasks/task_e_68537a47e528832099e94f2b5a152df7

Imported from GitHub pull request. Original GitHub pull request: #977 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/977 Original created: 2025-06-19T03:10:21Z Original updated: 2025-06-23T20:59:37Z Original head: carverauto/serviceradar:codex/verify-nats-kv-watcher-code Original base: 800-live-armis-poc-integration-into-serviceradar Original merged: 2025-06-19T03:11:25Z by @mfreeman451 --- ## Summary - add a test verifying that NatsStore's Watch method forwards updates from jetstream and cleans up on context cancel - create fake implementations of jetstream.KeyValue and KeyWatcher for testing ## Testing - `go test ./...` ------ https://chatgpt.com/codex/tasks/task_e_68537a47e528832099e94f2b5a152df7
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
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
carverauto/serviceradar!1753
No description provided.