Add step to process Render Test results.

scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Remoting_Tests.json

 [
   {
     "cmd": [
       "python",
       "-u",
       "\nimport sys, os\npath = sys.argv[1]\nmode = int(sys.argv[2])\nif not os.path.isdir(path):\n  if os.path.exists(path):\n    print \"%s exists but is not a dir\" % path\n    sys.exit(1)\n  os.makedirs(path, mode)\n",
       "[BUILDER_CACHE]/Android_Remoting_Tests",
       "511"
     ],
     "name": "makedirs checkout path",
@@ skipping 595 matching lines @@
       "[BUILDER_CACHE]/Android_Remoting_Tests/src/third_party/android_tools/sdk/platform-tools/adb"
     ],
     "cwd": "[BUILDER_CACHE]/Android_Remoting_Tests",
     "env": {
       "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account-goma-client.json",
       "PATH": "[BUILDER_CACHE]/Android_Remoting_Tests/src/third_party/android_tools/sdk/platform-tools:[BUILDER_CACHE]/Android_Remoting_Tests/src/build/android:%(PATH)s"
     },
     "name": "remoting_unittests"
   },
   {
+    "cmd": [],
+    "cwd": "[BUILDER_CACHE]/Android_Remoting_Tests",
+    "env": {
+      "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account-goma-client.json",
+      "PATH": "[BUILDER_CACHE]/Android_Remoting_Tests/src/third_party/android_tools/sdk/platform-tools:[BUILDER_CACHE]/Android_Remoting_Tests/src/build/android:%(PATH)s"
+    },
+    "name": "ChromotingTest",
+    "~followup_annotations": [
+      "@@@STEP_TEXT@<br/>failures:<br/>Test.Two<br/>@@@",
+      "@@@STEP_FAILURE@@@"
+    ]
+  },
+  {
     "cmd": [
       "python",
       "-u",
       "[BUILDER_CACHE]/Android_Remoting_Tests/src/out/Release/bin/run_remoting_test_apk",
       "--blacklist-file",
       "[BUILDER_CACHE]/Android_Remoting_Tests/src/out/bad_devices.json",
       "--verbose",
       "--json-results-file",
       "/path/to/tmp/json",
       "--adb-path",
       "[BUILDER_CACHE]/Android_Remoting_Tests/src/third_party/android_tools/sdk/platform-tools/adb"
     ],
     "cwd": "[BUILDER_CACHE]/Android_Remoting_Tests",
     "env": {
       "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account-goma-client.json",
       "PATH": "[BUILDER_CACHE]/Android_Remoting_Tests/src/third_party/android_tools/sdk/platform-tools:[BUILDER_CACHE]/Android_Remoting_Tests/src/build/android:%(PATH)s"
     },
-    "name": "ChromotingTest",
+    "name": "ChromotingTest.ChromotingTest",
     "~followup_annotations": [
+      "@@@STEP_NEST_LEVEL@1@@@",
       "step returned non-zero exit code: 1",
-      "@@@STEP_TEXT@<br/>failures:<br/>Test.Two<br/>@@@",
       "@@@STEP_FAILURE@@@"
     ]
   },
   {
     "cmd": [
       "python",
       "-u",
       "RECIPE_MODULE[build::test_results]/resources/upload_test_results.py",
       "--input-json",
       "{\"per_iteration_data\": [{\"Test.One\": [{\"elapsed_time_ms\": 0, \"output_snippet\": \"\", \"status\": \"SUCCESS\"}], \"Test.Three\": [{\"elapsed_time_ms\": 0, \"output_snippet\": \"\", \"status\": \"SUCCESS\"}], \"Test.Two\": [{\"elapsed_time_ms\": 0, \"output_snippet\": \"\", \"status\": \"FAILURE\"}]}]}",
       "--master-name",
       "chromium.fyi",
       "--builder-name",
       "Android Remoting Tests",
       "--build-number",
       "571",
       "--test-type",
       "ChromotingTest",
       "--test-results-server",
       "test-results.appspot.com",
       "--chrome-revision",
       "refs/heads/master@{#170242}"
     ],
     "cwd": "[BUILDER_CACHE]/Android_Remoting_Tests",
     "env": {
       "GOMA_SERVICE_ACCOUNT_JSON_FILE": "/creds/service_accounts/service-account-goma-client.json",
       "PATH": "[BUILDER_CACHE]/Android_Remoting_Tests/src/third_party/android_tools/sdk/platform-tools:[BUILDER_CACHE]/Android_Remoting_Tests/src/build/android:%(PATH)s"
     },
-    "name": "Upload to test-results [ChromotingTest]"
+    "name": "ChromotingTest.Upload to test-results [ChromotingTest]",
+    "~followup_annotations": [
+      "@@@STEP_NEST_LEVEL@1@@@"
+    ]
   },
   {
     "cmd": [
       "python",
       "-u",
       "RECIPE_PACKAGE_REPO[build]/scripts/slave/daemonizer.py",
       "--action",
       "stop",
       "--pid-file-path",
       "/tmp/device_monitor.pid"
@@ skipping 112 matching lines @@
       "@@@STEP_LOG_LINE@python.inline@for report in glob.glob(sys.argv[1]):@@@",
       "@@@STEP_LOG_LINE@python.inline@  with open(report, 'r') as f:@@@",
       "@@@STEP_LOG_LINE@python.inline@    for l in f.readlines():@@@",
       "@@@STEP_LOG_LINE@python.inline@      print l@@@",
       "@@@STEP_LOG_LINE@python.inline@  os.remove(report)@@@",
       "@@@STEP_LOG_END@python.inline@@@"
     ]
   },
   {
     "name": "$result",
-    "reason": "1 tests failed: ['ChromotingTest']",

[mikecase (-- gone --), 2016/10/26 17:33:01]

oops, I didnt see this. writing fix to re-raise step_failure I catch

[mikecase (-- gone --), 2016/10/26 17:37:05]

Fixed

     "recipe_result": null,
-    "status_code": 1
+    "status_code": 0
   }
 ]