Skip to content

Commit 5885155

Browse files
Fixed coordinator filtering based uppon test regexp (#254)
1 parent c4ae6fe commit 5885155

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

redis_benchmarks_specification/__self_contained_coordinator__/self_contained_coordinator.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -530,11 +530,11 @@ def process_self_contained_coordinator_stream(
530530
fn, github_token, pull_request, "redis", "redis", verbose
531531
)
532532

533-
test_regexp = ".*"
534-
if b"test_regexp" in testDetails:
535-
test_regexp = testDetails[b"test_regexp"]
533+
tests_regexp = ".*"
534+
if b"tests_regexp" in testDetails:
535+
tests_regexp = testDetails[b"tests_regexp"]
536536
logging.info(
537-
f"detected a regexp definition on the streamdata {test_regexp}"
537+
f"detected a regexp definition on the streamdata {tests_regexp}"
538538
)
539539

540540
command_groups_regexp = None
@@ -596,7 +596,7 @@ def process_self_contained_coordinator_stream(
596596
defaults_filename,
597597
priority_lower_limit,
598598
priority_upper_limit,
599-
test_regexp,
599+
tests_regexp,
600600
testsuite_spec_files,
601601
command_groups_regexp,
602602
)
@@ -1368,7 +1368,7 @@ def filter_test_files(
13681368
defaults_filename,
13691369
priority_lower_limit,
13701370
priority_upper_limit,
1371-
test_regexp,
1371+
tests_regexp,
13721372
testsuite_spec_files,
13731373
command_groups_regexp=None,
13741374
):
@@ -1377,17 +1377,17 @@ def filter_test_files(
13771377
if defaults_filename in test_file:
13781378
continue
13791379

1380-
if test_regexp != ".*":
1380+
if tests_regexp != ".*":
13811381
logging.info(
1382-
"Filtering all tests via a regular expression: {}".format(test_regexp)
1382+
"Filtering all tests via a regular expression: {}".format(tests_regexp)
13831383
)
1384-
tags_regex_string = re.compile(test_regexp)
1384+
tags_regex_string = re.compile(tests_regexp)
13851385

13861386
match_obj = re.search(tags_regex_string, test_file)
13871387
if match_obj is None:
13881388
logging.info(
13891389
"Skipping {} given it does not match regex {}".format(
1390-
test_file, test_regexp
1390+
test_file, tests_regexp
13911391
)
13921392
)
13931393
continue

0 commit comments

Comments
 (0)