[foreign-preview] RFR: 8281147: Other tests that use the foreign API are missing --enable-preview

Julia Boes jboes at openjdk.java.net
Thu Feb 3 10:39:34 UTC 2022


On Wed, 2 Feb 2022 22:55:49 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> This change adds the --enable-preview flag to tests in other areas (jdk.incubator.vector, java.util.stream) that use the (to be previewed) foreign API, as well as cleaning up the use of a no longer needed proxy class.
>
> test/jdk/jdk/incubator/vector/Byte128VectorLoadStoreTests.java line 26:
> 
>> 24: /*
>> 25:  * @test
>> 26:  * @enablePreview
> 
> Can we do the same trick you did for the stream API test, and add a new TEST.properties file in this folder and enable preview there? Or, I assume you tried and that didn't work?

Let's refrain from adding a properties file, given that there’s a template file already that defines test configuration (where I added `@enablePreview`), test/jdk/jdk/incubator/vector/templates/X-LoadStoreTest.java.template.

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

PR: https://git.openjdk.java.net/panama-foreign/pull/636


More information about the panama-dev mailing list