commit 096e1856666ecb97c8dc45a582bbc61b9b320fe0
author Allen Li <ayatane@chromium.org> Fri Jul 14 09:46:07 2017 -0700
committer Aviv Keshet <akeshet@chromium.org> Fri Jul 14 16:52:55 2017 +0000
tree f27c534b06799a4d0baa311509b1c292d3d3fc46
parent 4d92126654fd2ce2ca1c1370d56b46571cc40349

[autotest] Use namedtuple

BUG=chromium:743071
TEST=run_suite_unittest; local run_suite -m against a previous existing
suite

Change-Id: If3425514ab190ff036eeeb9d33311064aca91268
Reviewed-on: https://chromium-review.googlesource.com/572022
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>

site_utils/run_suite.py

diff --git a/site_utils/run_suite.py b/site_utils/run_suite.py
index 6af4373..eb3e51d 100755
--- a/site_utils/run_suite.py
+++ b/site_utils/run_suite.py
@@ -1660,9 +1660,10 @@
         except (error.CrosDynamicSuiteException,
                 error.RPCException, proxy.JSONRPCException) as e:
             logging.exception('Error Message: %s', e)
-            return (RETURN_CODES.INFRA_FAILURE, {'return_message': str(e)})
+            return SuiteResult(RETURN_CODES.INFRA_FAILURE,
+                               {'return_message': str(e)})
         except AttributeError:
-            return (RETURN_CODES.INVALID_OPTIONS, {})
+            return SuiteResult(RETURN_CODES.INVALID_OPTIONS, {})
 
     job_timer = diagnosis_utils.JobTimer(
             job_created_on, float(options.timeout_mins))
@@ -1678,7 +1679,7 @@
     if options.create_and_return:
         msg = '--create_and_return was specified, terminating now.'
         logging.info(msg)
-        return (RETURN_CODES.OK, {'return_message':msg})
+        return SuiteResult(RETURN_CODES.OK, {'return_message':msg})
 
     if options.no_wait:
         return _handle_job_nowait(job_id, options, instance_server)