Skip to content

Commit 9ecfd98

Browse files
committed
Merge branch 'sg/cvs-tests-with-x'
Allow running a couple of tests with "sh -x". * sg/cvs-tests-with-x: t9402-git-cvsserver-refs: don't check the stderr of a subshell t9400-git-cvsserver-server: don't rely on the output of 'test_cmp'
2 parents a8ba07c + c20bf94 commit 9ecfd98

File tree

2 files changed

+7
-9
lines changed

2 files changed

+7
-9
lines changed

t/t9400-git-cvsserver-server.sh

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -447,12 +447,10 @@ test_expect_success 'cvs update (-p)' '
447447
git push gitcvs.git >/dev/null &&
448448
cd cvswork &&
449449
GIT_CONFIG="$git_config" cvs update &&
450-
rm -f failures &&
451450
for i in merge no-lf empty really-empty; do
452-
GIT_CONFIG="$git_config" cvs update -p "$i" >$i.out
453-
test_cmp $i.out ../$i >>failures 2>&1
454-
done &&
455-
test -z "$(cat failures)"
451+
GIT_CONFIG="$git_config" cvs update -p "$i" >$i.out &&
452+
test_cmp $i.out ../$i || return 1
453+
done
456454
'
457455

458456
cd "$WORKDIR"

t/t9402-git-cvsserver-refs.sh

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -455,20 +455,20 @@ test_expect_success 'cvs up -r $(git rev-parse v1)' '
455455
'
456456

457457
test_expect_success 'cvs diff -r v1 -u' '
458-
( cd cvswork && cvs -f diff -r v1 -u ) >cvsDiff.out 2>cvs.log &&
458+
( cd cvswork && cvs -f diff -r v1 -u >../cvsDiff.out 2>../cvs.log ) &&
459459
test_must_be_empty cvsDiff.out &&
460460
test_must_be_empty cvs.log
461461
'
462462

463463
test_expect_success 'cvs diff -N -r v2 -u' '
464-
( cd cvswork && ! cvs -f diff -N -r v2 -u ) >cvsDiff.out 2>cvs.log &&
464+
( cd cvswork && ! cvs -f diff -N -r v2 -u >../cvsDiff.out 2>../cvs.log ) &&
465465
test_must_be_empty cvs.log &&
466466
test -s cvsDiff.out &&
467467
check_diff cvsDiff.out v2 v1 >check_diff.out 2>&1
468468
'
469469

470470
test_expect_success 'cvs diff -N -r v2 -r v1.2' '
471-
( cd cvswork && ! cvs -f diff -N -r v2 -r v1.2 -u ) >cvsDiff.out 2>cvs.log &&
471+
( cd cvswork && ! cvs -f diff -N -r v2 -r v1.2 -u >../cvsDiff.out 2>../cvs.log ) &&
472472
test_must_be_empty cvs.log &&
473473
test -s cvsDiff.out &&
474474
check_diff cvsDiff.out v2 v1.2 >check_diff.out 2>&1
@@ -487,7 +487,7 @@ test_expect_success 'apply early [cvswork3] diff to b3' '
487487
'
488488

489489
test_expect_success 'check [cvswork3] diff' '
490-
( cd cvswork3 && ! cvs -f diff -N -u ) >"$WORKDIR/cvsDiff.out" 2>cvs.log &&
490+
( cd cvswork3 && ! cvs -f diff -N -u >"$WORKDIR/cvsDiff.out" 2>../cvs.log ) &&
491491
test_must_be_empty cvs.log &&
492492
test -s cvsDiff.out &&
493493
test $(grep Index: cvsDiff.out | wc -l) = 3 &&

0 commit comments

Comments
 (0)