Add presubmit checks to look for BUG= and TEST= tags in change list descripti...

tests/presubmit_unittest.py

Index: tests/presubmit_unittest.py
===================================================================
--- tests/presubmit_unittest.py	(revision 15890)
+++ tests/presubmit_unittest.py	(working copy)
@@ -583,14 +583,41 @@
 
   def testMembersChanged(self):
     members = [
-      'CheckChangeHasNoTabs', 'CheckChangeHasQaField',
-      'CheckChangeHasTestedField', 'CheckDoNotSubmit',
+      'CheckChangeHasBugField', 'CheckChangeHasNoTabs',
+      'CheckChangeHasQaField', 'CheckChangeHasTestedField',
+      'CheckChangeHasTestField', 'CheckDoNotSubmit',
       'CheckDoNotSubmitInDescription', 'CheckDoNotSubmitInFiles',
       'CheckLongLines', 'CheckTreeIsOpen',
     ]
     # If this test fails, you should add the relevant test.
     self.compareMembers(presubmit_canned_checks, members)
 
+  def testCannedCheckChangeHasBugField(self):
+    change = self.MakeBasicChange('foo',
+                                  'Foo\nBUG=1234')
+    api = presubmit.InputApi(change, 'PRESUBMIT.py')
+    self.failIf(presubmit_canned_checks.CheckChangeHasBugField(
+        api, presubmit.OutputApi))
+
+    change = self.MakeBasicChange('foo',
+                                  'Foo\nNEVERTESTED=did some stuff')
+    api = presubmit.InputApi(change, 'PRESUBMIT.py')
+    self.failUnless(presubmit_canned_checks.CheckChangeHasBugField(
+        api, presubmit.OutputApi))
+
+  def testCannedCheckChangeHasTestField(self):
+    change = self.MakeBasicChange('foo',
+                                  'Foo\nTEST=did some stuff')
+    api = presubmit.InputApi(change, 'PRESUBMIT.py')
+    self.failIf(presubmit_canned_checks.CheckChangeHasTestField(
+        api, presubmit.OutputApi))
+
+    change = self.MakeBasicChange('foo',
+                                  'Foo\nNOTEST=did some stuff')
+    api = presubmit.InputApi(change, 'PRESUBMIT.py')
+    self.failUnless(presubmit_canned_checks.CheckChangeHasTestField(
+        api, presubmit.OutputApi))
+
   def testCannedCheckChangeHasTestedField(self):
     change = self.MakeBasicChange('foo',
                                   'Foo\nTESTED=did some stuff')