From 466bb220909f1836232d82091196a942f16c283c Mon Sep 17 00:00:00 2001 From: danielhanrahan Date: Mon, 13 Nov 2023 19:17:45 +0000 Subject: [PATCH] Fix PASS/FAIL reporting in performance test summary Performance test summary says PASS even if the memory limit is exceeded. It currently only looks at time. Issue-ID: CPS-1753 Signed-off-by: danielhanrahan Change-Id: I6b9d5ae8e8f560459ba77cdb8fe8b889d0dc7a8a --- .../org/onap/cps/integration/performance/base/PerfTestBase.groovy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy b/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy index b0a105c7f..a96d7f64b 100644 --- a/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy +++ b/integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy @@ -55,7 +55,7 @@ abstract class PerfTestBase extends CpsIntegrationSpecBase { abstract def createInitialData() def recordAndAssertResourceUsage(String shortTitle, double thresholdInSec, double recordedTimeInSec, memoryLimit, memoryUsageInMB) { - def pass = recordedTimeInSec <= thresholdInSec + def pass = recordedTimeInSec <= thresholdInSec && memoryUsageInMB <= memoryLimit if (shortTitle.length() > 40) { shortTitle = shortTitle.substring(0, 40) } -- 2.16.6