Default `--ll` to `None` in test harness
Only override `tractor.log._default_loglevel` when the flag is explicitly passed — lets per-spawn and per-example `loglevel` kwargs take effect instead of being clobbered by the hard-coded `'ERROR'` default. (this commit msg was generated in some part by [`claude-code`][claude-code-gh]) [claude-code-gh]: https://github.com/anthropics/claude-codesubint_forkserver_backend
parent
9431a81d37
commit
72a0465c52
|
|
@ -135,25 +135,30 @@ def pytest_addoption(
|
||||||
"--ll",
|
"--ll",
|
||||||
action="store",
|
action="store",
|
||||||
dest='loglevel',
|
dest='loglevel',
|
||||||
default='ERROR', help="logging level to set when testing"
|
default=None,
|
||||||
|
help="logging level to set when testing",
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
@pytest.fixture(scope='session', autouse=True)
|
@pytest.fixture(scope='session', autouse=True)
|
||||||
def loglevel(
|
def loglevel(
|
||||||
request: pytest.FixtureRequest,
|
request: pytest.FixtureRequest,
|
||||||
) -> str:
|
) -> str|None:
|
||||||
import tractor
|
import tractor
|
||||||
orig = tractor.log._default_loglevel
|
orig = tractor.log._default_loglevel
|
||||||
level = tractor.log._default_loglevel = request.config.option.loglevel
|
flag_level: str|None = request.config.option.loglevel
|
||||||
|
|
||||||
|
if flag_level is not None:
|
||||||
|
tractor.log._default_loglevel = flag_level
|
||||||
|
|
||||||
log = tractor.log.get_console_log(
|
log = tractor.log.get_console_log(
|
||||||
level=level,
|
level=flag_level,
|
||||||
name='tractor', # <- enable root logger
|
name='tractor', # <- enable root logger
|
||||||
)
|
)
|
||||||
log.info(
|
log.info(
|
||||||
f'Test-harness set runtime loglevel: {level!r}\n'
|
f'Test-harness set runtime loglevel: {flag_level!r}\n'
|
||||||
)
|
)
|
||||||
yield level
|
yield flag_level
|
||||||
tractor.log._default_loglevel = orig
|
tractor.log._default_loglevel = orig
|
||||||
|
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue