-
Notifications
You must be signed in to change notification settings - Fork 733
Improve reporting of the differences between long strings #3101
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
Merged
dennisdoomen
merged 1 commit into
fluentassertions:main
from
dennisdoomen:improve-string-comparison
Oct 3, 2025
Merged
Improve reporting of the differences between long strings #3101
dennisdoomen
merged 1 commit into
fluentassertions:main
from
dennisdoomen:improve-string-comparison
Oct 3, 2025
+640
−180
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Qodana for .NETIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at [email protected]
|
f53f48a to
575cff3
Compare
f99d1a8 to
269576e
Compare
Pull Request Test Coverage Report for Build 18198614323Details
💛 - Coveralls |
269576e to
4ee71a7
Compare
jnyrup
reviewed
Oct 2, 2025
4ee71a7 to
67716ad
Compare
67716ad to
b63ff16
Compare
jnyrup
approved these changes
Oct 3, 2025
This was referenced Oct 23, 2025
This was referenced Dec 29, 2025
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Given the following test:
Before this PR, the result would be
With the line length extended to 80 and the full subject and expectation included when it was truncated, this will be reported now
Note If
BeEquivalentToruns a string comparison, we will not include the full comparands, unless the new optionIncludingFullStringsInDifferenceis used.#3088