From 3e30d1a1d4d5be01629e0645e05bae19743b9d38 Mon Sep 17 00:00:00 2001 From: gnaponie Date: Jul 12 2019 08:02:19 +0000 Subject: Fix pylint Signed-off-by: gnaponie --- diff --git a/greenwave/policies.py b/greenwave/policies.py index e1ee597..b91237e 100644 --- a/greenwave/policies.py +++ b/greenwave/policies.py @@ -493,7 +493,6 @@ class PassingTestCaseRule(Rule): visited_arch_variants.add(arch_variant) answer = self._answer_for_result( result, - product_version, policy.subject_type, subject_identifier) answers.append(answer) @@ -507,7 +506,6 @@ class PassingTestCaseRule(Rule): for result in matching_results: answers.append(self._answer_for_result( result, - product_version, policy.subject_type, subject_identifier)) return answers @@ -524,7 +522,7 @@ class PassingTestCaseRule(Rule): } def _answer_for_result( - self, result, product_version, subject_type, subject_identifier): + self, result, subject_type, subject_identifier): if result['outcome'] in ('PASSED', 'INFO'): log.debug('Test result passed for the result_id %s and testcase %s,' ' because the outcome is %s', result['id'], self.test_case_name, diff --git a/greenwave/request_session.py b/greenwave/request_session.py index aad59e1..1003581 100644 --- a/greenwave/request_session.py +++ b/greenwave/request_session.py @@ -1,6 +1,7 @@ import requests from requests.adapters import HTTPAdapter +# pylint: disable=E0401 from requests.packages.urllib3.util.retry import Retry from greenwave import __version__ diff --git a/greenwave/resources.py b/greenwave/resources.py index 47f349b..0aaa3ac 100644 --- a/greenwave/resources.py +++ b/greenwave/resources.py @@ -196,7 +196,7 @@ def _retrieve_yaml_remote_rule_git_archive(rev, pkg_name, pkg_namespace): cmd = ['git', 'archive', f'--remote={dist_git_url}', rev, 'gating.yaml'] # Retry thrice if TimeoutExpired exception is raised MAX_RETRY = 3 - for tries in range(MAX_RETRY): + for _ in range(MAX_RETRY): try: git_archive = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE) output, error_output = git_archive.communicate(timeout=30) diff --git a/greenwave/tests/test_api_v1.py b/greenwave/tests/test_api_v1.py index e29d7d0..59ef253 100644 --- a/greenwave/tests/test_api_v1.py +++ b/greenwave/tests/test_api_v1.py @@ -63,6 +63,8 @@ def make_decision(**kwargs): def test_make_decision_retrieves_waivers_on_missing(mock_results, mock_waivers): + mock_results.return_value = [] + mock_waivers.return_value = [] response = make_decision() assert 200 == response.status_code assert '1 of 1 required test results missing' == response.json['summary'] @@ -71,6 +73,7 @@ def test_make_decision_retrieves_waivers_on_missing(mock_results, mock_waivers): def test_make_decision_retrieves_waivers_on_failed(mock_results, mock_waivers): mock_results.return_value = [make_result(outcome='FAILED')] + mock_waivers.return_value = [] response = make_decision() assert 200 == response.status_code assert '1 of 1 required tests failed' == response.json['summary'] @@ -79,6 +82,7 @@ def test_make_decision_retrieves_waivers_on_failed(mock_results, mock_waivers): def test_make_decision_retrieves_waivers_omitted_on_passed(mock_results, mock_waivers): mock_results.return_value = [make_result(outcome='PASSED')] + mock_waivers.return_value = [] response = make_decision() assert 200 == response.status_code assert 'All required tests passed' == response.json['summary'] @@ -86,6 +90,8 @@ def test_make_decision_retrieves_waivers_omitted_on_passed(mock_results, mock_wa def test_make_decision_retrieves_waivers_once_on_verbose_and_missing(mock_results, mock_waivers): + mock_results.return_value = [] + mock_waivers.return_value = [] response = make_decision(verbose=True) assert 200 == response.status_code assert '1 of 1 required test results missing' == response.json['summary'] diff --git a/greenwave/tests/test_resultsdb_consumer.py b/greenwave/tests/test_resultsdb_consumer.py index e66c235..227029e 100644 --- a/greenwave/tests/test_resultsdb_consumer.py +++ b/greenwave/tests/test_resultsdb_consumer.py @@ -331,6 +331,7 @@ def test_guess_product_version(): def test_guess_product_version_with_koji(): + # pylint: disable=W0212,W0612 class DummyKojiProxy(): def getBuild(self, subject_identifier): assert 'fake_koji_build' == subject_identifier @@ -353,6 +354,7 @@ def test_guess_product_version_with_koji(): 'badnvr-1.2.f30', )) def test_guess_product_version_failure(nvr): + # pylint: disable=W0212 product_version = greenwave.consumers.resultsdb._subject_product_version(nvr, 'koji_build') assert product_version is None diff --git a/greenwave/tests/test_waivers_retriever.py b/greenwave/tests/test_waivers_retriever.py index 4b201cb..6cfa3d9 100644 --- a/greenwave/tests/test_waivers_retriever.py +++ b/greenwave/tests/test_waivers_retriever.py @@ -16,6 +16,7 @@ _DUMMY_FILTERS = ['dummy_filter'] def test_waivers_retriever_retrieves_not_ignored_ids(): + # pylint: disable=W0212 retriever = WaiversRetriever(**_DUMMY_RETRIEVER_ARGUMENTS) retriever.ignore_ids = [100] waiver = dict( @@ -32,6 +33,7 @@ def test_waivers_retriever_retrieves_not_ignored_ids(): def test_waivers_retriever_ignores_ids(): + # pylint: disable=W0212 retriever = WaiversRetriever(**_DUMMY_RETRIEVER_ARGUMENTS) retriever.ignore_ids = [99] waiver = dict( @@ -48,6 +50,7 @@ def test_waivers_retriever_ignores_ids(): def test_waivers_retriever_ignores_no_waived(): + # pylint: disable=W0212 retriever = WaiversRetriever(**_DUMMY_RETRIEVER_ARGUMENTS) waiver = dict( id=99, diff --git a/pylintrc b/pylintrc index bfe20a0..5e4da35 100644 --- a/pylintrc +++ b/pylintrc @@ -1,2 +1,2 @@ [MESSAGES CONTROL] -disable=R,C,no-member,fixme,W0622 +disable=R,C,no-member,fixme,W0622,W0621