[crac] RFR: 8364449: [CRaC] Launch new main outside of the core C/R body [v2]

Timofei Pushkin tpushkin at openjdk.org
Thu Aug 7 08:37:05 UTC 2025


On Thu, 7 Aug 2025 08:29:06 GMT, Timofei Pushkin <tpushkin at openjdk.org> wrote:

>> test/jdk/jdk/crac/newArgs/CheckpointInNewMainTest.java line 59:
>> 
>>> 57:         final var out = builder.doRestore().outputAnalyzer();
>>> 58: 
>>> 59:         out.shouldContain(RESTORE_OLD_MSG).shouldContain(RESTORE_NEW_MSG);
>> 
>> I think that a common pattern is to suppress continuation of the old main. Could you call add a variant that calls `System.exit(0)` in `InternalMain` and assert that `RESTORE_OLD_MSG` is not printed in that case?
>
> Added this to `FailedCheckpointRestoreTest` — felt a bit more appropriate since it tests various points at which execution ends. Although the clearest way would be to add a separate test...

I better add a separate test indeed

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

PR Review Comment: https://git.openjdk.org/crac/pull/253#discussion_r2259558997


More information about the crac-dev mailing list