kernelci: api: helper: don't crash when adding artifacts to node

Nodes are created with `'artifacts': null`, meaning calling `update()`
could lead to an uncaught exception, and therefore a crash. Avoid this
situation by calling `update()` only when this key exists and yields a
non-None value.

Signed-off-by: Arnaud Ferraris <[email protected]>

Author: a-wai

kernelci/api/helper.py

@@ -557,9 +557,14 @@ def submit_results(self, results, root):
         root_from_db = self.api.node.get(root['id'])
         root_node = merge(root_from_db, root)
         root_node = root.copy()
+        # TODO: we should also go through other potential fields and copy
+        # them over. Maybe even just: root_node = merge(results['node'], root)
         root_node['result'] = results['node']['result']
         root_node['state'] = results['node'].get('state', 'done')
-        root_node['artifacts'].update(results['node']['artifacts'])
+        if root_node.get('artifacts'):
+            root_node['artifacts'].update(results['node']['artifacts'])
+        else:
+            root_node['artifacts'] = results['node']['artifacts']
         root_node['data'].update(results['node'].get('data', {}))
         root_node['processed_by_kcidb_bridge'] = False
         if 'holdoff' in results['node']: