Fixed sonar issues 32/110832/1
authoredyta <edyta.krukowska@nokia.com>
Mon, 3 Aug 2020 05:07:48 +0000 (07:07 +0200)
committeredyta <edyta.krukowska@nokia.com>
Mon, 3 Aug 2020 05:07:48 +0000 (07:07 +0200)
Issue-ID: DCAEGEN2-2254
Signed-off-by: Edyta Krukowska <edyta.krukowska@nokia.com>
Change-Id: Iad63113fd20040504fddb195572d7ce820b83cf6

services/external-schema-manager/src/main/java/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/SchemaReferenceMapper.java
services/external-schema-manager/src/main/java/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/StndDefinedValidator.java
services/external-schema-manager/src/main/java/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/UrlMapper.java
services/external-schema-manager/src/main/java/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/ValidatorCache.java
services/external-schema-manager/src/main/test/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/FileReaderTest.java
services/external-schema-manager/src/main/test/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/JsonFragmentRetrieverTest.java
services/external-schema-manager/src/main/test/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/SchemaReferenceJsonGeneratorTest.java
services/external-schema-manager/src/main/test/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/SchemaReferenceMapperTest.java
services/external-schema-manager/src/main/test/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/StndDefinedValidatorBuilderTest.java
services/external-schema-manager/src/main/test/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/StndDefinedValidatorTest.java
services/external-schema-manager/src/main/test/org/onap/dcaegen2/services/sdk/services/external/schema/manager/service/UrlMapperTest.java

index 79108f0..7963b7c 100644 (file)
@@ -20,6 +20,7 @@
 
 package org.onap.dcaegen2.services.sdk.services.external.schema.manager.service;
 
+import org.onap.dcaegen2.services.sdk.services.external.schema.manager.exception.NoLocalReferenceException;
 import org.onap.dcaegen2.services.sdk.services.external.schema.manager.model.SchemaReference;
 
 import java.io.File;
@@ -38,7 +39,7 @@ class SchemaReferenceMapper {
         return urlMapper;
     }
 
-    SchemaReference mapToLocalSchema(SchemaReference schemaReference) {
+    SchemaReference mapToLocalSchema(SchemaReference schemaReference) throws NoLocalReferenceException {
         String publicUrl = schemaReference.getUrl();
         String localUrl = urlMapper.mapToLocalUrl(publicUrl);
         return createLocalSchemaReference(localUrl, schemaReference.getInternalReference());
index 0b34339..95bb575 100644 (file)
@@ -53,7 +53,7 @@ public class StndDefinedValidator {
      * @param event as JsonNode
      * @return validation result
      * @throws IncorrectInternalFileReferenceException when reference to part of openApi yaml file with schemas is incorrect.
-     * @throws NoLocalReferenceException when mapping for public url is not present in schema mapping file.
+     * @throws NoLocalReferenceException               when mapping for public url is not present in schema mapping file.
      */
     public boolean validate(JsonNode event) throws IncorrectInternalFileReferenceException, NoLocalReferenceException {
         boolean validationResult = false;
@@ -65,7 +65,7 @@ public class StndDefinedValidator {
             validationResult = true;
         } catch (ValidationException ex) {
             logger.error(String.valueOf(ex.results()));
-        } catch (IOException ex){
+        } catch (IOException ex) {
             logger.error("Schema reference has invalid characters", ex);
         }
         return validationResult;
index 06864b3..5edaf46 100644 (file)
@@ -36,7 +36,7 @@ final class UrlMapper {
         return Map.copyOf(mappingsCache);
     }
 
-    String mapToLocalUrl(String publicUrl) {
+    String mapToLocalUrl(String publicUrl) throws NoLocalReferenceException {
         String externalUrl = mappingsCache.get(publicUrl);
         if (externalUrl == null) {
             throw new NoLocalReferenceException("Couldn't find mapping for public url. PublicURL: " + publicUrl);
index e67f0eb..b19768c 100644 (file)
@@ -21,6 +21,7 @@
 package org.onap.dcaegen2.services.sdk.services.external.schema.manager.service;
 
 import com.fasterxml.jackson.databind.JsonNode;
+import org.onap.dcaegen2.services.sdk.services.external.schema.manager.exception.NoLocalReferenceException;
 import org.onap.dcaegen2.services.sdk.services.external.schema.manager.model.SchemaReference;
 import org.onap.dcaegen2.services.sdk.services.external.schema.manager.exception.IncorrectInternalFileReferenceException;
 import org.openapi4j.core.exception.ResolutionException;
@@ -47,7 +48,7 @@ class ValidatorCache {
         return schemaReferenceMapper;
     }
 
-    synchronized SchemaValidator resolveValidator(JsonNode event, String schemaRefPath) throws IOException {
+    synchronized SchemaValidator resolveValidator(JsonNode event, String schemaRefPath) throws IOException, IncorrectInternalFileReferenceException, NoLocalReferenceException {
         SchemaReference schemaReference = resolveSchemaReference(event, schemaRefPath);
         schemaReference = schemaReferenceMapper.mapToLocalSchema(schemaReference);
         SchemaValidator validator = cache.get(schemaReference.getUrl());
@@ -67,7 +68,7 @@ class ValidatorCache {
         return validator != null;
     }
 
-    private SchemaValidator createNewValidator(SchemaReference schemaReference) throws IOException {
+    private SchemaValidator createNewValidator(SchemaReference schemaReference) throws IOException, IncorrectInternalFileReferenceException {
         logger.info("Creating new stndDefined schema validator");
         JsonNode schemaRefNode = SchemaReferenceJsonGenerator.getSchemaReferenceJson(schemaReference);
         SchemaValidator schemaValidator = handleValidatorCreation(schemaRefNode);
@@ -75,7 +76,7 @@ class ValidatorCache {
         return schemaValidator;
     }
 
-    private SchemaValidator handleValidatorCreation(JsonNode schemaNode) {
+    private SchemaValidator handleValidatorCreation(JsonNode schemaNode) throws IncorrectInternalFileReferenceException{
         try {
             return new SchemaValidator("StndDefinedSchemaValidator", schemaNode);
         } catch (ResolutionException e) {
index 5d391a6..8f9f54c 100644 (file)
@@ -31,7 +31,7 @@ class FileReaderTest {
     public static final String TEST_RESOURCES = "src/main/test/resources/";
 
     @Test
-    public void shouldReturnEmptyStringWhenFileNotFound() {
+    void shouldReturnEmptyStringWhenFileNotFound() {
         //given
         String expectedContent = "";
         String fileName = "dummyFileName";
@@ -44,7 +44,7 @@ class FileReaderTest {
     }
 
     @Test
-    public void shouldReturnFileContentWhenFileExists() {
+    void shouldReturnFileContentWhenFileExists() {
         //given
         String expectedContent = "{\n" +
                 "  \"someObject\": \"dummyValue\"\n" +
@@ -59,7 +59,7 @@ class FileReaderTest {
     }
 
     @Test
-    public void shouldReturnFalseWhenFileDoesNotExist() {
+    void shouldReturnFalseWhenFileDoesNotExist() {
         //when
         boolean doesFileExists = new FileReader("dummyFileName").doesFileExists();
 
@@ -68,7 +68,7 @@ class FileReaderTest {
     }
 
     @Test
-    public void shouldReturnTrueWhenFileExists() {
+    void shouldReturnTrueWhenFileExists() {
         //when
         boolean doesFileExists = new FileReader(TEST_RESOURCES + "file_with_one_line.json").doesFileExists();
 
index f1cae3f..b5e3df6 100644 (file)
@@ -41,7 +41,7 @@ class JsonFragmentRetrieverTest {
     private ObjectMapper objectMapper = new ObjectMapper();
 
     @Test
-    public void shouldReturnJsonFragmentAtValidPath() throws IOException {
+    void shouldReturnJsonFragmentAtValidPath() throws IOException {
         //given
         JsonNode jsonContent = objectMapper.readTree(VALID_JSON_CONTENT);
         JsonNode expectedJsonNode = objectMapper.readTree("true");
@@ -55,7 +55,7 @@ class JsonFragmentRetrieverTest {
     }
 
     @Test
-    public void shouldThrowErrorWhenPathDoesNotExistInJsonContent() throws IOException {
+    void shouldThrowErrorWhenPathDoesNotExistInJsonContent() throws IOException {
         //given
         JsonNode jsonContent = objectMapper.readTree(VALID_JSON_CONTENT);
         String dummyPath = "dummyPath";
index 42bc21c..4c8cd18 100644 (file)
@@ -33,7 +33,7 @@ import static org.junit.jupiter.api.Assertions.assertThrows;
 class SchemaReferenceJsonGeneratorTest {
 
     @Test
-    public void shouldReturnValidSchemaReferenceWhenUrlIsInValidFormat() throws IOException {
+    void shouldReturnValidSchemaReferenceWhenUrlIsInValidFormat() throws IOException {
         //given
         String validUrl = "src/main/test/resources/file_with_one_line.json#/elo";
         String schemaReferenceContent = "{\"$ref\":\"" + validUrl + "\"}";
@@ -49,7 +49,7 @@ class SchemaReferenceJsonGeneratorTest {
     }
 
     @Test
-    public void shouldThrowErrorWhenUrlIsInInvalidFormat() {
+    void shouldThrowErrorWhenUrlIsInInvalidFormat() {
         //given
         String invalidFormatUrl = "\"someDummyValue\n\t";
         SchemaReferenceResolver schemaReferenceResolver = new SchemaReferenceResolver(invalidFormatUrl);
index 1f5ddfb..1658b51 100644 (file)
@@ -30,7 +30,7 @@ class SchemaReferenceMapperTest {
     public static final String SCHEMAS_PATH = "src/main/test/resources";
 
     @Test
-    public void shouldReturnProperReferenceWhenSchemaReferenceHasNoHash() {
+    void shouldReturnProperReferenceWhenSchemaReferenceHasNoHash() {
         //given
         SchemaReferenceMapper schemaReferenceMapper = getSchemaReferenceMapper();
         String publicUrlWithoutHash = "http://someExternalUrl/external";
@@ -47,7 +47,7 @@ class SchemaReferenceMapperTest {
     }
 
     @Test
-    public void shouldReturnProperReferenceWhenSchemaReferenceContainsHash() {
+    void shouldReturnProperReferenceWhenSchemaReferenceContainsHash() {
         //given
         SchemaReferenceMapper schemaReferenceMapper = getSchemaReferenceMapper();
         String publicUrlWithHash = "http://someExternalUrl/external#someString";
index 5663154..5e21be5 100644 (file)
@@ -26,7 +26,7 @@ import java.util.Map;
 
 import static org.assertj.core.api.Assertions.assertThat;
 
-public class StndDefinedValidatorBuilderTest {
+class StndDefinedValidatorBuilderTest {
 
     private static final String TEST_RESOURCES = "src/main/test/resources/externalRepo/";
 
@@ -38,7 +38,7 @@ public class StndDefinedValidatorBuilderTest {
         Map<String, String> mappingsCache = getMappingsCache(validator);
 
         //then
-        assertThat(mappingsCache.size()).isEqualTo(6);
+        assertThat(mappingsCache).hasSize(6);
     }
 
     @Test
@@ -48,7 +48,7 @@ public class StndDefinedValidatorBuilderTest {
         Map<String, String> mappingsCache = getMappingsCache(validator);
 
         //then
-        assertThat(mappingsCache.size()).isEqualTo(5);
+        assertThat(mappingsCache).hasSize(5);
     }
 
     @Test
@@ -58,7 +58,7 @@ public class StndDefinedValidatorBuilderTest {
         Map<String, String> mappingsCache = getMappingsCache(validator);
 
         //then
-        assertThat(mappingsCache.size()).isEqualTo(4);
+        assertThat(mappingsCache).hasSize(4);
     }
 
     @Test
@@ -68,7 +68,7 @@ public class StndDefinedValidatorBuilderTest {
         Map<String, String> mappingsCache = getMappingsCache(validator);
 
         //then
-        assertThat(mappingsCache.size()).isEqualTo(3);
+        assertThat(mappingsCache).hasSize(3);
     }
 
     @Test
@@ -78,7 +78,7 @@ public class StndDefinedValidatorBuilderTest {
         Map<String, String> mappingsCache = getMappingsCache(validator);
 
         //then
-        assertThat(mappingsCache.size()).isEqualTo(0);
+        assertThat(mappingsCache.size()).isZero();
     }
 
     @Test
@@ -88,7 +88,7 @@ public class StndDefinedValidatorBuilderTest {
         Map<String, String> mappingsCache = getMappingsCache(validator);
 
         //then
-        assertThat(mappingsCache.size()).isEqualTo(0);
+        assertThat(mappingsCache.size()).isZero();
     }
 
     private StndDefinedValidator getValidator(String mappingFilePath) {
index be4bea9..f53b70c 100644 (file)
@@ -32,7 +32,7 @@ import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-public class StndDefinedValidatorTest {
+class StndDefinedValidatorTest {
 
     private static final String TEST_RESOURCES = "src/main/test/resources/";
     private static final String MAPPING_FILE_PATH = TEST_RESOURCES + "externalRepo/schema-map.json";
index 60214eb..0031094 100644 (file)
@@ -40,7 +40,7 @@ class UrlMapperTest {
     }
 
     @Test
-    public void shouldThrowExceptionWhenNoMappingExists() {
+    void shouldThrowExceptionWhenNoMappingExists() {
         //given
         String notMappedPublicUrl = "http://localhost:8080/notExisting";
 
@@ -50,7 +50,7 @@ class UrlMapperTest {
     }
 
     @Test
-    public void shouldThrowExceptionWhenLocalSchemaFileIsEmpty() {
+    void shouldThrowExceptionWhenLocalSchemaFileIsEmpty() {
         //given
         String publicUrlToEmptyLocal = "http://someExternalUrl/emptySchema";
 
@@ -60,7 +60,7 @@ class UrlMapperTest {
     }
 
     @Test
-    public void shouldThrowExceptionWhenFileHasInvalidYamlStructure() {
+    void shouldThrowExceptionWhenFileHasInvalidYamlStructure() {
         //given
         String publicUrlToInvalidYamlLocal = "http://someExternalUrl/invalidYamlFile";
 
@@ -70,7 +70,7 @@ class UrlMapperTest {
     }
 
     @Test
-    public void shouldThrowExceptionWhenLocalFileDoesNotExist() {
+    void shouldThrowExceptionWhenLocalFileDoesNotExist() {
         //given
         String publicUrlToNotExistingLocalFile = "http://someExternalUrl/missingFile";
 
@@ -80,7 +80,7 @@ class UrlMapperTest {
     }
 
     @Test
-    public void shouldReturnLocalUrlWhenFileValidAndFound() {
+    void shouldReturnLocalUrlWhenFileValidAndFound() {
         //given
         String publicUrl = "http://someExternalUrl/external";
 
@@ -90,7 +90,7 @@ class UrlMapperTest {
     }
 
     @Test
-    public void shouldNotThrowExceptionWhenMappingFileDoesNotExist() {
+    void shouldNotThrowExceptionWhenMappingFileDoesNotExist() {
         String invalidMappingFilePath = "src/main/test/resources/missing-schema.json";
 
         Assertions.assertDoesNotThrow(() -> new UrlMapperFactory().getUrlMapper(invalidMappingFilePath, SCHEMAS_PATH));