mirror of
https://github.com/brain-hackers/u-boot-brain
synced 2024-09-30 08:30:50 +09:00
test: Run driver-model tests using ut_run_list()
Use this function instead of implementing it separately for driver model. Make ut_run_tests() private since it is only used in test-main.c Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
parent
fe806861a9
commit
f97f85e661
@ -381,48 +381,6 @@ struct unit_test_state *test_get_state(void);
|
|||||||
*/
|
*/
|
||||||
void test_set_state(struct unit_test_state *uts);
|
void test_set_state(struct unit_test_state *uts);
|
||||||
|
|
||||||
/**
|
|
||||||
* ut_run_test_live_flat() - Run a test with both live and flat tree
|
|
||||||
*
|
|
||||||
* This calls ut_run_test() with livetree enabled, which is the standard setup
|
|
||||||
* for runnig tests. Then, for driver model test, it calls it again with
|
|
||||||
* livetree disabled. This allows checking of flattree being used when OF_LIVE
|
|
||||||
* is enabled, as is the case in U-Boot proper before relocation, as well as in
|
|
||||||
* SPL.
|
|
||||||
*
|
|
||||||
* @uts: Test state to update. The caller should ensure that this is zeroed for
|
|
||||||
* the first call to this function. On exit, @uts->fail_count is
|
|
||||||
* incremented by the number of failures (0, one hopes)
|
|
||||||
* @test: Test to run
|
|
||||||
* @name: Name of test, possibly skipping a prefix that should not be displayed
|
|
||||||
* @return 0 if all tests passed, -EAGAIN if the test should be skipped, -1 if
|
|
||||||
* any failed
|
|
||||||
*/
|
|
||||||
int ut_run_test_live_flat(struct unit_test_state *uts, struct unit_test *test,
|
|
||||||
const char *name);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* ut_run_tests() - Run a set of tests
|
|
||||||
*
|
|
||||||
* This runs the tests, handling any preparation and clean-up needed. It prints
|
|
||||||
* the name of each test before running it.
|
|
||||||
*
|
|
||||||
* @uts: Test state to update. The caller should ensure that this is zeroed for
|
|
||||||
* the first call to this function. On exit, @uts->fail_count is
|
|
||||||
* incremented by the number of failures (0, one hopes)
|
|
||||||
* @prefix: String prefix for the tests. Any tests that have this prefix will be
|
|
||||||
* printed without the prefix, so that it is easier to see the unique part
|
|
||||||
* of the test name. If NULL, no prefix processing is done
|
|
||||||
* @tests: List of tests to run
|
|
||||||
* @count: Number of tests to run
|
|
||||||
* @select_name: Name of a single test to run (from the list provided). If NULL
|
|
||||||
* then all tests are run
|
|
||||||
* @return 0 if all tests passed, -ENOENT if test @select_name was not found,
|
|
||||||
* -EBADF if any failed
|
|
||||||
*/
|
|
||||||
int ut_run_tests(struct unit_test_state *uts, const char *prefix,
|
|
||||||
struct unit_test *tests, int count, const char *select_name);
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* ut_run_tests() - Run a set of tests
|
* ut_run_tests() - Run a set of tests
|
||||||
*
|
*
|
||||||
|
@ -20,31 +20,14 @@
|
|||||||
|
|
||||||
DECLARE_GLOBAL_DATA_PTR;
|
DECLARE_GLOBAL_DATA_PTR;
|
||||||
|
|
||||||
static bool test_matches(const char *test_name, const char *find_name)
|
struct unit_test_state global_dm_test_state;
|
||||||
{
|
|
||||||
if (!find_name)
|
|
||||||
return true;
|
|
||||||
|
|
||||||
if (!strcmp(test_name, find_name))
|
|
||||||
return true;
|
|
||||||
|
|
||||||
/* All tests have this prefix */
|
|
||||||
if (!strncmp(test_name, "dm_test_", 8))
|
|
||||||
test_name += 8;
|
|
||||||
|
|
||||||
if (!strcmp(test_name, find_name))
|
|
||||||
return true;
|
|
||||||
|
|
||||||
return false;
|
|
||||||
}
|
|
||||||
|
|
||||||
int dm_test_run(const char *test_name)
|
int dm_test_run(const char *test_name)
|
||||||
{
|
{
|
||||||
struct unit_test *tests = ll_entry_start(struct unit_test, dm_test);
|
struct unit_test *tests = ll_entry_start(struct unit_test, dm_test);
|
||||||
const int n_ents = ll_entry_count(struct unit_test, dm_test);
|
const int n_ents = ll_entry_count(struct unit_test, dm_test);
|
||||||
struct unit_test_state uts_s = { .fail_count = 0 }, *uts = &uts_s;
|
struct unit_test_state uts_s = { .fail_count = 0 }, *uts = &uts_s;
|
||||||
struct unit_test *test;
|
struct device_node *of_root;
|
||||||
int found;
|
|
||||||
|
|
||||||
uts->fail_count = 0;
|
uts->fail_count = 0;
|
||||||
|
|
||||||
@ -60,29 +43,11 @@ int dm_test_run(const char *test_name)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!test_name)
|
of_root = gd_of_root();
|
||||||
printf("Running %d driver model tests\n", n_ents);
|
ut_run_list("driver model", "dm_test_", tests, n_ents, test_name);
|
||||||
else
|
|
||||||
|
|
||||||
found = 0;
|
|
||||||
uts->of_root = gd_of_root();
|
|
||||||
for (test = tests; test < tests + n_ents; test++) {
|
|
||||||
const char *name = test->name;
|
|
||||||
|
|
||||||
if (!test_matches(name, test_name))
|
|
||||||
continue;
|
|
||||||
|
|
||||||
ut_assertok(ut_run_test_live_flat(uts, test, test->name));
|
|
||||||
found++;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (test_name && !found)
|
|
||||||
printf("Test '%s' not found\n", test_name);
|
|
||||||
else
|
|
||||||
printf("Failures: %d\n", uts->fail_count);
|
|
||||||
|
|
||||||
/* Put everything back to normal so that sandbox works as expected */
|
/* Put everything back to normal so that sandbox works as expected */
|
||||||
gd_set_of_root(uts->of_root);
|
gd_set_of_root(of_root);
|
||||||
gd->dm_root = NULL;
|
gd->dm_root = NULL;
|
||||||
ut_assertok(dm_init(CONFIG_IS_ENABLED(OF_LIVE)));
|
ut_assertok(dm_init(CONFIG_IS_ENABLED(OF_LIVE)));
|
||||||
dm_scan_plat(false);
|
dm_scan_plat(false);
|
||||||
|
@ -111,6 +111,38 @@ static bool ut_test_run_on_flattree(struct unit_test *test)
|
|||||||
return !strstr(fname, "video") || strstr(test->name, "video_base");
|
return !strstr(fname, "video") || strstr(test->name, "video_base");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* test_matches() - Check if a test should be run
|
||||||
|
*
|
||||||
|
* This checks if the a test should be run. In the normal case of running all
|
||||||
|
* tests, @select_name is NULL.
|
||||||
|
*
|
||||||
|
* @prefix: String prefix for the tests. Any tests that have this prefix will be
|
||||||
|
* printed without the prefix, so that it is easier to see the unique part
|
||||||
|
* of the test name. If NULL, no prefix processing is done
|
||||||
|
* @test_name: Name of current test
|
||||||
|
* @select_name: Name of test to run (or NULL for all)
|
||||||
|
* @return true to run this test, false to skip it
|
||||||
|
*/
|
||||||
|
static bool test_matches(const char *prefix, const char *test_name,
|
||||||
|
const char *select_name)
|
||||||
|
{
|
||||||
|
if (!select_name)
|
||||||
|
return true;
|
||||||
|
|
||||||
|
if (!strcmp(test_name, select_name))
|
||||||
|
return true;
|
||||||
|
|
||||||
|
/* All tests have this prefix */
|
||||||
|
if (prefix && !strncmp(test_name, prefix, strlen(prefix)))
|
||||||
|
test_name += strlen(prefix);
|
||||||
|
|
||||||
|
if (!strcmp(test_name, select_name))
|
||||||
|
return true;
|
||||||
|
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* test_pre_run() - Handle any preparation needed to run a test
|
* test_pre_run() - Handle any preparation needed to run a test
|
||||||
*
|
*
|
||||||
@ -213,8 +245,25 @@ static int ut_run_test(struct unit_test_state *uts, struct unit_test *test,
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
int ut_run_test_live_flat(struct unit_test_state *uts, struct unit_test *test,
|
/**
|
||||||
const char *name)
|
* ut_run_test_live_flat() - Run a test with both live and flat tree
|
||||||
|
*
|
||||||
|
* This calls ut_run_test() with livetree enabled, which is the standard setup
|
||||||
|
* for runnig tests. Then, for driver model test, it calls it again with
|
||||||
|
* livetree disabled. This allows checking of flattree being used when OF_LIVE
|
||||||
|
* is enabled, as is the case in U-Boot proper before relocation, as well as in
|
||||||
|
* SPL.
|
||||||
|
*
|
||||||
|
* @uts: Test state to update. The caller should ensure that this is zeroed for
|
||||||
|
* the first call to this function. On exit, @uts->fail_count is
|
||||||
|
* incremented by the number of failures (0, one hopes)
|
||||||
|
* @test: Test to run
|
||||||
|
* @name: Name of test, possibly skipping a prefix that should not be displayed
|
||||||
|
* @return 0 if all tests passed, -EAGAIN if the test should be skipped, -1 if
|
||||||
|
* any failed
|
||||||
|
*/
|
||||||
|
static int ut_run_test_live_flat(struct unit_test_state *uts,
|
||||||
|
struct unit_test *test, const char *name)
|
||||||
{
|
{
|
||||||
int runs;
|
int runs;
|
||||||
|
|
||||||
@ -242,24 +291,39 @@ int ut_run_test_live_flat(struct unit_test_state *uts, struct unit_test *test,
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
int ut_run_tests(struct unit_test_state *uts, const char *prefix,
|
/**
|
||||||
struct unit_test *tests, int count, const char *select_name)
|
* ut_run_tests() - Run a set of tests
|
||||||
|
*
|
||||||
|
* This runs the tests, handling any preparation and clean-up needed. It prints
|
||||||
|
* the name of each test before running it.
|
||||||
|
*
|
||||||
|
* @uts: Test state to update. The caller should ensure that this is zeroed for
|
||||||
|
* the first call to this function. On exit, @uts->fail_count is
|
||||||
|
* incremented by the number of failures (0, one hopes)
|
||||||
|
* @prefix: String prefix for the tests. Any tests that have this prefix will be
|
||||||
|
* printed without the prefix, so that it is easier to see the unique part
|
||||||
|
* of the test name. If NULL, no prefix processing is done
|
||||||
|
* @tests: List of tests to run
|
||||||
|
* @count: Number of tests to run
|
||||||
|
* @select_name: Name of a single test to run (from the list provided). If NULL
|
||||||
|
* then all tests are run
|
||||||
|
* @return 0 if all tests passed, -ENOENT if test @select_name was not found,
|
||||||
|
* -EBADF if any failed
|
||||||
|
*/
|
||||||
|
static int ut_run_tests(struct unit_test_state *uts, const char *prefix,
|
||||||
|
struct unit_test *tests, int count,
|
||||||
|
const char *select_name)
|
||||||
{
|
{
|
||||||
struct unit_test *test;
|
struct unit_test *test;
|
||||||
int prefix_len = prefix ? strlen(prefix) : 0;
|
|
||||||
int found = 0;
|
int found = 0;
|
||||||
|
|
||||||
for (test = tests; test < tests + count; test++) {
|
for (test = tests; test < tests + count; test++) {
|
||||||
const char *test_name = test->name;
|
const char *test_name = test->name;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
/* Remove the prefix */
|
if (!test_matches(prefix, test_name, select_name))
|
||||||
if (prefix && !strncmp(test_name, prefix, prefix_len))
|
|
||||||
test_name += prefix_len;
|
|
||||||
|
|
||||||
if (select_name && strcmp(select_name, test_name))
|
|
||||||
continue;
|
continue;
|
||||||
ret = ut_run_test_live_flat(uts, test, test_name);
|
ret = ut_run_test_live_flat(uts, test, select_name);
|
||||||
found++;
|
found++;
|
||||||
if (ret == -EAGAIN)
|
if (ret == -EAGAIN)
|
||||||
continue;
|
continue;
|
||||||
@ -281,6 +345,7 @@ int ut_run_list(const char *category, const char *prefix,
|
|||||||
if (!select_name)
|
if (!select_name)
|
||||||
printf("Running %d %s tests\n", count, category);
|
printf("Running %d %s tests\n", count, category);
|
||||||
|
|
||||||
|
uts.of_root = gd_of_root();
|
||||||
ret = ut_run_tests(&uts, prefix, tests, count, select_name);
|
ret = ut_run_tests(&uts, prefix, tests, count, select_name);
|
||||||
|
|
||||||
if (ret == -ENOENT)
|
if (ret == -ENOENT)
|
||||||
|
Loading…
Reference in New Issue
Block a user