[crac] RFR: Convert CRaC tests from shell scripts to Java [v3]
Anton Kozlov
akozlov at openjdk.org
Fri Mar 10 09:15:00 UTC 2023
On Tue, 7 Mar 2023 14:49:27 GMT, Radim Vansa <duke at openjdk.org> wrote:
>> Right now this is a draft as it builds on top of https://github.com/openjdk/crac/pull/47
>>
>> Please see `CracTest` javadoc for detailed info.
>
> Radim Vansa has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove forgotten debugging command
test/jdk/jdk/crac/JarFileFactoryCacheTest/JarFileFactoryCacheTest.java line 61:
> 59: }
> 60: assert temp.toFile().delete();
> 61: }
Looks fine, although not very expected. The test is adjusted to generate the text file, so it is also deleted after the jar is generated.
A nit: probably `temp.resolve("test.txt")` worth to be computed once.
test/jdk/jdk/crac/SecureRandom/InterlockTest.java line 119:
> 117: public static void main(String[] args) throws Exception {
> 118: CracTest.run(InterlockTest.class, args);
> 119: }
Is it a leftover that can be deleted, or do we need to help CracTest to find the test class in this test?
test/jdk/jdk/crac/Selector/Test970/ChannelResource.java line 34:
> 32: public enum SelectionType {SELECT, SELECT_TIMEOUT, SELECT_NOW}
> 33:
> 34: ;
Something wrong with formatting here
test/jdk/jdk/crac/Selector/Test970/ChannelResource.java line 57:
> 55: }
> 56:
> 57: @java.lang.Override
Maybe just `@Override`?
-------------
PR: https://git.openjdk.org/crac/pull/50
More information about the crac-dev
mailing list