Skip to content

Migration of tests to VertxTestBase stateless mode#6052

Merged
vietj merged 26 commits intomasterfrom
migrate-http-test-to-stateless-test
Mar 31, 2026
Merged

Migration of tests to VertxTestBase stateless mode#6052
vietj merged 26 commits intomasterfrom
migrate-http-test-to-stateless-test

Conversation

@vietj
Copy link
Copy Markdown
Member

@vietj vietj commented Mar 30, 2026

Motivation:

Migrate existing usage of VertxTestBase to stateless assertions.

An usage is typically migrated with:

  1. Switch the test to stateless mode
  2. Run the test, it will log incorrect usage of AsyncTestBase assertions which cannot reliably fail the test
  3. Fix the incorrect unreliable assertions, make sure that those assertions are called from a thread capable of doing so
  4. Migrate the test from AsyncTestBase assertion usage and instead static asserts such as JUnit or AssertJ (this can be achieved by an agent)

This covers a portion of the (large) vertx test base.

@vietj vietj added this to the 5.1.0 milestone Mar 30, 2026
@vietj vietj force-pushed the migrate-http-test-to-stateless-test branch from 6eaba51 to eaa45d1 Compare March 31, 2026 09:41
vietj added 20 commits March 31, 2026 14:01
…can be safely be reused for tests in static mode.
Some portions of this content were created with the assistance of Claude Code.
Some portions of this content were created with the assistance of Claude Code.
Some portions of this content were created with the assistance of Claude Code.
…ic asserts.

Some portions of this content were created with the assistance of Claude Code.
…rts.

Some portions of this content were created with the assistance of Claude Code.
Some portions of this content were created with the assistance of Claude Code.
Some portions of this content were created with the assistance of Claude Code.
Some portions of this content were created with the assistance of Claude Code.
Some portions of this content were created with the assistance of Claude Code.
@vietj vietj force-pushed the migrate-http-test-to-stateless-test branch from eaa45d1 to 6f8016c Compare March 31, 2026 12:02
vietj added 6 commits March 31, 2026 14:20
Some portions of this content were created with the assistance of Claude Code.
Some portions of this content were created with the assistance of Claude Code.
Some portions of this content were created with the assistance of Claude Code.
@vietj vietj force-pushed the migrate-http-test-to-stateless-test branch from 451abf5 to d7af36f Compare March 31, 2026 14:18
@vietj vietj merged commit 7ed8c53 into master Mar 31, 2026
7 checks passed
@vietj vietj deleted the migrate-http-test-to-stateless-test branch March 31, 2026 14:45
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.

1 participant