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

Stuck between versions #1371

Closed
ValentijnMakkenze opened this issue Dec 19, 2023 · 9 comments
Closed

Stuck between versions #1371

ValentijnMakkenze opened this issue Dec 19, 2023 · 9 comments

Comments

@ValentijnMakkenze
Copy link
Contributor

Hi,

We want to switch to NUnit 4 but when we did we got this error:
No suitable tests found in ...
Either assembly contains no tests or proper test driver has not been found.

Turned out we where using NUnit.ConsoleRunner 3.16.3 so we downgraded to version 3.15.5
But now our tests hang because of issue #1183: Build hangs when test spawns processes which do not terminate properly
This issue is fixed in the 16 version but not in the 15 version.

Can issue #1183 be backported to the 15 series or NUnit 4 support be added to the 16 series?

Kind regards,

Valentijn

@OsirisTerje
Copy link
Member

I'm on travel now, so not at any computer. Do you see a stacktrace? And does it point to the engine ir the console? The 3.16 series have a loading issue where we don't know the resolution, and we're therefore careful with moving code from that to 3.15. We're going to do that eventually, but we're low on resources. If the crash is in the console, it is easy to move over. If it is in the engine it is harder. We can have a look if we can get 4 support into 3.16.

@ValentijnMakkenze
Copy link
Contributor Author

I cherry-picked #1183 to the release-3.15.3 branch and build that and put it on our buildserver and the tests run OK now.

@OsirisTerje
Copy link
Member

Awesome! Feel free to put up a pr on it, and we can see if it works on all tests, adapter incl.

@ValentijnMakkenze
Copy link
Contributor Author

I created a PR but wasn't quite sure which branch to target #1372

@ValentijnMakkenze
Copy link
Contributor Author

Created a new PR with target branch version315 ##1373

@ValentijnMakkenze
Copy link
Contributor Author

Can I do something to help release 3.15.6, with the fix for #1183 ?

@stevenaw
Copy link
Member

@ValentijnMakkenze
Can you see if the 3.17 pre-release build works for you?
https://github.com/nunit/nunit-console/releases/tag/V3.17.0-dev.3

It looks like your commits have also been included in the 3.17 dev build which should, I believe, be built off of 3.15.x and also include a select few commits from the 3.16.x branch. It should include .NET8 support as well, should you need it (#1374)

@ValentijnMakkenze
Copy link
Contributor Author

I tried the 3.17 pre-release and was able to build without problems.

Thanks

@stevenaw
Copy link
Member

Happy to hear it helped @ValentijnMakkenze !
Thanks for letting us know

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

No branches or pull requests

3 participants