Skip to content

Abort for invalid combinations - #3020

Merged
krmahadevan merged 1 commit into
testng-team:masterfrom
krmahadevan:add_edit_checks
Dec 22, 2023
Merged

krmahadevan merged 1 commit into
testng-team:masterfrom
krmahadevan:add_edit_checks

Conversation

@krmahadevan

Copy link
Copy Markdown
Member

GraphThreadPoolExecutor will NOT work if user is
trying to use a shared thread pool for just their
data driven tests (or) for all tests (both regular and data driven) .

So if these combinations are found, abort execution.

@krmahadevan
krmahadevan requested a review from juherr as a code owner December 21, 2023 06:16

@juherr juherr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe add a test for that

GraphThreadPoolExecutor will NOT work if user is
trying to use a shared thread pool for just their
data driven tests (or) for all tests (both regular and data driven)
.

So if these combinations are found, abort execution.
@krmahadevan

Copy link
Copy Markdown
Member Author

Maybe add a test for that

This is done. Can you please help take another look ?

@krmahadevan
krmahadevan requested a review from juherr December 22, 2023 04:58
@krmahadevan
krmahadevan merged commit ad0fb1c into testng-team:master Dec 22, 2023
@krmahadevan
krmahadevan deleted the add_edit_checks branch December 22, 2023 05:18
@krmahadevan krmahadevan added this to the 7.9.0 milestone Dec 22, 2023
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

Successfully merging this pull request may close these issues.

2 participants