diff options
author | 2021-01-20 07:26:02 -0800 | |
---|---|---|
committer | 2021-01-20 07:26:02 -0800 | |
commit | 9935321c0dbee83f0d505e23698b34871df84c07 (patch) | |
tree | 036cef40683caf7d551201701173c2bbb86d4c1e /infra/cifuzz/affected_fuzz_targets.py | |
parent | ddb0add036e34a85d6ab4a711d7c29d7fe15ac19 (diff) |
fmt/lnt
Diffstat (limited to 'infra/cifuzz/affected_fuzz_targets.py')
-rw-r--r-- | infra/cifuzz/affected_fuzz_targets.py | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/infra/cifuzz/affected_fuzz_targets.py b/infra/cifuzz/affected_fuzz_targets.py index b5f51ad6..dcad5a50 100644 --- a/infra/cifuzz/affected_fuzz_targets.py +++ b/infra/cifuzz/affected_fuzz_targets.py @@ -19,6 +19,7 @@ import sys import coverage +# pylint: disable=wrong-import-position,import-error sys.path.append(os.path.dirname(os.path.dirname(os.path.abspath(__file__)))) import utils @@ -39,7 +40,7 @@ def remove_unaffected_fuzz_targets(project_name, out_dir, files_changed, """ if not files_changed: # Don't remove any fuzz targets if there is no difference from HEAD. - logging.info('No files changed.') + logging.info('No files changed compared to HEAD.') return logging.info('Files changed in PR: %s', files_changed) @@ -50,15 +51,15 @@ def remove_unaffected_fuzz_targets(project_name, out_dir, files_changed, logging.error('No fuzz targets found in out dir.') return - coverage_getter = coverage.OSSFuzzCoveragGetter(project_name, - repo_path) + coverage_getter = coverage.OSSFuzzCoveragGetter(project_name, repo_path) if coverage_getter.fuzzer_stats_url: # Don't remove any fuzz targets unless we have data. logging.error('Could not download latest coverage report.') return - affected_fuzz_targets = get_affected_fuzz_targets( - coverage_getter, fuzz_target_paths, files_changed) + affected_fuzz_targets = get_affected_fuzz_targets(coverage_getter, + fuzz_target_paths, + files_changed) if not affected_fuzz_targets: logging.info('No affected fuzz targets detected, keeping all as fallback.') @@ -73,8 +74,8 @@ def remove_unaffected_fuzz_targets(project_name, out_dir, files_changed, try: os.remove(fuzz_target_path) except OSError as error: - logging.error('%s occurred while removing file %s', - error, fuzz_target_path) + logging.error('%s occurred while removing file %s', error, + fuzz_target_path) def is_fuzz_target_affected(coverage_getter, fuzz_target_path, files_changed): @@ -89,8 +90,7 @@ def is_fuzz_target_affected(coverage_getter, fuzz_target_path, files_changed): fuzz_target) return True - logging.info('Fuzz target %s is affected by: %s', - fuzz_target, covered_files) + logging.info('Fuzz target %s is affected by: %s', fuzz_target, covered_files) for filename in files_changed: if filename in covered_files: logging.info('Fuzz target %s is affected by changed file: %s', @@ -101,13 +101,13 @@ def is_fuzz_target_affected(coverage_getter, fuzz_target_path, files_changed): return False -def get_affected_fuzz_targets( - coverage_getter, fuzz_target_paths, files_changed): +def get_affected_fuzz_targets(coverage_getter, fuzz_target_paths, + files_changed): """Returns a list of paths of affected targets.""" affected_fuzz_targets = set() for fuzz_target_path in fuzz_target_paths: - if is_fuzz_target_affected( - coverage_getter, fuzz_target_path, files_changed): + if is_fuzz_target_affected(coverage_getter, fuzz_target_path, + files_changed): affected_fuzz_targets.add(fuzz_target_path) return affected_fuzz_targets |