Diff: test_api.py

Revision:

35:da9c89f8be7d

Parent:

31:8ea194f6145b

Child:

36:96847d42f010

diff -r 42f20e28adf5 -r da9c89f8be7d test_api.py
--- a/test_api.py	Mon Feb 13 09:29:13 2017 -0600
+++ b/test_api.py	Wed Feb 15 13:53:18 2017 -0600
@@ -2133,7 +2133,7 @@
     base_path = norm_relative_path(build_path, execution_directory)
 
     target_name = target if isinstance(target, str) else target.name
-    cfg, macros, features = get_config(base_source_paths, target_name, toolchain_name)
+    cfg, _, _ = get_config(base_source_paths, target_name, toolchain_name)
 
     baud_rate = 9600
     if 'platform.stdio-baud-rate' in cfg:
@@ -2201,12 +2201,16 @@
                             report[target_name][toolchain_name][test_key] = report_entry[test_key]
                         
                         # Set the overall result to a failure if a build failure occurred
-                        if not worker_result['result'] and not isinstance(worker_result['reason'], NotSupportedException):
+                        if ('reason' in worker_result and
+                            not worker_result['reason'] and
+                            not isinstance(worker_result['reason'], NotSupportedException)):
                             result = False
                             break
 
                         # Adding binary path to test build result
-                        if worker_result['result'] and 'bin_file' in worker_result:
+                        if ('result' in worker_result and
+                            worker_result['result'] and
+                            'bin_file' in worker_result):
                             bin_file = norm_relative_path(worker_result['bin_file'], execution_directory)
 
                             test_build['tests'][worker_result['kwargs']['project_id']] = {