Skip to content

Update the NVS testcases #3344

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.

Sign up for GitHub

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

Jump to bottom
Open
zhaoxingyu12 wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update the NVS testcases #3344

zhaoxingyu12 wants to merge 3 commits into from
+44 −40

Conversation

@zhaoxingyu12
Copy link
Contributor

@zhaoxingyu12 zhaoxingyu12 commented Jan 19, 2026

Note: Please adhere to Contributing Guidelines.

Summary

Update the NVS test cases because the mtd_config configuration item in Kconfig has been modified.

Impact

testcases for nvs module

Testing

This has been tested and passed in qemu

test_nvs_mount: test begin
test_nvs_mount: success
test_nvs_write: test begin
test_nvs_write: success
test_nvs_corrupt_expire: test begin
test_nvs_corrupt_expire: success
test_nvs_corrupted_write: test begin
test_nvs_corrupted_write: success
test_nvs_gc: test begin
test_nvs_gc: success
test_nvs_gc_3sectors: test begin
test_nvs_gc_3sectors: success
test_nvs_corrupted_sector_close: test begin
test_nvs_corrupted_sector_close: success
test_nvs_full_sector: test begin
test_nvs_full_sector: success
test_nvs_gc_corrupt_close_ate: test begin
test_nvs_gc_corrupt_close_ate: success
test_nvs_gc_corrupt_ate: test begin
test_nvs_gc_corrupt_ate: success
test_nvs_gc_touched_deleted_ate: test begin
test_nvs_gc_touched_deleted_ate: success
test_nvs_gc_touched_expired_ate: test begin
test_nvs_gc_touched_expired_ate: success
test_nvs_gc_not_touched_expired_ate: test begin
test_nvs_gc_not_touched_expired_ate: success

@zhaoxingyu12 zhaoxingyu12 force-pushed the master branch 2 times, most recently from 3d04e42 to bcf8259 Compare January 20, 2026 02:17
@zhaoxingyu12 zhaoxingyu12 reopened this Jan 20, 2026
@zhaoxingyu12 zhaoxingyu12 force-pushed the master branch 2 times, most recently from c5e6ba1 to 7f80c9c Compare January 20, 2026 11:14
@zhaoxingyu12 zhaoxingyu12 mentioned this pull request Jan 21, 2026
Open
@cederom
Copy link
Contributor

cederom commented Jan 21, 2026

ci/msvc failed at network access, restarted :-)

@cederom
Copy link
Contributor

cederom commented Jan 21, 2026

ci/msvc failed again, unrelated to this change, temporary or permanent compiler move by arm?

====================================================================================
Start CI build: 2026-01-21 01:57:10
------------------------------------------------------------------------------------
Windows_NT
Check ARM clang toolchain ...
clang version 20.1.8
Target: x86_64-pc-windows-msvc
Thread model: posix
InstalledDir: C:\Program Files\LLVM\bin

Check ARM GCC toolchain toolchain ...
Download: ARM GCC toolchain
An error occurred while executing the script: Failed to download the file: 
Access Denied

Access Denied
 
You don't have permission to access "http://developer.arm.com/-/media/Files/downloads/gnu/13.2.rel1/binrel/arm-gnu-toolchain-13.2.Rel1-mingw-w64-i686-arm-none-eabi.zip" on this server.
Reference #18.c6d11102.1768960633.90729b7f
https://errors.edgesuite.net/18.c6d11102.1768960633.90729b7f



Found Visual Studio 2022 installations
File testlist\windows.dat
ERROR: Missing test list file

USAGE: testbuild.ps1 -h [-n] [-A] [-G] [-N]
Where:
  -h will show this help test and terminate
  -n selects Windows native
  -p only print the list of configs without running any builds
  -A store the build executable artifact in ARTIFACTDIR (defaults to ../buildartifacts
  -C Skip tree cleanness check.
  -G Use "git clean -xfdq" instead of "make distclean" to clean the tree.
  -N Use CMake with Ninja as the backend.
  <testlist-file> selects the list of configurations to test.  No default

Error: Process completed with exit code 1.

@zhaoxingyu12
Copy link
Contributor Author

zhaoxingyu12 commented Jan 21, 2026

ci/msvc failed again, unrelated to this change, temporary or permanent compiler move by arm?

====================================================================================
Start CI build: 2026-01-21 01:57:10
------------------------------------------------------------------------------------
Windows_NT
Check ARM clang toolchain ...
clang version 20.1.8
Target: x86_64-pc-windows-msvc
Thread model: posix
InstalledDir: C:\Program Files\LLVM\bin

Check ARM GCC toolchain toolchain ...
Download: ARM GCC toolchain
An error occurred while executing the script: Failed to download the file: 
Access Denied

Access Denied
 
You don't have permission to access "http&#58;&#47;&#47;developer&#46;arm&#46;com&#47;&#45;&#47;media&#47;Files&#47;downloads&#47;gnu&#47;13&#46;2&#46;rel1&#47;binrel&#47;arm&#45;gnu&#45;toolchain&#45;13&#46;2&#46;Rel1&#45;mingw&#45;w64&#45;i686&#45;arm&#45;none&#45;eabi&#46;zip" on this server.
Reference&#32;&#35;18&#46;c6d11102&#46;1768960633&#46;90729b7f
https&#58;&#47;&#47;errors&#46;edgesuite&#46;net&#47;18&#46;c6d11102&#46;1768960633&#46;90729b7f



Found Visual Studio 2022 installations
File testlist\windows.dat
ERROR: Missing test list file

USAGE: testbuild.ps1 -h [-n] [-A] [-G] [-N]
Where:
  -h will show this help test and terminate
  -n selects Windows native
  -p only print the list of configs without running any builds
  -A store the build executable artifact in ARTIFACTDIR (defaults to ../buildartifacts
  -C Skip tree cleanness check.
  -G Use "git clean -xfdq" instead of "make distclean" to clean the tree.
  -N Use CMake with Ninja as the backend.
  <testlist-file> selects the list of configurations to test.  No default

Error: Process completed with exit code 1.

Should I restart the CI again?

@cederom
Copy link
Contributor

cederom commented Jan 21, 2026

nope, looks like a ci/net glitch unrealted to your change, we should retry after some time.

@zhaoxingyu12
Copy link
Contributor Author

zhaoxingyu12 commented Jan 21, 2026

nope, looks like a ci/net glitch unrealted to your change, we should retry after some time.

ok

zhaoxingyu12 and others added 3 commits January 21, 2026 23:27
@zhaoxingyu12
mtd/nvs: modify config name to MTD_CONFIG_NVS part1
abb5d53 
Because the MTD_CONFIG configuration item in
drivers/mtd/Kconfig has changed

Signed-off-by: zhaoxingyu1 <zhaoxingyu1@xiaomi.com>
@guohao15 @zhaoxingyu12
mtdconfig: support ram_mtdconfig device && lomtdconfig device
8444178 
For nvs test in qemu

Signed-off-by: guohao15 <guohao15@xiaomi.com>
@zhaoxingyu12
mtdconfig: lomtdconfig device change to depends on !MTD_CONFIG_NONE
84f496d 
Because  MTD_CONFIG configuration item in drivers/mtd/Kconfig
has changed.

Signed-off-by: zhaoxingyu1 <zhaoxingyu1@xiaomi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xiaoxiang781216 xiaoxiang781216 xiaoxiang781216 approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

Area: NSH Area: System Area: Testing Size: S

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@zhaoxingyu12 @cederom @xiaoxiang781216 @guohao15