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

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

Issue 1154593005: [Findit] Add a sub-pipeline to analyze failures caused by DEPS rolls. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Unittests will be added in next patch. Created 5 years, 7 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/build_failure_analysis_test.py
diff --git a/appengine/findit/waterfall/test/build_failure_analysis_test.py b/appengine/findit/waterfall/test/build_failure_analysis_test.py
index 8fbd51e8e545c923a83b148a34f233bb5db133cb..69a5ba8400970656cfe5fa99af3ffd308bf79f3c 100644
--- a/appengine/findit/waterfall/test/build_failure_analysis_test.py
+++ b/appengine/findit/waterfall/test/build_failure_analysis_test.py
@@ -150,7 +150,7 @@ class BuildFailureAnalysisTest(unittest.TestCase):
'failed': False,
}
result = build_failure_analysis.AnalyzeBuildFailure(
- failure_info, None, None)
+ failure_info, None, None, None)
self.assertEqual(0, len(result['failures']))
def testAnalyzeBuildFailure(self):
@@ -239,6 +239,10 @@ class BuildFailureAnalysisTest(unittest.TestCase):
],
},
}
+ deps_info = {
+ 'deps_rolls': {},
+ 'current_deps': {},
+ }
failure_signals_json = {
'a': {
'files': {
@@ -293,5 +297,5 @@ class BuildFailureAnalysisTest(unittest.TestCase):
}
analysis_result = build_failure_analysis.AnalyzeBuildFailure(
- failure_info, change_logs, failure_signals_json)
+ failure_info, change_logs, deps_info, failure_signals_json)
self.assertEqual(expected_analysis_result, analysis_result)

Powered by Google App Engine
This is Rietveld 408576698