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

Unified Diff: appengine/findit/crash/test/results_test.py

Issue 1914113002: [Findit] Enable project classifier and component classifier (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Rebase Created 4 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/crash/test/results_test.py
diff --git a/appengine/findit/crash/test/results_test.py b/appengine/findit/crash/test/results_test.py
index cd9f8651f633844ebd4c9b4a982fe99448964c93..c9058dfc0a318528ebd5e9c9875e520857e96af2 100644
--- a/appengine/findit/crash/test/results_test.py
+++ b/appengine/findit/crash/test/results_test.py
@@ -78,7 +78,7 @@ class ResultsTest(CrashTestSuite):
def testResultToDict(self):
- result = Result(DUMMY_CHANGELOG1, 'src/', '',
+ result = Result(DUMMY_CHANGELOG1, 'src/',
confidence=1, reason='some reason')
expected_result_json = {
@@ -96,14 +96,14 @@ class ResultsTest(CrashTestSuite):
def testResultToString(self):
- result = Result(DUMMY_CHANGELOG1, 'src/', '',
+ result = Result(DUMMY_CHANGELOG1, 'src/',
confidence=1, reason='some reason')
expected_result_str = ''
self.assertEqual(result.ToString(), expected_result_str)
result.file_to_stack_infos = {
- 'a.cc': [(StackFrame(0, 'src/', '', 'func', 'a.cc', []), 0)]
+ 'a.cc': [(StackFrame(0, 'src/', 'func', 'a.cc', 'src/a.cc', []), 0)]
}
expected_result_str = 'Changed file a.cc crashed in func (#0)'
@@ -111,35 +111,37 @@ class ResultsTest(CrashTestSuite):
def testMatchResultUpdate(self):
# Touched lines have intersection with crashed lines.
- result = MatchResult(DUMMY_CHANGELOG1, 'src/', '',
+ result = MatchResult(DUMMY_CHANGELOG1, 'src/',
confidence=1, reason='some reason')
- stack_infos = [(StackFrame(0, 'src/', '', 'func', 'a.cc', [7]), 0)]
+ stack_infos = [(StackFrame(0, 'src/', 'func', 'a.cc', 'src/a.cc', [7]), 0)]
result.Update('a.cc', stack_infos, DUMMY_BLAME)
self.assertEqual(result.min_distance, 0)
# Touched lines are before crashed lines.
- result = MatchResult(DUMMY_CHANGELOG1, 'src/', '',
+ result = MatchResult(DUMMY_CHANGELOG1, 'src/',
confidence=1, reason='some reason')
- stack_infos = [(StackFrame(0, 'src/', '', 'func', 'a.cc', [3]), 0)]
+ stack_infos = [(StackFrame(0, 'src/', 'func', 'a.cc', 'src/a.cc', [3]), 0)]
result.Update('a.cc', stack_infos, DUMMY_BLAME)
self.assertEqual(result.min_distance, 3)
# Touched lines are after crashed lines.
- result = MatchResult(DUMMY_CHANGELOG1, 'src/', '',
+ result = MatchResult(DUMMY_CHANGELOG1, 'src/',
confidence=1, reason='some reason')
- stack_infos = [(StackFrame(0, 'src/', '', 'func', 'a.cc', [10]), 0)]
+ stack_infos = [(StackFrame(0, 'src/', 'func', 'a.cc', 'src/a.cc', [10]), 0)]
result.Update('a.cc', stack_infos, DUMMY_BLAME)
self.assertEqual(result.min_distance, 2)
def testMatchResultsGenerateMatchResults(self):
match_results = MatchResults(ignore_cls=set(['2']))
- stack_infos1 = [(StackFrame(0, 'src/', '', 'func', 'a.cc', [7]), 0)]
- stack_infos2 = [(StackFrame(1, 'src/', '', 'func', 'b.cc', [11]), 0)]
+ stack_infos1 = [(StackFrame(
+ 0, 'src/', 'func', 'a.cc', 'src/a.cc', [7]), 0)]
+ stack_infos2 = [(StackFrame(
+ 1, 'src/', 'func', 'b.cc', 'src/b.cc', [11]), 0)]
match_results.GenerateMatchResults('a.cc', 'src/', stack_infos1,
[DUMMY_CHANGELOG1, DUMMY_CHANGELOG2],
DUMMY_BLAME)
@@ -148,7 +150,7 @@ class ResultsTest(CrashTestSuite):
[DUMMY_CHANGELOG1, DUMMY_CHANGELOG2],
DUMMY_BLAME2)
- expected_match_result = MatchResult(DUMMY_CHANGELOG1, 'src/', '')
+ expected_match_result = MatchResult(DUMMY_CHANGELOG1, 'src/')
expected_match_result.file_to_stack_infos = {
'a.cc': stack_infos1,
'b.cc': stack_infos2
« no previous file with comments | « appengine/findit/crash/test/project_classifier_test.py ('k') | appengine/findit/crash/test/stacktrace_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698