Re: [PATCHv5 6/6] test: py: add initial coverage for scp03 cmd

13 Feb 2021


      Hi Jorge,
On Tue, 9 Feb 2021 at 13:10, Jorge Ramirez-Ortiz jorge@foundries.io wrote:
...
From: Igor Opaniuk igor.opaniuk@foundries.io
Add initial test coverage for SCP03 command.
Signed-off-by: Igor Opaniuk igor.opaniuk@foundries.io
test/py/tests/test_scp03.py | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)
 create mode 100644 test/py/tests/test_scp03.py
Reviewed-by: Simon Glass sjg@chromium.org
This is fine, but note you can also write this test in C for sandbox,
and this might be easier if it gets more complicated. For example you
can check the state of devices from C.
Regards,
Simon