EvergenEnergy / sparkplug-host / 28985046629

85%

main: 85%

Pull #49

github

test: fix flaky integration tests by removing timing-based synchronization

The host was run with a fixed 300ms context timeout while tests waited
for its retained STATE birth certificate. On a slow runner the death
certificate (also retained) could replace it before the test client
subscribed, so online=true never arrived and the test timed out.

Rebirth tests had a similar race: they subscribed to the NCMD topic
after publishing the triggering messages, so a fast host could publish
the (non-retained) rebirth command before the subscription was active.

Replace both with deterministic synchronization:
- run the host under a cancellable context and only shut it down once
  the test has observed what it needs
- signal end-of-processing in runAndCollectAllMetrics with a sentinel
  edge node birth published after the test's own messages
- subscribe for rebirth requests before publishing the messages that
  trigger them

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

1 of 3 new or added lines in 2 files covered. (33.33%)

794 of 2164 relevant lines covered (36.69%)

16.83 hits per line