-
Notifications
You must be signed in to change notification settings - Fork 790
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
xUnit 2.3.0 VS Runner breaks Live Unit Testing feature of VS 2017 #1514
Comments
This is not a good sign. |
I have same problem here =( |
To unblock yourself, Downgrade xunit packages to 2.3.0-beta5-* which may cause performance degrade. |
We have used xunit 2.2.0 but after the Windows Fall Creator Update 2017 yesterday we got the following error:
So we updated to xunit 2.3.0 and there we had the same problem as described in the bug report. Downgrading to xunit 2.3.0-beta5-* resolved all issues. |
On the other hand, using the regular Visual Studio test runner yields the same error with xunit 2.3.0-beta5-build3769. |
…main, it's possible to use xunit for testing. Readd CefSharp.Test.csproj to solution Add some very basic unit tests using xunit, DisableTestParallelization, Set xunit.appDomain to denied in app.config to disable using appdomains (https://xunit.github.io/docs/configuring-with-xml.html) Add xunit vs test running packages, using 2.2.0 version as 2.3.0 was throwing an exception similar to xunit/xunit#1514 TODO: Take a look at /~https://github.com/NuGet/NuGetGallery/tree/master/tests/NuGetGallery.Facts for some ideas on structuring xunit tests (look at implementing Theory) NOTE: I am not providing this as a definitive set of tests, it's simply here to get someone else started in creating tests as it appears people have trouble dealing with AppDomain problem Only includes Tests for the OffScreen version at this stage.
I'm using xUnit 2.3.1, xunit.runner.visualstudio 2.3.1 with VS 2017 Enterprise 15.7.5 |
@majorimi Does it still repro with 2.4 RC 1? |
(if so, please open a new issue with a repro) |
Thanks for the reply @bradwilson. I'm not a big fan of using Pre-Release packages because they tend to break something. Since the feature is already broken I tried your suggestion. Even used a newer version than you suggested: 2.4.0-rc.2.build4045. Result is the same... Live unit testing does not work. Looking into the Output window I just got a very weird error message:
But it is completely bogus. I can build, run, debug the project. So there is no build error. Additional info: my project is a Visual Studio Extension. Can it be the root cause of the issue? |
I will look at the other issue w/ the repro. |
Repo project here:
/~https://github.com/kstreith/xunit23liveunittestbug
If you use the xunit.runner.visualstudio 2.3.0 package, live unit testing is broken. Open the ConsoleApp1.sln in LiveTestsBrokenWith_xUnit23Vs23Runner in VS 2017. Enable live unit testing. It will fail with the following messages in the Output window (Live Unit Testing output).
This is only an issue with the xunit.runner.visualstudio 2.3.0 package. If you look in the repo solution directory, I have another project. The LiveTestsWorksWith_xUnit23Vs22Runner project works correctly with Live Unit Tests, it has xunit 2.3.0 but uses the earlier version of the visualstudio runner, e.g. 2.2.0.
I used Visual Studio 15.3.3. Details on Live Unit Testing of Visual Studio: https://docs.microsoft.com/en-us/visualstudio/test/live-unit-testing
I'd like to be able to use the new visualstudio runner with live unit testing feature of VS.
The text was updated successfully, but these errors were encountered: