-
Notifications
You must be signed in to change notification settings - Fork 29
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
improve mismatch formatting #109
base: master
Are you sure you want to change the base?
Conversation
@nscheibe downside of this solution is that we have to switch from Assert.Equals to Assert.True since XUnit does not support a custom message on Assert.Equals |
Okay let me check it tonight. The only important thing is, that the same exception is thrown (otherwise its a breaking change)... But if its Assert.Equal or Assert.True... I personally dont care :) |
The exception will change that's why some tests are failing |
Are you still on this @RohrerF ? |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Summary of the changes
Addresses #42