From 09d2ec0b674d07b01b530ecc48c68132358c4915 Mon Sep 17 00:00:00 2001 From: Remigiusz Janeczek Date: Thu, 18 Mar 2021 10:32:23 +0100 Subject: Fix sonar code smells Issue-ID: SDC-3185 Signed-off-by: Remigiusz Janeczek Change-Id: Icfb466816c76c0444b7132651038768a832c03a2 --- .../api/SupportedVersionsControllerTest.java | 7 +++--- .../api/ValidationControllerTest.java | 26 ++++++++++++---------- .../helm/versions/ApiVersionsReaderTest.java | 4 ++-- 3 files changed, 20 insertions(+), 17 deletions(-) (limited to 'src/test/java/org') 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 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"); } -- cgit 1.2.3-korg