Skip to content
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

Target testing strategy (IEC-1) #16

Open
tore-espressif opened this issue Mar 3, 2022 · 1 comment
Open

Target testing strategy (IEC-1) #16

tore-espressif opened this issue Mar 3, 2022 · 1 comment
Assignees

Comments

@tore-espressif
Copy link
Collaborator

One note about this — I think it's not very convenient to specify the target list for test apps here in CI configuration. It will probably get quite messy if the repository gets more test applications, and each test application will have its own set of targets.

We could consider using find_apps.py and build_apps.py scripts from IDF to perform builds for all targets, but in that case we would need to support the merge_bin step as part of build_apps.py.

(No action needed for now, just bringing this up. I'm afraid the missing tests for S2 will be easy to forget about.)

Originally posted by @igrr in #13 (comment)

@tore-espressif
Copy link
Collaborator Author

would need to support the merge_bin step as part of build_apps.py.

the merge_bin step is not needed anymore with pytest-embedded v >= 0.7.0

@tore-espressif tore-espressif self-assigned this May 9, 2022
tore-espressif added a commit that referenced this issue Jul 13, 2022
Relates to #69

This commit is a good reason why #16 should be addressed
tore-espressif added a commit that referenced this issue Jul 14, 2022
Relates to #69

This commit is a good reason why #16 should be addressed
@tom-borcin tom-borcin changed the title Target testing strategy Target testing strategy Apr 21, 2023
@github-actions github-actions bot changed the title Target testing strategy Target testing strategy (IEC-1) Apr 21, 2023
peter-marcisovsky pushed a commit to espressif/esp-usb that referenced this issue Jan 18, 2024
Relates to espressif/idf-extra-components#69

This commit is a good reason why espressif/idf-extra-components#16 should be addressed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant