Dashboard - Add more logging around Sheriffs and alert creation.

dashboard/dashboard/edit_sheriffs_test.py

 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import unittest
 
 import webapp2
 import webtest
 
 from google.appengine.api import users
@@ skipping 195 matching lines @@
             'email': '',
             'internal_only': False,
             'labels': 'hello,world',
             'summarize': True,
             'patterns': 'a/b/c\nx/y/z',
         },
     }
     actual = self.GetEmbeddedVariable(response, 'SHERIFF_DATA')
     self.assertEqual(expected, actual)
 
+  def testPost_SendsNotificationEmail(self):
+    self._AddSampleTestData()
+    self._AddSheriff('Chromium Perf Sheriff', patterns=['*/*/*/*'])
+    self.testapp.post('/edit_sheriffs', {
+        'add-edit': 'edit',
+        'edit-name': 'Chromium Perf Sheriff',
+        'patterns': '*/*/*/ddd\n\n*/*/*/ccc',
+        'xsrf_token': xsrf.GenerateToken(users.get_current_user()),
+    })
+    sheriff_entity = sheriff.Sheriff.query().fetch()[0]
+    self.assertEqual(['*/*/*/ccc', '*/*/*/ddd'], sheriff_entity.patterns)
+
+    messages = self.mail_stub.get_sent_messages()
+    self.assertEqual(1, len(messages))
+    self.assertEqual('gasper-alerts@google.com', messages[0].sender)
+    self.assertEqual('chrome-performance-monitoring-alerts@google.com',
+                     messages[0].to)
+    self.assertEqual(
+        'Added or updated Sheriff: Chromium Perf Sheriff by foo@bar.com',
+        messages[0].subject)
+    expected_email = """The configuration of None was changed by foo@bar.com.
+
+Key: Chromium Perf Sheriff
+
+Added test paths:
+[]
+
+Removed test paths:
+[
+  "TheMaster/TheBot/Suite1/bbb",
+  "TheMaster/TheBot/Suite1/aaa"
+]"""
+    print messages[0].body
+    self.assertIn(expected_email, str(messages[0].body))
+
 
 if __name__ == '__main__':
   unittest.main()