-
Notifications
You must be signed in to change notification settings - Fork 266
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
[Testing Platform] Batch Filter #3528
Comments
This is assuming that the tests are always returned in the same order right? I don't see a direct big value in this shape (I would prefer to split based on list of UIDs) but I don't think that's a bad suggestion so I'll mark it as feature request and see if we have some more interest before we work on implementation. |
Yes you're right they'd need to be in the same order. But if framework providers register it like they do your treenode filter, they can choose whether to support it or not. My tests are source generated so should be deterministic afaik |
Splitting on uids works too. Wouldn't require specific ordering then, but would be a very verbose dotnet run command |
Also are you planning to let frameworks build their own filters? That's all still internal code isn't it? |
Yes, my idea was to be able to provide a filter file with either simply the list of uids OR to have a structured file so we are not only expecting UIDs but we could potentially do mix up of filters.
Yes and yes. We could start opening with the |
I was actually thinking about mixes of filters earlier and whether that was worth supporting. You'd need a new type like |
Yes definitely! |
I think that to have complete filtering we should offer
Users can OR them, treenode is already supporting the globbing part we should add something like |
You already have |
Yep it's only used by the protocol atm. If we expose the api in the cli we can fill that object too. |
This would work alongside #3527.
A filter type that could specify a start index and an end index, and pass that filter through to test adapters.
Test adapters could implement this and skip/batch as appropriate. All the testing platform needs to do is pass through a new filter type with the start/end indexes on it.
This would enable users/build systems to run batches of tests on different processes/agents to speed up test runs using parallelism.
Logic could be as follows:
--dotnet run --count-tests
dotnet run --batch-filter 0 99
dotnet run --batch-filter 100 199
dotnet run --batch-filter 200 299
dotnet run --batch-filter 300 399
dotnet run --batch-filter 400 499
The text was updated successfully, but these errors were encountered: