- Notifications
You must be signed in to change notification settings - Fork7.8k
feat(test): Enhance NVS test#11481
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
github-actionsbot commentedJun 18, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
👋Hello JakubAndrysek, we appreciate your contribution to this project! 📘 Please review the project'sContributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you haveread and signed theContributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
github-actionsbot commentedJun 18, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Test Results144 files 144 suites 48m 19s ⏱️ For more details on these failures, seethis check. Results for commit13aac76. ♻️ This comment has been updated with latest results. |
JakubAndrysek commentedJun 19, 2025
lucasssvaz commentedJun 19, 2025
These are the compilation tests and not the runtime tests. Either way HW tests are passing and I've checked the wokwi run on your fork. Everything looks good. |
95ae8cf intoespressif:masterUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Description of Change
This pull request introduces enhancements to the
tests/validation/nvsmodule, focusing on extending the functionality of the NVS (Non-Volatile Storage) validation and improving test coverage. The changes include adding support for more data types, implementing value incrementation, and updating the test cases accordingly.Tests scenarios
I have tested my Pull Request on Arduino-esp32 core v3.2.0 with Wokwi on all available chips.