From 36806ac304bac477a463b7eb9f8d3fb664c8b4e3 Mon Sep 17 00:00:00 2001
From: waynedunican <wayne.dunican@est.tech>
Date: Fri, 24 Feb 2023 08:44:58 +0000
Subject: Add SLA validations to API

Issue-ID: POLICY-4528
Change-Id: I517cb477bbccb36d5e2000b60450a2c015d4940b
Signed-off-by: Wayne Dunican <wayne.dunican@est.tech>
---
 csit/resources/scripts/setup-api.sh |  3 ++-
 csit/resources/tests/api-slas.robot | 45 +++++++++++++++++++++++++++++++++++++
 csit/resources/tests/api-test.robot |  5 +++++
 3 files changed, 52 insertions(+), 1 deletion(-)
 create mode 100644 csit/resources/tests/api-slas.robot

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
-- 
cgit 1.2.3-korg