We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
PR rh-jmc-team#43 modified error strings in implementations, but not in tests.
com.redhat.rhjmc.containerjfr.commands.internal.ConnectCommandTest > shouldNotValidateNull() FAILED org.mockito.exceptions.verification.ArgumentsAreDifferent at ConnectCommandTest.java:84 com.redhat.rhjmc.containerjfr.commands.internal.ConnectCommandTest > shouldNotValidateEmptyString() FAILED org.mockito.exceptions.verification.ArgumentsAreDifferent at ConnectCommandTest.java:90 com.redhat.rhjmc.containerjfr.commands.internal.ConnectCommandTest > shouldExpectOneArg() FAILED org.mockito.exceptions.verification.ArgumentsAreDifferent at ConnectCommandTest.java:67 com.redhat.rhjmc.containerjfr.commands.internal.ConnectCommandTest > shouldNotValidateInvalidIdentifiers(String)[1] FAILED org.mockito.exceptions.verification.ArgumentsAreDifferent at ConnectCommandTest.java:78 com.redhat.rhjmc.containerjfr.commands.internal.ConnectCommandTest > shouldNotValidateInvalidIdentifiers(String)[2] FAILED org.mockito.exceptions.verification.ArgumentsAreDifferent at ConnectCommandTest.java:78 com.redhat.rhjmc.containerjfr.commands.internal.ConnectCommandTest > shouldNotValidateInvalidIdentifiers(String)[3] FAILED org.mockito.exceptions.verification.ArgumentsAreDifferent at ConnectCommandTest.java:78 417 tests completed, 6 failed testsuite com.redhat.rhjmc.containerjfr.commands.internal.SaveRecordingCommandTest: testcase com.redhat.rhjmc.containerjfr.commands.internal.ConnectCommandTest > shouldNotValidateNull(): Argument(s) are different! Wanted: cw.println( "Expected one argument: host name/URL" ); -> at com.redhat.rhjmc.containerjfr.core.tui.ClientWriter.println(ClientWriter.java:11) > Task :test FAILED
This is really trivial though.
The text was updated successfully, but these errors were encountered:
Thanks for catching this, I'll fix it momentarily.
Sorry, something went wrong.
Correct unit test assertions
5af8417
Strings printed by ConnectCommand on invalid input were changed but the tests not modified to match. Fixes #45
Correct unit test assertions (#46)
b6eb0d7
Successfully merging a pull request may close this issue.
PR rh-jmc-team#43 modified error strings in implementations, but not in tests.
This is really trivial though.
The text was updated successfully, but these errors were encountered: