From 7e7b00f1a5d14ceca62fd18c6e11456fc9eddd21 Mon Sep 17 00:00:00 2001 From: Jonathan Tan Date: Tue, 24 Jul 2018 15:35:16 -0700 Subject: Send only 1 flush pkt at the end of v2 fetch When processing a fetch using protocol v2, UploadPack#fetchV2 sends an extraneous flush pkt when also sending a packfile (#sendPack sending its own flush pkt). Update that method to only send the flush pkt if the packfile is not being sent. Change-Id: I7117a264bccd2d7f3a048645fcb8425a9d78d526 Signed-off-by: Jonathan Tan --- .../tst/org/eclipse/jgit/transport/UploadPackTest.java | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'org.eclipse.jgit.test/tst/org') diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/UploadPackTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/UploadPackTest.java index 2a3e1ca56d..9f49073eae 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/UploadPackTest.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/UploadPackTest.java @@ -658,6 +658,10 @@ public class UploadPackTest { new StringWriter(), NullOutputStream.INSTANCE); PackParser pp = client.newObjectInserter().newPackParser(sb); pp.parse(NullProgressMonitor.INSTANCE); + + // Ensure that there is nothing left in the stream. + assertThat(recvStream.read(), is(-1)); + return pp.getReceivedPackStatistics(); } -- cgit v1.2.3