-
Notifications
You must be signed in to change notification settings - Fork 9
Run test py #235
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run test py #235
Conversation
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
|
Code size report: |
Signed-off-by: jaenrig-ifx <enriquezgarcia.external@infineon.com>
d9923b8 to
ebef6a8
Compare
|
@ramya-subramanyam once merge here. In psoc edge I need to check if the get_devs.py script works. If that is the case, we can reuse this directly. |
get_devs.py is working already so we can reuse it. |
ramya-subramanyam
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very good! The logs are much clearer.
Yes we can next bring it to psoc-edge from common repo
| This class takes care of running the different MicroPython test types. | ||
| It supports the following: | ||
| - single: single device tests | ||
| - single_post_delay: single device tests with a delay between tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which is this type?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed, i will add a README in the newly created repo with this basic info.
| - board: CY8CKIT-062S2-AI | ||
|
|
||
| - name: vfs-flash-large | ||
| type: custom |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type field is not mandatory for each test? If not then how would the script know what to run?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed, only custom and multi need to be explicitly added. Also will be documented.
|
|
||
| ``` | ||
| ./run_psoc6_tests.sh --test-suite ci-tests --board CY8CPROTO-062-4343W --hil ifx-mpy-hil | ||
| python tools/psoc6/run_test_plan.py wifi --max-retries 1 --hil-devs local-devs.yml --board CY8CKIT-062S2-AI |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So you can specifically have retries defined for specific tests.. Nice!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The retries will apply to all the listed tests (all if no list specified as positional argument after the run_test_plan.py).
run_test_plan.pysupporting the split of test info as yml file and the different HIL tests enabled for PSOC6.test-plan.yml