summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordanielhanrahan <daniel.hanrahan@est.tech>2023-11-13 19:17:45 +0000
committerdanielhanrahan <daniel.hanrahan@est.tech>2023-11-13 19:17:45 +0000
commit466bb220909f1836232d82091196a942f16c283c (patch)
treebd786a07512082735fc3bb09daf55e404ef52ec2
parented9d179947325bbb9b81378b469341d023b46cd8 (diff)
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 <daniel.hanrahan@est.tech> Change-Id: I6b9d5ae8e8f560459ba77cdb8fe8b889d0dc7a8a
-rw-r--r--integration-test/src/test/groovy/org/onap/cps/integration/performance/base/PerfTestBase.groovy2
1 files changed, 1 insertions, 1 deletions
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)
}