Fix new sonars in drools-pdp

Addressed the following sonar issues:
- extract common test method
- too many assertions in test method
- don't always return the same value

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

1 files changed, 6 insertions, 1 deletions

diff --git a/policy-core/src/main/java/org/onap/policy/drools/core/PolicyContainer.java b/policy-core/src/main/java/org/onap/policy/drools/core/PolicyContainer.java
index 6547ca73..5bce3505 100644
--- a/policy-core/src/main/java/org/onap/policy/drools/core/PolicyContainer.java
+++ b/policy-core/src/main/java/org/onap/policy/drools/core/PolicyContainer.java
@@ -537,7 +537,12 @@ public class PolicyContainer implements Startable {
      * {@inheritDoc}.
      */
     @Override
-    public synchronized boolean start() {
+    public synchronized boolean start() {   // NOSONAR
+        /*
+         * disabling sonar about returning the same value, because we prefer the code to
+         * be structured this way
+         */
+
         if (isStarted) {
             return true;
         }