diff --git a/presubmit_canned_checks.py b/presubmit_canned_checks.py index 37c5143be5..9cd84d6e63 100644 --- a/presubmit_canned_checks.py +++ b/presubmit_canned_checks.py @@ -873,7 +873,7 @@ def CheckOwners(input_api, output_api, source_file_filter=None): output_fn = output_api.PresubmitNotifyResult affected_files = set([f.LocalPath() for f in - input_api.AffectedFiles(file_filter=source_file_filter)]) + input_api.change.AffectedFiles(file_filter=source_file_filter)]) owners_db = input_api.owners_db owner_email, reviewers = GetCodereviewOwnerAndReviewers( @@ -1091,7 +1091,7 @@ def PanProjectChecks(input_api, output_api, if owners_check: snapshot("checking owners") results.extend(input_api.canned_checks.CheckOwners( - input_api, output_api, source_file_filter=sources)) + input_api, output_api, source_file_filter=None)) snapshot("checking long lines") results.extend(input_api.canned_checks.CheckLongLines( diff --git a/tests/presubmit_unittest.py b/tests/presubmit_unittest.py index 2c036e27e4..78319e609d 100755 --- a/tests/presubmit_unittest.py +++ b/tests/presubmit_unittest.py @@ -2195,7 +2195,7 @@ class CannedChecksUnittest(PresubmitTestsBase): if not is_committing or (not tbr and issue): affected_file.LocalPath().AndReturn('foo/xyz.cc') - input_api.AffectedFiles(file_filter=None).AndReturn([affected_file]) + change.AffectedFiles(file_filter=None).AndReturn([affected_file]) if issue and not rietveld_response and not gerrit_response: rietveld_response = { "owner_email": change.author_email,