[Findit] Catch ValueError when get swarming result.

appengine/findit/waterfall/swarming_util.py

Index: appengine/findit/waterfall/swarming_util.py
diff --git a/appengine/findit/waterfall/swarming_util.py b/appengine/findit/waterfall/swarming_util.py
index d8cad51b9c39cc10c3c6fca143f51a0cd009ac7d..9adf9bc0536d3cffa9cf7b4ced35d83870893601 100644
--- a/appengine/findit/waterfall/swarming_util.py
+++ b/appengine/findit/waterfall/swarming_util.py
@@ -252,8 +252,10 @@ def _RetrieveOutputJsonFile(output_json_content, http_client):
     get_content = base64.b64decode(json_content['content'])
   else:  # pragma: no cover
     get_content = None  # Just for precausion.
-  return json.loads(zlib.decompress(get_content)) if get_content else None
-
+  try:
+    return json.loads(zlib.decompress(get_content)) if get_content else None
+  except ValueError:
+    return None

[lijeffrey, 2016/07/19 04:43:43]

Do we want to add some logging that get_content is unexpectedly not in json
format?

[chanli, 2016/07/20 20:12:33]

Done.

 
 def _DownloadTestResults(isolated_data, http_client):
   """Downloads the output.json file and returns the json object."""