-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
tests(cpu): Add deep sleep and misc tests for CPU #10112
base: master
Are you sure you want to change the base?
Conversation
👋 Hello lucasssvaz, we appreciate your contribution to this project! Click to see more instructions ...
Review and merge process you can expect ...
|
Test Results125 files - 14 125 suites - 14 16m 13s ⏱️ - 6m 3s For more details on these errors, see this check. Results for commit 47a1329. ± Comparison against base commit b05f18d. This pull request removes 5 and adds 3 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
@lucasssvaz - I would suggest adding a light sleep test as well. |
Sure! Do you have any good light sleep examples that we can use in the tests ? |
Check this issue #10065 - it may be useful for a test case. |
0a5fa9b
to
fdad0c3
Compare
4ddcd08
to
8c6164e
Compare
By completing this PR sufficiently, you help us to review this Pull Request quicker and also help improve the quality of Release Notes
Checklist
This entire section above can be deleted if all items are checked.
Description of Change
Please describe your proposed Pull Request and it's impact.
Tests scenarios
Please describe on what Hardware and Software combinations you have tested this Pull Request and how.
(eg. I have tested my Pull Request on Arduino-esp32 core v2.0.2 with ESP32 and ESP32-S2 Board with this scenario)
Related links
Please provide links to related issue, PRs etc.
(eg. Closes #number of issue)