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

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

Issue 2663063007: [Predator] Switch from anonymous dict to CrashData. (Closed)
Patch Set: Rebase and fix delta test. 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/chrome_crash_data_test.py
diff --git a/appengine/findit/crash/test/chrome_crash_data_test.py b/appengine/findit/crash/test/chrome_crash_data_test.py
index 18a6a470c72dd9393c62a32da0a56a11ddc0b06c..f208e672f4f56851d20a7bcad3a1b34e8beeb4e1 100644
--- a/appengine/findit/crash/test/chrome_crash_data_test.py
+++ b/appengine/findit/crash/test/chrome_crash_data_test.py
@@ -22,7 +22,7 @@ class ChromeCrashDataTest(StacktraceTestSuite):
def testProperties(self):
"""Tests ``ChromeCrashData`` specific properties."""
- raw_crash_data = self.GetDummyCrashData()
+ raw_crash_data = self.GetDummyChromeCrashData()
crash_data = ChromeCrashData(raw_crash_data, None)
self.assertEqual(crash_data.channel,
raw_crash_data['customized_data']['channel'])
@@ -33,7 +33,7 @@ class ChromeCrashDataTest(StacktraceTestSuite):
"""Tests that ``stacktrace`` is None when failed to pars stacktrace."""
self.mock(ChromeDependencyFetcher, 'GetDependency', lambda *_: {})
crash_data = ChromeCrashData(
- self.GetDummyCrashData(),
+ self.GetDummyChromeCrashData(),
ChromeDependencyFetcher(self.GetMockRepoFactory()))
self.mock(ChromeCrashParser, 'Parse', lambda *args, **kwargs: None)
self.assertIsNone(crash_data.stacktrace)
@@ -42,7 +42,7 @@ class ChromeCrashDataTest(StacktraceTestSuite):
"""Tests parsing ``stacktrace``."""
self.mock(ChromeDependencyFetcher, 'GetDependency', lambda *_: {})
crash_data = ChromeCrashData(
- self.GetDummyCrashData(),
+ self.GetDummyChromeCrashData(),
ChromeDependencyFetcher(self.GetMockRepoFactory()))
stack = CallStack(0)
stacktrace = Stacktrace([stack], stack)
@@ -52,7 +52,7 @@ class ChromeCrashDataTest(StacktraceTestSuite):
def testParseStacktraceReturnsCache(self):
"""Tests that ``stacktrace`` returns cached ``_stacktrace`` value."""
- crash_data = ChromeCrashData(self.GetDummyCrashData(), None)
+ crash_data = ChromeCrashData(self.GetDummyChromeCrashData(), None)
stack = CallStack(1)
stacktrace = Stacktrace([stack], stack)
crash_data._stacktrace = stacktrace
@@ -61,7 +61,7 @@ class ChromeCrashDataTest(StacktraceTestSuite):
def testDetectRegressionRangeFailed(self):
"""Tests that ``regression_range`` is None when detection failed."""
crash_data = ChromeCrashData(
- self.GetDummyCrashData(),
+ self.GetDummyChromeCrashData(),
ChromeDependencyFetcher(self.GetMockRepoFactory()))
with mock.patch('crash.detect_regression_range.DetectRegressionRange',
@@ -71,7 +71,7 @@ class ChromeCrashDataTest(StacktraceTestSuite):
def testDetectRegressionRangeSucceeded(self):
"""Tests detecting ``regression_range``."""
crash_data = ChromeCrashData(
- self.GetDummyCrashData(),
+ self.GetDummyChromeCrashData(),
ChromeDependencyFetcher(self.GetMockRepoFactory()))
regression_range = ('1', '3')
@@ -81,22 +81,21 @@ class ChromeCrashDataTest(StacktraceTestSuite):
def testDetectRegressionRangeReturnsCache(self):
"""Tests that ``regression_range`` returns cached ``_regression_range``."""
- crash_data = ChromeCrashData(self.GetDummyCrashData(), None)
+ crash_data = ChromeCrashData(self.GetDummyChromeCrashData(), None)
regression_range = ('1', '5')
crash_data._regression_range = regression_range
self.assertEqual(crash_data.regression_range, regression_range)
def testCrashedVersionDepsReturnsCache(self):
"""Tests that ``_CrashedVersionDeps`` returns cached value."""
- crash_data = ChromeCrashData(self.GetDummyCrashData(), None)
+ crash_data = ChromeCrashData(self.GetDummyChromeCrashData(), None)
crashed_version_deps = {'src/': Dependency('src/', 'https://repo', 'rev')}
crash_data._crashed_version_deps = crashed_version_deps
self.assertEqual(crash_data._CrashedVersionDeps(), crashed_version_deps)
def testDependencies(self):
"""Tests that ``dependencies`` returns filtered ``_CrashedVersionDeps``."""
- crash_data = ChromeCrashData(self.GetDummyCrashData(), None)
-
+ crash_data = ChromeCrashData(self.GetDummyChromeCrashData(), None)
chromium_dep = Dependency('src/', 'https://repo', 'rev1')
crash_data._crashed_version_deps = {
chromium_dep.path: chromium_dep,
@@ -110,21 +109,21 @@ class ChromeCrashDataTest(StacktraceTestSuite):
def testDependenciesReturnsCache(self):
"""Tests that ``dependencies`` returns cached ``_dependencies`` value."""
- crash_data = ChromeCrashData(self.GetDummyCrashData(), None)
+ crash_data = ChromeCrashData(self.GetDummyChromeCrashData(), None)
crashed_version_deps = {'src/': Dependency('src/', 'https://repo', 'rev')}
crash_data._dependencies = crashed_version_deps
self.assertEqual(crash_data.dependencies, crashed_version_deps)
def testDependencyRollsReturnsCache(self):
"""Tests that ``dependency_rolls`` returns cached ``_dependency_rolls``."""
- crash_data = ChromeCrashData(self.GetDummyCrashData(), None)
+ crash_data = ChromeCrashData(self.GetDummyChromeCrashData(), None)
dep_roll = {'src/': DependencyRoll('src/', 'https://repo', 'rev0', 'rev3')}
crash_data._dependency_rolls = dep_roll
self.assertEqual(crash_data.dependency_rolls, dep_roll)
def testDependencyRollsWhenRegressionRangeIsEmpty(self):
"""Tests that ``regression_rolls`` is {} when regression_range is empty."""
- crash_data = ChromeCrashData(self.GetDummyCrashData(), None)
+ crash_data = ChromeCrashData(self.GetDummyChromeCrashData(), None)
crash_data._regression_range = None
self.assertEqual(crash_data.dependency_rolls, {})
@@ -140,7 +139,7 @@ class ChromeCrashDataTest(StacktraceTestSuite):
self.mock(ChromeDependencyFetcher, 'GetDependencyRollsDict',
lambda *_: regression_rolls)
crash_data = ChromeCrashData(
- self.GetDummyCrashData(),
+ self.GetDummyChromeCrashData(),
ChromeDependencyFetcher(self.GetMockRepoFactory()))
crash_data._regression_range = ('rev1', 'rev6')
« no previous file with comments | « appengine/findit/crash/test/changelist_classifier_test.py ('k') | appengine/findit/crash/test/crash_data_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698