From c184cf1bda7a8a5d1cd489fb8080d2ee507611de Mon Sep 17 00:00:00 2001 From: Bartosz Gardziejewski Date: Thu, 8 Oct 2020 11:48:34 +0200 Subject: Fix get executions JSON parsing error. Signed-off-by: Bartosz Gardziejewski Change-Id: I1ee1c070cc0c97eebfad5cdb5e737d8136631cbf Issue-ID: VNFSDK-697 --- .../org/onap/vtp/VTPExecutionResourceTest.java | 13 ++++---- .../execution/VTPExecutionResultsSupplierTest.java | 39 +++++++++++++++++----- 2 files changed, 37 insertions(+), 15 deletions(-) (limited to 'vnfmarket-be/vnf-sdk-marketplace/src/test/java/org') diff --git a/vnfmarket-be/vnf-sdk-marketplace/src/test/java/org/onap/vtp/VTPExecutionResourceTest.java b/vnfmarket-be/vnf-sdk-marketplace/src/test/java/org/onap/vtp/VTPExecutionResourceTest.java index 1a3200ab..4f15fcf5 100644 --- a/vnfmarket-be/vnf-sdk-marketplace/src/test/java/org/onap/vtp/VTPExecutionResourceTest.java +++ b/vnfmarket-be/vnf-sdk-marketplace/src/test/java/org/onap/vtp/VTPExecutionResourceTest.java @@ -20,6 +20,7 @@ import com.google.common.collect.Lists; import com.google.gson.Gson; import com.google.gson.JsonArray; import com.google.gson.JsonElement; +import com.google.gson.JsonObject; import com.google.gson.JsonParser; import org.glassfish.jersey.media.multipart.ContentDisposition; import org.glassfish.jersey.media.multipart.FormDataBodyPart; @@ -153,12 +154,12 @@ public class VTPExecutionResourceTest { String testProduct = "VTP Scenario 2"; String testCommand = "s1.ts1.testcase-2"; String testSuiteName = "testsuite-2"; - String testRequestId = "test-02-request-id"; + String testRequestId = "test-wrong-request-id"; String testExecutionId = testRequestId + "-execution-id"; String testProfile = "open-cli-schema"; String expectedStatus = "FAIL"; JsonElement expectedResult = new Gson().fromJson("" + - "[{ \"error\": \"unable to find execution results\"}]", JsonArray.class); + "{ \"error\": \"unable to find execution results\"}", JsonObject.class); prepareMockRpcMethods( testStartTime, testEndTime, testProduct, testCommand, testSuiteName, @@ -193,11 +194,11 @@ public class VTPExecutionResourceTest { String testProfile = "open-cli-schema"; String expectedStatus = "FAIL"; JsonElement expectedResult = new Gson().fromJson("" + - "[{ " + + "{ " + "\"error\": \"fail to load execution result\"," + - "\"reason\":\"Expected a com.google.gson.JsonArray but was com.google.gson.JsonPrimitive\"" + - "}]", - JsonArray.class + "\"reason\":\"com.google.gson.stream.MalformedJsonException: Use JsonReader.setLenient(true) to accept malformed JSON at line 1 column 8 path $\"" + + "}", + JsonObject.class ); prepareMockRpcMethods( diff --git a/vnfmarket-be/vnf-sdk-marketplace/src/test/java/org/onap/vtp/execution/VTPExecutionResultsSupplierTest.java b/vnfmarket-be/vnf-sdk-marketplace/src/test/java/org/onap/vtp/execution/VTPExecutionResultsSupplierTest.java index 7be8a7d4..83ee1739 100644 --- a/vnfmarket-be/vnf-sdk-marketplace/src/test/java/org/onap/vtp/execution/VTPExecutionResultsSupplierTest.java +++ b/vnfmarket-be/vnf-sdk-marketplace/src/test/java/org/onap/vtp/execution/VTPExecutionResultsSupplierTest.java @@ -19,6 +19,7 @@ package org.onap.vtp.execution; import com.google.gson.Gson; import com.google.gson.JsonArray; import com.google.gson.JsonElement; +import com.google.gson.JsonObject; import org.junit.Test; import static org.junit.Assert.assertEquals; @@ -28,7 +29,7 @@ public class VTPExecutionResultsSupplierTest { private static final String TEST_PATH_TO_EXECUTION = "src/test/resources/executions"; @Test - public void whenGetExecutionOutputsFromFileIsCalledWithPathToCorrectFileThenContentShouldBeLoadedAsJson() { + public void whenGetExecutionOutputsFromFileIsCalledWithPathToCorrectFileThenContentShouldBeLoadedAsJsonArray() { // given VTPExecutionResultsSupplier vtpExecutionResultsSupplier = new VTPExecutionResultsSupplier(TEST_PATH_TO_EXECUTION); @@ -48,20 +49,40 @@ public class VTPExecutionResultsSupplierTest { } @Test - public void whenGetExecutionOutputsFromFileIsCalledWithPathToNonExistingFileThenProperMessageShouldBeReturned() { + public void whenGetExecutionOutputsFromFileIsCalledWithPathToCorrectFileThenContentShouldBeLoadedAsJsonObject() { // given VTPExecutionResultsSupplier vtpExecutionResultsSupplier = new VTPExecutionResultsSupplier(TEST_PATH_TO_EXECUTION); String pathToCorrectFile = "test-02-request-id-execution-id"; + JsonElement expectedResult = new Gson().fromJson("" + + "{" + + "\"test_1\": \"error01\"," + + "\"test_2\": \"error02\" " + + "}", JsonObject.class); + + // when + JsonElement executionOutputsFromFile = + vtpExecutionResultsSupplier.getExecutionOutputsFromFile(pathToCorrectFile); + + // then + assertEquals(executionOutputsFromFile, expectedResult); + } + + @Test + public void whenGetExecutionOutputsFromFileIsCalledWithPathToNonExistingFileThenProperMessageShouldBeReturned() { + // given + VTPExecutionResultsSupplier vtpExecutionResultsSupplier = + new VTPExecutionResultsSupplier(TEST_PATH_TO_EXECUTION); + String pathToCorrectFile = "test-wrong-request-id-execution-id"; JsonElement expectedErrorMessage = new Gson().fromJson("" + - "[{ \"error\": \"unable to find execution results\"}]", JsonArray.class); + "{ \"error\": \"unable to find execution results\"}", JsonObject.class); // when JsonElement executionOutputsFromFile = vtpExecutionResultsSupplier.getExecutionOutputsFromFile(pathToCorrectFile); // then - assertEquals(executionOutputsFromFile, expectedErrorMessage); + assertEquals(expectedErrorMessage, executionOutputsFromFile); } @Test @@ -71,11 +92,11 @@ public class VTPExecutionResultsSupplierTest { new VTPExecutionResultsSupplier(TEST_PATH_TO_EXECUTION); String pathToCorrectFile = "test-incorrect-request-id-execution-id-data"; JsonElement expectedErrorMessage = new Gson().fromJson("" + - "[{ " + + "{ " + "\"error\": \"fail to load execution result\"," + - "\"reason\":\"Expected a com.google.gson.JsonArray but was com.google.gson.JsonPrimitive\"" + - "}]", - JsonArray.class + "\"reason\":\"com.google.gson.stream.MalformedJsonException: Use JsonReader.setLenient(true) to accept malformed JSON at line 1 column 8 path $\"" + + "}", + JsonObject.class ); // when @@ -83,6 +104,6 @@ public class VTPExecutionResultsSupplierTest { vtpExecutionResultsSupplier.getExecutionOutputsFromFile(pathToCorrectFile); // then - assertEquals(executionOutputsFromFile, expectedErrorMessage); + assertEquals(expectedErrorMessage, executionOutputsFromFile); } } -- cgit 1.2.3-korg