Skip to content

Clear out test restrictions for Python 3.9 #649

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 19, 2025
Merged

Clear out test restrictions for Python 3.9 #649

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions tests/unit/ctl/test_render_app.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import json
import os
import sys
from dataclasses import dataclass
from pathlib import Path

Expand All @@ -17,8 +16,6 @@

FIXTURE_BASE_DIR = Path(Path(os.path.abspath(__file__)).parent / ".." / ".." / "fixtures" / "repos")

requires_python_310 = pytest.mark.skipif(sys.version_info < (3, 10), reason="Requires Python 3.10 or higher")


@dataclass
class RenderAppFailure:
Expand Down Expand Up @@ -55,7 +52,6 @@ class RenderAppFailure:
"test_case",
[pytest.param(tc, id=tc.name) for tc in RENDER_APP_FAIL_TEST_CASES],
)
@requires_python_310
def test_validate_template_not_found(test_case: RenderAppFailure, httpx_mock: HTTPXMock) -> None:
"""Ensure that the correct errors are caught"""
httpx_mock.add_response(
Expand Down Expand Up @@ -83,7 +79,6 @@ def test_validate_template_not_found(test_case: RenderAppFailure, httpx_mock: HT
(None, None, True, "git-branch"),
],
)
@requires_python_310
def test_render_branch_selection(monkeypatch, httpx_mock: HTTPXMock, cli_branch, env_branch, from_git, expected_branch):
"""Test that the render command uses the correct branch source."""

Expand Down
8 changes: 0 additions & 8 deletions tests/unit/ctl/test_transform_app.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
import json
import os
import shutil
import sys
import tempfile
from pathlib import Path

Expand All @@ -23,8 +22,6 @@
Path(os.path.abspath(__file__)).parent / ".." / ".." / "fixtures" / "integration" / "test_infrahubctl"
)

requires_python_310 = pytest.mark.skipif(sys.version_info < (3, 10), reason="Requires Python 3.10 or higher")


@pytest.fixture
def tags_transform_dir():
Expand All @@ -51,7 +48,6 @@ class TestInfrahubctlTransform:
"""Groups the 'infrahubctl transform' test cases."""

@staticmethod
@requires_python_310
def test_transform_not_exist_in_infrahub_yml(tags_transform_dir: str) -> None:
"""Case transform is not specified in the infrahub.yml file."""
transform_name = "not_existing_transform"
Expand All @@ -61,7 +57,6 @@ def test_transform_not_exist_in_infrahub_yml(tags_transform_dir: str) -> None:
assert output.exit_code == 1

@staticmethod
@requires_python_310
def test_transform_python_file_not_defined(tags_transform_dir: str) -> None:
"""Case transform python file not defined."""
# Remove transform file
Expand All @@ -76,7 +71,6 @@ def test_transform_python_file_not_defined(tags_transform_dir: str) -> None:
assert output.exit_code == 1

@staticmethod
@requires_python_310
def test_transform_python_class_not_defined(tags_transform_dir: str) -> None:
"""Case transform python class not defined."""
# Rename transform inside of python file so the class name searched for no longer exists
Expand All @@ -96,7 +90,6 @@ def test_transform_python_class_not_defined(tags_transform_dir: str) -> None:
assert output.exit_code == 1

@staticmethod
@requires_python_310
def test_gql_query_not_defined(tags_transform_dir: str) -> None:
"""Case GraphQL Query is not defined"""
# Remove GraphQL Query file
Expand All @@ -110,7 +103,6 @@ def test_gql_query_not_defined(tags_transform_dir: str) -> None:
assert output.exit_code == 1

@staticmethod
@requires_python_310
def test_infrahubctl_transform_cmd_success(httpx_mock: HTTPXMock, tags_transform_dir: str) -> None:
"""Case infrahubctl transform command executes successfully"""
httpx_mock.add_response(
Expand Down