Add missing tests for Deploy Notifier (SSE)

cspr-rad / kairos

Apache License 2.0

2 stars 0 forks source link

Add missing tests for Deploy Notifier (SSE) #148

Closed koxu1996 closed 1 month ago

koxu1996 commented 1 month ago

Fixes #85.

koxu1996 commented 1 month ago

@marijanp Provided tests with mocks are mostly the evidence that edge cases are handled according to my expectations: scenarios like double handshake should never happen. Of course we need integration tests to have higher confidence that code is actually working. If I have time I will experiment with running NCTL for notifier tests.