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

897 NUnit TestKit #898

Merged
merged 2 commits into from
Apr 24, 2015
Merged

897 NUnit TestKit #898

merged 2 commits into from
Apr 24, 2015

Conversation

ashic
Copy link
Contributor

@ashic ashic commented Apr 24, 2015

This is an NUnit implementation of the TestKit.

Resolves #897 .

@Aaronontheweb
Copy link
Member

welp, looks like we don't have a good excuse to keep shitting all over XUnit anymore :p

Thanks for your submission! We'll bundle this as part of the v1.0.1 release next week along with the XUnit2 testkit in #888

cc @annymsMthd and @HCanber - does this look kosher to you?

@annymsMthd
Copy link
Contributor

Looks good to me. Nice to see some NUnit up in here:)

@ashic
Copy link
Contributor Author

ashic commented Apr 24, 2015

Seems like an unrelated test failure. Intermittent test?

@Aaronontheweb
Copy link
Member

@ashic yes, it's one of the outstanding racy unit tests that we still haven't quite fixed from #853

@annymsMthd
Copy link
Contributor

#888 has a fix for that one

@Aaronontheweb
Copy link
Member

@annymsMthd
whiskey-approval

Aaronontheweb added a commit that referenced this pull request Apr 24, 2015
@Aaronontheweb Aaronontheweb merged commit 249d242 into akkadotnet:dev Apr 24, 2015
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.

Nunit TestKit
3 participants