Reverse dependency map shouldn't be created when test_all is set

utils/tests_fetcher.py

@@ -1038,17 +1038,19 @@ def infer_tests_to_run(
     """
     if not test_all:
         modified_files = get_modified_python_files(diff_with_last_commit=diff_with_last_commit)
+        reverse_map = create_reverse_dependency_map()
+        impacted_files = modified_files.copy()
+        for f in modified_files:
+            if f in reverse_map:
+                impacted_files.extend(reverse_map[f])
     else:
-        modified_files = [str(k) for k in PATH_TO_TESTS.glob("*/*") if str(k).endswith(".py") and "test_" in str(k)]
+        impacted_files = modified_files = [
+            str(k) for k in PATH_TO_TESTS.glob("*/*") if str(k).endswith(".py") and "test_" in str(k)
+        ]
         print("\n### test_all is TRUE, FETCHING ALL FILES###\n")
     print(f"\n### MODIFIED FILES ###\n{_print_list(modified_files)}")
 
     # Create the map that will give us all impacted modules.
-    reverse_map = create_reverse_dependency_map()
-    impacted_files = modified_files.copy()
-    for f in modified_files:
-        if f in reverse_map:
-            impacted_files.extend(reverse_map[f])
 
     # Remove duplicates
     impacted_files = sorted(set(impacted_files))