From d3438454879ec3f3c31ca8af2fc5903b7f8697ae Mon Sep 17 00:00:00 2001 From: Pamela Dragosh Date: Tue, 26 Mar 2019 11:19:12 -0400 Subject: Fix comments and add debugging Added some fixes from previous review and some debugging for the JUnits. The merge failed in the JUnits and I could not see why. I think the wrong request was sent, but I'm not sure why the JUnits woulnd't fail locally or in the verify job. We need to do a lot of testing with these policy translations. Also fixed the guard policies to use correct VF Module Create recipe. Issue-ID: POLICY-1273 Change-Id: I813bcca92436d356c43da5613eab7ba7746a4e3d Signed-off-by: Pamela Dragosh --- .../application/guard/GuardPdpApplicationTest.java | 103 +++++++++++---------- .../vDNS.policy.guard.frequency.output.tosca.yaml | 2 +- .../vDNS.policy.guard.minmax.output.tosca.yaml | 2 +- 3 files changed, 56 insertions(+), 51 deletions(-) (limited to 'applications/guard') diff --git a/applications/guard/src/test/java/org/onap/policy/xacml/pdp/application/guard/GuardPdpApplicationTest.java b/applications/guard/src/test/java/org/onap/policy/xacml/pdp/application/guard/GuardPdpApplicationTest.java index ff137e13..981afee7 100644 --- a/applications/guard/src/test/java/org/onap/policy/xacml/pdp/application/guard/GuardPdpApplicationTest.java +++ b/applications/guard/src/test/java/org/onap/policy/xacml/pdp/application/guard/GuardPdpApplicationTest.java @@ -75,6 +75,7 @@ public class GuardPdpApplicationTest { */ @BeforeClass public static void setUp() throws Exception { + LOGGER.info("Setting up class"); // // Setup our temporary folder // @@ -108,7 +109,7 @@ public class GuardPdpApplicationTest { strDump.append(application.supportedPolicyTypes()); strDump.append(System.lineSeparator()); } - LOGGER.debug("{}", strDump); + LOGGER.info("{}", strDump); // // Tell it to initialize based on the properties file // we just built for it. @@ -118,6 +119,7 @@ public class GuardPdpApplicationTest { @Test public void test1Basics() throws CoderException, IOException { + LOGGER.info("**************** Running test1 ****************"); // // Load Single Decision Request // @@ -163,6 +165,7 @@ public class GuardPdpApplicationTest { @Test public void test2NoPolicies() { + LOGGER.info("**************** Running test2 ****************"); // // Ask for a decision // @@ -175,6 +178,7 @@ public class GuardPdpApplicationTest { @Test public void test3FrequencyLimiter() throws CoderException, FileNotFoundException, IOException { + LOGGER.info("**************** Running test3 ****************"); // // Now load the vDNS frequency limiter Policy - make sure // the pdp can support it and have it load @@ -190,36 +194,37 @@ public class GuardPdpApplicationTest { // Load the policies // service.loadPolicies(toscaObject); - // - // Ask for a decision - should get permit - // - DecisionResponse response = service.makeDecision(requestGuardPermit); - LOGGER.info("Looking for Permit Decision {}", response); - - assertThat(response).isNotNull(); - assertThat(response.getStatus()).isNotNull(); - assertThat(response.getStatus()).isEqualTo("Permit"); - // - // Dump it out as Json - // - LOGGER.info(gson.encode(response)); - // - // Ask for a decision - should get deny - // - response = service.makeDecision(requestGuardDeny2); - LOGGER.info("Looking for Deny Decision {}", response); - assertThat(response).isNotNull(); - assertThat(response.getStatus()).isNotNull(); - assertThat(response.getStatus()).isEqualTo("Deny"); - // - // Dump it out as Json - // - LOGGER.info(gson.encode(response)); } + // + // Ask for a decision - should get permit + // + DecisionResponse response = service.makeDecision(requestGuardPermit); + LOGGER.info("Looking for Permit Decision {}", response); + + assertThat(response).isNotNull(); + assertThat(response.getStatus()).isNotNull(); + assertThat(response.getStatus()).isEqualTo("Permit"); + // + // Dump it out as Json + // + LOGGER.info(gson.encode(response)); + // + // Ask for a decision - should get deny + // + response = service.makeDecision(requestGuardDeny); + LOGGER.info("Looking for Deny Decision {}", response); + assertThat(response).isNotNull(); + assertThat(response.getStatus()).isNotNull(); + assertThat(response.getStatus()).isEqualTo("Deny"); + // + // Dump it out as Json + // + LOGGER.info(gson.encode(response)); } @Test public void test4MinMax() throws CoderException, FileNotFoundException, IOException { + LOGGER.info("**************** Running test4 ****************"); // // Now load the vDNS min max Policy - make sure // the pdp can support it and have it load @@ -238,34 +243,34 @@ public class GuardPdpApplicationTest { // // Ask for a decision - should get permit // - DecisionResponse response = service.makeDecision(requestGuardPermit); - LOGGER.info("Looking for Permit Decision {}", response); - - assertThat(response).isNotNull(); - assertThat(response.getStatus()).isNotNull(); - assertThat(response.getStatus()).isEqualTo("Permit"); - // - // Dump it out as Json - // - LOGGER.info(gson.encode(response)); - // - // Ask for a decision - should get deny - // - response = service.makeDecision(requestGuardDeny); - LOGGER.info("Looking for Deny Decision {}", response); - assertThat(response).isNotNull(); - assertThat(response.getStatus()).isNotNull(); - assertThat(response.getStatus()).isEqualTo("Deny"); - // - // Dump it out as Json - // - LOGGER.info(gson.encode(response)); } + DecisionResponse response = service.makeDecision(requestGuardPermit); + LOGGER.info("Looking for Permit Decision {}", response); + + assertThat(response).isNotNull(); + assertThat(response.getStatus()).isNotNull(); + assertThat(response.getStatus()).isEqualTo("Permit"); + // + // Dump it out as Json + // + LOGGER.info(gson.encode(response)); + // + // Ask for a decision - should get deny + // + response = service.makeDecision(requestGuardDeny); + LOGGER.info("Looking for Deny Decision {}", response); + assertThat(response).isNotNull(); + assertThat(response.getStatus()).isNotNull(); + assertThat(response.getStatus()).isEqualTo("Deny"); + // + // Dump it out as Json + // + LOGGER.info(gson.encode(response)); } @Test public void test5MissingFields() throws FileNotFoundException, IOException { - LOGGER.debug("Running test5"); + LOGGER.info("**************** Running test5 ****************"); // // Most likely we would not get a policy with missing fields passed to // us from the API. But in case that happens, or we decide that some fields diff --git a/applications/guard/src/test/resources/vDNS.policy.guard.frequency.output.tosca.yaml b/applications/guard/src/test/resources/vDNS.policy.guard.frequency.output.tosca.yaml index 03afd5e2..a0552d42 100644 --- a/applications/guard/src/test/resources/vDNS.policy.guard.frequency.output.tosca.yaml +++ b/applications/guard/src/test/resources/vDNS.policy.guard.frequency.output.tosca.yaml @@ -10,7 +10,7 @@ topology_template: policy-version: 1 properties: actor: SO - recipe: scaleOut + recipe: VF Module Create targets: .* clname: ControlLoop-vDNS-6f37f56d-a87d-4b85-b6a9-cc953cf779b3 limit: 1 diff --git a/applications/guard/src/test/resources/vDNS.policy.guard.minmax.output.tosca.yaml b/applications/guard/src/test/resources/vDNS.policy.guard.minmax.output.tosca.yaml index 5ac76015..97282a6f 100644 --- a/applications/guard/src/test/resources/vDNS.policy.guard.minmax.output.tosca.yaml +++ b/applications/guard/src/test/resources/vDNS.policy.guard.minmax.output.tosca.yaml @@ -10,7 +10,7 @@ topology_template: policy-version: 1 properties: actor: SO - recipe: scaleOut + recipe: VF Module Create targets: .* clname: ControlLoop-vDNS-6f37f56d-a87d-4b85-b6a9-cc953cf779b3 min: 1 -- cgit 1.2.3-korg