mirror of
https://github.com/git/git
synced 2024-08-25 10:45:59 +00:00
test-lib-functions: drop redundant diagnostic print
`test_path_is_missing` was introduced back in2caf20c52b
("test-lib: user-friendly alternatives to test [-d|-f|-e]", 2010-08-10). It took the path that was supposed to be missing, as well as an optional "diagnosis" that would be echoed if the path was found to be alive. Commit45a2686441
("test-lib-functions: remove bug-inducing "diagnostics" helper param", 2021-02-12) dropped this diagnostic functionality from several `test_path_is_foo` helpers, but note how it tweaked the README entry on `test_path_is_missing` without actually adjusting its implementation. Commite7884b353b
("test-lib-functions: assert correct parameter count", 2021-02-12) then followed up by asserting that we get just a single argument. This history leaves us in a state where we assert that we have exactly one argument, then go on to anyway check for arguments, echoing them all. It's clear that we can simplify this code. We should also note that we run `ls -ld "$1"`, so printing the filename a second time doesn't really buy us anything. Thus, we can drop the whole `if` block as redundant. Signed-off-by: Martin Ågren <martin.agren@gmail.com> Signed-off-by: Taylor Blau <me@ttaylorr.com>
This commit is contained in:
parent
c03801e19c
commit
cc8f95c042
|
@ -921,10 +921,6 @@ test_path_is_missing () {
|
|||
then
|
||||
echo "Path exists:"
|
||||
ls -ld "$1"
|
||||
if test $# -ge 1
|
||||
then
|
||||
echo "$*"
|
||||
fi
|
||||
false
|
||||
fi
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue