fix assert pmd hack

csviri · csviri · commit 01ff6e667ab5 · 2025-07-14T10:42:12.000+02:00
Signed-off-by: Attila Mészáros &lt;a_meszaros@apple.com&gt;
diff --git a/spark-operator/src/test/java/org/apache/spark/k8s/operator/config/SparkOperatorConfigMapReconcilerTest.java b/spark-operator/src/test/java/org/apache/spark/k8s/operator/config/SparkOperatorConfigMapReconcilerTest.java
@@ -65,6 +65,7 @@ void stopController() {
   }
 
   @Test
+  @SuppressWarnings("PMD.UnitTestShouldIncludeAssert")
   void sanityTest() {
     client.resource(testConfigMap()).create();
 
@@ -73,8 +74,6 @@ void sanityTest() {
             () -> {
               assertThat(RECONCILER_INTERVAL_SECONDS.getValue()).isEqualTo(60L);
             });
-    // adding this here to make pmd happy
-    assertThat(RECONCILER_INTERVAL_SECONDS.getValue()).isEqualTo(60L);
   }
 
   ConfigMap testConfigMap() {

Original file line number	Diff line number	Diff line change
`@@ -65,6 +65,7 @@ void stopController() {`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`@Test`
	`68`	`+ @SuppressWarnings("PMD.UnitTestShouldIncludeAssert")`
`68`	`69`	`void sanityTest() {`
`69`	`70`	`client.resource(testConfigMap()).create();`
`70`	`71`
`@@ -73,8 +74,6 @@ void sanityTest() {`
`73`	`74`	`() -> {`
`74`	`75`	`assertThat(RECONCILER_INTERVAL_SECONDS.getValue()).isEqualTo(60L);`
`75`	`76`	`});`
`76`		`- // adding this here to make pmd happy`
`77`		`- assertThat(RECONCILER_INTERVAL_SECONDS.getValue()).isEqualTo(60L);`
`78`	`77`	`}`
`79`	`78`
`80`	`79`	`ConfigMap testConfigMap() {`