-
Notifications
You must be signed in to change notification settings - Fork 324
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
Remove xunit customer runner from tests, come up with a similar way to do the BDD style tests. #371
Comments
What about just using machine.specifications? |
+1 all our bdd test on top of NES & our products are based on mspec, we are very satisfied. |
This project was actually using mspec and we moved away from it. The custom BDD code for XUnit was added to make it as close to the existing mspec tests as possible to simplify moving away from it. I've used mspec for a while. The primary issue is that it's too easy to mess up and have hard to read buggy tests. You initialize one static field outside of a context and a group of tests fail... sometimes... Also, in every project I've used it, there's been a ton of friction with on-boarding new devs. It's a unique way of writing tests that's not easy to understand at first. I say we stick w/ XUnit and find a non-custom approach to BDD. |
maybe was a bug on the previous versions, we have 700+ mspec tests on our current project without any issue. |
@kblooie we need to run the persistence tests with different options for a given provider (eg: mongo with or without server side checkpoint), there's a simple way to do this with current test framework? |
Changed how testing works: created some support classes so we can chose between XUnit, NUnit or MSTests. |
Custom runner doesn't work in ncrunch and the whole API has change in xunit 2.0
The text was updated successfully, but these errors were encountered: