diff --git a/infra/cifuzz/affected_fuzz_targets.py b/infra/cifuzz/affected_fuzz_targets.py index dcad5a507..b9468f4a6 100644 --- a/infra/cifuzz/affected_fuzz_targets.py +++ b/infra/cifuzz/affected_fuzz_targets.py @@ -51,8 +51,8 @@ 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) - if coverage_getter.fuzzer_stats_url: + coverage_getter = coverage.OssFuzzCoverageGetter(project_name, repo_path) + if not coverage_getter.fuzzer_stats_url: # Don't remove any fuzz targets unless we have data. logging.error('Could not download latest coverage report.') return diff --git a/infra/cifuzz/coverage.py b/infra/cifuzz/coverage.py index 63e189360..91113ef38 100644 --- a/infra/cifuzz/coverage.py +++ b/infra/cifuzz/coverage.py @@ -27,7 +27,7 @@ import utils LATEST_REPORT_INFO_PATH = 'oss-fuzz-coverage/latest_report_info/' -class OSSFuzzCoverageGetter: +class OssFuzzCoverageGetter: """Gets coverage data for a project from OSS-Fuzz.""" def __init__(self, project_name, repo_path): diff --git a/infra/cifuzz/coverage_test.py b/infra/cifuzz/coverage_test.py index ed7524d57..2a6248c18 100644 --- a/infra/cifuzz/coverage_test.py +++ b/infra/cifuzz/coverage_test.py @@ -17,7 +17,7 @@ from unittest import mock import coverage - +# pylint: disable=protected-access class GetFuzzerStatsDirUrlTest(unittest.TestCase): """Tests _get_fuzzer_stats_dir_url."""