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..e2b6907bb53c82aac67c27df79eff5f0ebf5fab8 100644 |
--- a/appengine/findit/waterfall/test/build_failure_analysis_test.py |
+++ b/appengine/findit/waterfall/test/build_failure_analysis_test.py |
@@ -113,7 +113,7 @@ class BuildFailureAnalysisTest(unittest.TestCase): |
} |
justification = build_failure_analysis._CheckFiles( |
- FailureSignal.FromDict(failure_signal_json), change_log_json) |
+ FailureSignal.FromDict(failure_signal_json), change_log_json, {}) |
self.assertIsNotNone(justification) |
# The score is 14 because: |
# +5 added a/b/f1.cc (same file src/a/b/f1.cc in failure_signal log) |
@@ -142,7 +142,7 @@ class BuildFailureAnalysisTest(unittest.TestCase): |
} |
justification = build_failure_analysis._CheckFiles( |
- FailureSignal.FromDict(failure_signal_json), change_log_json) |
+ FailureSignal.FromDict(failure_signal_json), change_log_json, {}) |
self.assertIsNone(justification) |
def testAnalyzeSuccessfulBuild(self): |
@@ -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) |