Skip to content

Commit bb9efa4

Browse files
author
Jeny Sadadia
committed
src/send_kcidb: fixes of the previous commit
Fix `_parse_coverage_node` method: 1. Fix `field_name` 2. Fix extraction of parsed build node Signed-off-by: Jeny Sadadia <[email protected]>
1 parent 4ed796e commit bb9efa4

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/send_kcidb.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -577,7 +577,8 @@ def _parse_coverage_node(self, origin, node):
577577
if build_node['kind'] != 'kbuild':
578578
self.log.debug(f"{node['id']} is not a top-level coverage node, skipping")
579579
return []
580-
parsed_coverage_node = self._parse_build_node(origin, build_node)
580+
parsed_node = self._parse_build_node(origin, build_node)
581+
parsed_coverage_node = parsed_node[0]
581582

582583
# Add misc fields for coverage results
583584
child_nodes = self._api.node.find({'parent': node['id'], 'kind': 'test'})
@@ -588,7 +589,7 @@ def _parse_coverage_node(self, origin, node):
588589
self.log.warning(f"No measurement in node '{child['name']}' ({child['id']})")
589590
continue
590591
field_name = self._replace_restricted_chars(child['name'], r'^[a-zA-Z0-9_]*$')
591-
parsed_coverage_node['misc'][field] = child['data']['misc']['measurement']
592+
parsed_coverage_node['misc'][field_name] = child['data']['misc']['measurement']
592593

593594
# Add HTML coverage report to the build node's artifacts
594595
artifacts = node.get('artifacts')

0 commit comments

Comments
 (0)