Skip FIPS JVMs in testReloadCredentialsFromKeystore (elastic#116814)

DaveCTurner · DaveCTurner · commit b0a69565b24c · 2024-11-20T09:25:13.000+11:00
This test doesn't need to run in FIPS mode, and apparently it fails sometimes, so with this commit we skip it. Closes elastic#116811 (cherry picked from commit ea90fbc) # Conflicts: # muted-tests.yml
diff --git a/modules/repository-s3/src/javaRestTest/java/org/elasticsearch/repositories/s3/RepositoryS3RestIT.java b/modules/repository-s3/src/javaRestTest/java/org/elasticsearch/repositories/s3/RepositoryS3RestIT.java
@@ -52,6 +52,8 @@ protected String getTestRestCluster() {
     }
 
     public void testReloadCredentialsFromKeystore() throws IOException {
+        assumeFalse("doesn't work in a FIPS JVM, but that's ok", inFipsJvm());
+
         // Register repository (?verify=false because we don't have access to the blob store yet)
         final var repositoryName = randomIdentifier();
         registerRepository(

Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,8 @@ protected String getTestRestCluster() {`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`public void testReloadCredentialsFromKeystore() throws IOException {`
	`55`	`+ assumeFalse("doesn't work in a FIPS JVM, but that's ok", inFipsJvm());`
	`56`	`+`
`55`	`57`	`// Register repository (?verify=false because we don't have access to the blob store yet)`
`56`	`58`	`final var repositoryName = randomIdentifier();`
`57`	`59`	`registerRepository(`