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

Unified Diff: dashboard/dashboard/add_histograms_test.py

Issue 2988823002: DeviceInfo into GenericSets (Closed)
Patch Set: Created 3 years, 5 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: dashboard/dashboard/add_histograms_test.py
diff --git a/dashboard/dashboard/add_histograms_test.py b/dashboard/dashboard/add_histograms_test.py
index 2deb9ce64b66bf31f9cffece18e83c42b5ace60f..e993ebe64dc7fd4abb3a8594657ebdf81209fff5 100644
--- a/dashboard/dashboard/add_histograms_test.py
+++ b/dashboard/dashboard/add_histograms_test.py
@@ -570,9 +570,14 @@ class AddHistogramsTest(testing_common.TestCase):
def testDeduplicateAndPut_Same(self):
d = {
- 'guid': 'abc',
- 'osName': 'linux',
- 'type': 'DeviceInfo'
+ 'buildNumber': 0,
+ 'buildbotMasterName': '',
+ 'buildbotName': '',
+ 'displayBotName': 'bot',
+ 'displayMasterName': 'master',
+ 'guid': 'e9c2891d-2b04-413f-8cf4-099827e67626',
+ 'logUri': '',
+ 'type': 'BuildbotInfo'
}
test_key = utils.TestKey('Chromium/win7/foo')
entity = histogram.SparseDiagnostic(
@@ -590,9 +595,14 @@ class AddHistogramsTest(testing_common.TestCase):
def testDeduplicateAndPut_Different(self):
d = {
- 'guid': 'abc',
- 'osName': 'linux',
- 'type': 'DeviceInfo'
+ 'buildNumber': 0,
+ 'buildbotMasterName': '',
+ 'buildbotName': '',
+ 'displayBotName': 'bot',
+ 'displayMasterName': 'master',
+ 'guid': 'e9c2891d-2b04-413f-8cf4-099827e67626',
+ 'logUri': '',
+ 'type': 'BuildbotInfo'
}
test_key = utils.TestKey('Chromium/win7/foo')
entity = histogram.SparseDiagnostic(
@@ -601,7 +611,7 @@ class AddHistogramsTest(testing_common.TestCase):
entity.put()
d2 = d.copy()
d2['guid'] = 'def'
- d2['osName'] = 'mac'
+ d2['displayBotName'] = 'mac'
entity2 = histogram.SparseDiagnostic(
data=d2, test=test_key,
start_revision=1, end_revision=sys.maxint, id='def')
@@ -611,9 +621,14 @@ class AddHistogramsTest(testing_common.TestCase):
def testDeduplicateAndPut_New(self):
d = {
- 'guid': 'abc',
- 'osName': 'linux',
- 'type': 'DeviceInfo'
+ 'buildNumber': 0,
+ 'buildbotMasterName': '',
+ 'buildbotName': '',
+ 'displayBotName': 'bot',
+ 'displayMasterName': 'master',
+ 'guid': 'e9c2891d-2b04-413f-8cf4-099827e67626',
+ 'logUri': '',
+ 'type': 'BuildbotInfo'
}
test_key = utils.TestKey('Chromium/win7/foo')
entity = histogram.SparseDiagnostic(

Powered by Google App Engine
This is Rietveld 408576698