From 99837f40d77dd41f80eeb4649c43479e898bcaff Mon Sep 17 00:00:00 2001 From: Pamela Dragosh Date: Thu, 21 Mar 2019 09:58:52 -0400 Subject: [PATCH] Add module for error responses Module will hold a common error message object used by API's to clients for returning an overall error message and list of detail messages and warnings. Added an example and finished a JUnit test. Changed to response code from HTTP Added assertj and used StandardCoder Issue-ID: POLICY-1195 Change-Id: Id0fcf4831b8d03223c590f8085f207dbbbe559b4 Signed-off-by: Pamela Dragosh --- models-errors/pom.xml | 41 ++++++++++++++ .../models/errors/concepts/ErrorResponse.java | 50 +++++++++++++++++ .../models/errors/concepts/ErrorResponseTest.java | 62 ++++++++++++++++++++++ .../policy/models/errors/concepts/TestModels.java | 42 +++++++++++++++ .../src/test/resources/error.response.simple.json | 11 ++++ pom.xml | 7 +++ 6 files changed, 213 insertions(+) create mode 100644 models-errors/pom.xml create mode 100644 models-errors/src/main/java/org/onap/policy/models/errors/concepts/ErrorResponse.java create mode 100644 models-errors/src/test/java/org/onap/policy/models/errors/concepts/ErrorResponseTest.java create mode 100644 models-errors/src/test/java/org/onap/policy/models/errors/concepts/TestModels.java create mode 100644 models-errors/src/test/resources/error.response.simple.json diff --git a/models-errors/pom.xml b/models-errors/pom.xml new file mode 100644 index 000000000..ab998537e --- /dev/null +++ b/models-errors/pom.xml @@ -0,0 +1,41 @@ + + + + 4.0.0 + + org.onap.policy.models + policy-models + 2.0.0-SNAPSHOT + + + models-errors + + ${project.artifactId} + The models for Policy API's to return Error/warning message details. + + + org.onap.policy.common + gson + ${policy.common.version} + + + diff --git a/models-errors/src/main/java/org/onap/policy/models/errors/concepts/ErrorResponse.java b/models-errors/src/main/java/org/onap/policy/models/errors/concepts/ErrorResponse.java new file mode 100644 index 000000000..b072ba1f6 --- /dev/null +++ b/models-errors/src/main/java/org/onap/policy/models/errors/concepts/ErrorResponse.java @@ -0,0 +1,50 @@ +/* + * ============LICENSE_START======================================================= + * ONAP Policy Decision Models + * ================================================================================ + * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.errors.concepts; + +import com.google.gson.annotations.SerializedName; + +import java.util.List; + +import javax.ws.rs.core.Response; + +import lombok.Data; + +/** + * This class is for defining a common error/warning + * response from API calls in the Policy Framework. + * + * @author pameladragosh + * + */ +@Data +public class ErrorResponse { + + @SerializedName("code") + private Response.Status responseCode; + @SerializedName("error") + private String errorMessage; + @SerializedName("details") + private List errorDetails; + @SerializedName("warnings") + private List warningDetails; + +} diff --git a/models-errors/src/test/java/org/onap/policy/models/errors/concepts/ErrorResponseTest.java b/models-errors/src/test/java/org/onap/policy/models/errors/concepts/ErrorResponseTest.java new file mode 100644 index 000000000..ff2baa9a3 --- /dev/null +++ b/models-errors/src/test/java/org/onap/policy/models/errors/concepts/ErrorResponseTest.java @@ -0,0 +1,62 @@ +/* + * ============LICENSE_START======================================================= + * ONAP Policy Decision Models + * ================================================================================ + * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.errors.concepts; + +import static org.assertj.core.api.Assertions.assertThatCode; +import static org.junit.Assert.assertEquals; + +import java.util.Arrays; + +import javax.ws.rs.core.Response; + +import org.junit.Test; +import org.onap.policy.common.utils.coder.StandardCoder; +import org.slf4j.Logger; +import org.slf4j.LoggerFactory; + +public class ErrorResponseTest { + + public static final Logger logger = LoggerFactory.getLogger(ErrorResponseTest.class); + + @Test + public void test() { + assertThatCode(() -> { + ErrorResponse error = new ErrorResponse(); + + error.setResponseCode(Response.Status.NOT_ACCEPTABLE); + error.setErrorMessage("Missing metadata section"); + + error.setErrorDetails(Arrays.asList("You must have a metadata section with policy-id value")); + + error.setWarningDetails(Arrays.asList("Please make sure topology template field is included.")); + + StandardCoder coder = new StandardCoder(); + String jsonOutput = coder.encode(error); + + logger.debug("Resulting json output {}", jsonOutput); + + ErrorResponse deserializedResponse = coder.decode(jsonOutput, ErrorResponse.class); + + assertEquals(deserializedResponse, error); + }).doesNotThrowAnyException(); + } + +} diff --git a/models-errors/src/test/java/org/onap/policy/models/errors/concepts/TestModels.java b/models-errors/src/test/java/org/onap/policy/models/errors/concepts/TestModels.java new file mode 100644 index 000000000..03dd8e6a4 --- /dev/null +++ b/models-errors/src/test/java/org/onap/policy/models/errors/concepts/TestModels.java @@ -0,0 +1,42 @@ +/* + * ============LICENSE_START======================================================= + * ONAP Policy Decision Models + * ================================================================================ + * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved. + * ================================================================================ + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * ============LICENSE_END========================================================= + */ + +package org.onap.policy.models.errors.concepts; + +import com.openpojo.reflection.filters.FilterPackageInfo; +import com.openpojo.validation.Validator; +import com.openpojo.validation.ValidatorBuilder; +import com.openpojo.validation.test.impl.GetterTester; +import com.openpojo.validation.test.impl.SetterTester; + +import org.junit.Test; +import org.onap.policy.common.utils.validation.ToStringTester; + +public class TestModels { + + @Test + public void testDecisionModels() { + final Validator validator = ValidatorBuilder.create().with(new ToStringTester()).with(new SetterTester()) + .with(new GetterTester()).build(); + validator.validate(TestModels.class.getPackage().getName(), new FilterPackageInfo()); + } + + +} diff --git a/models-errors/src/test/resources/error.response.simple.json b/models-errors/src/test/resources/error.response.simple.json new file mode 100644 index 000000000..d9420c07d --- /dev/null +++ b/models-errors/src/test/resources/error.response.simple.json @@ -0,0 +1,11 @@ +{ + "code" : "404", + "error" : "Policy Type does not exist", + "details" : [ + "Missing metadata section", + "The property value foo bar is missing" + ], + "warnings" : [ + "The topology_template is missing" + ] +} diff --git a/pom.xml b/pom.xml index 8d80849fb..969474627 100644 --- a/pom.xml +++ b/pom.xml @@ -56,6 +56,7 @@ models-dao models-tosca models-pap + models-errors models-decisions models-provider models-examples @@ -117,6 +118,12 @@ test + + org.assertj + assertj-core + test + + org.eclipse.persistence eclipselink -- 2.16.6