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.
@stevenaw This is for discussion, to show issues we face.
Contributes to #53
I added a specific constraint for comparing strings (
EqualStringConstraint
)The idea was that I then could remove the string specific modifiers (
IgnoreCase
,IgnoreWhiteSpace
) from the actualEqualConstraint
This didn't work because the
EqualConstraint
modifiers, don't just apply to the type at hand, but to members of (nested) collection and dictionaries. Whether it applies is determined at runtime.See the
CollectionEqualsTests
:The only thing this
EqualsStringConstraint
brings us is that this constraint doesn't allow passing in anWithin
constraint.So the one test for that:
This no longer compiles. Hereby moving an error from runtime to compile time.
Fixes #4875