diff options
author | 2023-11-13 19:17:45 +0000 | |
---|---|---|
committer | 2023-11-13 19:17:45 +0000 | |
commit | 466bb220909f1836232d82091196a942f16c283c (patch) | |
tree | bd786a07512082735fc3bb09daf55e404ef52ec2 | |
parent | ed9d179947325bbb9b81378b469341d023b46cd8 (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.groovy | 2 |
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) } |