diff options
author | elinuxhenrik <henrik.b.andersson@est.tech> | 2020-08-24 14:18:33 +0200 |
---|---|---|
committer | Henrik Andersson <henrik.b.andersson@est.tech> | 2020-08-24 14:08:24 +0000 |
commit | 749f96c4c4877bee554b4cf7d5d696e3e88c0af8 (patch) | |
tree | 8997822f49e734d83f4fa3056bc2f09d54d8dc77 | |
parent | c4d1f8ad32a1a5e12780b30f32a64256a3c0b0f1 (diff) |
Fix Sonar issues
Change-Id: I868d22847853ba7646164c2234b608ec319085ab
Issue-ID: CCSDK-2502
Signed-off-by: elinuxhenrik <henrik.b.andersson@est.tech>
2 files changed, 7 insertions, 8 deletions
diff --git a/a1-policy-management/src/test/java/org/onap/ccsdk/oran/a1policymanagementservice/ApplicationTestV2.java b/a1-policy-management/src/test/java/org/onap/ccsdk/oran/a1policymanagementservice/ApplicationTestV2.java index bc47baf9..3a7068d6 100644 --- a/a1-policy-management/src/test/java/org/onap/ccsdk/oran/a1policymanagementservice/ApplicationTestV2.java +++ b/a1-policy-management/src/test/java/org/onap/ccsdk/oran/a1policymanagementservice/ApplicationTestV2.java @@ -500,12 +500,12 @@ class ApplicationTestV2 { url = "/policy-schemas?ric_id=ric1&policytype_id=type1"; rsp = restClient().get(url).block(); PolicySchemaList list = gson.fromJson(rsp, PolicySchemaList.class); - assertThat(list.schemas.size()).isEqualTo(1); + assertThat(list.schemas).hasSize(1); url = "/policy-schemas?ric_id=ric1&policytype_id=type2"; rsp = restClient().get(url).block(); list = gson.fromJson(rsp, PolicySchemaList.class); - assertThat(list.schemas.size()).isEqualTo(0); + assertThat(list.schemas).isEmpty(); // Get schema for non existing RIC url = "/policy-schemas?ric_id=ric1XXX"; diff --git a/a1-policy-management/src/test/java/org/onap/ccsdk/oran/a1policymanagementservice/clients/SdncOscA1ClientTest.java b/a1-policy-management/src/test/java/org/onap/ccsdk/oran/a1policymanagementservice/clients/SdncOscA1ClientTest.java index 1fc4a421..08e99847 100644 --- a/a1-policy-management/src/test/java/org/onap/ccsdk/oran/a1policymanagementservice/clients/SdncOscA1ClientTest.java +++ b/a1-policy-management/src/test/java/org/onap/ccsdk/oran/a1policymanagementservice/clients/SdncOscA1ClientTest.java @@ -22,6 +22,7 @@ package org.onap.ccsdk.oran.a1policymanagementservice.clients; import static org.junit.Assert.fail; import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertThrows; import static org.mockito.ArgumentMatchers.anyString; import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; @@ -95,12 +96,10 @@ class SdncOscA1ClientTest { @Test void createClientWithWrongProtocol_thenErrorIsThrown() { - try { - new SdncOscA1Client(A1ProtocolType.STD_V1_1, null, null, new AsyncRestClient("", null)); - fail("Should have thrown exception."); - } catch (IllegalArgumentException e) { - return; - } + AsyncRestClient asyncRestClient = new AsyncRestClient("", null); + assertThrows(IllegalArgumentException.class, () -> { + new SdncOscA1Client(A1ProtocolType.STD_V1_1, null, null, asyncRestClient); + }); } @Test |