mirror of
https://github.com/git/git
synced 2024-08-25 02:35:46 +00:00
git p4 test: run as user "author"
The tests use author@example.com as the canonical submitter,
but he does not have an entry in the p4 users database.
This causes the generated change description to complain
that the git and p4 users disagree. The complaint message
is still valid, but isn't useful in tests. It was introduced
in 848de9c
(git-p4: warn if git authorship won't be retained,
2011-05-13).
Fix t9813 to use @example.com instead of @localhost due to
change in p4_add_user(). Move the function into the git p4
test library so author can be added at initialization time.
Signed-off-by: Pete Wyckoff <pw@padd.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
0577849d2b
commit
0055b56e10
|
@ -47,9 +47,10 @@ P4DPORT=$((10669 + ($testid - $git_p4_test_start)))
|
||||||
|
|
||||||
P4PORT=localhost:$P4DPORT
|
P4PORT=localhost:$P4DPORT
|
||||||
P4CLIENT=client
|
P4CLIENT=client
|
||||||
|
P4USER=author
|
||||||
P4EDITOR=:
|
P4EDITOR=:
|
||||||
unset P4CHARSET
|
unset P4CHARSET
|
||||||
export P4PORT P4CLIENT P4EDITOR P4CHARSET
|
export P4PORT P4CLIENT P4USER P4EDITOR P4CHARSET
|
||||||
|
|
||||||
db="$TRASH_DIRECTORY/db"
|
db="$TRASH_DIRECTORY/db"
|
||||||
cli="$TRASH_DIRECTORY/cli"
|
cli="$TRASH_DIRECTORY/cli"
|
||||||
|
@ -96,12 +97,24 @@ start_p4d() {
|
||||||
return 1
|
return 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
|
# build a p4 user so author@example.com has an entry
|
||||||
|
p4_add_user author
|
||||||
|
|
||||||
# build a client
|
# build a client
|
||||||
client_view "//depot/... //client/..." &&
|
client_view "//depot/... //client/..." &&
|
||||||
|
|
||||||
return 0
|
return 0
|
||||||
}
|
}
|
||||||
|
|
||||||
|
p4_add_user() {
|
||||||
|
name=$1 &&
|
||||||
|
p4 user -f -i <<-EOF
|
||||||
|
User: $name
|
||||||
|
Email: $name@example.com
|
||||||
|
FullName: Dr. $name
|
||||||
|
EOF
|
||||||
|
}
|
||||||
|
|
||||||
kill_p4d() {
|
kill_p4d() {
|
||||||
pid=$(cat "$pidfile")
|
pid=$(cat "$pidfile")
|
||||||
# it had better exist for the first kill
|
# it had better exist for the first kill
|
||||||
|
|
|
@ -19,16 +19,6 @@ test_expect_success 'create files' '
|
||||||
)
|
)
|
||||||
'
|
'
|
||||||
|
|
||||||
p4_add_user() {
|
|
||||||
name=$1 fullname=$2 &&
|
|
||||||
p4 user -f -i <<-EOF &&
|
|
||||||
User: $name
|
|
||||||
Email: $name@localhost
|
|
||||||
FullName: $fullname
|
|
||||||
EOF
|
|
||||||
p4 passwd -P secret $name
|
|
||||||
}
|
|
||||||
|
|
||||||
p4_grant_admin() {
|
p4_grant_admin() {
|
||||||
name=$1 &&
|
name=$1 &&
|
||||||
{
|
{
|
||||||
|
@ -51,8 +41,8 @@ make_change_by_user() {
|
||||||
|
|
||||||
# Test username support, submitting as user 'alice'
|
# Test username support, submitting as user 'alice'
|
||||||
test_expect_success 'preserve users' '
|
test_expect_success 'preserve users' '
|
||||||
p4_add_user alice Alice &&
|
p4_add_user alice &&
|
||||||
p4_add_user bob Bob &&
|
p4_add_user bob &&
|
||||||
p4_grant_admin alice &&
|
p4_grant_admin alice &&
|
||||||
git p4 clone --dest="$git" //depot &&
|
git p4 clone --dest="$git" //depot &&
|
||||||
test_when_finished cleanup_git &&
|
test_when_finished cleanup_git &&
|
||||||
|
@ -60,8 +50,8 @@ test_expect_success 'preserve users' '
|
||||||
cd "$git" &&
|
cd "$git" &&
|
||||||
echo "username: a change by alice" >>file1 &&
|
echo "username: a change by alice" >>file1 &&
|
||||||
echo "username: a change by bob" >>file2 &&
|
echo "username: a change by bob" >>file2 &&
|
||||||
git commit --author "Alice <alice@localhost>" -m "a change by alice" file1 &&
|
git commit --author "Alice <alice@example.com>" -m "a change by alice" file1 &&
|
||||||
git commit --author "Bob <bob@localhost>" -m "a change by bob" file2 &&
|
git commit --author "Bob <bob@example.com>" -m "a change by bob" file2 &&
|
||||||
git config git-p4.skipSubmitEditCheck true &&
|
git config git-p4.skipSubmitEditCheck true &&
|
||||||
P4EDITOR=touch P4USER=alice P4PASSWD=secret git p4 commit --preserve-user &&
|
P4EDITOR=touch P4USER=alice P4PASSWD=secret git p4 commit --preserve-user &&
|
||||||
p4_check_commit_author file1 alice &&
|
p4_check_commit_author file1 alice &&
|
||||||
|
@ -78,7 +68,7 @@ test_expect_success 'refuse to preserve users without perms' '
|
||||||
cd "$git" &&
|
cd "$git" &&
|
||||||
git config git-p4.skipSubmitEditCheck true &&
|
git config git-p4.skipSubmitEditCheck true &&
|
||||||
echo "username-noperms: a change by alice" >>file1 &&
|
echo "username-noperms: a change by alice" >>file1 &&
|
||||||
git commit --author "Alice <alice@localhost>" -m "perms: a change by alice" file1 &&
|
git commit --author "Alice <alice@example.com>" -m "perms: a change by alice" file1 &&
|
||||||
P4EDITOR=touch P4USER=bob P4PASSWD=secret &&
|
P4EDITOR=touch P4USER=bob P4PASSWD=secret &&
|
||||||
export P4EDITOR P4USER P4PASSWD &&
|
export P4EDITOR P4USER P4PASSWD &&
|
||||||
test_must_fail git p4 commit --preserve-user &&
|
test_must_fail git p4 commit --preserve-user &&
|
||||||
|
@ -94,9 +84,9 @@ test_expect_success 'preserve user where author is unknown to p4' '
|
||||||
cd "$git" &&
|
cd "$git" &&
|
||||||
git config git-p4.skipSubmitEditCheck true &&
|
git config git-p4.skipSubmitEditCheck true &&
|
||||||
echo "username-bob: a change by bob" >>file1 &&
|
echo "username-bob: a change by bob" >>file1 &&
|
||||||
git commit --author "Bob <bob@localhost>" -m "preserve: a change by bob" file1 &&
|
git commit --author "Bob <bob@example.com>" -m "preserve: a change by bob" file1 &&
|
||||||
echo "username-unknown: a change by charlie" >>file1 &&
|
echo "username-unknown: a change by charlie" >>file1 &&
|
||||||
git commit --author "Charlie <charlie@localhost>" -m "preserve: a change by charlie" file1 &&
|
git commit --author "Charlie <charlie@example.com>" -m "preserve: a change by charlie" file1 &&
|
||||||
P4EDITOR=touch P4USER=alice P4PASSWD=secret &&
|
P4EDITOR=touch P4USER=alice P4PASSWD=secret &&
|
||||||
export P4EDITOR P4USER P4PASSWD &&
|
export P4EDITOR P4USER P4PASSWD &&
|
||||||
test_must_fail git p4 commit --preserve-user &&
|
test_must_fail git p4 commit --preserve-user &&
|
||||||
|
@ -121,24 +111,24 @@ test_expect_success 'not preserving user with mixed authorship' '
|
||||||
(
|
(
|
||||||
cd "$git" &&
|
cd "$git" &&
|
||||||
git config git-p4.skipSubmitEditCheck true &&
|
git config git-p4.skipSubmitEditCheck true &&
|
||||||
p4_add_user derek Derek &&
|
p4_add_user derek &&
|
||||||
|
|
||||||
make_change_by_user usernamefile3 Derek derek@localhost &&
|
make_change_by_user usernamefile3 Derek derek@example.com &&
|
||||||
P4EDITOR=cat P4USER=alice P4PASSWD=secret &&
|
P4EDITOR=cat P4USER=alice P4PASSWD=secret &&
|
||||||
export P4EDITOR P4USER P4PASSWD &&
|
export P4EDITOR P4USER P4PASSWD &&
|
||||||
git p4 commit |\
|
git p4 commit |\
|
||||||
grep "git author derek@localhost does not match" &&
|
grep "git author derek@example.com does not match" &&
|
||||||
|
|
||||||
make_change_by_user usernamefile3 Charlie charlie@localhost &&
|
make_change_by_user usernamefile3 Charlie charlie@example.com &&
|
||||||
git p4 commit |\
|
git p4 commit |\
|
||||||
grep "git author charlie@localhost does not match" &&
|
grep "git author charlie@example.com does not match" &&
|
||||||
|
|
||||||
make_change_by_user usernamefile3 alice alice@localhost &&
|
make_change_by_user usernamefile3 alice alice@example.com &&
|
||||||
git p4 commit |\
|
git p4 commit |\
|
||||||
test_must_fail grep "git author.*does not match" &&
|
test_must_fail grep "git author.*does not match" &&
|
||||||
|
|
||||||
git config git-p4.skipUserNameCheck true &&
|
git config git-p4.skipUserNameCheck true &&
|
||||||
make_change_by_user usernamefile3 Charlie charlie@localhost &&
|
make_change_by_user usernamefile3 Charlie charlie@example.com &&
|
||||||
git p4 commit |\
|
git p4 commit |\
|
||||||
test_must_fail grep "git author.*does not match" &&
|
test_must_fail grep "git author.*does not match" &&
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue