Skip to content
This repository has been archived by the owner. It is now read-only.

Xunit2 support #1

Merged
merged 26 commits into from Nov 1, 2015
Merged

Xunit2 support #1

merged 26 commits into from Nov 1, 2015

Conversation

@citizenmatt
Copy link
Contributor

citizenmatt commented May 2, 2014

This PR tracks everything that's required for xunit2 support:

  • Basic running of tests with xunit2
  • Enable parallelisation (relies on outcome of xunit/xunit#75)
  • Use xunit2 for test discovery in source code. Replace existing xunit1 dependency
  • Serialise tests discovered in source through to the runner, so discovery only happens once
  • Explicit support for collections, to allow grouping in the ReSharper UI. Also allow for error reporting if a collection fixture fails
  • Report shadow copy cache folder to ReSharper to allow cache cleanup in case of process abort. Relies on xunit/xunit#81
  • Reference provider for MemberData parameters - code completion, ctrl-click navigation, etc.
  • Update external annotations for xunit2 assemblies
  • Update live templates for xunit2. How to know when to use xunit1 vs xunit2 templates? (e.g. Theory has a different namespace)
@citizenmatt citizenmatt added this to the xunit2 milestone May 2, 2014
@citizenmatt citizenmatt self-assigned this May 2, 2014
citizenmatt added a commit that referenced this pull request Jul 23, 2014
Escape control chars in strings. /cc #1
citizenmatt added 22 commits Aug 12, 2014
The xunit runner utility can produce false positives when parsing
multiple nested exceptions. Fixed in 2.1, but this adds some defensive
coding to at least not throw exceptions and fail to finish running the
tests (the message we display might be a bit rubbish though).

Fixes #50
The SDK's tests try to find the code base of the test runner as a
relative path to the executing assembly codebase, which doesn't work if
shadow copy is enabled.
Not sure if this is a new requirement or oversight...
IsType and IsAssignableFrom return values, but it's not an error to
ignore the return value.

Fixes #69
@BrunoJuchli
Copy link

BrunoJuchli commented Aug 16, 2015

I would really love support for custom tests. Is there any way i can help with the integration of xunit2 for discovery?

@citizenmatt
Copy link
Contributor Author

citizenmatt commented Nov 1, 2015

Enough's enough. This pull request has gone on long enough. Time to move back to master.

Merging + closing this. Missing features are #88, #87, #86, #85.

@citizenmatt citizenmatt merged commit 69646ab into master Nov 1, 2015
@citizenmatt citizenmatt deleted the xunit2 branch Nov 1, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.