Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(150)

Unified Diff: appengine/findit/waterfall/test/swarming_util_test.py

Issue 2312413002: [Findit] Record error if swarming task completes with exit_code as 2. (Closed)
Patch Set: Add None check to avoid keyvalue error (related change) Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: appengine/findit/waterfall/test/swarming_util_test.py
diff --git a/appengine/findit/waterfall/test/swarming_util_test.py b/appengine/findit/waterfall/test/swarming_util_test.py
index b4252b797ffbc13016c0a0cc4d1dfd98304bc1ae..9ac70162be055625c6fae6c7efc83247ef89c711 100644
--- a/appengine/findit/waterfall/test/swarming_util_test.py
+++ b/appengine/findit/waterfall/test/swarming_util_test.py
@@ -578,4 +578,8 @@ class SwarmingUtilTest(wf_testcase.WaterfallTestCase):
self.assertIsNone(swarming_util.GetTagValue(tags, 'c'))
def testGenerateIsolatedDataOutputsrefNone(self):
- self.assertEqual({}, swarming_util._GenerateIsolatedData(None))
+ self.assertEqual({}, swarming_util._GenerateIsolatedData(None))
+
+ def testFetchOutputJsonInfoFromIsolatedServerReturnNone(self):
+ self.assertIsNone(swarming_util._FetchOutputJsonInfoFromIsolatedServer(
+ None, self.http_client))

Powered by Google App Engine
This is Rietveld 408576698