Skip to content

Commit

Permalink
Merge branch 'bc/hash-independent-tests-part-4'
Browse files Browse the repository at this point in the history
Test fix.

* bc/hash-independent-tests-part-4:
  t0000: reword comments for "local" test
  t: decrease nesting in test_oid_to_path
  • Loading branch information
gitster committed Aug 9, 2019
2 parents d2422a9 + 7f0b590 commit fcdd9cc
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 9 deletions.
14 changes: 6 additions & 8 deletions t/t0000-basic.sh
Original file line number Diff line number Diff line change
Expand Up @@ -25,16 +25,14 @@ try_local_x () {
echo "$x"
}

# This test is an experiment to check whether any Git users are using
# Shells that don't support the "local" keyword. "local" is not
# Check whether the shell supports the "local" keyword. "local" is not
# POSIX-standard, but it is very widely supported by POSIX-compliant
# shells, and if it doesn't cause problems for people, we would like
# to be able to use it in Git code.
# shells, and we rely on it within Git's test framework.
#
# For now, this is the only test that requires "local". If your shell
# fails this test, you can ignore the failure, but please report the
# problem to the Git mailing list <[email protected]>, as it might
# convince us to continue avoiding the use of "local".
# If your shell fails this test, the results of other tests may be
# unreliable. You may wish to report the problem to the Git mailing
# list <[email protected]>, as it could cause us to reconsider
# relying on "local".
test_expect_success 'verify that the running shell supports "local"' '
x="notlocal" &&
echo "local" >expected1 &&
Expand Down
3 changes: 2 additions & 1 deletion t/test-lib-functions.sh
Original file line number Diff line number Diff line change
Expand Up @@ -1433,7 +1433,8 @@ test_oid () {
# Insert a slash into an object ID so it can be used to reference a location
# under ".git/objects". For example, "deadbeef..." becomes "de/adbeef..".
test_oid_to_path () {
echo "${1%${1#??}}/${1#??}"
local basename=${1#??}
echo "${1%$basename}/$basename"
}

# Choose a port number based on the test script's number and store it in
Expand Down

0 comments on commit fcdd9cc

Please sign in to comment.