summaryrefslogtreecommitdiffstats
path: root/BRMSGateway/src/test
diff options
context:
space:
mode:
authorliamfallon <liam.fallon@ericsson.com>2018-05-08 11:28:40 +0100
committerliamfallon <liam.fallon@ericsson.com>2018-05-08 11:53:07 +0100
commit0dea75c506058a9e999d30ec1916c7530504a8d6 (patch)
tree1c7253dc4594c7275bd73fc049ca632af12ab17e /BRMSGateway/src/test
parent973929cfd6cd3e5ba3a837fa56d35b210307c9ac (diff)
Configuration as argument to BRMS Gateway
Added the ability specify the parameter file for the BRMS Gateway as a argument to allow different configurations to be used during unit test. This will allow more thorought unit tests to be written. Replaced System.exit() calls with exeception throws becasue System.exit() call bring down the entire JVM during testing, terminating the test at that point. Changed the package path on four unit test files to the correct path for unit tests for the BRMS gateway. Added a unit test for sanity test of the configuraiton file argument. Issue-ID: POLICY-773 Change-Id: Ic095a5131ddb846eaf3b11157853fab71908c629 Signed-off-by: liamfallon <liam.fallon@ericsson.com>
Diffstat (limited to 'BRMSGateway/src/test')
-rw-r--r--BRMSGateway/src/test/java/org/onap/policy/brms/BrmsJpaTest.java (renamed from BRMSGateway/src/test/java/org/onap/brmsgw/test/BrmsJpaTest.java)2
-rw-r--r--BRMSGateway/src/test/java/org/onap/policy/brms/BrmsPushTest.java (renamed from BRMSGateway/src/test/java/org/onap/brmsgw/test/BrmsPushTest.java)6
-rw-r--r--BRMSGateway/src/test/java/org/onap/policy/brms/ControllerPojoTest.java (renamed from BRMSGateway/src/test/java/org/onap/brmsgw/test/ControllerPojoTest.java)2
-rw-r--r--BRMSGateway/src/test/java/org/onap/policy/brms/NotificationPojoTest.java (renamed from BRMSGateway/src/test/java/org/onap/brmsgw/test/NotificationPojoTest.java)2
-rw-r--r--BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayMainTest.java59
-rw-r--r--BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayTest.java2
6 files changed, 66 insertions, 7 deletions
diff --git a/BRMSGateway/src/test/java/org/onap/brmsgw/test/BrmsJpaTest.java b/BRMSGateway/src/test/java/org/onap/policy/brms/BrmsJpaTest.java
index 32ed314d5..2d5aaa496 100644
--- a/BRMSGateway/src/test/java/org/onap/brmsgw/test/BrmsJpaTest.java
+++ b/BRMSGateway/src/test/java/org/onap/policy/brms/BrmsJpaTest.java
@@ -18,7 +18,7 @@
* ============LICENSE_END=========================================================
*/
-package org.onap.brmsgw.test;
+package org.onap.policy.brms;
import static org.junit.Assert.assertEquals;
diff --git a/BRMSGateway/src/test/java/org/onap/brmsgw/test/BrmsPushTest.java b/BRMSGateway/src/test/java/org/onap/policy/brms/BrmsPushTest.java
index 36e633638..4a1b4f8d4 100644
--- a/BRMSGateway/src/test/java/org/onap/brmsgw/test/BrmsPushTest.java
+++ b/BRMSGateway/src/test/java/org/onap/policy/brms/BrmsPushTest.java
@@ -18,7 +18,7 @@
* ============LICENSE_END=========================================================
*/
-package org.onap.brmsgw.test;
+package org.onap.policy.brms;
import static org.junit.Assert.assertNotNull;
@@ -100,8 +100,8 @@ public class BrmsPushTest {
public void brmsHandlerFailTest6() throws PolicyException {
PropertyChange prop = new PropertyChange();
prop.key = "repositoryURL";
- prop.value =
- "http://nexus:8081/nexus/content/repositories/releases, http://nexus:8081/nexus/content/repositories/releases";
+ prop.value = "http://nexus:8081/nexus/content/repositories/releases,"
+ + "http://nexus:8081/nexus/content/repositories/releases";
prop.remove = false;
final List<PropertyChange> props = new LinkedList<>();
props.add(prop);
diff --git a/BRMSGateway/src/test/java/org/onap/brmsgw/test/ControllerPojoTest.java b/BRMSGateway/src/test/java/org/onap/policy/brms/ControllerPojoTest.java
index 21534ddcd..e00689dc8 100644
--- a/BRMSGateway/src/test/java/org/onap/brmsgw/test/ControllerPojoTest.java
+++ b/BRMSGateway/src/test/java/org/onap/policy/brms/ControllerPojoTest.java
@@ -18,7 +18,7 @@
* ============LICENSE_END=========================================================
*/
-package org.onap.brmsgw.test;
+package org.onap.policy.brms;
import static org.junit.Assert.assertEquals;
diff --git a/BRMSGateway/src/test/java/org/onap/brmsgw/test/NotificationPojoTest.java b/BRMSGateway/src/test/java/org/onap/policy/brms/NotificationPojoTest.java
index f83b749f7..ef6fff087 100644
--- a/BRMSGateway/src/test/java/org/onap/brmsgw/test/NotificationPojoTest.java
+++ b/BRMSGateway/src/test/java/org/onap/policy/brms/NotificationPojoTest.java
@@ -18,7 +18,7 @@
* ============LICENSE_END=========================================================
*/
-package org.onap.brmsgw.test;
+package org.onap.policy.brms;
import static org.junit.Assert.assertEquals;
diff --git a/BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayMainTest.java b/BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayMainTest.java
new file mode 100644
index 000000000..e20986fa6
--- /dev/null
+++ b/BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayMainTest.java
@@ -0,0 +1,59 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * ONAP Policy Engine
+ * ================================================================================
+ * Copyright (C) 2018 Ericsson. 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.brms.api;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.fail;
+
+import org.junit.Test;
+import org.onap.policy.api.PolicyException;
+
+public class BrmsGatewayMainTest {
+
+ @Test
+ public void testTooManyArguments() {
+ try {
+ String[] args = {"aaa", "bbb"};
+ BrmsGateway.main(args);
+ fail("test should throw an exception");
+ }
+ catch (PolicyException e) {
+ assertEquals("usage: org.onap.policy.brms.api.BrmsGateway [configFile]", e.getMessage());
+ }
+ catch (Exception e) {
+ fail("test should throw a PolicyException");
+ }
+
+ try {
+ String[] args = {"aaa"};
+ BrmsGateway.main(args);
+ fail("test should throw an exception");
+ }
+ catch (PolicyException e) {
+ assertEquals("Check your property file: PE300 - Data Issue: "
+ + "Config File doesn't Exist in the specified Path aaa", e.getMessage());
+ }
+ catch (Exception e) {
+ fail("test should throw a PolicyException");
+ }
+ }
+
+}
diff --git a/BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayTest.java b/BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayTest.java
index b3f836fc5..fd8a7ed3b 100644
--- a/BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayTest.java
+++ b/BRMSGateway/src/test/java/org/onap/policy/brms/api/BrmsGatewayTest.java
@@ -52,7 +52,7 @@ public class BrmsGatewayTest {
// Run app
try {
- final String[] args = null;
+ final String[] args = new String[0];
BrmsGateway.main(args);
} catch (final Exception ex) {
fail("Not expected an exception: " + ex);