diff options
| author |   ToineSiebelink <toine.siebelink@est.tech> | 2024-07-10 09:24:22 +0100 |
|---|---|---|
| committer | ToineSiebelink <toine.siebelink@est.tech> | 2024-07-10 10:04:04 +0100 |
| commit | 4962922401c1c779c6b7d1161a995942750566ed (patch) | |
| tree | e5f36152a6d3382246a5b22c9eb693c0e4280a03 /policy-executor-stub | |
| parent | 8f3103dd1eaf5d537cd085dd8fab5ef39d7d914d (diff) | |
Policy Executor API, make Authorization header optional
Issue-ID: CPS-22291
Change-Id: I2b8f9b97c92dfc640c8b40b7053c609bbdcb2f6e
Signed-off-by: ToineSiebelink <toine.siebelink@est.tech>
Diffstat (limited to 'policy-executor-stub')
2 files changed, 14 insertions, 10 deletions
diff --git a/policy-executor-stub/src/main/java/org/onap/cps/policyexecutor/stub/controller/PolicyExecutorStubController.java b/policy-executor-stub/src/main/java/org/onap/cps/policyexecutor/stub/controller/PolicyExecutorStubController.java index 7989c3fc8c..a5ec6dcac9 100644 --- a/policy-executor-stub/src/main/java/org/onap/cps/policyexecutor/stub/controller/PolicyExecutorStubController.java +++ b/policy-executor-stub/src/main/java/org/onap/cps/policyexecutor/stub/controller/PolicyExecutorStubController.java @@ -41,9 +41,9 @@ public class PolicyExecutorStubController implements PolicyExecutorApi { @Override public ResponseEntity<PolicyExecutionResponse> executePolicyAction( - final String authorization, final String action, - final PolicyExecutionRequest policyExecutionRequest) { + final PolicyExecutionRequest policyExecutionRequest, + final String authorization) { if (policyExecutionRequest.getPayload().isEmpty()) { return new ResponseEntity<>(HttpStatus.BAD_REQUEST); } @@ -56,15 +56,19 @@ public class PolicyExecutorStubController implements PolicyExecutorApi { return new ResponseEntity<>(HttpStatusCode.valueOf(errorCode)); } - final PolicyExecutionResponse policyExecutionResponse = new PolicyExecutionResponse(); - policyExecutionResponse.setDecisionId(String.valueOf(++decisionCounter)); + final String decisionId = String.valueOf(++decisionCounter); + final String decision; + final String message; if (firstTargetFdn.toLowerCase(Locale.getDefault()).contains("cps-is-great")) { - policyExecutionResponse.setDecision("permit"); + decision = "permit"; + message = "All good"; } else { - policyExecutionResponse.setDecision("deny"); - policyExecutionResponse.setMessage("Only FDNs containing 'cps-is-great' are permitted"); + decision = "deny"; + message = "Only FDNs containing 'cps-is-great' are permitted"; } + final PolicyExecutionResponse policyExecutionResponse = + new PolicyExecutionResponse(decisionId, decision, message); return ResponseEntity.ok(policyExecutionResponse); } } diff --git a/policy-executor-stub/src/test/groovy/org/onap/cps/policyexecutor/stub/controller/PolicyExecutorStubControllerSpec.groovy b/policy-executor-stub/src/test/groovy/org/onap/cps/policyexecutor/stub/controller/PolicyExecutorStubControllerSpec.groovy index 593394fdc7..871db81ac8 100644 --- a/policy-executor-stub/src/test/groovy/org/onap/cps/policyexecutor/stub/controller/PolicyExecutorStubControllerSpec.groovy +++ b/policy-executor-stub/src/test/groovy/org/onap/cps/policyexecutor/stub/controller/PolicyExecutorStubControllerSpec.groovy @@ -66,7 +66,7 @@ class PolicyExecutorStubControllerSpec extends Specification { where: 'the following targets are used' targetFdn || expectedDecsisonId | expectedDecsison | expectedMessage 'some fdn' || '1' | 'deny' | "Only FDNs containing 'cps-is-great' are permitted" - 'fdn with cps-is-great' || '2' | 'permit' | null + 'fdn with cps-is-great' || '2' | 'permit' | "All good" } def 'Execute Policy Action with a HTTP Error Code.'() { @@ -94,8 +94,8 @@ class PolicyExecutorStubControllerSpec extends Specification { .contentType(MediaType.APPLICATION_JSON) .content(requestBody)) .andReturn().response - then: 'response status is Bad Request' - assert response.status == HttpStatus.BAD_REQUEST.value() + then: 'response status is OK' + assert response.status == HttpStatus.OK.value() } def 'Execute Policy Action with Empty Payload.'() { |