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

Unified Diff: dashboard/dashboard/file_bug_test.py

Issue 2138843003: Add support for setting sheriff-specific bug labels. (Closed) Base URL: https://github.com/catapult-project/catapult@master
Patch Set: Fix unit tests Created 4 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
« no previous file with comments | « dashboard/dashboard/file_bug.py ('k') | dashboard/dashboard/models/sheriff.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dashboard/dashboard/file_bug_test.py
diff --git a/dashboard/dashboard/file_bug_test.py b/dashboard/dashboard/file_bug_test.py
index cd3e520e627bd55cb8520c7e8cec9609c7719fe0..1bef12cb989bd88a8c035ba2198486bc161808db 100644
--- a/dashboard/dashboard/file_bug_test.py
+++ b/dashboard/dashboard/file_bug_test.py
@@ -72,7 +72,8 @@ class FileBugTest(testing_common.TestCase):
def _AddSampleAlerts(self):
"""Adds sample data and returns a dict of rev to anomaly key."""
# Add sample sheriff, masters, bots, and tests.
- sheriff_key = sheriff.Sheriff(id='Sheriff').put()
+ sheriff_key = sheriff.Sheriff(
+ id='Sheriff', labels=['Performance-Sheriff']).put()
testing_common.AddTests(['ChromiumPerf'], ['linux'], {
'scrolling': {
'first_paint': {},
@@ -132,6 +133,7 @@ class FileBugTest(testing_common.TestCase):
alert_keys[0].urlsafe(), alert_keys[1].urlsafe()))
self.assertIn('label1-foo', response.body)
self.assertIn('Performance>Blink', response.body)
+ self.assertIn('Performance-Sheriff', response.body)
@mock.patch(
'google.appengine.api.app_identity.get_default_version_hostname',
« no previous file with comments | « dashboard/dashboard/file_bug.py ('k') | dashboard/dashboard/models/sheriff.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698