RFR: 7904076: Include timestamp in the summary logs for junit and testng tests [v2]

Jaikiran Pai jpai at openjdk.org
Sat Sep 6 14:09:17 UTC 2025


> Can I please get a review of this change which enhances the log messages printed by jtreg when summarizing the test execution?
> 
> With this change, the started/finished log messages will now look like:
> 
> For testng:
> 
> [09:10:12.473] test p.q.Pass.m1(): success [10ms]
> [09:10:13.123] test p.q.Pass.m2(): success [202ms]
> 
> 
> For junit:
> 
> [09:55:22.278] STARTED    p.JUnitTest::test 'test'
> [09:55:22.279] SUCCESSFUL p.JUnitTest::test 'test' [1ms]
> 
> 
> The timestamp associated with these logs would help during investigating test failures or test timeouts. No new self test has been added given the nature of this change.

Jaikiran Pai has updated the pull request incrementally with three additional commits since the last revision:

 - another remaining final
 - Christian's review - remove final
 - adjust testng self test

-------------

Changes:
  - all: https://git.openjdk.org/jtreg/pull/286/files
  - new: https://git.openjdk.org/jtreg/pull/286/files/bb948df3..51476a2d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jtreg&pr=286&range=01
 - incr: https://webrevs.openjdk.org/?repo=jtreg&pr=286&range=00-01

  Stats: 40 lines in 3 files changed: 2 ins; 13 del; 25 mod
  Patch: https://git.openjdk.org/jtreg/pull/286.diff
  Fetch: git fetch https://git.openjdk.org/jtreg.git pull/286/head:pull/286

PR: https://git.openjdk.org/jtreg/pull/286


More information about the jtreg-dev mailing list