Index: tests/presubmit_unittest.py |
=================================================================== |
--- tests/presubmit_unittest.py (revision 78512) |
+++ tests/presubmit_unittest.py (working copy) |
@@ -1867,13 +1867,16 @@ |
def OwnersTest(self, is_committing, tbr=False, change_tags=None, |
suggested_reviewers=None, approvers=None, |
- uncovered_files=None, expected_reviewers=None, expected_output=''): |
+ uncovered_files=None, expected_reviewers=None, expected_output='', |
+ host_url=None): |
affected_file = self.mox.CreateMock(presubmit.SvnAffectedFile) |
affected_file.LocalPath().AndReturn('foo.cc') |
change = self.mox.CreateMock(presubmit.Change) |
change.AffectedFiles(None).AndReturn([affected_file]) |
input_api = self.MockInputApi(change, False) |
+ if host_url: |
+ input_api.host_url = host_url |
fake_db = self.mox.CreateMock(owners.Database) |
fake_db.email_regexp = input_api.re.compile(owners.BASIC_EMAIL_REGEXP) |
input_api.owners_db = fake_db |
@@ -1887,7 +1890,7 @@ |
rietveld_response = ('{"owner": "john@example.com",' |
'"messages": [' + ','.join(messages) + ']}') |
input_api.urllib2.urlopen( |
- input_api.host_url + '/api/1?messages=true').AndReturn( |
+ 'http://localhost/api/1?messages=true').AndReturn( |
StringIO.StringIO(rietveld_response)) |
input_api.json = presubmit.json |
fake_db.files_not_covered_by(set(['foo.cc']), approvers).AndReturn( |
@@ -1937,6 +1940,12 @@ |
uncovered_files=set(), |
expected_output='--tbr was specified, skipping OWNERS check\n') |
+ def testCannedCheckOwners_MissingSchemeInHostURL(self): |
+ self.OwnersTest(is_committing=True, |
+ approvers=set(['ben@example.com']), |
+ uncovered_files=set(), host_url='localhost') |
+ |
+ |
if __name__ == '__main__': |
import unittest |
unittest.main() |