Merge pull request #344 from opsmill/fac-fix-testcontainer-ver

fix(testing): remove version fixture

Author: dgarros

.github/workflows/ci.yml

@@ -288,9 +288,6 @@ jobs:
           pip install invoke toml codecov
       - name: "Install Package"
         run: "poetry install --all-extras"
-      - name: "Set environment variables for python_testcontainers"
-        run: |
-          echo INFRAHUB_TESTING_IMAGE_VER=latest >> $GITHUB_ENV
       - name: "Integration Tests"
         run: |
           poetry run pytest --cov infrahub_sdk tests/integration/

infrahub_sdk/testing/docker.py

@@ -8,13 +8,16 @@
 
 from .. import Config, InfrahubClient, InfrahubClientSync
 
-INFRAHUB_VERSION = os.getenv("INFRAHUB_TESTING_IMAGE_VER", "latest")
+INFRAHUB_VERSION = os.getenv("INFRAHUB_TESTING_IMAGE_VER")
 
 
 def skip_version(min_infrahub_version: str | None = None, max_infrahub_version: str | None = None) -> bool:
     """
     Check if a test should be skipped depending on infrahub version.
     """
+    if INFRAHUB_VERSION is None:
+        return True
+
     try:
         version = Version(INFRAHUB_VERSION)
     except InvalidVersion:
@@ -31,10 +34,6 @@ def skip_version(min_infrahub_version: str | None = None, max_infrahub_version:
 
 
 class TestInfrahubDockerClient(TestInfrahubDocker):
-    @pytest.fixture(scope="class")
-    def infrahub_version(self) -> str:
-        return INFRAHUB_VERSION
-
     @pytest.fixture(scope="class")
     def client(self, infrahub_port: int) -> InfrahubClient:
         return InfrahubClient(

poetry.lock

@@ -64,8 +64,7 @@ pytest-xdist = "^3.3.1"
 types-python-slugify = "^8.0.0.3"
 invoke = "^2.2.0"
 towncrier = "^24.8.0"
-infrahub-testcontainers = "~1.1.9"
-
+infrahub-testcontainers = { version = "^1.2.5", python = ">=3.10" }
 astroid = "~3.1"
 
 [tool.poetry.extras]