Skip to content

Commit 3f360de

Browse files
luked99gitster
authored andcommitted
git-p4: remove ticket expiry test
The git-p4 login ticket expiry test causes unreliable test runs. Since the handling of ticket expiry in git-p4 is far from polished anyway, let's remove it for now. A better way to actually run the test is to create a python "fake" version of "p4" which returns whatever expiry results the test requires. Ideally git-p4 would look at the expiry time before starting any long operations, and cleanup gracefully if there is not enough time left. But that's quite hard to do. Signed-off-by: Luke Diamand <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 0d0ac38 commit 3f360de

File tree

1 file changed

+0
-27
lines changed

1 file changed

+0
-27
lines changed

t/t9833-errors.sh

Lines changed: 0 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -45,33 +45,6 @@ test_expect_success 'ticket logged out' '
4545
)
4646
'
4747

48-
test_expect_success 'create group with short ticket expiry' '
49-
P4TICKETS="$cli/tickets" &&
50-
echo "newpassword" | p4 login &&
51-
p4_add_user short_expiry_user &&
52-
p4 -u short_expiry_user passwd -P password &&
53-
p4 group -i <<-EOF &&
54-
Group: testgroup
55-
Timeout: 3
56-
Users: short_expiry_user
57-
EOF
58-
59-
p4 users | grep short_expiry_user
60-
'
61-
62-
test_expect_success 'git operation with expired ticket' '
63-
P4TICKETS="$cli/tickets" &&
64-
P4USER=short_expiry_user &&
65-
echo "password" | p4 login &&
66-
(
67-
cd "$git" &&
68-
git p4 sync &&
69-
sleep 5 &&
70-
test_must_fail git p4 sync 2>errmsg &&
71-
grep "failure accessing depot" errmsg
72-
)
73-
'
74-
7548
test_expect_success 'kill p4d' '
7649
kill_p4d
7750
'

0 commit comments

Comments
 (0)