-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Enable python tests during coverage #37421
Conversation
Changed Files
|
PR #37421: Size comparison from a998bf6 to 07750fc Full report (14 builds for cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, tizen)
|
07750fc
to
8014527
Compare
PR #37421: Size comparison from a998bf6 to 8014527 Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
8014527
to
0c71000
Compare
PR #37421: Size comparison from 7adac6e to 0c71000 Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
@@ -172,8 +172,13 @@ if [ "$skip_gn" == false ]; then | |||
# 3) Run Python tests if requested | |||
# | |||
if [ "$ENABLE_PYTHON" == true ]; then | |||
echo "Building python and applications ..." | |||
scripts/run_in_build_env.sh \ | |||
"./scripts/tests/local.py build" |
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.
does this build with coverage?
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.
Yes, it is triggered by './scripts/tests/local.py build'
Add Python Integration Tests into Test Coverage
Fixes #37220
Testing