t5539: update a flaky test
commitf0e802ca200b1296495d2ee5c55cd8f8083486bc
authorJeff King <peff@peff.net>
Mon, 14 Jul 2014 05:40:22 +0000 (14 01:40 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 15 Jul 2014 18:27:08 +0000 (15 11:27 -0700)
tree18ff9351dddf6223dff5913312827608ed5da04b
parent73f43f220f0276012de50c84413fd61bf6aa307b
t5539: update a flaky test

The test creates some unrelated commits in two separate repositories,
and then fetches from one to the other. Since the commit creation
happens in a subshell, the first commit in each ends up with the
same test_tick value. When fetch-pack looks at the two root commits
"unrelated1" and "new-too", the exact sequence of ACKs is different
depending on which one it pulls out of the queue first.

With the current code, it happens to be "unrelated1" (though this is not
at all guaranteed by the prio_queue data structure, it is deterministic
for this particular sequence of input). We see the ready-ACK, and the
test succeeds.

With the stable queue, we reliably get "new-too" out (since it is our
local tip, it is added to the queue before we even talk to the remote).
We never see a ready-ACK, and the test fails due to the grep on the
TRACE_PACKET output at the end (the fetch itself succeeds as expected).

I'm really not quite clear on what's supposed to be going on in the
test. I can make it pass with this change.
t/t5539-fetch-http-shallow.sh