Fix flaky tests

simonelbaz · simonelbaz · commit a32ea6bc84d5 · 2026-04-29T16:46:10.000Z
Signed-off-by: Simon ELBAZ &lt;elbazsimon9@gmail.com&gt;
diff --git a/data-prepper-plugins/failures-common/src/test/java/org/opensearch/dataprepper/plugins/dlq/s3/S3DlqWriterConfigTest.java b/data-prepper-plugins/failures-common/src/test/java/org/opensearch/dataprepper/plugins/dlq/s3/S3DlqWriterConfigTest.java
@@ -68,11 +68,10 @@ public void getS3ClientWithValidStsRoleArn(final String stsRoleArn) throws NoSuc
     }
 
     @ParameterizedTest
-    @NullSource
     @ValueSource(booleans = {false, true})
     public void getS3ClientWithValidAccessStyle(final boolean forcePathStyle) throws NoSuchFieldException, IllegalAccessException {
         final S3DlqWriterConfig config = new S3DlqWriterConfig();
-        reflectivelySetField(config, "force_path_style", forcePathStyle);
+        reflectivelySetField(config, "forcePathStyle", forcePathStyle);
         final S3Client s3Client = config.getS3Client();
         assertThat(s3Client, is(notNullValue()));
     }

Original file line number	Diff line number	Diff line change
`@@ -68,11 +68,10 @@ public void getS3ClientWithValidStsRoleArn(final String stsRoleArn) throws NoSuc`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`@ParameterizedTest`
`71`		`- @NullSource`
`72`	`71`	`@ValueSource(booleans = {false, true})`
`73`	`72`	`public void getS3ClientWithValidAccessStyle(final boolean forcePathStyle) throws NoSuchFieldException, IllegalAccessException {`
`74`	`73`	`final S3DlqWriterConfig config = new S3DlqWriterConfig();`
`75`		`- reflectivelySetField(config, "force_path_style", forcePathStyle);`
	`74`	`+ reflectivelySetField(config, "forcePathStyle", forcePathStyle);`
`76`	`75`	`final S3Client s3Client = config.getS3Client();`
`77`	`76`	`assertThat(s3Client, is(notNullValue()));`
`78`	`77`	`}`