summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorStephen Warren <swarren@wwwdotorg.org>2014-03-01 22:18:00 -0700
committerTom Rini <trini@ti.com>2014-03-07 10:59:06 -0500
commitcfd13e8dda9d2db3f6bdf32d623aecf10ee1ba50 (patch)
tree407efdba8744978388eacbb09a2bdf0b3272f6f6 /test
parentfe9ca3d3287185e388de55904420cc7915e4a3b1 (diff)
downloadu-boot-imx-cfd13e8dda9d2db3f6bdf32d623aecf10ee1ba50.zip
u-boot-imx-cfd13e8dda9d2db3f6bdf32d623aecf10ee1ba50.tar.gz
u-boot-imx-cfd13e8dda9d2db3f6bdf32d623aecf10ee1ba50.tar.bz2
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.c14
1 files changed, 9 insertions, 5 deletions
diff --git a/test/command_ut.c b/test/command_ut.c
index b6b6976..aaa1ee2 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