Skip to content

Ensure setup is only called once in VirtualThreadEventBusTest, VirtualThreadContextTest and NetBandwidthLimitingTest [4.x]#5859

Merged
vietj merged 1 commit intoeclipse-vertx:4.xfrom
ahus1:is-5857-backport-4.x
Dec 30, 2025
Merged

Ensure setup is only called once in VirtualThreadEventBusTest, VirtualThreadContextTest and NetBandwidthLimitingTest [4.x]#5859
vietj merged 1 commit intoeclipse-vertx:4.xfrom
ahus1:is-5857-backport-4.x

Conversation

@ahus1
Copy link
Contributor

@ahus1 ahus1 commented Dec 27, 2025

Closes #5857

(cherry picked from commit 57b9c68)

Motivation:

Setup should not be called twice due to the @Before anotation

@ahus1 ahus1 changed the title Ensure setup is only called onceEnsure setup is only called once in VirtualThreadEventBusTest, VirtualThreadContextTest and NetBandwidthLimitingTest [4.x] Ensure setup is only called once in VirtualThreadEventBusTest, VirtualThreadContextTest and NetBandwidthLimitingTest [4.x] Dec 27, 2025
@ahus1
Copy link
Contributor Author

ahus1 commented Dec 27, 2025

@vietj - this is a backport of my changes to the master branch that you reviewed earlier.

There is a failing test and I opened the following issue to track it. I already found the cause and proposed a fix, still I might need more discussion.

I hope this PR can still be merged as the CI failure is unrelated to this PR.

Please review when you have the time. Thanks!

@ahus1 ahus1 marked this pull request as ready for review December 27, 2025 12:24
@vietj vietj added this to the 4.5.24 milestone Dec 30, 2025
@vietj vietj merged commit 5762bdf into eclipse-vertx:4.x Dec 30, 2025
5 of 7 checks passed
@ahus1 ahus1 deleted the is-5857-backport-4.x branch December 30, 2025 10:08
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