Skip to content

Commit 47ab768

Browse files
committed
Ignore records with unset performance when grouping test cases
1 parent 55926cd commit 47ab768

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

reframe/frontend/reporting/__init__.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -511,6 +511,11 @@ def _group_testcases(testcases, group_by, extra_cols):
511511
for pvar, reftuple in tc['perfvalues'].items():
512512
pvar = pvar.split(':')[-1]
513513
pval, pref, plower, pupper, punit = reftuple
514+
if pval is None:
515+
# Ignore `None` performance values
516+
# (performance tests that failed sanity)
517+
continue
518+
514519
plower = pref * (1 + plower) if plower is not None else -math.inf
515520
pupper = pref * (1 + pupper) if pupper is not None else math.inf
516521
record = {

unittests/resources/checks/frontend_checks.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,10 +63,15 @@ class NoPrgEnvCheck(BaseFrontendCheck):
6363

6464
@rfm.simple_test
6565
class SanityFailureCheck(BaseFrontendCheck):
66+
'''A performance test that fails in sanity.'''
6667
@sanity_function
6768
def validate_output(self):
6869
return sn.assert_found('foo', self.stdout)
6970

71+
@performance_function('Gflop/s')
72+
def perf(self):
73+
return sn.extractsingle(r'perf: (\d+)', self.stdout, 1, int)
74+
7075

7176
@rfm.simple_test
7277
class PerformanceFailureCheck(BaseFrontendCheck):

0 commit comments

Comments
 (0)