use PEP-8 naming for method names in owners.py, tests/owners_unittest.py

tests/owners_unittest.py

Index: tests/owners_unittest.py
diff --git a/tests/owners_unittest.py b/tests/owners_unittest.py
index 7531f83af3c391c24e45aefb153afa78bc833779..fcae3ce9345024e63396e7c0557d93b2ce6da9b0 100755
--- a/tests/owners_unittest.py
+++ b/tests/owners_unittest.py
@@ -55,76 +55,76 @@ class OwnersDatabaseTest(unittest.TestCase):
     os_path = os_path or self.repo
     return owners.Database(root, fopen, os_path)
 
-  def test_Constructor(self):
+  def test_constructor(self):
     self.assertNotEquals(self.db(), None)
 
-  def assert_CoveredBy(self, files, reviewers):
+  def assert_covered_by(self, files, reviewers):
     db = self.db()
-    self.assertTrue(db.FilesAreCoveredBy(set(files), set(reviewers)))
+    self.assertTrue(db.files_are_covered_by(set(files), set(reviewers)))
 
-  def test_CoveredBy_Everyone(self):
-    self.assert_CoveredBy(['DEPS'], [john])
-    self.assert_CoveredBy(['DEPS'], [darin])
+  def test_covered_by__everyone(self):
+    self.assert_covered_by(['DEPS'], [john])
+    self.assert_covered_by(['DEPS'], [darin])
 
-  def test_CoveredBy_Explicit(self):
-    self.assert_CoveredBy(['content/content.gyp'], [john])
-    self.assert_CoveredBy(['chrome/gpu/OWNERS'], [ken])
+  def test_covered_by__explicit(self):
+    self.assert_covered_by(['content/content.gyp'], [john])
+    self.assert_covered_by(['chrome/gpu/OWNERS'], [ken])
 
-  def test_CoveredBy_OwnersPropagatesDown(self):
-    self.assert_CoveredBy(['chrome/gpu/OWNERS'], [ben])
-    self.assert_CoveredBy(['/chrome/renderer/gpu/gpu_channel_host.h'], [peter])
+  def test_covered_by__owners_propagates_down(self):
+    self.assert_covered_by(['chrome/gpu/OWNERS'], [ben])
+    self.assert_covered_by(['/chrome/renderer/gpu/gpu_channel_host.h'], [peter])
 
-  def assert_NotCoveredBy(self, files, reviewers, unreviewed_files):
+  def assert_not_covered_by(self, files, reviewers, unreviewed_files):
     db = self.db()
-    self.assertEquals(db.FilesNotCoveredBy(set(files), set(reviewers)),
+    self.assertEquals(db.files_not_covered_by(set(files), set(reviewers)),
                       set(unreviewed_files))
 
-  def test_NotCoveredBy_NeedAtLeastOneReviewer(self):
-    self.assert_NotCoveredBy(['DEPS'], [], ['DEPS'])
+  def test_not_covered_by__need_at_least_one_reviewer(self):
+    self.assert_not_covered_by(['DEPS'], [], ['DEPS'])
 
-  def test_NotCoveredBy_OwnersPropagatesDown(self):
-    self.assert_NotCoveredBy(
+  def test_not_covered_by__owners_propagates_down(self):
+    self.assert_not_covered_by(
       ['chrome/gpu/gpu_channel.h', 'chrome/renderer/gpu/gpu_channel_host.h'],
       [ben], [])
 
-  def test_NotCoveredBy_PartialCovering(self):
-    self.assert_NotCoveredBy(
+  def test_not_covered_by__partial_covering(self):
+    self.assert_not_covered_by(
       ['content/content.gyp', 'chrome/renderer/gpu/gpu_channel_host.h'],
       [peter], ['content/content.gyp'])
 
-  def test_NotCoveredBy_SetNoParentWorks(self):
-    self.assert_NotCoveredBy(['content/content.gyp'], [ben],
+  def test_not_covered_by__set_noparent_works(self):
+    self.assert_not_covered_by(['content/content.gyp'], [ben],
                              ['content/content.gyp'])
 
-  def assert_ReviewersFor(self, files, expected_reviewers):
+  def assert_reviewers_for(self, files, expected_reviewers):
     db = self.db()
-    self.assertEquals(db.ReviewersFor(set(files)), set(expected_reviewers))
+    self.assertEquals(db.reviewers_for(set(files)), set(expected_reviewers))
 
-  def test_ReviewersFor_BasicFunctionality(self):
-    self.assert_ReviewersFor(['chrome/gpu/gpu_channel.h'],
+  def test_reviewers_for__basic_functionality(self):
+    self.assert_reviewers_for(['chrome/gpu/gpu_channel.h'],
                              [ken, ben, brett, owners.EVERYONE])
 
-  def test_ReviewersFor_SetNoParentWorks(self):
-    self.assert_ReviewersFor(['content/content.gyp'], [john, darin])
+  def test_reviewers_for__set_noparent_works(self):
+    self.assert_reviewers_for(['content/content.gyp'], [john, darin])
 
-  def test_ReviewersFor_WildcardDir(self):
-    self.assert_ReviewersFor(['DEPS'], [owners.EVERYONE])
+  def test_reviewers_for__wildcard_dir(self):
+    self.assert_reviewers_for(['DEPS'], [owners.EVERYONE])
 
-  def assert_SyntaxError(self, owners_file_contents):
+  def assert_syntax_error(self, owners_file_contents):
     db = self.db()
     self.files['/foo/OWNERS'] = owners_file_contents
     self.files['/foo/DEPS'] = ''
-    self.assertRaises(owners.SyntaxErrorInOwnersFile, db.ReviewersFor,
+    self.assertRaises(owners.SyntaxErrorInOwnersFile, db.reviewers_for,
       ['/foo/DEPS'])
 
-  def test_SyntaxError_UnknownToken(self):
-    self.assert_SyntaxError('{}\n')
+  def test_syntax_error__unknown_token(self):
+    self.assert_syntax_error('{}\n')
 
-  def test_SyntaxError_UnknownSet(self):
-    self.assert_SyntaxError('set myfatherisbillgates\n')
+  def test_syntax_error__unknown_set(self):
+    self.assert_syntax_error('set myfatherisbillgates\n')
 
-  def test_SyntaxError_BadEmail(self):
-    self.assert_SyntaxError('ben\n')
+  def test_syntax_error__bad_email(self):
+    self.assert_syntax_error('ben\n')
 
 
 if __name__ == '__main__':