diff --git a/infra/cifuzz/fuzz_target.py b/infra/cifuzz/fuzz_target.py index c531f732b13d..66014ba51c71 100644 --- a/infra/cifuzz/fuzz_target.py +++ b/infra/cifuzz/fuzz_target.py @@ -161,7 +161,7 @@ def is_reproducible(self, test_case, target_path): Returns: (True, True) if crash is reproducible and we were able to run the binary. - (True, False) if we were not able to attempt reproduction. Do not rely + (False, False) if we were not able to attempt reproduction. Do not rely on the first return value in this case. (False, True) if we were able to attempt reproduction but the crash did not reproduce. @@ -255,7 +255,7 @@ def is_crash_reportable(self, test_case): if not ran_target: # This happens if the project has OSS-Fuzz builds, but the fuzz target - # doesn't. + # is not in it. logging.info(could_not_test_on_oss_fuzz_message) return True diff --git a/infra/cifuzz/fuzz_target_test.py b/infra/cifuzz/fuzz_target_test.py index 7dc03a2731f3..0147c8d6d110 100644 --- a/infra/cifuzz/fuzz_target_test.py +++ b/infra/cifuzz/fuzz_target_test.py @@ -46,8 +46,8 @@ EXECUTE_FAILURE_RETVAL = ('', '', 1) -# TODO(metzman): Use patch from clusterfuzz/src/python/tests/test_libs/ -# so that we don't need to accept this as an argument in every test method. +# TODO(metzman): Use patch from test_libs/helpers.py in clusterfuzz so that we +# don't need to accept this as an argument in every test method. @unittest.mock.patch('utils.get_container_name', return_value='container') class IsReproducibleUnitTest(fake_filesystem_unittest.TestCase): """Test is_reproducible function in the fuzz_target module."""