[nrf fromtree] tests: fs: nvs: Add a test scenario for data CRC

Adapt the test_nvs_gc_corrupt_close_ate test to forge the data CRC.

Signed-off-by: Adrien Ricciardi <[email protected]>
(cherry picked from commit e13840c)
Signed-off-by: Andrzej Głąbek <[email protected]>

Author: RICCIARDI-Adrien

tests/subsys/fs/nvs/src/main.c

@@ -560,7 +560,8 @@ ZTEST_F(nvs, test_nvs_full_sector)
 				     len);
 		} else {
 			zassert_true(len == sizeof(data_read),
-				     "nvs_read failed: %d", i, len);
+				     "nvs_read #%d failed: len is %zd instead of %zu",
+				     i, len, sizeof(data_read));
 			zassert_equal(data_read, i,
 				      "read unexpected data: %d instead of %d",
 				      data_read, i);
@@ -639,6 +640,9 @@ ZTEST_F(nvs, test_nvs_gc_corrupt_close_ate)
 	uint32_t data;
 	ssize_t len;
 	int err;
+#ifdef CONFIG_NVS_DATA_CRC
+	uint32_t data_crc;
+#endif
 
 	close_ate.id = 0xffff;
 	close_ate.offset = fixture->fs.sector_size - sizeof(struct nvs_ate) * 5;
@@ -648,6 +652,9 @@ ZTEST_F(nvs, test_nvs_gc_corrupt_close_ate)
 	ate.id = 0x1;
 	ate.offset = 0;
 	ate.len = sizeof(data);
+#ifdef CONFIG_NVS_DATA_CRC
+	ate.len += sizeof(data_crc);
+#endif
 	ate.crc8 = crc8_ccitt(0xff, &ate,
 			      offsetof(struct nvs_ate, crc8));
 
@@ -666,6 +673,12 @@ ZTEST_F(nvs, test_nvs_gc_corrupt_close_ate)
 	data = 0xaa55aa55;
 	err = flash_write(fixture->fs.flash_device, fixture->fs.offset, &data, sizeof(data));
 	zassert_true(err == 0,  "flash_write failed: %d", err);
+#ifdef CONFIG_NVS_DATA_CRC
+	data_crc = crc32_ieee((const uint8_t *) &data, sizeof(data));
+	err = flash_write(fixture->fs.flash_device, fixture->fs.offset + sizeof(data), &data_crc,
+			  sizeof(data_crc));
+	zassert_true(err == 0,  "flash_write for data CRC failed: %d", err);
+#endif
 
 	/* Mark sector 1 as closed */
 	err = flash_write(fixture->fs.flash_device,

tests/subsys/fs/nvs/testcase.yaml

@@ -11,3 +11,15 @@ tests:
       - CONFIG_NVS_LOOKUP_CACHE=y
       - CONFIG_NVS_LOOKUP_CACHE_SIZE=64
     platform_allow: native_sim
+  filesystem.nvs.data_crc:
+    extra_args:
+      - CONFIG_NVS_DATA_CRC=y
+    platform_allow:
+      - native_sim
+      - qemu_x86
+  filesystem.nvs.data_crc_cache:
+    extra_args:
+      - CONFIG_NVS_DATA_CRC=y
+      - CONFIG_NVS_LOOKUP_CACHE=y
+      - CONFIG_NVS_LOOKUP_CACHE_SIZE=64
+    platform_allow: native_sim