Skip to content
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

[WIP] Support of NUnit3 #1

Merged
merged 1 commit into from
Jan 27, 2016
Merged

Conversation

Evangelink
Copy link

Based on NUnit3 source code I propose the following migration of the
custom constraints.

  • We should add tests to prove the WriteDescriptionTo method migration
    is doing what is intended.
  • The F#-friendly formatting has been partially merged. We need to
    decide if the EqualConstraintResult class is good enough or if we
    inherit it and make it F#-friendly.

Based on NUnit3 source code I propose the following migration of the
custom constraints.
- We should add tests to prove the WriteDescriptionTo method migration
is doing what is intended.
- The F#-friendly formatting has been partially merged. We need to
decide if the EqualConstraintResult class is good enough or if we
inherit it and make it F#-friendly.
@sergey-tihon
Copy link
Owner

Seems like Description pretty-print only expected value but not the actual one

@Evangelink
Copy link
Author

@sergey-tihon It seems to be the behavior of the other constraint classes. Besides the WriteDescriptionTo method was not displaying the actual value, was it?

@sergey-tihon
Copy link
Owner

But WriteActualValueTo did it previously

sergey-tihon added a commit that referenced this pull request Jan 27, 2016
@sergey-tihon sergey-tihon merged commit 3414dba into sergey-tihon:master Jan 27, 2016
@sergey-tihon
Copy link
Owner

@Evangelink Thank for this PR
Lets continue discussion here fsprojects#83

sergey-tihon pushed a commit that referenced this pull request Oct 30, 2019
Add collection supersetOf assertion for NUnit, xUnit and MsTest.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants