diff --git a/tests/conftest.py b/tests/conftest.py index 1f92a83..4299d97 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -87,6 +87,20 @@ def get_required_futs() -> list[EnumFut]: assert EnumFut(fut), fut if "mcu" in metafunc.fixturenames: + + def warning( + firmware_spec: FirmwareSpecBase | None, + futs: list[EnumFut], + ) -> None: + msg = "No tentacle where selected" + if firmware_spec is not None: + msg += f" for testing firmware '{firmware_spec.board_variant}'" + msg += "." + if len(futs) > 0: + futs_text = ", ".join(f.name for f in _required_futs) + msg += f" Required futs: {futs_text}." + logger.warning(msg) + list_tentacles: list[TentacleShowcase] = [] for firmware_spec in get_firmware_specs( config=metafunc.config, @@ -99,9 +113,7 @@ def get_required_futs() -> list[EnumFut]: ) tentacles = list(filter(firmware_spec.match_board, tentacles)) if len(tentacles) == 0: - futs_text = ", ".join(f.name for f in _required_futs) - msg = f"No tentacle where selected for testing firmware '{firmware_spec.board_variant}'. Required futs: {futs_text}" - logger.warning(msg) + warning(firmware_spec=firmware_spec, futs=_required_futs) for tentacle in tentacles: # TODO: This might be not required anymore!!! # Need to create a copy to the tentacle as we @@ -111,9 +123,7 @@ def get_required_futs() -> list[EnumFut]: list_tentacles.append(tentacle) if len(list_tentacles) == 0: - msg = f"No tentacle where selected for testing firmware '{firmware_spec.board_variant}'." - # raise ValueError(msg) - logger.warning(msg) + warning(firmware_spec=firmware_spec, futs=[]) return # print(f"LEN={len(list_tentacles)}") metafunc.parametrize("mcu", list_tentacles, ids=lambda t: t.pytest_id)