diff options
| author | Stephen Warren <swarren@wwwdotorg.org> | 2014-03-01 22:18:00 -0700 |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2014-03-07 10:59:06 -0500 |
| commit | cfd13e8dda9d2db3f6bdf32d623aecf10ee1ba50 (patch) | |
| tree | 407efdba8744978388eacbb09a2bdf0b3272f6f6 /test | |
| parent | fe9ca3d3287185e388de55904420cc7915e4a3b1 (diff) | |
| download | blackbird-obmc-uboot-cfd13e8dda9d2db3f6bdf32d623aecf10ee1ba50.tar.gz blackbird-obmc-uboot-cfd13e8dda9d2db3f6bdf32d623aecf10ee1ba50.zip | |
unit-test: make "test -e" test independent of $CWD
The unit-test for hush's "test -e" currently relies upon being run in
the U-Boot build directory, because it tests for the existence of a file
that exists in that directory.
Fix this by explicitly creating the file we use for the existence test,
and deleting it afterwards so that multiple successive unit-test
invocations succeed. This required adding an os.c function to erase
files.
Reported-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Stephen Warren <swarren@wwwdotorg.org>
Diffstat (limited to 'test')
| -rw-r--r-- | test/command_ut.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/test/command_ut.c b/test/command_ut.c index b6b6976616..aaa1ee25d0 100644 --- a/test/command_ut.c +++ b/test/command_ut.c @@ -7,6 +7,9 @@ #define DEBUG #include <common.h> +#ifdef CONFIG_SANDBOX +#include <os.h> +#endif static const char test_cmd[] = "setenv list 1\n setenv list ${list}2; " "setenv list ${list}3\0" @@ -156,11 +159,12 @@ static int do_ut_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) setenv("ut_var_test", NULL); #ifdef CONFIG_SANDBOX - /* - * File existence - * This assume U-Boot sandbox is run from the U-Boot build directory - */ - HUSH_TEST(e, "-e host - u-boot", y); + /* File existence */ + HUSH_TEST(e, "-e host - creating_this_file_breaks_uboot_unit_test", n); + run_command("sb save host - creating_this_file_breaks_uboot_unit_test 0 1", 0); + HUSH_TEST(e, "-e host - creating_this_file_breaks_uboot_unit_test", y); + /* Perhaps this could be replaced by an "rm" shell command one day */ + assert(!os_unlink("creating_this_file_breaks_uboot_unit_test")); HUSH_TEST(e, "-e host - creating_this_file_breaks_uboot_unit_test", n); #endif #endif |
