sts-sponsors team mailing list archive

[Alberto Donato <mp+443673@xxxxxxxxxxxxxxxxxx>]

Alberto Donato has proposed merging ~ack/maas-ci/+git/system-tests:image-to-test-fixture-fix into ~maas-committers/maas-ci/+git/system-tests:master.

Commit message:
fix image_to_test fixture when no image is provided



Requested reviews:
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~ack/maas-ci/+git/system-tests/+merge/443673
-- 
Your team MAAS Committers is requested to review the proposed merge of ~ack/maas-ci/+git/system-tests:image-to-test-fixture-fix into ~maas-committers/maas-ci/+git/system-tests:master.

diff --git a/systemtests/conftest.py b/systemtests/conftest.py
index 0b9fac9..b8d4799 100644
--- a/systemtests/conftest.py
+++ b/systemtests/conftest.py
@@ -288,8 +288,8 @@ def hardware_sync_machine(
 
 
 @pytest.fixture(scope="module")
-def image_to_test(request: pytest.FixtureRequest) -> Iterator[TestableImage]:
-    image: TestableImage = request.param
+def image_to_test(request: pytest.FixtureRequest) -> Iterator[TestableImage | None]:
+    image: TestableImage | None = getattr(request, "param", None)
     yield image