target: tests: Address review comments on PR#667

ARM-software / devlib

Library for interaction with and instrumentation of remote devices.

Apache License 2.0

45 stars 77 forks source link

Closed metin-arm closed 1 month ago

metin-arm commented 3 months ago

PR#667: https://github.com/ARM-software/devlib/pull/667

Implement separate initialize / tear down methods and make various cleanups in test/test_target.py
Make docstrings Sphinx compatible
Make TargetRunner and its subclasses private
Cleanup tests/test_target.py
Replace print()'s with appropriate logging calls
Implement NOPTargetRunner class for simplifying tests
Improve Python v3.7 compatibility
Relax host machine type checking
Escape user input strings

and more..

metin-arm commented 2 months ago

Only updated tools/docker/target_configs.yml in the last push (384c2aefa47).

douglas-raillard-arm commented 2 months ago

LGTM