[crac] RFR: Add CRaC-specific tests to GHA [v7]
    Anton Kozlov 
    akozlov at openjdk.org
       
    Fri Mar 24 08:24:05 UTC 2023
    
    
  
On Thu, 23 Mar 2023 15:21:53 GMT, Radim Vansa <duke at openjdk.org> wrote:
>> Existing GitHub Actions run test tier1 but since most changes in this project focus on the CRaC capabilities we should run them in an automated fashion, too.
>> Right now the tests are mostly failing: this should be addressed in https://github.com/openjdk/crac/pull/47
>
> Radim Vansa has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Let ReseedTest use stdout rather than exit codes
>   
>   There was a 1:256 chance of test randomly failing because the random
>   generators produced the same number, despite differently seeded.
test/jdk/jdk/crac/SecureRandom/ReseedTest.java line 75:
> 73: 
> 74:         System.out.println(sr.nextInt());
> 75:         System.exit(0);
This change will also help when something is wrong with the restore, the previous version might confuse that situation with legitimate return with non-zero error. 
On the second thought, could you extract this into a separate PR?
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/48#discussion_r1147247593
    
    
More information about the crac-dev
mailing list