diff options
author | Junio C Hamano <gitster@pobox.com> | 2008-08-08 02:26:28 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2008-08-17 00:41:52 -0700 |
commit | bfdbee98109c5ad2dbbc392e7eed1ae688acc039 (patch) | |
tree | cc430b09c2901ce38ac638e9e785aabb88e2e500 /generate-cmdlist.sh | |
parent | git-p4: chdir now properly sets PWD environment variable in msysGit (diff) | |
download | tgif-bfdbee98109c5ad2dbbc392e7eed1ae688acc039.tar.xz |
tests: use $TEST_DIRECTORY to refer to the t/ directory
Many test scripts assumed that they will start in a 'trash' subdirectory
that is a single level down from the t/ directory, and referred to their
test vector files by asking for files like "../t9999/expect". This will
break if we move the 'trash' subdirectory elsewhere.
To solve this, we earlier introduced "$TEST_DIRECTORY" so that they can
refer to t/ directory reliably. This finally makes all the tests use
it to refer to the outside environment.
With this patch, and a one-liner not included here (because it would
contradict with what Dscho really wants to do):
| diff --git a/t/test-lib.sh b/t/test-lib.sh
| index 70ea7e0..60e69e4 100644
| --- a/t/test-lib.sh
| +++ b/t/test-lib.sh
| @@ -485,7 +485,7 @@ fi
| . ../GIT-BUILD-OPTIONS
|
| # Test repository
| -test="trash directory"
| +test="trash directory/another level/yet another"
| rm -fr "$test" || {
| trap - exit
| echo >&5 "FATAL: Cannot prepare test area"
all the tests still pass, but we would want extra sets of eyeballs on this
type of change to really make sure.
[jc: with help from Stephan Beyer on http-push tests I do not run myself;
credits for locating silly quoting errors go to Olivier Marin.]
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'generate-cmdlist.sh')
0 files changed, 0 insertions, 0 deletions