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

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

Issue 2673733002: [Predator] Add CrashData class to process raw json crash data. (Closed)
Patch Set: Rename CrashBuffer to CrashData Created 3 years, 10 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/findit_for_chromecrash_test.py
diff --git a/appengine/findit/crash/test/findit_for_chromecrash_test.py b/appengine/findit/crash/test/findit_for_chromecrash_test.py
index dbe3a6a45e8aa98682e1b49582786a58792102c5..4cd853e5b72dfde424df1e8cb0a83024002562ca 100644
--- a/appengine/findit/crash/test/findit_for_chromecrash_test.py
+++ b/appengine/findit/crash/test/findit_for_chromecrash_test.py
@@ -21,7 +21,6 @@ from crash.project_classifier import ProjectClassifier
from crash.suspect import Suspect
from crash.stacktrace import CallStack
from crash.stacktrace import Stacktrace
-from crash.test.crash_pipeline_test import DummyCrashData
from crash.test.predator_testcase import PredatorTestCase
from crash.type_enums import CrashClient
from gae_libs.http.http_client_appengine import HttpClientAppengine
@@ -99,15 +98,15 @@ class FinditForFracasTest(PredatorTestCase):
self.assertEqual(_FinditForFracas().RenamePlatform('linux'), 'unix')
def testCheckPolicyUnsupportedPlatform(self):
- self.assertIsNone(_FinditForFracas().CheckPolicy(DummyCrashData(
+ self.assertIsNone(_FinditForFracas().CheckPolicy(self.GetDummyCrashData(
platform = 'unsupported_platform')))
def testCheckPolicyBlacklistedSignature(self):
- self.assertIsNone(_FinditForFracas().CheckPolicy(DummyCrashData(
+ self.assertIsNone(_FinditForFracas().CheckPolicy(self.GetDummyCrashData(
signature = 'Blacklist marker signature')))
def testCheckPolicyPlatformRename(self):
- new_crash_data = _FinditForFracas().CheckPolicy(DummyCrashData(
+ new_crash_data = _FinditForFracas().CheckPolicy(self.GetDummyCrashData(
platform = 'linux'))
self.assertIsNotNone(new_crash_data,
'FinditForFracas.CheckPolicy unexpectedly returned None')
@@ -127,7 +126,7 @@ class FinditForFracasTest(PredatorTestCase):
analysis)
def testInitializeAnalysisForFracas(self):
- crash_data = DummyCrashData(platform = 'linux')
+ crash_data = self.GetDummyCrashData(platform = 'linux')
crash_identifiers = crash_data['crash_identifiers']
findit_client = _FinditForFracas()
@@ -138,7 +137,7 @@ class FinditForFracasTest(PredatorTestCase):
self.assertIsNotNone(analysis,
'FinditForFracas.GetAnalysis unexpectedly returned None')
- self.assertEqual(analysis.crashed_version, crash_data['crashed_version'])
+ self.assertEqual(analysis.crashed_version, crash_data['chrome_version'])
self.assertEqual(analysis.signature, crash_data['signature'])
self.assertEqual(analysis.platform, crash_data['platform'])
self.assertEqual(analysis.stack_trace, crash_data['stack_trace'])
@@ -148,11 +147,12 @@ class FinditForFracasTest(PredatorTestCase):
self.assertEqual(analysis.channel, channel)
def testNeedsNewAnalysisIsTrueIfNoAnalysisYet(self):
- self.assertTrue(_FinditForFracas()._NeedsNewAnalysis(DummyCrashData()))
+ self.assertTrue(_FinditForFracas()._NeedsNewAnalysis(
+ self.GetDummyCrashData()))
def testNeedsNewAnalysisIsTrueIfLastOneFailed(self):
findit_client = _FinditForFracas()
- crash_data = DummyCrashData()
+ crash_data = self.GetDummyCrashData()
analysis = findit_client.CreateAnalysis(crash_data['crash_identifiers'])
analysis.status = analysis_status.ERROR
analysis.put()
@@ -160,7 +160,7 @@ class FinditForFracasTest(PredatorTestCase):
def testNeedsNewAnalysisIsFalseIfLastOneIsNotFailed(self):
findit_client = _FinditForFracas()
- crash_data = DummyCrashData()
+ crash_data = self.GetDummyCrashData()
crash_identifiers = crash_data['crash_identifiers']
for status in (analysis_status.PENDING, analysis_status.RUNNING,
analysis_status.COMPLETED, analysis_status.SKIPPED):

Powered by Google App Engine
This is Rietveld 408576698