From 8cda07fb9f7f840f274ece49f06ef98e449ea66b Mon Sep 17 00:00:00 2001 From: Jonathan Metzman Date: Wed, 20 Jan 2021 07:59:32 -0800 Subject: [PATCH] move over test --- infra/cifuzz/affected_fuzz_targets.py | 4 ++-- infra/cifuzz/coverage.py | 2 +- infra/cifuzz/coverage_test.py | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) 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."""