-
Notifications
You must be signed in to change notification settings - Fork 57
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
Westlad/ping pong fix #526
Conversation
Hey, I'm the one to blame for the ping-pong tests failing, sorry about that! |
I wouldn't worry. Tests always break eventually, unless they are part of the automated testing. |
Testing that a failed ping pong test is picked up.
fix: replace with correct code after action test
I see you have removed the Dockerfile inside |
Good point @daveroga. As this is a fairly urgent fix (for dev), I'll do a separate PR to remove the extraneous file. |
Fixes ping-pong tests after some recent PRs broke it. Also runs an automated test on ping-pong to avoid future occurrences.
For instructions on running the ping-pong test, see
test/ping-pong/README.md
.