-
Notifications
You must be signed in to change notification settings - Fork 53
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
bug: Inconsistent testing #1826
Comments
Seen this multiple time now.
|
I noticed some tests use We could get into colisions when using explicit ports with other projects (I am not sure how dedicated the Action runner is), but I am also not sure how parallelization works when running tests in Nim - is it possible that it somehow tries to parallelize the test run to speed it up? In any case, it might make sense to try to remove all hardcoded ports, if possible as a first attempt to mitigate this, maybe?:) |
and also
|
Ran into #1883 Seams like Maybe the best would be to add a |
Problem
Tests fail without code change.
Impact
Very annoying and might lead to us to dismiss failed tests.
To reproduce
git commit --amend
until test pass/failExpected behavior
Test are solid and only fail due to code changes.
nwaku version/commit hash
52894a8 ✔️
I rebase on top
4fdd0e9 ❌
then I change the commit message
28bf211 ✔️
Additional context
In 4fdd0e9
2 test failed
[FAILED] Messages are relayed between two websocket nodes
[FAILED] Messages relaying fails with non-overlapping transports (TCP or Websockets)
The text was updated successfully, but these errors were encountered: