mhagger@xxxxxxxxxxxx writes:
> From: Michael Haggerty <mhagger@xxxxxxxxxxxx>
>
> There is currently a bug: if passed an absolute top-level path that
> doesn't exist (e.g., "/foo") it incorrectly interprets the path as a
> relative path (e.g., returns "$(pwd)/foo"). So mark the test as
> failing.
>
> Signed-off-by: Michael Haggerty <mhagger@xxxxxxxxxxxx>
> ---
> t/t0000-basic.sh | 12 +++++++++++-
> 1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/t/t0000-basic.sh b/t/t0000-basic.sh
> index 1a51634..ad002ee 100755
> --- a/t/t0000-basic.sh
> +++ b/t/t0000-basic.sh
> @@ -458,7 +458,17 @@ test_expect_success 'real path rejects the empty string' '
> test_must_fail test-path-utils real_path ""
> '
>
> -test_expect_success SYMLINKS 'real path works as expected' '
> +test_expect_failure 'real path works on absolute paths' '
> + nopath="hopefully-absent-path" &&
> + test "/" = "$(test-path-utils real_path "/")" &&
> + test "/$nopath" = "$(test-path-utils real_path "/$nopath")" &&
You could perhaps do
sfx=0 &&
while test -e "/$nopath$sfx"
do
sfx=$(( $sfx + 1 ))
done && nopath=$nopath$sfx &&
test "/$nopath" = "$(test-path-utils real_path "/$nopath")" &&
if you really cared.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html