Skip to content

Commit e63fdc9

Browse files
Jeny SadadiaJenySadadia
Jeny Sadadia
authored andcommitted
src/send_kcidb: handle skipped tests
Do not retrieve artifacts or metadata from parent node for skipped tests as in pratice only kernel revision, test runtime and platform will be available for skipped tests. Signed-off-by: Jeny Sadadia <[email protected]>
1 parent c8f8b28 commit e63fdc9

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/send_kcidb.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -335,6 +335,12 @@ def _parse_test_node(self, origin, test_node):
335335
}
336336
}
337337

338+
if test_node['result']:
339+
parsed_test_node['status'] = self._parse_node_result(test_node)
340+
if parsed_test_node['status'] == 'SKIP':
341+
# No artifacts and metadata will be available for skipped tests
342+
return parsed_test_node, dummy_build
343+
338344
job_metadata = self._get_job_metadata(test_node)
339345
if job_metadata:
340346
parsed_test_node['environment']['misc']['job_id'] = job_metadata.get(
@@ -358,9 +364,6 @@ def _parse_test_node(self, origin, test_node):
358364
parsed_test_node['log_excerpt'] = self._get_log_excerpt(
359365
log_url)
360366

361-
if test_node['result']:
362-
parsed_test_node['status'] = self._parse_node_result(test_node)
363-
364367
if test_node['result'] != 'pass':
365368
error_metadata = self._get_error_metadata(test_node)
366369
if error_metadata:

0 commit comments

Comments
 (0)