From 83a6145f2fd08e6e256c4342799476c90d7de2e0 Mon Sep 17 00:00:00 2001 From: Ivan Frade Date: Wed, 18 Sep 2019 14:38:37 -0700 Subject: [PATCH] UploadPackReachabilityTest: Use assertThrows instead of thrown In https://git.eclipse.org/r/c/144009/ UploadPack tests moved from thrown to assertThrows, but newly introduced tests are still using the thrown. Update test so all of them use assertThrows. Change-Id: I0ff19a6f8ba9e978d8ffc7a912c0572d9f00c7fa Signed-off-by: Ivan Frade --- .../transport/UploadPackReachabilityTest.java | 29 +++++++++---------- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/UploadPackReachabilityTest.java b/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/UploadPackReachabilityTest.java index 1a855d1d9..7122082d8 100644 --- a/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/UploadPackReachabilityTest.java +++ b/org.eclipse.jgit.test/tst/org/eclipse/jgit/transport/UploadPackReachabilityTest.java @@ -63,7 +63,6 @@ import org.eclipse.jgit.transport.resolver.ServiceNotAuthorizedException; import org.eclipse.jgit.transport.resolver.ServiceNotEnabledException; import org.eclipse.jgit.transport.resolver.UploadPackFactory; -import org.hamcrest.Matchers; import org.junit.After; import org.junit.Before; import org.junit.Rule; @@ -189,11 +188,11 @@ public void testFetchUnreachableBlobWithoutBitmap() throws Exception { assertFalse(client.getObjectDatabase().has(blob.toObjectId())); try (Transport tn = testProtocol.open(uri, client, "server")) { - thrown.expect(TransportException.class); - thrown.expectMessage(Matchers - .containsString("want " + blob.name() + " not valid")); + TransportException e = assertThrows(TransportException.class, () -> tn.fetch(NullProgressMonitor.INSTANCE, - Collections.singletonList(new RefSpec(blob.name()))); + Collections.singletonList(new RefSpec(blob.name())))); + assertThat(e.getMessage(), + containsString("want " + blob.name() + " not valid")); } } @@ -266,11 +265,11 @@ public void testFetchUnreachableCommitWithBitmap() throws Exception { assertFalse(client.getObjectDatabase().has(commit.toObjectId())); try (Transport tn = testProtocol.open(uri, client, "server")) { - thrown.expect(TransportException.class); - thrown.expectMessage(Matchers - .containsString("want " + commit.name() + " not valid")); - tn.fetch(NullProgressMonitor.INSTANCE, - Collections.singletonList(new RefSpec(commit.name()))); + TransportException e = assertThrows(TransportException.class, + () -> tn.fetch(NullProgressMonitor.INSTANCE, + Collections.singletonList(new RefSpec(commit.name())))); + assertThat(e.getMessage(), + containsString("want " + commit.name() + " not valid")); } } @@ -285,11 +284,11 @@ public void testFetchUnreachableCommitWithoutBitmap() throws Exception { assertFalse(client.getObjectDatabase().has(commit.toObjectId())); try (Transport tn = testProtocol.open(uri, client, "server")) { - thrown.expect(TransportException.class); - thrown.expectMessage(Matchers - .containsString("want " + commit.name() + " not valid")); - tn.fetch(NullProgressMonitor.INSTANCE, - Collections.singletonList(new RefSpec(commit.name()))); + TransportException e = assertThrows(TransportException.class, + () -> tn.fetch(NullProgressMonitor.INSTANCE, Collections + .singletonList(new RefSpec(commit.name())))); + assertThat(e.getMessage(), + containsString("want " + commit.name() + " not valid")); } }