Skip to content

Port NUnit based tests to Core CLR#748

Merged
KevinRansom merged 3 commits into
dotnet:coreclrfrom
PatrickMcDonald:coreunit-coreclr-nunit
Dec 21, 2015
Merged

Port NUnit based tests to Core CLR#748
KevinRansom merged 3 commits into
dotnet:coreclrfrom
PatrickMcDonald:coreunit-coreclr-nunit

Conversation

@PatrickMcDonald

Copy link
Copy Markdown
Contributor

PR for #690

This version keeps NUnit 3.0 as the test framework.

The Core CLR version of FSharp.Core.Unittests now compiles to an exe, and uses NUnitLite as the test runner.

Alternative to #699, which uses xUnit.net

@KevinRansom

Copy link
Copy Markdown
Contributor

@PatrickMcDonald This is really nice!!!!

@KevinRansom

Copy link
Copy Markdown
Contributor

@PatrickMcDonald Yesterday I updated all of the project jsons to use the latest RC2 version of the .Net framework. That may!!!! explain the failures.

Kevin

@enricosada

Copy link
Copy Markdown
Contributor

great work @PatrickMcDonald !!

@KevinRansom

Copy link
Copy Markdown
Contributor

@PatrickMcDonald, @enricosada --- hey guys, I love this PR, the changes look great, although I expect you have both stomped over similar sets of files. What additional work is needed before I pull this. Or should I just pull it and fixup the issues?

Kevin

@enricosada

Copy link
Copy Markdown
Contributor

I'll update my pr if you pull this one.
This pr is really good, if this doesnt build and you want to pull in, i'll check this weekend and fix issues.
Is nice yo have nunit runner working in coreclr, i can use that for fsharp suite

@PatrickMcDonald

Copy link
Copy Markdown
Contributor Author

There's just the 2 tests failing, both due to Reflection errors I think, but I'm not sure how to fix them.. Upgrading to Core CLR RC2 did not fix. The choices are to fix or ignore the tests or leave them failing. I'd welcome guidance on how to proceed!

@enricosada

Copy link
Copy Markdown
Contributor

I think you can disable the failing tests and open an issue with details about failure.
For example there are some failing tests with NO_CI tags.
So pr can be merged and we can fix later.

@KevinRansom KevinRansom added this to the CoreCLR milestone Dec 13, 2015
@KevinRansom

Copy link
Copy Markdown
Contributor

Okay I am going to try to bring these into the coreclr branch ....
wish me luck :-)

I know you have done all of the work, but I feel like being needy and high maintenance today.

@PatrickMcDonald

Copy link
Copy Markdown
Contributor Author

hehe, good luck!

@KevinRansom KevinRansom merged commit 385ac5d into dotnet:coreclr Dec 21, 2015
@KevinRansom

Copy link
Copy Markdown
Contributor

Thank you so much for this, its awesome.

@enricosada

Copy link
Copy Markdown
Contributor

👍 great!

@PatrickMcDonald PatrickMcDonald deleted the coreunit-coreclr-nunit branch December 21, 2015 13:40
@dsyme

dsyme commented Jan 5, 2016

Copy link
Copy Markdown
Contributor

fab

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.

5 participants