Skip to content

Added log to TestHelper and fixed ChunkedInputClosedOnErrorTest - #6110

Merged
dmatej merged 3 commits into
eclipse-ee4j:2.xfrom
dmatej:TestHelperLog
Jul 25, 2026
Merged

Added log to TestHelper and fixed ChunkedInputClosedOnErrorTest#6110
dmatej merged 3 commits into
eclipse-ee4j:2.xfrom
dmatej:TestHelperLog

Conversation

@dmatej

@dmatej dmatej commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Added log to TestHelper to separate logs of generated tests

  • prints ie. this:
Jul 23, 2026 7:29:03 PM org.glassfish.jersey.test.spi.TestHelper lambda$toTestContainer$3 
INFO: Invoking test jaxbTest (SimpleBean, JsonbTestProvider).test

As the build failed again, I backported yet two commits from later versions so ChunkedInputClosedOnErrorTest should stop randomly failing.

- prints ie. this:
  Jul 23, 2026 7:29:03 PM org.glassfish.jersey.test.spi.TestHelper lambda$toTestContainer$3
  INFO: Invoking test jaxbTest (SimpleBean, JsonbTestProvider).test
- Before that it was impossible to find out which test failed.
- Useful for PR eclipse-ee4j#6035

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
@dmatej dmatej added this to the 2.49 milestone Jul 23, 2026
@dmatej
dmatej requested a review from a team July 23, 2026 18:28
@dmatej dmatej self-assigned this Jul 23, 2026
- The test randomly failed and it still does, however now it checks
  the exception, runs more iterations (100 or more instead of 10), and is more
  readable.

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
@dmatej dmatej changed the title Added log to TestHelper to separate logs of generated tests Added log to TestHelper and fixed ChunkedInputClosedOnErrorTest Jul 24, 2026
- flush doesn't guarantee the end of input, however it should be close.

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
@dmatej

dmatej commented Jul 25, 2026

Copy link
Copy Markdown
Contributor Author

@mkarg Can you approve this too?

@dmatej
dmatej merged commit 609ca4f into eclipse-ee4j:2.x Jul 25, 2026
7 checks passed
@dmatej
dmatej deleted the TestHelperLog branch July 25, 2026 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants