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

Unified Diff: appengine/findit/model/crash/test/chrome_crash_analysis_test.py

Issue 2663063007: [Predator] Switch from anonymous dict to CrashData. (Closed)
Patch Set: Rebase. 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/model/crash/test/chrome_crash_analysis_test.py
diff --git a/appengine/findit/model/crash/test/chrome_crash_analysis_test.py b/appengine/findit/model/crash/test/chrome_crash_analysis_test.py
index 7d0982977a8439c93265936f0bdec84695fac9d4..697b47656f95cee1673fa31207d9761f4417b68b 100644
--- a/appengine/findit/model/crash/test/chrome_crash_analysis_test.py
+++ b/appengine/findit/model/crash/test/chrome_crash_analysis_test.py
@@ -4,6 +4,7 @@
from datetime import datetime
+from crash.chrome_crash_buffer import ChromeCrashBuffer
from crash.test.predator_testcase import PredatorTestCase
from model import analysis_status
from model import result_status
@@ -43,3 +44,40 @@ class ChromeCrashAnalysisTest(PredatorTestCase):
{'historical_metadata': {'chrome_version': '50.0.1200.0',
'cpm': 0.5},
'channel': 'canary'})
+
+ def testInitializeWithCrashBuffer(self):
+ findit = self.GetMockFindit()
+ channel = 'dummy channel'
+ historical_metadata = []
+ crash_data = self.GetDummyCrashData(channel=channel,
+ historical_metadata=historical_metadata)
+ class MockChromeCrashBuffer(ChromeCrashBuffer):
+
+ def __init__(self, crash_data):
+ super(MockChromeCrashBuffer, self).__init__(crash_data, None)
+
+ @property
+ def stacktrace(self):
+ return None
+
+ @property
+ def regression_range(self):
+ return None
+
+ @property
+ def dependencies(self):
+ return {}
+
+ @property
+ def dependency_rolls(self):
+ return {}
+
+ self.mock(findit, 'GetCrashBuffer',
+ lambda crash_data: MockChromeCrashBuffer( # pylint: disable=W0108
+ crash_data))
+
+ crash_buffer = findit.GetCrashBuffer(crash_data)
+ analysis = ChromeCrashAnalysis()
+ analysis.Initialize(crash_buffer)
+ self.assertEqual(analysis.channel, channel)
+ self.assertEqual(analysis.historical_metadata, historical_metadata)

Powered by Google App Engine
This is Rietveld 408576698