From: danielhanrahan Date: Mon, 13 Nov 2023 19:17:45 +0000 (+0000) Subject: Fix PASS/FAIL reporting in performance test summary X-Git-Tag: 3.4.1~38^2 X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=466bb220909f1836232d82091196a942f16c283c;p=cps.git 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 --- 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) }