-
Notifications
You must be signed in to change notification settings - Fork 103
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
Add partition filter parsing to the adapter #1138
Comments
@Plasma Do you want to grab this one too? |
@OsirisTerje I may have some bandwidth to take this, can you point me at the correct file I should be looking at? |
@OsirisTerje I took a look at this, and it appears to already work with the latest master/pre-release version on myget.
I see the partitioning working when I run it like this
Is there somewhere I can track for the 5.0.0 release? I'll see if we're able to use the myget version until its published on nuget |
It does look like the partition isn't passed directly though though, the test discovery appears to transform it into
which isn't ideal |
We added partitioning to the framework in nunit/nunit#4391
https://github.com/nunit/nunit/blob/6fa2c0cc21c08e2b229c248b1af228e4596f4e71/src/NUnitFramework/framework/Internal/TestFilter.cs#L191-L193
Parsing support was added to the console in nunit/nunit-console#1346
@OsirisTerje noted that we need to add parsing support in the adapter as well nunit/nunit#4391 (comment)
The text was updated successfully, but these errors were encountered: