mirror of
https://github.com/git-for-windows/git.git
synced 2026-06-19 16:55:31 -05:00
cd352cab5b
Replace bare grep with test_grep in test assertions across the
suite, including sourced test helpers (lib-*.sh, *-tests.sh).
test_grep prints the contents of the file being searched on
failure, making debugging easier than a bare grep which fails
silently.
Only assertion-style greps are converted: grep used as a filter
in pipelines, command substitutions, conditionals, or with
redirected I/O is left as-is with a "# lint-ok" annotation.
Existing '! test_grep' calls are rewritten to 'test_grep !' so
that the diagnostic output is preserved on failure.
The conversion was generated using a grep-assertion linter
(greplint.pl, added in the following commit) to identify bare
grep calls at command position. To reproduce:
# Step 1: mark bare greps that should not be converted
sed -i '/! grep "$m" \.git\/packed-refs/s/$/ # lint-ok: file may not exist (reftable)/' \
t/t1400-update-ref.sh
sed -i '/! grep dirty file3 &&/{/lint-ok/!s/$/ # lint-ok: file may not exist after --quit/}' \
t/t3420-rebase-autostash.sh
sed -i '/grep -vf before commits\.raw/s/$/ # lint-ok: data filter/' \
t/t5326-multi-pack-bitmaps.sh
sed -i '/! grep $d shallow-client\/\.git\/shallow/s/$/ # lint-ok: file may not exist after repack/' \
t/t5537-fetch-shallow.sh
sed -i '/grep -E "^\[0-9a-f\].*|| :/s/$/ # lint-ok: data filter/' \
t/t5702-protocol-v2.sh
sed -i '/! grep gitdir squatting-clone/s/$/ # lint-ok: file may not exist after failed clone/' \
t/t7450-bad-git-dotfiles.sh
# Step 2: reorder pre-existing '! test_grep' to 'test_grep !'
# (must come before steps 3-4 so greplint does not see them)
sed -i 's/! test_grep/test_grep !/' t/t0031-lockfile-pid.sh
sed -i 's/! test_grep/test_grep !/' t/t5300-pack-object.sh
sed -i 's/! test_grep/test_grep !/' t/t5319-multi-pack-index.sh
# Step 3: convert '! grep' -> 'test_grep !'
perl t/greplint.pl t/*.sh 2>&1 | cut -d: -f1,2 |
while IFS=: read f l; do
sed -i "${l}s/! *grep/test_grep !/" "$f"
done
# Step 4: convert remaining 'grep' -> 'test_grep'
perl t/greplint.pl t/*.sh 2>&1 | cut -d: -f1,2 |
while IFS=: read f l; do
sed -i "${l}s/grep/test_grep/" "$f"
done
To verify, run: make -C t test-greplint
Signed-off-by: Michael Montalbo <mmontalbo@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
434 lines
10 KiB
Bash
Executable File
434 lines
10 KiB
Bash
Executable File
#!/bin/sh
|
|
|
|
test_description='git p4 tests'
|
|
|
|
GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
|
|
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
|
|
|
|
. ./lib-git-p4.sh
|
|
|
|
test_expect_success 'start p4d' '
|
|
start_p4d
|
|
'
|
|
|
|
test_expect_success 'add p4 files' '
|
|
(
|
|
cd "$cli" &&
|
|
echo file1 >file1 &&
|
|
p4 add file1 &&
|
|
p4 submit -d "file1" &&
|
|
echo file2 >file2 &&
|
|
p4 add file2 &&
|
|
p4 submit -d "file2"
|
|
)
|
|
'
|
|
|
|
test_expect_success 'basic git p4 clone' '
|
|
git p4 clone --dest="$git" //depot &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
git log --oneline >lines &&
|
|
test_line_count = 1 lines
|
|
)
|
|
'
|
|
|
|
test_expect_success 'depot typo error' '
|
|
test_must_fail git p4 clone --dest="$git" /depot 2>errs &&
|
|
test_grep "Depot paths must start with" errs
|
|
'
|
|
|
|
test_expect_success 'git p4 clone @all' '
|
|
git p4 clone --dest="$git" //depot@all &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
git log --oneline >lines &&
|
|
test_line_count = 2 lines
|
|
)
|
|
'
|
|
|
|
test_expect_success 'git p4 sync uninitialized repo' '
|
|
test_create_repo "$git" &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_must_fail git p4 sync 2>errs &&
|
|
test_grep "Perhaps you never did" errs
|
|
)
|
|
'
|
|
|
|
#
|
|
# Create a git repo by hand. Add a commit so that HEAD is valid.
|
|
# Test imports a new p4 repository into a new git branch.
|
|
#
|
|
test_expect_success 'git p4 sync new branch' '
|
|
test_create_repo "$git" &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_commit head &&
|
|
git p4 sync --branch=refs/remotes/p4/depot //depot@all &&
|
|
git log --oneline p4/depot >lines &&
|
|
test_line_count = 2 lines
|
|
)
|
|
'
|
|
|
|
#
|
|
# Setup as before, and then explicitly sync imported branch, using a
|
|
# different ref format.
|
|
#
|
|
test_expect_success 'git p4 sync existing branch without changes' '
|
|
test_create_repo "$git" &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_commit head &&
|
|
git p4 sync --branch=depot //depot@all &&
|
|
git p4 sync --branch=refs/remotes/p4/depot >out &&
|
|
test_grep "No changes to import!" out
|
|
)
|
|
'
|
|
|
|
#
|
|
# Same as before, relative branch name.
|
|
#
|
|
test_expect_success 'git p4 sync existing branch with relative name' '
|
|
test_create_repo "$git" &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_commit head &&
|
|
git p4 sync --branch=branch1 //depot@all &&
|
|
git p4 sync --branch=p4/branch1 >out &&
|
|
test_grep "No changes to import!" out
|
|
)
|
|
'
|
|
|
|
#
|
|
# Same as before, with a nested branch path, referenced different ways.
|
|
#
|
|
test_expect_success 'git p4 sync existing branch with nested path' '
|
|
test_create_repo "$git" &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_commit head &&
|
|
git p4 sync --branch=p4/some/path //depot@all &&
|
|
git p4 sync --branch=some/path >out &&
|
|
test_grep "No changes to import!" out
|
|
)
|
|
'
|
|
|
|
#
|
|
# Same as before, with a full ref path outside the p4/* namespace.
|
|
#
|
|
test_expect_success 'git p4 sync branch explicit ref without p4 in path' '
|
|
test_create_repo "$git" &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_commit head &&
|
|
git p4 sync --branch=refs/remotes/someremote/depot //depot@all &&
|
|
git p4 sync --branch=refs/remotes/someremote/depot >out &&
|
|
test_grep "No changes to import!" out
|
|
)
|
|
'
|
|
|
|
test_expect_success 'git p4 sync nonexistent ref' '
|
|
test_create_repo "$git" &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_commit head &&
|
|
git p4 sync --branch=depot //depot@all &&
|
|
test_must_fail git p4 sync --branch=depot2 2>errs &&
|
|
test_grep "Perhaps you never did" errs
|
|
)
|
|
'
|
|
|
|
test_expect_success 'git p4 sync existing non-p4-imported ref' '
|
|
test_create_repo "$git" &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_commit head &&
|
|
git p4 sync --branch=depot //depot@all &&
|
|
test_must_fail git p4 sync --branch=refs/heads/master 2>errs &&
|
|
test_grep "Perhaps you never did" errs
|
|
)
|
|
'
|
|
|
|
test_expect_success 'clone two dirs' '
|
|
(
|
|
cd "$cli" &&
|
|
mkdir sub1 sub2 &&
|
|
echo sub1/f1 >sub1/f1 &&
|
|
echo sub2/f2 >sub2/f2 &&
|
|
p4 add sub1/f1 &&
|
|
p4 submit -d "sub1/f1" &&
|
|
p4 add sub2/f2 &&
|
|
p4 submit -d "sub2/f2"
|
|
) &&
|
|
git p4 clone --dest="$git" //depot/sub1 //depot/sub2 &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
git ls-files >lines &&
|
|
test_line_count = 2 lines &&
|
|
git log --oneline p4/master >lines &&
|
|
test_line_count = 1 lines
|
|
)
|
|
'
|
|
|
|
test_expect_success 'clone two dirs, @all' '
|
|
(
|
|
cd "$cli" &&
|
|
echo sub1/f3 >sub1/f3 &&
|
|
p4 add sub1/f3 &&
|
|
p4 submit -d "sub1/f3"
|
|
) &&
|
|
git p4 clone --dest="$git" //depot/sub1@all //depot/sub2@all &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
git ls-files >lines &&
|
|
test_line_count = 3 lines &&
|
|
git log --oneline p4/master >lines &&
|
|
test_line_count = 3 lines
|
|
)
|
|
'
|
|
|
|
test_expect_success 'clone two dirs, @all, conflicting files' '
|
|
(
|
|
cd "$cli" &&
|
|
echo sub2/f3 >sub2/f3 &&
|
|
p4 add sub2/f3 &&
|
|
p4 submit -d "sub2/f3"
|
|
) &&
|
|
git p4 clone --dest="$git" //depot/sub1@all //depot/sub2@all &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
git ls-files >lines &&
|
|
test_line_count = 3 lines &&
|
|
git log --oneline p4/master >lines &&
|
|
test_line_count = 4 lines &&
|
|
echo sub2/f3 >expected &&
|
|
test_cmp expected f3
|
|
)
|
|
'
|
|
|
|
test_expect_success 'clone two dirs, each edited by submit, single git commit' '
|
|
(
|
|
cd "$cli" &&
|
|
echo sub1/f4 >sub1/f4 &&
|
|
p4 add sub1/f4 &&
|
|
echo sub2/f4 >sub2/f4 &&
|
|
p4 add sub2/f4 &&
|
|
p4 submit -d "sub1/f4 and sub2/f4"
|
|
) &&
|
|
git p4 clone --dest="$git" //depot/sub1@all //depot/sub2@all &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
git ls-files >lines &&
|
|
test_line_count = 4 lines &&
|
|
git log --oneline p4/master >lines &&
|
|
test_line_count = 5 lines
|
|
)
|
|
'
|
|
|
|
revision_ranges="2000/01/01,#head \
|
|
1,2080/01/01 \
|
|
2000/01/01,2080/01/01 \
|
|
2000/01/01,1000 \
|
|
1,1000"
|
|
|
|
test_expect_success 'clone using non-numeric revision ranges' '
|
|
test_when_finished cleanup_git &&
|
|
for r in $revision_ranges
|
|
do
|
|
rm -fr "$git" &&
|
|
test ! -d "$git" &&
|
|
git p4 clone --dest="$git" //depot@$r &&
|
|
(
|
|
cd "$git" &&
|
|
git ls-files >lines &&
|
|
test_line_count = 8 lines
|
|
) || return 1
|
|
done
|
|
'
|
|
|
|
test_expect_success 'clone with date range, excluding some changes' '
|
|
test_when_finished cleanup_git &&
|
|
before=$(date +%Y/%m/%d:%H:%M:%S) &&
|
|
sleep 2 &&
|
|
(
|
|
cd "$cli" &&
|
|
:>date_range_test &&
|
|
p4 add date_range_test &&
|
|
p4 submit -d "Adding file"
|
|
) &&
|
|
git p4 clone --dest="$git" //depot@1,$before &&
|
|
(
|
|
cd "$git" &&
|
|
test_path_is_missing date_range_test
|
|
)
|
|
'
|
|
|
|
test_expect_success 'exit when p4 fails to produce marshaled output' '
|
|
mkdir badp4dir &&
|
|
test_when_finished "rm badp4dir/p4 && rmdir badp4dir" &&
|
|
cat >badp4dir/p4 <<-EOF &&
|
|
#!$SHELL_PATH
|
|
exit 1
|
|
EOF
|
|
chmod 755 badp4dir/p4 &&
|
|
(
|
|
PATH="$TRASH_DIRECTORY/badp4dir:$PATH" &&
|
|
export PATH &&
|
|
test_expect_code 1 git p4 clone --dest="$git" //depot >errs 2>&1
|
|
) &&
|
|
test_grep ! Traceback errs
|
|
'
|
|
|
|
# Hide a file from p4d, make sure we catch its complaint. This won't fail in
|
|
# p4 changes, files, or describe; just in p4 print. If P4CLIENT is unset, the
|
|
# message will include "Librarian checkout".
|
|
test_expect_success 'exit gracefully for p4 server errors' '
|
|
# Note that newer Perforce versions started to store files
|
|
# compressed in directories. The case statement handles both
|
|
# old and new layout.
|
|
case "$(echo "$db"/depot/file1*)" in
|
|
*,v)
|
|
test_when_finished "mv \"$db\"/depot/file1,v,hidden \"$db\"/depot/file1,v" &&
|
|
mv "$db"/depot/file1,v "$db"/depot/file1,v,hidden;;
|
|
*,d)
|
|
path="$(echo "$db"/depot/file1,d/*.gz)" &&
|
|
test_when_finished "mv \"$path\",hidden \"$path\"" &&
|
|
mv "$path" "$path",hidden;;
|
|
*)
|
|
BUG "unhandled p4d layout";;
|
|
esac &&
|
|
test_when_finished cleanup_git &&
|
|
test_expect_code 1 git p4 clone --dest="$git" //depot@1 >out 2>err &&
|
|
test_grep "Error from p4 print" err
|
|
'
|
|
|
|
test_expect_success 'clone --bare should make a bare repository' '
|
|
rm -rf "$git" &&
|
|
git p4 clone --dest="$git" --bare //depot &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
test_path_is_missing .git &&
|
|
git config --get --bool core.bare true &&
|
|
git rev-parse --verify refs/remotes/p4/master &&
|
|
git rev-parse --verify refs/remotes/p4/HEAD &&
|
|
git rev-parse --verify refs/heads/main &&
|
|
git rev-parse --verify HEAD
|
|
)
|
|
'
|
|
|
|
# Sleep a bit so that the top-most p4 change did not happen "now". Then
|
|
# import the repo and make sure that the initial import has the same time
|
|
# as the top-most change.
|
|
test_expect_success 'initial import time from top change time' '
|
|
p4change=$(p4 -G changes -m 1 //depot/... | marshal_dump change) &&
|
|
p4time=$(p4 -G changes -m 1 //depot/... | marshal_dump time) &&
|
|
sleep 3 &&
|
|
git p4 clone --dest="$git" //depot &&
|
|
test_when_finished cleanup_git &&
|
|
(
|
|
cd "$git" &&
|
|
gittime=$(git show -s --pretty=format:%at HEAD) &&
|
|
echo $p4time $gittime &&
|
|
test $p4time = $gittime
|
|
)
|
|
'
|
|
|
|
test_expect_success 'unresolvable host in P4PORT should display error' '
|
|
test_when_finished cleanup_git &&
|
|
git p4 clone --dest="$git" //depot &&
|
|
(
|
|
cd "$git" &&
|
|
P4PORT=nosuchhost:65537 &&
|
|
export P4PORT &&
|
|
test_expect_code 1 git p4 sync >out 2>err &&
|
|
test_grep "connect to nosuchhost" err
|
|
)
|
|
'
|
|
|
|
# Test following scenarios:
|
|
# - Without ".git/hooks/p4-pre-submit" , submit should continue
|
|
# - With the hook returning 0, submit should continue
|
|
# - With the hook returning 1, submit should abort
|
|
test_expect_success 'run hook p4-pre-submit before submit' '
|
|
test_when_finished cleanup_git &&
|
|
git p4 clone --dest="$git" //depot &&
|
|
(
|
|
cd "$git" &&
|
|
echo "hello world" >hello.txt &&
|
|
git add hello.txt &&
|
|
git commit -m "add hello.txt" &&
|
|
git config git-p4.skipSubmitEdit true &&
|
|
git p4 submit --dry-run >out &&
|
|
test_grep "Would apply" out
|
|
) &&
|
|
test_hook -C "$git" p4-pre-submit <<-\EOF &&
|
|
exit 0
|
|
EOF
|
|
(
|
|
cd "$git" &&
|
|
git p4 submit --dry-run >out &&
|
|
test_grep "Would apply" out
|
|
) &&
|
|
test_hook -C "$git" --clobber p4-pre-submit <<-\EOF &&
|
|
exit 1
|
|
EOF
|
|
(
|
|
cd "$git" &&
|
|
test_must_fail git p4 submit --dry-run >errs 2>&1 &&
|
|
test_grep ! "Would apply" errs
|
|
)
|
|
'
|
|
|
|
test_expect_success 'submit from detached head' '
|
|
test_when_finished cleanup_git &&
|
|
git p4 clone --dest="$git" //depot &&
|
|
(
|
|
cd "$git" &&
|
|
git checkout p4/master &&
|
|
>detached_head_test &&
|
|
git add detached_head_test &&
|
|
git commit -m "add detached_head" &&
|
|
git config git-p4.skipSubmitEdit true &&
|
|
git p4 submit &&
|
|
git p4 rebase &&
|
|
git log p4/master | grep detached_head
|
|
)
|
|
'
|
|
|
|
test_expect_success 'submit from worktree' '
|
|
test_when_finished cleanup_git &&
|
|
git p4 clone --dest="$git" //depot &&
|
|
(
|
|
cd "$git" &&
|
|
git worktree add ../worktree-test
|
|
) &&
|
|
(
|
|
cd "$git/../worktree-test" &&
|
|
test_commit "worktree-commit" &&
|
|
git config git-p4.skipSubmitEdit true &&
|
|
git p4 submit
|
|
) &&
|
|
(
|
|
cd "$cli" &&
|
|
p4 sync &&
|
|
test_path_is_file worktree-commit.t
|
|
)
|
|
'
|
|
|
|
test_done
|