changed filter to be more conservative - any no-whitelisted file runs all tests

1 files changed, 7 insertions, 3 deletions

diff --git a/tools/run_tests/run_tests_matrix.py b/tools/run_tests/run_tests_matrix.py
index 00a2d65d3d..3cea0f5011 100755
--- a/tools/run_tests/run_tests_matrix.py
+++ b/tools/run_tests/run_tests_matrix.py
@@ -235,6 +235,10 @@ argp.add_argument('--filter_pr_tests',
 		  action='store_const',	
 		  const=True,	  
 		  help='Filters out tests irrelavant to pull request changes.')
+argp.add_argument('--base_branch',
+                  default='origin/master',
+                  type=str,
+                  help='Branch that pull request is requesting to merge into')
 args = argp.parse_args()
 
 extra_args = []
@@ -270,12 +274,12 @@ print
 
 if args.filter_pr_tests:
   print 'IMPORTANT: Test filtering is not active; this is only for testing.'
-  relevant_jobs = filter_tests(jobs)
+  relevant_jobs = filter_tests(jobs, args.base_branch)
   print
   if len(relevant_jobs) == len(jobs):
-    print 'No tests were filtered.'
+    print '(TESTING) No tests will be skipped.'
   else:
-    print 'These tests were filtered:'
+    print '(TESTING) These tests will be skipped:'
     for job in list(set(jobs) - set(relevant_jobs)):
       print '  %s' % job.shortname
   print