Skip to content

Consolidate fake exceptions in HTTP/2 tests into Http2TestUtil#16712

Merged
normanmaurer merged 1 commit into
netty:4.2from
fru1tworld:fix/http2-fake-exception-cleanup
Apr 29, 2026
Merged

Consolidate fake exceptions in HTTP/2 tests into Http2TestUtil#16712
normanmaurer merged 1 commit into
netty:4.2from
fru1tworld:fix/http2-fake-exception-cleanup

Conversation

@fru1tworld
Copy link
Copy Markdown
Contributor

Motivation:

HTTP/2 tests create ad-hoc fake exceptions to drive error paths.

Modification:

Add Http2TestUtil.FAKE_EXCEPTION (stackless singleton) and replace local instances across 8 test classes.

Result:

Fixes #4274

@normanmaurer normanmaurer added this to the 4.2.13.Final milestone Apr 28, 2026
@normanmaurer normanmaurer 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 Apr 28, 2026
@normanmaurer normanmaurer merged commit 27b6a1b into netty:4.2 Apr 29, 2026
23 checks passed
@netty-project-bot
Copy link
Copy Markdown
Contributor

Auto-port PR for 4.1: #16725

@netty-project-bot
Copy link
Copy Markdown
Contributor

Could not create auto-port PR.
Got conflicts when cherry-picking onto 5.0.

@normanmaurer
Copy link
Copy Markdown
Member

Port for 5.0 #16726

normanmaurer added a commit that referenced this pull request Apr 29, 2026
… (#16726)

Motivation:

HTTP/2 tests create ad-hoc fake exceptions to drive error paths.

Modification:

Add `Http2TestUtil.FAKE_EXCEPTION` (stackless singleton) and replace
local instances across 8 test classes.

Result:

Fixes #4274

Co-authored-by: fru1tworld <[email protected]>
normanmaurer pushed a commit that referenced this pull request Apr 29, 2026
…TestUtil (#16725)

Auto-port of #16712 to 4.1
Cherry-picked commit: 27b6a1b

---
Motivation:

HTTP/2 tests create ad-hoc fake exceptions to drive error paths.

Modification:

Add `Http2TestUtil.FAKE_EXCEPTION` (stackless singleton) and replace
local instances across 8 test classes.

Result:

Fixes #4274

Co-authored-by: fru1tworld <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-cherry-pick-5.0 This PR should be cherry-picked to 5.0 once merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP/2 Fake Exception

3 participants