Mar 10, 2026 · Mar 10, 2026 · Mar 10, 2026 · Mar 10, 2026
diff --git a/Lib/test/test_tools/test_compute_changes.py b/Lib/test/test_tools/test_compute_changes.py
 elifp.is_file():
 f=p
 else:
 self.fail(f"LIBRARY_FUZZER_PATHS contains an invalid entry:{p!r}")
 continue
 result=process_changed_files({f})
 self.assertTrue(result.run_ci_fuzz_stdlib)
 self.assertTrue(is_fuzzable_library_file(f))
Original file line number	Diff line number	Diff line change
Expand Up		@@ -55,7 +55,7 @@ def test_ci_fuzz_stdlib(self):
		elifp.is_file():
		f=p
		else:
		self.fail(f"LIBRARY_FUZZER_PATHS contains an invalid entry:{p!r}")
		continue
Copy link Member webknjazMar 10, 2026 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. @StanFromIreland do we still want to log this, though? Or perhaps have a mechanism for skipping checks / checking against different sets of files in certain envs? Copy link Member StanFromIrelandMar 10, 2026• edited Loading Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Or perhaps have a mechanism for skipping checks / checking against different sets of files in certain envs? As I noted in the issue, this will greatly complicate the test (and increase the maintenance burden, who knows what envs people use), for something that is not particularly important. As such, I think skipping is fine, logging them too, however, sounds good. chris-eibl reacted with thumbs up emoji Copy link MemberAuthor chris-eiblMar 10, 2026 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. "Simple prints" will get lost in the noise of the output, and be false-positives in the OPs case. I don't think print(f"Warning: LIBRARY_FUZZER_PATHS might contain an invalid entry:{p!r}") will do any good. So I think this is fine to go. It was the previous behaviour, anyway. I just thought it is a low hanging fruit to crop when the else was removed in#145232 and we had spurious failures ...
		result=process_changed_files({f})
		self.assertTrue(result.run_ci_fuzz_stdlib)
		self.assertTrue(is_fuzzable_library_file(f))
Expand Down