From 2534ddee99a29aeb052812d0d4daa0db30199c0e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timoth=C3=A9e=20Ravier?= Date: Tue, 3 Oct 2023 13:28:20 +0000 Subject: [PATCH] Fix minor Ruff/flake8 warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ``` pungi/checks.py:575:17: F601 [*] Dictionary key literal `"type"` repeated pungi/phases/pkgset/pkgsets.py:617:12: E721 Do not compare types, use `isinstance()` tests/test_pkgset_source_koji.py:241:16: E721 Do not compare types, use `isinstance()` tests/test_pkgset_source_koji.py:244:16: E721 Do not compare types, use `isinstance()` tests/test_pkgset_source_koji.py:370:16: E721 Do not compare types, use `isinstance()` tests/test_pkgset_source_koji.py:374:20: E721 Do not compare types, use `isinstance()` ``` Signed-off-by: Timothée Ravier --- pungi/checks.py | 1 - pungi/phases/pkgset/pkgsets.py | 2 +- tests/test_pkgset_source_koji.py | 8 ++++---- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/pungi/checks.py b/pungi/checks.py index abdbf46c..88692dd6 100644 --- a/pungi/checks.py +++ b/pungi/checks.py @@ -572,7 +572,6 @@ def make_schema(): }, "required": ["kickstart"], "additionalProperties": False, - "type": "object", }, "osbs_config": { "type": "object", diff --git a/pungi/phases/pkgset/pkgsets.py b/pungi/phases/pkgset/pkgsets.py index 2d5ab3ea..3cc68f25 100644 --- a/pungi/phases/pkgset/pkgsets.py +++ b/pungi/phases/pkgset/pkgsets.py @@ -614,7 +614,7 @@ class KojiPackageSet(PackageSetBase): result_srpms = [] include_packages = set(include_packages or []) - if type(event) is dict: + if isinstance(event, dict): event = event["id"] msg = "Getting latest RPMs (tag: %s, event: %s, inherit: %s)" % ( diff --git a/tests/test_pkgset_source_koji.py b/tests/test_pkgset_source_koji.py index 6141652c..26c12854 100644 --- a/tests/test_pkgset_source_koji.py +++ b/tests/test_pkgset_source_koji.py @@ -238,10 +238,10 @@ class TestGetPackageSetFromKoji(helpers.PungiTestCase): self.compose, self.koji_wrapper, event, module_info_str ) - assert type(result) is list + assert isinstance(result, list) assert len(result) == 1 module = result[0] - assert type(module) is dict + assert isinstance(module, dict) self.assertIn("module_stream", module) self.assertIn("module_version", module) self.assertIn("module_context", module) @@ -367,11 +367,11 @@ class TestGetPackageSetFromKoji(helpers.PungiTestCase): self.compose, self.koji_wrapper, event, module_info_str ) - assert type(result) is list + assert isinstance(result, list) assert len(result) == 2 module = result[0] for module in result: - assert type(module) is dict + assert isinstance(module, dict) self.assertIn("module_stream", module) self.assertIn("module_version", module) self.assertIn("module_context", module)