[PATCH v2 3/7] test: cmd: add unit tests for part command
Dario Binacchi
dario.binacchi at amarulasolutions.com
Sat Apr 4 17:23:30 CEST 2026
Add unit tests for the 'part' command, specifically for the 'number',
'start', and 'size' subcommands.
These tests establish a baseline for the current partition lookup
functionality by name. This foundation will be used by subsequent
patches to extend the command, ensuring consistent behavior as new
features are introduced.
Signed-off-by: Dario Binacchi <dario.binacchi at amarulasolutions.com>
---
(no changes since v1)
test/cmd/Makefile | 1 +
test/cmd/part.c | 157 ++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 158 insertions(+)
create mode 100644 test/cmd/part.c
diff --git a/test/cmd/Makefile b/test/cmd/Makefile
index 8c9f112782d0..41fe21a4541e 100644
--- a/test/cmd/Makefile
+++ b/test/cmd/Makefile
@@ -26,6 +26,7 @@ obj-$(CONFIG_CMD_LOADM) += loadm.o
obj-$(CONFIG_CMD_MEMINFO) += meminfo.o
obj-$(CONFIG_CMD_MEMORY) += mem_copy.o
obj-$(CONFIG_CMD_MEM_SEARCH) += mem_search.o
+obj-$(CONFIG_CMD_PART) += part.o
ifdef CONFIG_CMD_PCI
obj-$(CONFIG_CMD_PCI_MPS) += pci_mps.o
endif
diff --git a/test/cmd/part.c b/test/cmd/part.c
new file mode 100644
index 000000000000..e60e0226b64b
--- /dev/null
+++ b/test/cmd/part.c
@@ -0,0 +1,157 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Test for part command
+ *
+ * Copyright (C) 2026 Amarula Solutions
+ * Written by Dario Binacchi <dario.binacchi at amarulasolutions.com>
+ */
+
+#include <dm/test.h>
+#include <env.h>
+#include <part.h>
+#include <test/ut.h>
+#include <vsprintf.h>
+
+static struct disk_partition gpt_parts[] = {
+ {
+ .start = 48,
+ .size = 1,
+ .name = "test1",
+ .uuid = "c5bce7a2-03f0-4d03-9048-01ff23b9d527",
+ },
+ {
+ .start = 49,
+ .size = 2,
+ .name = "test2",
+ .uuid = "9df346e8-2c53-4cd8-b9ac-3af83f9a9b74",
+ },
+};
+
+static char disk_guid[UUID_STR_LEN + 1] = "8d60b397-1bb6-4d33-80ee-b1587d24c2f8";
+
+static int setup_gpt_partitions(struct unit_test_state *uts, unsigned int mmc_dev_num)
+{
+ struct blk_desc *mmc_dev_desc;
+ char dev_str[10];
+ int i;
+
+ snprintf(dev_str, sizeof(dev_str), "%u", mmc_dev_num);
+
+ ut_asserteq(mmc_dev_num, blk_get_device_by_str("mmc", dev_str, &mmc_dev_desc));
+
+ if (CONFIG_IS_ENABLED(RANDOM_UUID)) {
+ for (i = 0; i < ARRAY_SIZE(gpt_parts); i++)
+ gen_rand_uuid_str(gpt_parts[i].uuid, UUID_STR_FORMAT_STD);
+
+ gen_rand_uuid_str(disk_guid, UUID_STR_FORMAT_STD);
+ }
+
+ ut_assertok(gpt_restore(mmc_dev_desc, disk_guid, gpt_parts,
+ ARRAY_SIZE(gpt_parts)));
+ return 0;
+}
+
+static int dm_test_cmd_part_number(struct unit_test_state *uts)
+{
+ unsigned int mmc_dev_num = 2;
+ char expected[10];
+ int i, ret;
+
+ ut_assertok(setup_gpt_partitions(uts, mmc_dev_num));
+
+ for (i = 0; i < ARRAY_SIZE(gpt_parts); i++) {
+ env_set("partnum", NULL);
+ ut_assertok(run_commandf("part number mmc %u %s partnum",
+ mmc_dev_num, gpt_parts[i].name));
+ snprintf(expected, sizeof(expected), "0x%x", i + 1);
+ ut_asserteq_str(expected, env_get("partnum"));
+ }
+
+ env_set("partnum", NULL);
+ ret = run_commandf("part number mmc %u bogus partnum", mmc_dev_num);
+ ut_assert(ret != 0);
+ ut_assertnull(env_get("partnum"));
+
+ return 0;
+}
+DM_TEST(dm_test_cmd_part_number, UTF_SCAN_PDATA | UTF_SCAN_FDT);
+
+static int dm_test_cmd_part_start(struct unit_test_state *uts)
+{
+ unsigned int mmc_dev_num = 2;
+ char expected[32];
+ int i, ret;
+
+ ut_assertok(setup_gpt_partitions(uts, mmc_dev_num));
+
+ for (i = 0; i < ARRAY_SIZE(gpt_parts); i++) {
+ env_set("partstart", NULL);
+ ut_assertok(run_commandf("part start mmc %u %d partstart",
+ mmc_dev_num, i + 1));
+ snprintf(expected, sizeof(expected), "%lx",
+ (unsigned long)gpt_parts[i].start);
+ ut_asserteq_str(expected, env_get("partstart"));
+ }
+
+ env_set("partstart", NULL);
+ ret = run_commandf("part start mmc %u 3 partstart", mmc_dev_num);
+ ut_assert(ret != 0);
+ ut_assertnull(env_get("partstart"));
+
+ for (i = 0; i < ARRAY_SIZE(gpt_parts); i++) {
+ env_set("partstart", NULL);
+ ut_assertok(run_commandf("part start mmc %u %s partstart",
+ mmc_dev_num, gpt_parts[i].name));
+ snprintf(expected, sizeof(expected), "%lx",
+ (unsigned long)gpt_parts[i].start);
+ ut_asserteq_str(expected, env_get("partstart"));
+ }
+
+ env_set("partstart", NULL);
+ ret = run_commandf("part start mmc %u bogus partstart", mmc_dev_num);
+ ut_assert(ret != 0);
+ ut_assertnull(env_get("partstart"));
+
+ return 0;
+}
+DM_TEST(dm_test_cmd_part_start, UTF_SCAN_PDATA | UTF_SCAN_FDT);
+
+static int dm_test_cmd_part_size(struct unit_test_state *uts)
+{
+ unsigned int mmc_dev_num = 2;
+ char expected[32];
+ int i, ret;
+
+ ut_assertok(setup_gpt_partitions(uts, mmc_dev_num));
+
+ for (i = 0; i < ARRAY_SIZE(gpt_parts); i++) {
+ env_set("partsize", NULL);
+ ut_assertok(run_commandf("part size mmc %u %d partsize",
+ mmc_dev_num, i + 1));
+ snprintf(expected, sizeof(expected), "%lx",
+ (unsigned long)gpt_parts[i].size);
+ ut_asserteq_str(expected, env_get("partsize"));
+ }
+
+ env_set("partsize", NULL);
+ ret = run_commandf("part size mmc %u 3 partsize", mmc_dev_num);
+ ut_assert(ret != 0);
+ ut_assertnull(env_get("partsize"));
+
+ for (i = 0; i < ARRAY_SIZE(gpt_parts); i++) {
+ env_set("partsize", NULL);
+ ut_assertok(run_commandf("part size mmc %u %s partsize",
+ mmc_dev_num, gpt_parts[i].name));
+ snprintf(expected, sizeof(expected), "%lx",
+ (unsigned long)gpt_parts[i].size);
+ ut_asserteq_str(expected, env_get("partsize"));
+ }
+
+ env_set("partsize", NULL);
+ ret = run_commandf("part size mmc %u bogus partsize", mmc_dev_num);
+ ut_assert(ret != 0);
+ ut_assertnull(env_get("partsize"));
+
+ return 0;
+}
+DM_TEST(dm_test_cmd_part_size, UTF_SCAN_PDATA | UTF_SCAN_FDT);
--
2.43.0
More information about the U-Boot
mailing list