From: waynedunican <wayne.dunican@est.tech>
Date: Fri, 24 Feb 2023 08:44:58 +0000 (+0000)
Subject: Add SLA validations to API
X-Git-Tag: 2.6.2~21^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F05%2F133405%2F2;p=policy%2Fdocker.git

Add SLA validations to API

Issue-ID: POLICY-4528
Change-Id: I517cb477bbccb36d5e2000b60450a2c015d4940b
Signed-off-by: Wayne Dunican <wayne.dunican@est.tech>
---

diff --git a/csit/resources/scripts/setup-api.sh b/csit/resources/scripts/setup-api.sh
index 2a984643..5a5cad20 100755
--- a/csit/resources/scripts/setup-api.sh
+++ b/csit/resources/scripts/setup-api.sh
@@ -28,7 +28,8 @@ unset http_proxy https_proxy
 # wait for the app to start up
 bash "${SCRIPTS}"/wait_for_rest.sh localhost ${API_PORT}
 
-export SUITES="api-test.robot"
+export SUITES="api-test.robot
+api-slas.robot"
 
 ROBOT_VARIABLES="-v POLICY_API_IP:localhost:${API_PORT} -v PROMETHEUS_IP:localhost:${PROMETHEUS_PORT}
 -v DATA:${DATA} -v NODETEMPLATES:${NODETEMPLATES}"
diff --git a/csit/resources/tests/api-slas.robot b/csit/resources/tests/api-slas.robot
new file mode 100644
index 00000000..c5feb728
--- /dev/null
+++ b/csit/resources/tests/api-slas.robot
@@ -0,0 +1,45 @@
+*** Settings ***
+Library    Collections
+Library    RequestsLibrary
+Library    OperatingSystem
+Library    json
+Resource    ${CURDIR}/common-library.robot
+
+*** Keywords ***
+ValidateResponseTimeForApi
+    [Arguments]  ${uri}  ${method}
+    [Documentation]  Check if uri response is under the 500ms required time for api metrics
+    ValidateResponseTime  api-metrics  ${uri}  ${method}  500
+
+*** Test Cases ***
+WaitForPrometheusServer
+    [Documentation]  Sleep time to wait for Prometheus server to gather all metrics
+    Sleep    1 minute
+
+ValidateResponseTimeForHealthcheck
+    [Documentation]  Validate component healthcheck response time
+    ValidateResponseTimeForApi  /healthcheck  GET
+
+ValidateResponseTimeForStatistics
+    [Documentation]  Validate statistics response time
+    ValidateResponseTimeForApi  /statistics  GET
+
+ValidateResponseTimeQueryPolicies
+    [Documentation]  Validate query policies response time
+    ValidateResponseTimeForApi  /policies  GET
+
+ValidateResponseTimeQueryPolicyVersion
+    [Documentation]  Validate query policy by version response time
+    ValidateResponseTimeForApi  /policies/{policyId}/versions/{policyVersion}  GET
+
+ValidateResponseTimeCreatePolicy
+    [Documentation]  Validate response time for creating a policy
+    ValidateResponseTimeForApi  /policytypes/{policyTypeId}/versions/{policyTypeVersion}/policies  POST
+
+ValidateResponseTimeCreatePolicyType
+    [Documentation]  Validate response time for creating a policyType
+    ValidateResponseTimeForApi  /policytypes  POST
+
+ValidateResponseTimeDeletePolicy
+    [Documentation]  Validate response time for deletion of policies
+    ValidateResponseTimeForApi  /policies/{policyId}/versions/{policyVersion}  DELETE
\ No newline at end of file
diff --git a/csit/resources/tests/api-test.robot b/csit/resources/tests/api-test.robot
index e8679a7f..f3c87de3 100644
--- a/csit/resources/tests/api-test.robot
+++ b/csit/resources/tests/api-test.robot
@@ -140,6 +140,11 @@ GetReq
      ${resp}=  PerformGetRequest  ${POLICY_API_IP}  ${url}  200  null  ${auth}
      [return]  ${resp}
 
+ValidateResponseTimeForApi
+    [Arguments]  ${url}  ${method}
+    [Documentation]  Check if url response is under required time for api metrics
+    ValidateResponseTime  api-metrics  ${url}  ${method}  500
+
 DeleteReq
      [Arguments]  ${url}  ${expectedstatus}
      ${auth}=  PolicyAdminAuth