Fix sonar code smells

Issue-ID: SDC-3185
Signed-off-by: Remigiusz Janeczek <remigiusz.janeczek@nokia.com>
Change-Id: Icfb466816c76c0444b7132651038768a832c03a2

3 files changed, 20 insertions, 17 deletions

diff --git a/src/test/java/org/onap/sdc/helmvalidator/api/SupportedVersionsControllerTest.java b/src/test/java/org/onap/sdc/helmvalidator/api/SupportedVersionsControllerTest.java
index ef93a90..4b653b0 100644
--- a/src/test/java/org/onap/sdc/helmvalidator/api/SupportedVersionsControllerTest.java
+++ b/src/test/java/org/onap/sdc/helmvalidator/api/SupportedVersionsControllerTest.java
@@ -60,8 +60,9 @@ class SupportedVersionsControllerTest {
             .supportedVersions();
         List<String> supportedVersions = supportedVersionsResponse.getBody().get(VERSIONS);
 
-        assertThat(supportedVersions).isNotNull();
-        assertThat(supportedVersions).hasSize(EXPECTED_SIZE);
-        assertThat(supportedVersions).contains(SAMPLE_VERSION1, SAMPLE_VERSION2, SAMPLE_VERSION3);
+        assertThat(supportedVersions)
+            .isNotNull()
+            .hasSize(EXPECTED_SIZE)
+            .containsExactly(SAMPLE_VERSION1, SAMPLE_VERSION2, SAMPLE_VERSION3);
     }
 }
diff --git a/src/test/java/org/onap/sdc/helmvalidator/api/ValidationControllerTest.java b/src/test/java/org/onap/sdc/helmvalidator/api/ValidationControllerTest.java
index 03f976a..3c87d7a 100644
--- a/src/test/java/org/onap/sdc/helmvalidator/api/ValidationControllerTest.java
+++ b/src/test/java/org/onap/sdc/helmvalidator/api/ValidationControllerTest.java
@@ -156,12 +156,13 @@ class ValidationControllerTest {
             .andReturn();
         String contentAsString = mvcResult.getResponse().getContentAsString();
 
-        assertThat(contentAsString).contains(VALID);
-        assertThat(contentAsString).contains(DEPLOYABLE);
-        assertThat(contentAsString).contains(RENDER_ERRORS);
-        assertThat(contentAsString).contains(LINT_WARNING);
-        assertThat(contentAsString).contains(LINT_ERROR);
-        assertThat(contentAsString).contains(VERSION_USED);
+        assertThat(contentAsString)
+            .contains(VALID)
+            .contains(DEPLOYABLE)
+            .contains(RENDER_ERRORS)
+            .contains(LINT_WARNING)
+            .contains(LINT_ERROR)
+            .contains(VERSION_USED);
         assertThat(mvcResult.getResponse().getStatus()).isEqualTo(HttpStatus.OK.value());
     }
 
@@ -184,12 +185,13 @@ class ValidationControllerTest {
             .andReturn();
         String contentAsString = mvcResult.getResponse().getContentAsString();
 
-        assertThat(contentAsString).doesNotContain(VALID);
-        assertThat(contentAsString).contains(DEPLOYABLE);
-        assertThat(contentAsString).contains(RENDER_ERRORS);
-        assertThat(contentAsString).doesNotContain(LINT_WARNING);
-        assertThat(contentAsString).doesNotContain(LINT_ERROR);
-        assertThat(contentAsString).contains(VERSION_USED);
+        assertThat(contentAsString)
+            .doesNotContain(VALID)
+            .contains(DEPLOYABLE)
+            .contains(RENDER_ERRORS)
+            .doesNotContain(LINT_WARNING)
+            .doesNotContain(LINT_ERROR)
+            .contains(VERSION_USED);
         assertThat(mvcResult.getResponse().getStatus()).isEqualTo(HttpStatus.OK.value());
     }
 }
diff --git a/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java b/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java
index 7343e59..79ea56a 100644
--- a/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java
+++ b/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java
@@ -68,9 +68,9 @@ class ApiVersionsReaderTest {
     @Test
     void shouldThrowExceptionWhenApiVersionIsNotProvided() throws IOException {
         prepareTestTar(null);
-
+        final String chartPath = TEST_TAR_PATH.toString();
         Exception exception = assertThrows(ApiVersionNotFoundException.class,
-            () -> apiVersionsReader.readVersion(TEST_TAR_PATH.toString()));
+            () -> apiVersionsReader.readVersion(chartPath));
 
         assertThat(exception).hasMessageContaining("Cannot find apiVersion value in a main chart");
     }