Rietveld: don't email on "Dry Run: CQ is trying da patch"

appengine/chromium_rietveld/tests/test_utils.py

Index: appengine/chromium_rietveld/tests/test_utils.py
diff --git a/appengine/chromium_rietveld/tests/test_utils.py b/appengine/chromium_rietveld/tests/test_utils.py
index aa7dfb4c458ae56ccb344d78017eaf75628ba6b2..f94bab8943ab30eeee2851c3b0d99d28fa9d099e 100755
--- a/appengine/chromium_rietveld/tests/test_utils.py
+++ b/appengine/chromium_rietveld/tests/test_utils.py
@@ -48,6 +48,24 @@ class UtilsTest(unittest.TestCase):
     msg = 'Dry run failed because https://weird.url/patch-status/123/4'
     self.assertEqual(utils.parse_cq_status_url_message(msg), (None, None, None))
 
+  def test_is_cq_dry_run_start_message(self):
+    self.assertTrue(utils.is_cq_dry_run_start_message(
+        'Dry Run: CQ is trying da patch. Follow status at\n\nhttps://u.r/l'))
+    self.assertTrue(utils.is_cq_dry_run_start_message(
+        'Dry Run: CQ is trying da patch.\nReference: codereview/xxxx/yyy'))
+
+    self.assertFalse(utils.is_cq_dry_run_start_message(
+        'CQ is trying da patch. Follow status at\n\nhttps://u.r/l'))
+    self.assertFalse(utils.is_cq_dry_run_start_message(
+        'Dry run: Try jobs failed...'))
+    self.assertFalse(utils.is_cq_dry_run_start_message(
+        'Try jobs failed because builder name is Dry Run...'))
+
+  def test_is_cq_dry_message(self):
+    self.assertTrue(utils.is_cq_dry_message('Dry run: Try jobs failed...'))
+    self.assertFalse(utils.is_cq_dry_message(
+        'Try jobs failed because builder name is Dry Run...'))
+
 
 if __name__ == '__main__':
   unittest.main()