-
Notifications
You must be signed in to change notification settings - Fork 154
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mostly tests work #348
Mostly tests work #348
Conversation
Some cleaning as well
Currently only for NATSNoServersException
Will replace with normal fact and try the built-in Azure DevOps features for rerunning tests. |
Changes to VSTest and enables re-run of integration tests.
@ColinSullivan1 @watfordgnf reverted the custom fact and instead changed CI & CD to use VSTest task for executing tasks and configured integration tests to allow re-run of failing tests. Nothing more planned in this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
Mostly test improvements and cleaning:
Using new customNatsFact
in integration tests which currently retries (1 time) uponNATSNoServersException
. Obviously trying to stabilize the tests would be better.Sample of NatsFact outcome
Custom fact has been removed from PR.
Test failed (reported as skipped) then fails in retry:Test failed (reported as skipped) then succeeds in retry: