Skip to content

Fix RunInFastThreadLocalThreadExtension#16314

Merged
normanmaurer merged 3 commits into
netty:4.2from
chrisvest:4.2-ftl-extension
Feb 21, 2026
Merged

Fix RunInFastThreadLocalThreadExtension#16314
normanmaurer merged 3 commits into
netty:4.2from
chrisvest:4.2-ftl-extension

Conversation

@chrisvest
Copy link
Copy Markdown
Member

Motivation:
This JUnit extension is meant to run all the tests in FastThreadLocalThreads. However, it did not intercept repeated or parameterized test invocations.

Modification:
Make the extension also intercept templated tests and dynamic tests.

Result:
Repeated and parameterized tests now also run in FastThreadLocalThreads when using this extension.

Motivation:
This JUnit extension is meant to run all the tests in FastThreadLocalThreads.
However, it did not intercept repeated or parameterized test invocations.

Modification:
Make the extension also intercept templated tests and dynamic tests.

Result:
Repeated and parameterized tests now also run in FastThreadLocalThreads when using this extension.
@chrisvest chrisvest added this to the 4.2.11.Final milestone Feb 20, 2026
@chrisvest chrisvest added needs-cherry-pick-4.1 This PR should be cherry-picked to 4.1 once merged. needs-cherry-pick-5.0 This PR should be cherry-picked to 5.0 once merged. labels Feb 20, 2026
@normanmaurer normanmaurer merged commit 2530b66 into netty:4.2 Feb 21, 2026
19 checks passed
netty-project-bot pushed a commit that referenced this pull request Feb 21, 2026
Motivation:
This JUnit extension is meant to run all the tests in
FastThreadLocalThreads. However, it did not intercept repeated or
parameterized test invocations.

Modification:
Make the extension also intercept templated tests and dynamic tests.

Result:
Repeated and parameterized tests now also run in FastThreadLocalThreads
when using this extension.

Co-authored-by: Norman Maurer <[email protected]>
(cherry picked from commit 2530b66)
@netty-project-bot
Copy link
Copy Markdown
Contributor

Auto-port PR for 4.1: #16328

@github-actions github-actions Bot removed the needs-cherry-pick-4.1 This PR should be cherry-picked to 4.1 once merged. label Feb 21, 2026
netty-project-bot pushed a commit that referenced this pull request Feb 21, 2026
Motivation:
This JUnit extension is meant to run all the tests in
FastThreadLocalThreads. However, it did not intercept repeated or
parameterized test invocations.

Modification:
Make the extension also intercept templated tests and dynamic tests.

Result:
Repeated and parameterized tests now also run in FastThreadLocalThreads
when using this extension.

Co-authored-by: Norman Maurer <[email protected]>
(cherry picked from commit 2530b66)
@netty-project-bot
Copy link
Copy Markdown
Contributor

Auto-port PR for 5.0: #16329

@github-actions github-actions Bot removed the needs-cherry-pick-5.0 This PR should be cherry-picked to 5.0 once merged. label Feb 21, 2026
@chrisvest chrisvest deleted the 4.2-ftl-extension branch February 21, 2026 16:40
chrisvest added a commit that referenced this pull request Feb 21, 2026
Auto-port of #16314 to 5.0
Cherry-picked commit: 2530b66

---
Motivation:
This JUnit extension is meant to run all the tests in
FastThreadLocalThreads. However, it did not intercept repeated or
parameterized test invocations.

Modification:
Make the extension also intercept templated tests and dynamic tests.

Result:
Repeated and parameterized tests now also run in FastThreadLocalThreads
when using this extension.

Co-authored-by: Chris Vest <[email protected]>
Co-authored-by: Norman Maurer <[email protected]>
chrisvest added a commit that referenced this pull request Feb 21, 2026
Auto-port of #16314 to 4.1
Cherry-picked commit: 2530b66

---
Motivation:
This JUnit extension is meant to run all the tests in
FastThreadLocalThreads. However, it did not intercept repeated or
parameterized test invocations.

Modification:
Make the extension also intercept templated tests and dynamic tests.

Result:
Repeated and parameterized tests now also run in FastThreadLocalThreads
when using this extension.

---------

Co-authored-by: Chris Vest <[email protected]>
Co-authored-by: Norman Maurer <[email protected]>
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.

3 participants