On Sat, 27 Jun 2026 15:45:34 GMT, Jaikiran Pai <[email protected]> wrote:
> Can I please get a review of this test-only change which addresses the issues > in its parameterized tests, as noted here > https://github.com/openjdk/jdk/pull/31123/changes#r3486082890? > > While at it, I also did a general clean up of the test and added comments to > the test methods and the classes used in that test to clarify what's being > covered by this test. > > The test continues to pass after this change and the test methods get run > more number of times (as expected) due to fixing the parameters being > generated for the parameterized tests. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). This pull request has now been integrated. Changeset: 166b6532 Author: Jaikiran Pai <[email protected]> Committer: Chen Liang <[email protected]> URL: https://git.openjdk.org/valhalla/commit/166b6532dc1a278cdf12b28673aae1c9906a9681 Stats: 429 lines in 1 file changed: 264 ins; 70 del; 95 mod 8387374: [lworld] Address test issues in test/jdk/java/io/Serializable/valueObjects/ValueSerializationTest.java Reviewed-by: liach, vklang ------------- PR: https://git.openjdk.org/valhalla/pull/2590
