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

Runner Discovered Suites #2319

Merged
merged 5 commits into from
Jun 8, 2024

Conversation

cheeseng
Copy link
Contributor

Expose discovered suites through Runner's discoveredSuites.

@bvenners
Copy link
Contributor

@neko-kai Can you test out this PR to see if it adequately addresses your needs?

@neko-kai
Copy link

@bvenners Yes, I just tested – it fully solves the problem! (7mind/izumi#2120)

@cheeseng cheeseng requested a review from bvenners May 31, 2024 04:26
@cheeseng
Copy link
Contributor Author

@bvenners I resolved the conflicts, I can merge this in after your review.

Thanks.

(if (wildcard.isEmpty && membersOnly.isEmpty) taskDefs else (filterWildcard(wildcard, taskDefs) ++ filterMembersOnly(membersOnly, taskDefs)).distinct).map(createTask)
val tasks = (if (wildcard.isEmpty && membersOnly.isEmpty) taskDefs else (filterWildcard(wildcard, taskDefs) ++ filterMembersOnly(membersOnly, taskDefs)).distinct).map(createTask)

//runnerInstance().internalDiscoveredSuites.set(Some(discoveredSuites))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cheeseng Why is this line of code commented out?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bvenners The discovered suites are set in line 170 already, I think I just forgot to remove this commented line, this feature is supported on native as well.

@cheeseng cheeseng merged commit 0ff2439 into scalatest:main Jun 8, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants