google / google/openhtf

By default, the OpenHTF base logger propagates until initialize_record_logger is called

Open
#787 0 comments 0 reactions 1 assignee Claimed by @Kenadia View on GitHub
bug
Dominant language
Python
Stars
722
Forks
237
Avg merge
11h 31m
Merged PRs (30d)
4

Description

The problem is that `configure_cli_logging` does not configure the `openhtf` logger if `CLI_LOGGING_VERBOSITY` is `0`. The `openhtf` logger will be configured later once `initialize_record_logger` is called.

This can cause stray logs to appear if an OpenHTF user adds handlers to the root logger before the first `TestState` is created.

**Example:**

Add:

```python
import logging
logging.basicConfig(level=logging.DEBUG)
```

...to hello_world.py at the top of the main block. Then the following log line appears at the start of the test run:

> INFO:openhtf.core.test_descriptor:Executing test:

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.