From d261840dbdfe1e5bf5534baf56a507385ff6b0c3 Mon Sep 17 00:00:00 2001 From: Fu Hanxi Date: Tue, 8 Mar 2022 11:37:42 +0800 Subject: [PATCH] ci: pytest idf ci could accept "--target" as uppercase --- conftest.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/conftest.py b/conftest.py index 1bf5b55643..effd5f779d 100644 --- a/conftest.py +++ b/conftest.py @@ -22,6 +22,7 @@ from typing import Callable, List, Optional import pytest from _pytest.config import Config from _pytest.fixtures import FixtureRequest +from _pytest.main import Session from _pytest.nodes import Item from _pytest.python import Function from pytest_embedded.plugin import parse_configuration @@ -95,7 +96,7 @@ def build_dir( """ param_or_cli: str = getattr( request, 'param', None - ) or request.config.option.__dict__.get('build_dir') + ) or request.config.getoption('build_dir') if param_or_cli is not None: # respect the param and the cli return param_or_cli @@ -146,6 +147,12 @@ def pytest_addoption(parser: pytest.Parser) -> None: ) +@pytest.hookimpl(tryfirst=True) +def pytest_sessionstart(session: Session) -> None: + if session.config.option.target: + session.config.option.target = session.config.getoption('target').lower() + + @pytest.hookimpl(tryfirst=True) def pytest_collection_modifyitems(config: Config, items: List[Function]) -> None: target = config.getoption('target', None) # use the `build` dir