Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transform @DisabledOnFipsAndJava17 annotation into @DisabledOnFipsAndNative #1372

Closed
michalvavrik opened this issue Oct 17, 2024 · 0 comments · Fixed by #1380
Closed

Transform @DisabledOnFipsAndJava17 annotation into @DisabledOnFipsAndNative #1372

michalvavrik opened this issue Oct 17, 2024 · 0 comments · Fixed by #1380

Comments

@michalvavrik
Copy link
Member

We have https://github.com/quarkus-qe/quarkus-test-framework/blob/main/quarkus-test-core/src/main/java/io/quarkus/test/scenarios/annotations/DisabledOnFipsAndJava17.java that is not used anymore and it is unlikely a new bug comes that is specific for Java 17 and FIPS. We should transform it into @DisabledOnFipsAndNative that is currently repeated twice in our TS:

Maybe make it bit more flexible, like FIPS and some other condition that is added as value? Only @DisabledOnFipsAndNative is required by this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant