Last sonar fix for drools-apps

Remaining issues will be examined/addressed in Guilin.

Issue-ID: POLICY-2426
Signed-off-by: Jim Hahn <jrh3@att.com>
Change-Id: I2354122ed40e27de472a346001aff4e46d27ccf6

diff --git a/controlloop/common/rules-test/pom.xml b/controlloop/common/rules-test/pom.xml
index ddd897c..8b24f29 100644 (file)
--- a/controlloop/common/rules-test/pom.xml
+++ b/controlloop/common/rules-test/pom.xml
@@ -94,12 +94,6 @@
             <artifactId>simulators</artifactId>
             <version>${policy.models.version}</version>
         </dependency>
-        <dependency>
-            <groupId>org.onap.policy.models.policy-models-interactions.model-impl</groupId>
-            <artifactId>events</artifactId>
-            <version>${policy.models.version}</version>
-            <scope>provided</scope>
-        </dependency>
         <dependency>
             <groupId>org.onap.policy.drools-applications.controlloop.common</groupId>
             <artifactId>eventmanager</artifactId>

diff --git a/controlloop/common/rules-test/src/test/java/org/onap/policy/controlloop/common/rules/test/NamedRunnerTest.java b/controlloop/common/rules-test/src/test/java/org/onap/policy/controlloop/common/rules/test/NamedRunnerTest.java
index fe9ff80..6ee526e 100644 (file)
--- a/controlloop/common/rules-test/src/test/java/org/onap/policy/controlloop/common/rules/test/NamedRunnerTest.java
+++ b/controlloop/common/rules-test/src/test/java/org/onap/policy/controlloop/common/rules/test/NamedRunnerTest.java
@@ -59,8 +59,13 @@ public class NamedRunnerTest {
         checkTest();
     }
 
+    /*
+     * Note: this test is purposely marked with the "Ignore" annotation to verify that the
+     * NamedRunner skips over it, hence the sonar issue is being suppressed.
+     */
     @Test
     @Ignore
+    @SuppressWarnings("java:S1607")
     public void testIgnore() {
         fail("should not run");
     }