mirror of
https://source.denx.de/u-boot/u-boot.git
synced 2026-06-13 15:03:58 +03:00
Merge patch series "Tidy up console recording in tests"
Simon Glass <sjg@chromium.org> says: This series started as a small fix for checking for an empty line, but in the process several other problems were found and fixed: - fix tests which use console recording but don't set the flag - drop unnecessary resetting of the console in tests - drop unnecessary blank line before MMC output - update the docs a little - fix buildman test failure on newer Pythons - a few other minor things This series also renames the confusing flag names, so that they are easier to remember - just a UTF_ (unit-test flags) prefix.
This commit is contained in:
@@ -28,7 +28,6 @@ static int bootdev_test_cmd_list(struct unit_test_state *uts)
|
||||
{
|
||||
int probed;
|
||||
|
||||
console_record_reset_enable();
|
||||
for (probed = 0; probed < 2; probed++) {
|
||||
int probe_ch = probed ? '+' : ' ';
|
||||
|
||||
@@ -49,7 +48,7 @@ static int bootdev_test_cmd_list(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_cmd_list, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_cmd_list, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootdev select' and 'info' commands */
|
||||
static int bootdev_test_cmd_select(struct unit_test_state *uts)
|
||||
@@ -59,7 +58,6 @@ static int bootdev_test_cmd_select(struct unit_test_state *uts)
|
||||
/* get access to the CLI's cur_bootdev */
|
||||
ut_assertok(bootstd_get_priv(&std));
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_asserteq(1, run_command("bootdev info", 0));
|
||||
ut_assert_nextlinen("Please use");
|
||||
ut_assert_console_end();
|
||||
@@ -99,7 +97,7 @@ static int bootdev_test_cmd_select(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_cmd_select, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_cmd_select, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check bootdev labels */
|
||||
static int bootdev_test_labels(struct unit_test_state *uts)
|
||||
@@ -131,8 +129,7 @@ static int bootdev_test_labels(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_labels, UT_TESTF_DM | UT_TESTF_SCAN_FDT |
|
||||
UT_TESTF_ETH_BOOTDEV);
|
||||
BOOTSTD_TEST(bootdev_test_labels, UTF_DM | UTF_SCAN_FDT | UTF_ETH_BOOTDEV);
|
||||
|
||||
/* Check bootdev_find_by_any() */
|
||||
static int bootdev_test_any(struct unit_test_state *uts)
|
||||
@@ -158,7 +155,6 @@ static int bootdev_test_any(struct unit_test_state *uts)
|
||||
* 9 [ + ] OK mmc mmc1.bootdev
|
||||
* a [ ] OK mmc mmc0.bootdev
|
||||
*/
|
||||
console_record_reset_enable();
|
||||
ut_assertok(bootdev_find_by_any("8", &dev, &mflags));
|
||||
ut_asserteq(UCLASS_BOOTDEV, device_get_uclass_id(dev));
|
||||
ut_asserteq(BOOTFLOW_METHF_SINGLE_DEV, mflags);
|
||||
@@ -186,8 +182,8 @@ static int bootdev_test_any(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_any, UT_TESTF_DM | UT_TESTF_SCAN_FDT |
|
||||
UT_TESTF_ETH_BOOTDEV);
|
||||
BOOTSTD_TEST(bootdev_test_any, UTF_DM | UTF_SCAN_FDT | UTF_ETH_BOOTDEV |
|
||||
UTF_CONSOLE);
|
||||
|
||||
/*
|
||||
* Check bootdev ordering with the bootdev-order property and boot_targets
|
||||
@@ -274,7 +270,7 @@ static int bootdev_test_order(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_order, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_order, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check default bootdev ordering */
|
||||
static int bootdev_test_order_default(struct unit_test_state *uts)
|
||||
@@ -301,7 +297,7 @@ static int bootdev_test_order_default(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_order_default, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_order_default, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check bootdev ordering with the uclass priority */
|
||||
static int bootdev_test_prio(struct unit_test_state *uts)
|
||||
@@ -323,7 +319,6 @@ static int bootdev_test_prio(struct unit_test_state *uts)
|
||||
ut_assertok(bootstd_test_drop_bootdev_order(uts));
|
||||
|
||||
/* 3 MMC and 3 USB bootdevs: MMC should come before USB */
|
||||
console_record_reset_enable();
|
||||
ut_assertok(bootflow_scan_first(NULL, NULL, &iter, 0, &bflow));
|
||||
ut_asserteq(-ENODEV, bootflow_scan_next(&iter, &bflow));
|
||||
ut_asserteq(6, iter.num_devs);
|
||||
@@ -350,7 +345,7 @@ static int bootdev_test_prio(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_prio, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_prio, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check listing hunters */
|
||||
static int bootdev_test_hunter(struct unit_test_state *uts)
|
||||
@@ -363,7 +358,6 @@ static int bootdev_test_hunter(struct unit_test_state *uts)
|
||||
/* get access to the used hunters */
|
||||
ut_assertok(bootstd_get_priv(&std));
|
||||
|
||||
console_record_reset_enable();
|
||||
bootdev_list_hunters(std);
|
||||
ut_assert_nextline("Prio Used Uclass Hunter");
|
||||
ut_assert_nextlinen("----");
|
||||
@@ -390,7 +384,7 @@ static int bootdev_test_hunter(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_hunter, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_hunter, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootdev hunt' command */
|
||||
static int bootdev_test_cmd_hunt(struct unit_test_state *uts)
|
||||
@@ -403,7 +397,6 @@ static int bootdev_test_cmd_hunt(struct unit_test_state *uts)
|
||||
/* get access to the used hunters */
|
||||
ut_assertok(bootstd_get_priv(&std));
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootdev hunt -l", 0));
|
||||
ut_assert_nextline("Prio Used Uclass Hunter");
|
||||
ut_assert_nextlinen("----");
|
||||
@@ -464,8 +457,8 @@ static int bootdev_test_cmd_hunt(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_cmd_hunt, UT_TESTF_DM | UT_TESTF_SCAN_FDT |
|
||||
UT_TESTF_ETH_BOOTDEV);
|
||||
BOOTSTD_TEST(bootdev_test_cmd_hunt, UTF_DM | UTF_SCAN_FDT | UTF_ETH_BOOTDEV |
|
||||
UTF_CONSOLE);
|
||||
|
||||
/* Check searching for bootdevs using the hunters */
|
||||
static int bootdev_test_hunt_scan(struct unit_test_state *uts)
|
||||
@@ -485,7 +478,7 @@ static int bootdev_test_hunt_scan(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_hunt_scan, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_hunt_scan, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check that only bootable partitions are processed */
|
||||
static int bootdev_test_bootable(struct unit_test_state *uts)
|
||||
@@ -522,7 +515,7 @@ static int bootdev_test_bootable(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_bootable, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_bootable, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check hunting for bootdev of a particular priority */
|
||||
static int bootdev_test_hunt_prio(struct unit_test_state *uts)
|
||||
@@ -530,7 +523,6 @@ static int bootdev_test_hunt_prio(struct unit_test_state *uts)
|
||||
usb_started = false;
|
||||
test_set_skip_delays(true);
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(bootdev_hunt_prio(BOOTDEVP_4_SCAN_FAST, false));
|
||||
ut_assert_nextline("scanning bus for devices...");
|
||||
ut_assert_skip_to_line(" Type: Hard Disk");
|
||||
@@ -547,7 +539,7 @@ static int bootdev_test_hunt_prio(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_hunt_prio, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_hunt_prio, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check hunting for bootdevs with a particular label */
|
||||
static int bootdev_test_hunt_label(struct unit_test_state *uts)
|
||||
@@ -562,7 +554,6 @@ static int bootdev_test_hunt_label(struct unit_test_state *uts)
|
||||
ut_assertok(bootstd_get_priv(&std));
|
||||
|
||||
/* scan an unknown uclass */
|
||||
console_record_reset_enable();
|
||||
old = (void *)&mflags; /* arbitrary pointer to check against dev */
|
||||
dev = old;
|
||||
mflags = 123;
|
||||
@@ -600,7 +591,7 @@ static int bootdev_test_hunt_label(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_hunt_label, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootdev_test_hunt_label, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check iterating to the next label in a list */
|
||||
static int bootdev_test_next_label(struct unit_test_state *uts)
|
||||
@@ -627,7 +618,6 @@ static int bootdev_test_next_label(struct unit_test_state *uts)
|
||||
dev = NULL;
|
||||
mflags = 123;
|
||||
ut_assertok(bootdev_next_label(&iter, &dev, &mflags));
|
||||
console_record_reset_enable();
|
||||
ut_assert_console_end();
|
||||
ut_assertnonnull(dev);
|
||||
ut_asserteq_str("mmc0.bootdev", dev->name);
|
||||
@@ -677,8 +667,8 @@ static int bootdev_test_next_label(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_next_label, UT_TESTF_DM | UT_TESTF_SCAN_FDT |
|
||||
UT_TESTF_ETH_BOOTDEV | UT_TESTF_SF_BOOTDEV);
|
||||
BOOTSTD_TEST(bootdev_test_next_label, UTF_DM | UTF_SCAN_FDT | UTF_ETH_BOOTDEV |
|
||||
UTF_SF_BOOTDEV | UTF_CONSOLE);
|
||||
|
||||
/* Check iterating to the next prioirty in a list */
|
||||
static int bootdev_test_next_prio(struct unit_test_state *uts)
|
||||
@@ -703,7 +693,6 @@ static int bootdev_test_next_prio(struct unit_test_state *uts)
|
||||
iter.flags = BOOTFLOWIF_SHOW;
|
||||
|
||||
dev = NULL;
|
||||
console_record_reset_enable();
|
||||
ut_assertok(bootdev_next_prio(&iter, &dev));
|
||||
ut_assertnonnull(dev);
|
||||
ut_asserteq_str("mmc2.bootdev", dev->name);
|
||||
@@ -762,5 +751,5 @@ static int bootdev_test_next_prio(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootdev_test_next_prio, UT_TESTF_DM | UT_TESTF_SCAN_FDT |
|
||||
UT_TESTF_SF_BOOTDEV);
|
||||
BOOTSTD_TEST(bootdev_test_next_prio, UTF_DM | UTF_SCAN_FDT | UTF_SF_BOOTDEV |
|
||||
UTF_CONSOLE);
|
||||
|
||||
@@ -50,7 +50,6 @@ static int inject_response(struct unit_test_state *uts)
|
||||
/* Check 'bootflow scan/list' commands */
|
||||
static int bootflow_cmd(struct unit_test_state *uts)
|
||||
{
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootdev select 1", 0));
|
||||
ut_assert_console_end();
|
||||
ut_assertok(run_command("bootflow scan -lH", 0));
|
||||
@@ -76,14 +75,13 @@ static int bootflow_cmd(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_cmd, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow scan' with a label / seq */
|
||||
static int bootflow_cmd_label(struct unit_test_state *uts)
|
||||
{
|
||||
test_set_eth_enable(false);
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootflow scan -lH mmc1", 0));
|
||||
ut_assert_nextline("Scanning for bootflows with label 'mmc1'");
|
||||
ut_assert_skip_to_line("(1 bootflow, 1 valid)");
|
||||
@@ -123,15 +121,14 @@ static int bootflow_cmd_label(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd_label, UT_TESTF_DM | UT_TESTF_SCAN_FDT |
|
||||
UT_TESTF_ETH_BOOTDEV);
|
||||
BOOTSTD_TEST(bootflow_cmd_label, UTF_DM | UTF_SCAN_FDT | UTF_ETH_BOOTDEV |
|
||||
UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow scan/list' commands using all bootdevs */
|
||||
static int bootflow_cmd_glob(struct unit_test_state *uts)
|
||||
{
|
||||
ut_assertok(bootstd_test_drop_bootdev_order(uts));
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootflow scan -lGH", 0));
|
||||
ut_assert_nextline("Scanning for bootflows in all bootdevs");
|
||||
ut_assert_nextline("Seq Method State Uclass Part Name Filename");
|
||||
@@ -156,14 +153,13 @@ static int bootflow_cmd_glob(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd_glob, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_cmd_glob, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow scan -e' */
|
||||
static int bootflow_cmd_scan_e(struct unit_test_state *uts)
|
||||
{
|
||||
ut_assertok(bootstd_test_drop_bootdev_order(uts));
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootflow scan -aleGH", 0));
|
||||
ut_assert_nextline("Scanning for bootflows in all bootdevs");
|
||||
ut_assert_nextline("Seq Method State Uclass Part Name Filename");
|
||||
@@ -207,12 +203,11 @@ static int bootflow_cmd_scan_e(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd_scan_e, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_cmd_scan_e, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow info' */
|
||||
static int bootflow_cmd_info(struct unit_test_state *uts)
|
||||
{
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootdev select 1", 0));
|
||||
ut_assert_console_end();
|
||||
ut_assertok(run_command("bootflow scan", 0));
|
||||
@@ -248,12 +243,11 @@ static int bootflow_cmd_info(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd_info, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_cmd_info, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow scan -b' to boot the first available bootdev */
|
||||
static int bootflow_scan_boot(struct unit_test_state *uts)
|
||||
{
|
||||
console_record_reset_enable();
|
||||
ut_assertok(inject_response(uts));
|
||||
ut_assertok(run_command("bootflow scan -b", 0));
|
||||
ut_assert_nextline(
|
||||
@@ -270,7 +264,7 @@ static int bootflow_scan_boot(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_scan_boot, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_scan_boot, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check iterating through available bootflows */
|
||||
static int bootflow_iter(struct unit_test_state *uts)
|
||||
@@ -368,7 +362,7 @@ static int bootflow_iter(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_iter, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_iter, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
#if defined(CONFIG_SANDBOX) && defined(CONFIG_BOOTMETH_GLOBAL)
|
||||
/* Check using the system bootdev */
|
||||
@@ -386,7 +380,6 @@ static int bootflow_system(struct unit_test_state *uts)
|
||||
|
||||
/* We should get a single 'bootmgr' method right at the end */
|
||||
bootstd_clear_glob();
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootflow scan -lH", 0));
|
||||
ut_assert_skip_to_line(
|
||||
" 0 efi_mgr ready (none) 0 <NULL> ");
|
||||
@@ -396,8 +389,8 @@ static int bootflow_system(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_system, UT_TESTF_DM | UT_TESTF_SCAN_PDATA |
|
||||
UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_system, UTF_DM | UTF_SCAN_PDATA | UTF_SCAN_FDT |
|
||||
UTF_CONSOLE);
|
||||
#endif
|
||||
|
||||
/* Check disabling a bootmethod if it requests it */
|
||||
@@ -416,7 +409,6 @@ static int bootflow_iter_disable(struct unit_test_state *uts)
|
||||
ut_assertok(bootstd_test_drop_bootdev_order(uts));
|
||||
|
||||
bootstd_clear_glob();
|
||||
console_record_reset_enable();
|
||||
ut_assertok(inject_response(uts));
|
||||
ut_assertok(run_command("bootflow scan -lbH", 0));
|
||||
|
||||
@@ -438,7 +430,7 @@ static int bootflow_iter_disable(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_iter_disable, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_iter_disable, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow scan' with a bootmeth ordering including a global bootmeth */
|
||||
static int bootflow_scan_glob_bootmeth(struct unit_test_state *uts)
|
||||
@@ -452,7 +444,6 @@ static int bootflow_scan_glob_bootmeth(struct unit_test_state *uts)
|
||||
* Make sure that the -G flag makes the scan fail, since this is not
|
||||
* supported when an ordering is provided
|
||||
*/
|
||||
console_record_reset_enable();
|
||||
ut_assertok(bootmeth_set_order("efi firmware0"));
|
||||
ut_assertok(run_command("bootflow scan -lGH", 0));
|
||||
ut_assert_nextline("Scanning for bootflows in all bootdevs");
|
||||
@@ -479,12 +470,12 @@ static int bootflow_scan_glob_bootmeth(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_scan_glob_bootmeth, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_scan_glob_bootmeth, UTF_DM | UTF_SCAN_FDT |
|
||||
UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow boot' to boot a selected bootflow */
|
||||
static int bootflow_cmd_boot(struct unit_test_state *uts)
|
||||
{
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootdev select 1", 0));
|
||||
ut_assert_console_end();
|
||||
ut_assertok(run_command("bootflow scan", 0));
|
||||
@@ -508,7 +499,7 @@ static int bootflow_cmd_boot(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd_boot, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_cmd_boot, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/**
|
||||
* prep_mmc_bootdev() - Set up an mmc bootdev so we can access other distros
|
||||
@@ -585,7 +576,6 @@ static int scan_mmc_bootdev(struct unit_test_state *uts, const char *mmc_dev,
|
||||
|
||||
ut_assertok(prep_mmc_bootdev(uts, mmc_dev, bind_cros, &old_order));
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootflow scan", 0));
|
||||
ut_assert_console_end();
|
||||
|
||||
@@ -613,7 +603,6 @@ static int scan_mmc_android_bootdev(struct unit_test_state *uts, const char *mmc
|
||||
|
||||
ut_assertok(prep_mmc_bootdev(uts, mmc_dev, true, &old_order));
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootflow scan", 0));
|
||||
/* Android bootflow might print one or two 'ANDROID:*' logs */
|
||||
ut_check_skipline(uts);
|
||||
@@ -675,7 +664,7 @@ static int bootflow_cmd_menu(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd_menu, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_cmd_menu, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow scan -m' to select a bootflow using a menu */
|
||||
static int bootflow_scan_menu(struct unit_test_state *uts)
|
||||
@@ -723,8 +712,7 @@ static int bootflow_scan_menu(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_scan_menu,
|
||||
UT_TESTF_DM | UT_TESTF_SCAN_FDT | UT_TESTF_CONSOLE_REC);
|
||||
BOOTSTD_TEST(bootflow_scan_menu, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootflow scan -mb' to select and boot a bootflow using a menu */
|
||||
static int bootflow_scan_menu_boot(struct unit_test_state *uts)
|
||||
@@ -770,8 +758,7 @@ static int bootflow_scan_menu_boot(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_scan_menu_boot,
|
||||
UT_TESTF_DM | UT_TESTF_SCAN_FDT | UT_TESTF_CONSOLE_REC);
|
||||
BOOTSTD_TEST(bootflow_scan_menu_boot, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check searching for a single bootdev using the hunters */
|
||||
static int bootflow_cmd_hunt_single(struct unit_test_state *uts)
|
||||
@@ -783,7 +770,6 @@ static int bootflow_cmd_hunt_single(struct unit_test_state *uts)
|
||||
|
||||
ut_assertok(bootstd_test_drop_bootdev_order(uts));
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootflow scan -l mmc1", 0));
|
||||
ut_assert_nextline("Scanning for bootflows with label 'mmc1'");
|
||||
ut_assert_skip_to_line("(1 bootflow, 1 valid)");
|
||||
@@ -794,7 +780,8 @@ static int bootflow_cmd_hunt_single(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd_hunt_single, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_cmd_hunt_single, UTF_DM | UTF_SCAN_FDT |
|
||||
UTF_CONSOLE);
|
||||
|
||||
/* Check searching for a uclass label using the hunters */
|
||||
static int bootflow_cmd_hunt_label(struct unit_test_state *uts)
|
||||
@@ -808,7 +795,6 @@ static int bootflow_cmd_hunt_label(struct unit_test_state *uts)
|
||||
test_set_eth_enable(false);
|
||||
ut_assertok(bootstd_test_drop_bootdev_order(uts));
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootflow scan -l mmc", 0));
|
||||
|
||||
/* check that the hunter was used */
|
||||
@@ -831,7 +817,7 @@ static int bootflow_cmd_hunt_label(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmd_hunt_label, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_cmd_hunt_label, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/**
|
||||
* check_font() - Check that the font size for an item matches expectations
|
||||
@@ -891,7 +877,7 @@ static int bootflow_menu_theme(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_menu_theme, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootflow_menu_theme, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/**
|
||||
* check_arg() - Check both the normal case and the buffer-overflow case
|
||||
@@ -1127,7 +1113,6 @@ static int bootflow_cmdline(struct unit_test_state *uts)
|
||||
{
|
||||
ut_assertok(run_command("bootflow scan mmc", 0));
|
||||
ut_assertok(run_command("bootflow sel 0", 0));
|
||||
console_record_reset_enable();
|
||||
|
||||
ut_asserteq(1, run_command("bootflow cmdline get fred", 0));
|
||||
ut_assert_nextline("Argument not found");
|
||||
@@ -1151,13 +1136,11 @@ static int bootflow_cmdline(struct unit_test_state *uts)
|
||||
|
||||
ut_asserteq(0, run_command("bootflow cmdline set mary abc", 0));
|
||||
ut_asserteq(0, run_command("bootflow cmdline set mary", 0));
|
||||
ut_assert_nextline_empty();
|
||||
|
||||
ut_assert_console_end();
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cmdline, 0);
|
||||
BOOTSTD_TEST(bootflow_cmdline, UTF_CONSOLE);
|
||||
|
||||
/* test a few special changes to a long command line */
|
||||
static int bootflow_cmdline_special(struct unit_test_state *uts)
|
||||
@@ -1198,7 +1181,7 @@ static int bootflow_cros(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_cros, 0);
|
||||
BOOTSTD_TEST(bootflow_cros, UTF_CONSOLE);
|
||||
|
||||
/* Test Android bootmeth */
|
||||
static int bootflow_android(struct unit_test_state *uts)
|
||||
@@ -1221,4 +1204,4 @@ static int bootflow_android(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootflow_android, 0);
|
||||
BOOTSTD_TEST(bootflow_android, UTF_CONSOLE);
|
||||
|
||||
@@ -16,7 +16,6 @@
|
||||
/* Check 'bootmeth list' command */
|
||||
static int bootmeth_cmd_list(struct unit_test_state *uts)
|
||||
{
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootmeth list", 0));
|
||||
ut_assert_nextline("Order Seq Name Description");
|
||||
ut_assert_nextlinen("---");
|
||||
@@ -31,13 +30,12 @@ static int bootmeth_cmd_list(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootmeth_cmd_list, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootmeth_cmd_list, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootmeth order' command */
|
||||
static int bootmeth_cmd_order(struct unit_test_state *uts)
|
||||
{
|
||||
/* Select just one bootmethod */
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootmeth order extlinux", 0));
|
||||
ut_assert_console_end();
|
||||
ut_assertnonnull(env_get("bootmeths"));
|
||||
@@ -104,7 +102,7 @@ static int bootmeth_cmd_order(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootmeth_cmd_order, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootmeth_cmd_order, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootmeth order' command with global bootmeths */
|
||||
static int bootmeth_cmd_order_glob(struct unit_test_state *uts)
|
||||
@@ -112,7 +110,6 @@ static int bootmeth_cmd_order_glob(struct unit_test_state *uts)
|
||||
if (!IS_ENABLED(CONFIG_BOOTMETH_GLOBAL))
|
||||
return -EAGAIN;
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("bootmeth order \"efi firmware0\"", 0));
|
||||
ut_assert_console_end();
|
||||
ut_assertok(run_command("bootmeth list", 0));
|
||||
@@ -128,7 +125,7 @@ static int bootmeth_cmd_order_glob(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootmeth_cmd_order_glob, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootmeth_cmd_order_glob, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check 'bootmeths' env var */
|
||||
static int bootmeth_env(struct unit_test_state *uts)
|
||||
@@ -138,7 +135,6 @@ static int bootmeth_env(struct unit_test_state *uts)
|
||||
ut_assertok(bootstd_get_priv(&std));
|
||||
|
||||
/* Select just one bootmethod */
|
||||
console_record_reset_enable();
|
||||
ut_assertok(env_set("bootmeths", "extlinux"));
|
||||
ut_asserteq(1, std->bootmeth_count);
|
||||
|
||||
@@ -154,7 +150,7 @@ static int bootmeth_env(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootmeth_env, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootmeth_env, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check the get_state_desc() method */
|
||||
static int bootmeth_state(struct unit_test_state *uts)
|
||||
@@ -170,4 +166,4 @@ static int bootmeth_state(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(bootmeth_state, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(bootmeth_state, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
@@ -25,8 +25,6 @@ static int cedit_base(struct unit_test_state *uts)
|
||||
|
||||
ut_assertok(run_command("cedit load hostfs - cedit.dtb", 0));
|
||||
|
||||
console_record_reset_enable();
|
||||
|
||||
/*
|
||||
* ^N Move down to second menu
|
||||
* ^M Open menu
|
||||
@@ -52,7 +50,7 @@ static int cedit_base(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(cedit_base, 0);
|
||||
BOOTSTD_TEST(cedit_base, UTF_CONSOLE);
|
||||
|
||||
/* Check the cedit write_fdt and read_fdt commands */
|
||||
static int cedit_fdt(struct unit_test_state *uts)
|
||||
@@ -70,7 +68,6 @@ static int cedit_fdt(struct unit_test_state *uts)
|
||||
void *fdt;
|
||||
int i;
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("cedit load hostfs - cedit.dtb", 0));
|
||||
|
||||
ut_asserteq(ID_SCENE1, cedit_prepare(cur_exp, &vid_priv, &scn));
|
||||
@@ -122,7 +119,7 @@ static int cedit_fdt(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(cedit_fdt, 0);
|
||||
BOOTSTD_TEST(cedit_fdt, UTF_CONSOLE);
|
||||
|
||||
/* Check the cedit write_env and read_env commands */
|
||||
static int cedit_env(struct unit_test_state *uts)
|
||||
@@ -134,7 +131,6 @@ static int cedit_env(struct unit_test_state *uts)
|
||||
struct scene *scn;
|
||||
char *str;
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("cedit load hostfs - cedit.dtb", 0));
|
||||
|
||||
ut_asserteq(ID_SCENE1, cedit_prepare(cur_exp, &vid_priv, &scn));
|
||||
@@ -177,7 +173,7 @@ static int cedit_env(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(cedit_env, 0);
|
||||
BOOTSTD_TEST(cedit_env, UTF_CONSOLE);
|
||||
|
||||
/* Check the cedit write_cmos and read_cmos commands */
|
||||
static int cedit_cmos(struct unit_test_state *uts)
|
||||
@@ -187,7 +183,6 @@ static int cedit_cmos(struct unit_test_state *uts)
|
||||
extern struct expo *cur_exp;
|
||||
struct scene *scn;
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(run_command("cedit load hostfs - cedit.dtb", 0));
|
||||
|
||||
ut_asserteq(ID_SCENE1, cedit_prepare(cur_exp, &vid_priv, &scn));
|
||||
@@ -218,4 +213,4 @@ static int cedit_cmos(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(cedit_cmos, 0);
|
||||
BOOTSTD_TEST(cedit_cmos, UTF_CONSOLE);
|
||||
|
||||
@@ -114,7 +114,7 @@ static int expo_base(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(expo_base, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(expo_base, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check creating a scene */
|
||||
static int expo_scene(struct unit_test_state *uts)
|
||||
@@ -165,7 +165,7 @@ static int expo_scene(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(expo_scene, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(expo_scene, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check creating a scene with objects */
|
||||
static int expo_object(struct unit_test_state *uts)
|
||||
@@ -225,7 +225,7 @@ static int expo_object(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(expo_object, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(expo_object, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check setting object attributes and using themes */
|
||||
static int expo_object_attr(struct unit_test_state *uts)
|
||||
@@ -286,7 +286,7 @@ static int expo_object_attr(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(expo_object_attr, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(expo_object_attr, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/**
|
||||
* struct test_iter_priv - private data for expo-iterator test
|
||||
@@ -432,7 +432,7 @@ static int expo_object_menu(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(expo_object_menu, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(expo_object_menu, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
/* Check rendering a scene */
|
||||
static int expo_render_image(struct unit_test_state *uts)
|
||||
@@ -445,7 +445,6 @@ static int expo_render_image(struct unit_test_state *uts)
|
||||
struct expo *exp;
|
||||
int id;
|
||||
|
||||
console_record_reset_enable();
|
||||
ut_assertok(uclass_first_device_err(UCLASS_VIDEO, &dev));
|
||||
|
||||
ut_assertok(expo_new(EXPO_NAME, NULL, &exp));
|
||||
@@ -633,7 +632,7 @@ static int expo_render_image(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(expo_render_image, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(expo_render_image, UTF_DM | UTF_SCAN_FDT | UTF_CONSOLE);
|
||||
|
||||
/* Check building an expo from a devicetree description */
|
||||
static int expo_test_build(struct unit_test_state *uts)
|
||||
@@ -712,4 +711,4 @@ static int expo_test_build(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(expo_test_build, UT_TESTF_DM);
|
||||
BOOTSTD_TEST(expo_test_build, UTF_DM);
|
||||
|
||||
@@ -374,7 +374,7 @@ static int upl_test_info(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
UPL_TEST(upl_test_info, UT_TESTF_CONSOLE_REC);
|
||||
UPL_TEST(upl_test_info, UTF_CONSOLE);
|
||||
|
||||
/* Test 'upl read' and 'upl_write' commands */
|
||||
static int upl_test_read_write(struct unit_test_state *uts)
|
||||
@@ -396,7 +396,7 @@ static int upl_test_read_write(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
UPL_TEST(upl_test_read_write, UT_TESTF_CONSOLE_REC);
|
||||
UPL_TEST(upl_test_read_write, UTF_CONSOLE);
|
||||
|
||||
/* Test UPL passthrough */
|
||||
static int upl_test_info_norun(struct unit_test_state *uts)
|
||||
@@ -425,7 +425,7 @@ static int upl_test_info_norun(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
UPL_TEST(upl_test_info_norun, UT_TESTF_CONSOLE_REC | UT_TESTF_MANUAL);
|
||||
UPL_TEST(upl_test_info_norun, UTF_CONSOLE | UTF_MANUAL);
|
||||
|
||||
int do_ut_upl(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
|
||||
{
|
||||
|
||||
@@ -51,5 +51,5 @@ static int vbe_test_fixup_norun(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(vbe_test_fixup_norun, UT_TESTF_DM | UT_TESTF_SCAN_FDT |
|
||||
UT_TESTF_FLAT_TREE | UT_TESTF_MANUAL);
|
||||
BOOTSTD_TEST(vbe_test_fixup_norun, UTF_DM | UTF_SCAN_FDT | UTF_FLAT_TREE |
|
||||
UTF_MANUAL);
|
||||
|
||||
@@ -85,4 +85,4 @@ static int vbe_simple_test_base(struct unit_test_state *uts)
|
||||
|
||||
return 0;
|
||||
}
|
||||
BOOTSTD_TEST(vbe_simple_test_base, UT_TESTF_DM | UT_TESTF_SCAN_FDT);
|
||||
BOOTSTD_TEST(vbe_simple_test_base, UTF_DM | UTF_SCAN_FDT);
|
||||
|
||||
Reference in New Issue
Block a user