SonarQube fixes 75/116875/2
authorToineSiebelink <toine.siebelink@est.tech>
Thu, 14 Jan 2021 11:29:40 +0000 (11:29 +0000)
committerToineSiebelink <toine.siebelink@est.tech>
Thu, 14 Jan 2021 14:22:06 +0000 (14:22 +0000)
- Ignore SonarQube violations related to inheritance depth for exception classes
- Increase coverage service module

Issue-ID: CPS-89

Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Change-Id: I7998dee0f9abea0638639e0587b08febad6075f5

cps-service/src/main/java/org/onap/cps/spi/exceptions/AnchorAlreadyDefinedException.java
cps-service/src/main/java/org/onap/cps/spi/exceptions/DataspaceAlreadyDefinedException.java
cps-service/src/main/java/org/onap/cps/spi/exceptions/DataspaceNotFoundException.java
cps-service/src/main/java/org/onap/cps/spi/exceptions/NotFoundInDataspaceException.java
cps-service/src/main/java/org/onap/cps/spi/exceptions/SchemaSetAlreadyDefinedException.java
cps-service/src/main/java/org/onap/cps/spi/exceptions/SchemaSetNotFoundException.java
cps-service/src/test/groovy/org/onap/cps/spi/exceptions/CpsExceptionsSpec.groovy [new file with mode: 0644]

index efdb86e..2a4abad 100644 (file)
 
 package org.onap.cps.spi.exceptions;
 
+/**
+ * Anchor already defined exception. Indicates the an anchor with same name already exists in the same dataspace
+ */
+
+@SuppressWarnings("squid:S110")  // Team agreed to accept 6 levels of inheritance for CPS Exceptions
 public class AnchorAlreadyDefinedException extends CpsAdminException {
 
     private static final long serialVersionUID = 5744381546778730691L;
index d6d933c..fcc085a 100644 (file)
@@ -22,6 +22,8 @@ package org.onap.cps.spi.exceptions;
 /**
  * Dataspace already defined exception. Indicates the dataspace with same name already exists.
  */
+
+@SuppressWarnings("squid:S110")  // Team agreed to accept 6 levels of inheritance for CPS Exceptions
 public class DataspaceAlreadyDefinedException extends CpsAdminException {
 
     private static final long serialVersionUID = -5813793951842079228L;
index 58fcde3..42b77a2 100644 (file)
 
 package org.onap.cps.spi.exceptions;
 
-
 /**
  * Dataspace Not Found Exception. Indicates the requested data being absent.
  */
+
+@SuppressWarnings("squid:S110")  // Team agreed to accept 6 levels of inheritance for CPS Exceptions
 public class DataspaceNotFoundException extends CpsAdminException {
 
     private static final long serialVersionUID = -1852996415384288431L;
index 66cce1b..56a2f87 100644 (file)
 
 package org.onap.cps.spi.exceptions;
 
-
 /**
- * CP Service exception. Indicates the requested data being absent in a given dataspace
+ * Not found in dataspace exception. Indicates the requested data being absent in a given dataspace
  */
 public class NotFoundInDataspaceException extends CpsException {
-    
+
     private static final long serialVersionUID = 3838769447129047713L;
 
     public NotFoundInDataspaceException(final String dataspaceName, final String descriptionOfObject) {
index b376672..9376d68 100644 (file)
@@ -19,6 +19,9 @@
 
 package org.onap.cps.spi.exceptions;
 
+/**
+ * Schema set already defined exception. Indicates the a schema set with same name already exists in the same dataspace
+ */
 public class SchemaSetAlreadyDefinedException extends CpsAdminException {
 
     private static final long serialVersionUID = 501929839139881112L;
index f92ca37..cf63f92 100644 (file)
 
 package org.onap.cps.spi.exceptions;
 
+/**
+ * Schema set not found exception. Indicates the schema set is not found in a given dataspace
+ */
+
+@SuppressWarnings("squid:S110")  // Team agreed to accept 6 levels of inheritance for CPS Exceptions
 public class SchemaSetNotFoundException extends CpsAdminException {
 
     private static final long serialVersionUID = 7422782395935450035L;
@@ -33,4 +38,4 @@ public class SchemaSetNotFoundException extends CpsAdminException {
         super("Schema Set not found.",
                 String.format("Schema Set with name %s was not found for dataspace %s.", schemaSetName, dataspaceName));
     }
-}
\ No newline at end of file
+}
diff --git a/cps-service/src/test/groovy/org/onap/cps/spi/exceptions/CpsExceptionsSpec.groovy b/cps-service/src/test/groovy/org/onap/cps/spi/exceptions/CpsExceptionsSpec.groovy
new file mode 100644 (file)
index 0000000..5c6a16d
--- /dev/null
@@ -0,0 +1,107 @@
+/*
+ * ============LICENSE_START=======================================================
+ *  Copyright (C) 2020 Nordix Foundation
+ *  ================================================================================
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *        http://www.apache.org/licenses/LICENSE-2.0
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ *
+ *  SPDX-License-Identifier: Apache-2.0
+ *  ============LICENSE_END=========================================================
+ */
+package org.onap.cps.spi.exceptions
+
+import spock.lang.Specification
+
+class CpsExceptionsSpec extends Specification {
+    def dataspaceName = 'some data space'
+    def anchorName = 'some anchor'
+    def schemaSetName = 'some schema set'
+    def rootCause = new Throwable()
+    def providedMessage = 'some message'
+    def providedDetails = 'some details'
+
+    def 'Creating an exception that the Anchor already exist.'() {
+        given: 'an exception dat the Anchor already exist is created'
+            def exception = new AnchorAlreadyDefinedException(dataspaceName, anchorName, rootCause)
+        expect: 'the exception details contains the correct message with Anchor name and Dataspace name'
+            exception.details == "Anchor with name ${anchorName} already exists for dataspace ${dataspaceName}."
+        and: 'the correct root cause is maintained'
+            exception.cause == rootCause
+    }
+
+    def 'Creating an exception that the dataspace already exists.'() {
+        given: 'an exception that the dataspace already exists is created'
+            def exception = new DataspaceAlreadyDefinedException(dataspaceName, rootCause)
+        expect: 'the exception details contains the correct message with dataspace name'
+            exception.details == "Dataspace with name ${dataspaceName} already exists."
+        and: 'the correct root cause is maintained'
+            exception.cause == rootCause
+    }
+
+    def 'Creating a exception that a dataspace is not found.'() {
+        expect: 'the exception details contains the correct message with dataspace name'
+            (new DataspaceNotFoundException(dataspaceName)).details
+                    == "Dataspace with name ${dataspaceName} does not exist."
+    }
+
+    def'Creating a data validation exception.'() {
+        given: 'a data validation exception is created'
+            def exception = new DataValidationException(providedMessage, providedDetails, rootCause)
+        expect: 'the exception has the provided message'
+            exception.message == providedMessage
+        and: 'the exception has the provided details'
+            exception.details == providedDetails
+        and: 'the correct root cause is maintained'
+            exception.cause == rootCause
+    }
+
+    def'Creating a model validation exception.'() {
+        given: 'a data validation exception is created'
+            def exception = new ModelValidationException(providedMessage, providedDetails)
+        expect: 'the exception has the provided message'
+            exception.message == providedMessage
+        and: 'the exception has the provided details'
+            exception.details == providedDetails
+    }
+
+    def 'Creating a model validation exception with a root cause.'() {
+        given: 'a model validation exception is created'
+            def exception = new ModelValidationException(providedMessage, providedDetails, rootCause)
+        expect: 'the exception has the provided message'
+            exception.message == providedMessage
+        and: 'the exception has the provided details'
+            exception.details == providedDetails
+        and: 'the correct root cause is maintained'
+            exception.cause == rootCause
+    }
+
+    def 'Creating a exception for an object not found in a dataspace.'() {
+        def descriptionOfObject = 'some object'
+        expect: 'the exception details contains the correct message with dataspace name and description of the object'
+            (new NotFoundInDataspaceException(dataspaceName,descriptionOfObject)).details
+                    == "${descriptionOfObject} does not exist in dataspace ${dataspaceName}."
+    }
+
+    def 'Creating an exception that the schema set already exists.'() {
+        given: 'an exception that the schema set already exists is created'
+            def exception = new SchemaSetAlreadyDefinedException(dataspaceName, schemaSetName, rootCause)
+        expect: 'the exception details contains the correct message with dataspace and schema set names'
+            exception.details == "Schema Set with name ${schemaSetName} already exists for dataspace ${dataspaceName}."
+        and: 'the correct root cause is maintained'
+            exception.cause == rootCause
+    }
+
+    def 'Creating a exception that a schema set cannot be found.'() {
+        expect: 'the exception details contains the correct message with dataspace and schema set names'
+            (new SchemaSetNotFoundException(dataspaceName,schemaSetName)).details
+                    == "Schema Set with name ${schemaSetName} was not found for dataspace ${dataspaceName}."
+    }
+}