add owners_file() convenience constructor function for testing, make user names a bit more generic

Review URL: http://codereview.chromium.org/6632014

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@77178 0039d316-1c4b-4281-b951-d872f2087c98

tests/owners_unittest.py

@@ -10,52 +10,41 @@ import unittest
 import owners
 from tests import filesystem_mock
 
+ben = 'ben@example.com'
+brett = 'brett@example.com'
+darin = 'darin@example.com'
+john = 'john@example.com'
+ken = 'ken@example.com'
+peter = 'peter@example.com'
+
+def owners_file(*email_addresses, **kwargs):
+  s = ''
+  if kwargs.get('noparent'):
+    s = 'set noparent\n'
+  return s + '\n'.join(email_addresses) + '\n'
+
 
 def test_repo():
   return filesystem_mock.MockFileSystem(files={
     '/DEPS' : '',
-    '/OWNERS':
+    '/OWNERS': owners_file('*'),
-      ('# OWNERS'
+    '/base/vlog.h': '',
-       '*\n'),
+    '/chrome/OWNERS': owners_file(ben, brett),
-    '/base/vlog.h':
+    '/chrome/gpu/OWNERS': owners_file(ken),
-      '// vlog.h\n',
+    '/chrome/gpu/gpu_channel.h': '',
-    '/chrome/OWNERS':
+    '/chrome/renderer/OWNERS': owners_file(peter),
-      ('ben@example.com\n'
+    '/chrome/renderer/gpu/gpu_channel_host.h': '',
-       'brettw@example.com\n'),
+    '/chrome/renderer/safe_browsing/scorer.h': '',
-    '/chrome/gpu/OWNERS':
+    '/content/OWNERS': owners_file(john, darin, noparent=True),
-      ('kbr@example.com\n'),
+    '/content/content.gyp': '',
-    '/chrome/gpu/gpu_channel.h':
-      '// gpu_channel.h\n',
-    '/chrome/renderer/OWNERS':
-      ('pkasting@example.com\n'),
-    '/chrome/renderer/gpu/gpu_channel_host.h':
-      '// gpu_channel_host.h\n',
-    '/chrome/renderer/safe_browsing/scorer.h':
-      '// scorer.h\n',
-    '/content/OWNERS':
-      ('set noparent\n'
-       'jam@example.com\n'
-       'darin@example.com\n'),
-    '/content/content.gyp':
-      '# content.gyp\n',
   })
 
 
-ben = 'ben@example.com'
-brett = 'brettw@example.com'
-darin = 'darin@example.com'
-jam = 'jam@example.com'
-kbr = 'kbr@example.com'
-pkasting = 'pkasting@example.com'
-
-
 class OwnersDatabaseTest(unittest.TestCase):
   def setUp(self):
     self.repo = test_repo()
     self.files = self.repo.files
     self.root = '/'
-
-    # pylint: disable=W0108
     self.fopen = self.repo.open_for_reading
 
   def db(self, root=None, fopen=None, os_path=None):
@@ -82,14 +71,14 @@ class OwnersDatabaseTest(unittest.TestCase):
 
   def test_owners_for(self):
     self.assertReviewersFor(['DEPS'], [owners.ANYONE])
-    self.assertReviewersFor(['content/content.gyp'], [jam, darin])
+    self.assertReviewersFor(['content/content.gyp'], [john, darin])
-    self.assertReviewersFor(['chrome/gpu/gpu_channel.h'], [kbr])
+    self.assertReviewersFor(['chrome/gpu/gpu_channel.h'], [ken])
 
   def test_covered_by(self):
-    self.assertCoveredBy(['DEPS'], [jam])
+    self.assertCoveredBy(['DEPS'], [john])
     self.assertCoveredBy(['DEPS'], [darin])
-    self.assertCoveredBy(['content/content.gyp'], [jam])
+    self.assertCoveredBy(['content/content.gyp'], [john])
-    self.assertCoveredBy(['chrome/gpu/OWNERS'], [kbr])
+    self.assertCoveredBy(['chrome/gpu/OWNERS'], [ken])
     self.assertCoveredBy(['chrome/gpu/OWNERS'], [ben])
 
   def test_not_covered_by(self):
@@ -98,7 +87,7 @@ class OwnersDatabaseTest(unittest.TestCase):
       ['content/content.gyp'])
     self.assertNotCoveredBy(
       ['chrome/gpu/gpu_channel.h', 'chrome/renderer/gpu/gpu_channel_host.h'],
-      [pkasting], ['chrome/gpu/gpu_channel.h'])
+      [peter], ['chrome/gpu/gpu_channel.h'])
     self.assertNotCoveredBy(
       ['chrome/gpu/gpu_channel.h', 'chrome/renderer/gpu/gpu_channel_host.h'],
       [ben], [])
@@ -123,11 +112,10 @@ class OwnersDatabaseTest(unittest.TestCase):
       '/bar/OWNERS', '/bar/DEPS')
 
   def test_owners_propagates_down(self):
-    self.assertCoveredBy(['/chrome/renderer/gpu/gpu_channel_host.h'],
+    self.assertCoveredBy(['/chrome/renderer/gpu/gpu_channel_host.h'], [peter])
-      [pkasting])
 
   def test_set_noparent(self):
-    self.assertNotCoveredBy(['/content/content.gyp'], [pkasting],
+    self.assertNotCoveredBy(['/content/content.gyp'], [peter],
       ['/content/content.gyp'])