X-Git-Url: https://gerrit.onap.org/r/gitweb?p=policy%2Fengine.git;a=blobdiff_plain;f=POLICY-SDK-APP%2Fsrc%2Ftest%2Fjava%2Forg%2Fonap%2Fpolicy%2Fadmin%2FCheckPDPTest.java;fp=POLICY-SDK-APP%2Fsrc%2Ftest%2Fjava%2Forg%2Fonap%2Fpolicy%2Fadmin%2FCheckPDPTest.java;h=e97e89eae27980f3142c0a7694d7639db7265f04;hp=47358a8f2c5e44dffd5d66c53247cdc0be619b2a;hb=9301fed7c290f52208922f780fdfe4b5a880b5aa;hpb=9df8b88412f6e890320bc446ba1c83a13e99822d

diff --git a/POLICY-SDK-APP/src/test/java/org/onap/policy/admin/CheckPDPTest.java b/POLICY-SDK-APP/src/test/java/org/onap/policy/admin/CheckPDPTest.java
index 47358a8f2..e97e89eae 100644
--- a/POLICY-SDK-APP/src/test/java/org/onap/policy/admin/CheckPDPTest.java
+++ b/POLICY-SDK-APP/src/test/java/org/onap/policy/admin/CheckPDPTest.java
@@ -19,19 +19,54 @@
  */
 package org.onap.policy.admin;
 
-import static org.junit.Assert.*;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
 
 import java.io.File;
 
+import org.junit.FixMethodOrder;
 import org.junit.Test;
+import org.junit.runners.MethodSorters;
 
+@FixMethodOrder(MethodSorters.NAME_ASCENDING)
 public class CheckPDPTest {
 
 	@Test
-	public final void testCheckPDP() {
+	public final void test1NoPropertySet() {
+		try {
+			System.clearProperty("xacml.rest.pdp.idfile");
+			assertFalse(CheckPDP.validateID("http://localhost:8082/pdp/"));
+
+			System.setProperty("xacml.rest.pdp.idfile", new File(".").getCanonicalPath() + File.separator + "src"+ File.separator + "test" + File.separator + "resources" + File.separator + "idonotexist.properties");
+			assertFalse(CheckPDP.validateID("http://localhost:8082/pdp/"));
+
+			System.setProperty("xacml.rest.pdp.idfile", new File(".").getCanonicalPath() + File.separator + "src"+ File.separator + "test" + File.separator + "resources" + File.separator + "doesnothaveproperties.atall");
+			assertFalse(CheckPDP.validateID("http://localhost:8082/pdp/"));
+
+			System.setProperty("xacml.rest.pdp.idfile", new File(".").getCanonicalPath() + File.separator + "src"+ File.separator + "test" + File.separator + "resources" + File.separator + "testbad.properties");
+			assertFalse(CheckPDP.validateID("http://localhost:8082/pdp/"));
+			
+			System.setProperty("xacml.rest.pdp.idfile", new File(".").getCanonicalPath() + File.separator + "src"+ File.separator + "test" + File.separator + "resources" + File.separator + "empty.properties");
+			assertFalse(CheckPDP.validateID("http://localhost:8082/pdp/"));
+						
+			System.setProperty("xacml.rest.pdp.idfile", new File(".").getCanonicalPath() + File.separator + "src"+ File.separator + "test" + File.separator + "resources" + File.separator + "testnotenoughvalues.properties");
+			assertFalse(CheckPDP.validateID("http://localhost:8082/pdp/"));
+						
+			assertNull(CheckPDP.getPdpMap());
+			assertNull(CheckPDP.getEncoding("http://localhost:8082/pdp/"));
+
+		} catch (Exception e) {
+			fail("Error occured in CheckPDP test");
+		}
+	}
+	
+	@Test
+	public final void test2CheckPDP() {
 		try {
 			System.setProperty("xacml.rest.pdp.idfile", new File(".").getCanonicalPath() + File.separator + "src"+ File.separator + "test" + File.separator + "resources" + File.separator + "test.properties");
-			CheckPDP.validateID("http://localhost:8082/pdp/");
+			assertTrue(CheckPDP.validateID("http://localhost:8082/pdp/"));
 			assertTrue(CheckPDP.getPdpMap().containsKey("http://localhost:8082/pdp/"));
 			assertTrue(CheckPDP.getEncoding("http://localhost:8082/pdp/").equals("dGVzdHBkcDphbHBoYTQ1Ng=="));
 		} catch (Exception e) {