diff --git a/org.eclipse.jgit.lfs.server.test/META-INF/MANIFEST.MF b/org.eclipse.jgit.lfs.server.test/META-INF/MANIFEST.MF index 60d50f619..9d4181fb9 100644 --- a/org.eclipse.jgit.lfs.server.test/META-INF/MANIFEST.MF +++ b/org.eclipse.jgit.lfs.server.test/META-INF/MANIFEST.MF @@ -34,5 +34,6 @@ Import-Package: javax.servlet;version="[3.1.0,4.0.0)", org.eclipse.jgit.util;version="[4.10.0,4.11.0)", org.hamcrest.core;version="[1.1.0,2.0.0)", org.junit;version="[4.0.0,5.0.0)", + org.junit.rules;version="[4.11.0,5.0.0)", org.junit.runner;version="[4.0.0,5.0.0)", org.junit.runners;version="[4.0.0,5.0.0)" diff --git a/org.eclipse.jgit.lfs.server.test/tst/org/eclipse/jgit/lfs/server/fs/DownloadTest.java b/org.eclipse.jgit.lfs.server.test/tst/org/eclipse/jgit/lfs/server/fs/DownloadTest.java index 69d1a0480..0f7004b5c 100644 --- a/org.eclipse.jgit.lfs.server.test/tst/org/eclipse/jgit/lfs/server/fs/DownloadTest.java +++ b/org.eclipse.jgit.lfs.server.test/tst/org/eclipse/jgit/lfs/server/fs/DownloadTest.java @@ -45,7 +45,6 @@ import static org.apache.http.HttpStatus.SC_NOT_FOUND; import static org.apache.http.HttpStatus.SC_UNPROCESSABLE_ENTITY; import static org.junit.Assert.assertEquals; -import static org.junit.Assert.fail; import java.io.IOException; import java.nio.file.Path; @@ -56,10 +55,15 @@ import org.eclipse.jgit.lfs.lib.AnyLongObjectId; import org.eclipse.jgit.lfs.test.LongObjectIdTestUtils; import org.eclipse.jgit.util.FileUtils; +import org.junit.Rule; import org.junit.Test; +import org.junit.rules.ExpectedException; public class DownloadTest extends LfsServerTest { + @Rule + public ExpectedException exception = ExpectedException.none(); + @Test public void testDownload() throws Exception { String TEXT = "test"; @@ -76,15 +80,12 @@ public void testDownloadInvalidPathInfo() String TEXT = "test"; String id = putContent(TEXT).name().substring(0, 60); Path f = Paths.get(getTempDirectory().toString(), "download"); - try { - getContent(id, f); - fail("expected RuntimeException"); - } catch (RuntimeException e) { - String error = String.format( - "Invalid pathInfo '/%s' does not match '/{SHA-256}'", id); - assertEquals(formatErrorMessage(SC_UNPROCESSABLE_ENTITY, error), - e.getMessage()); - } + String error = String.format( + "Invalid pathInfo '/%s' does not match '/{SHA-256}'", id); + exception.expect(RuntimeException.class); + exception.expectMessage( + formatErrorMessage(SC_UNPROCESSABLE_ENTITY, error)); + getContent(id, f); } @Test @@ -93,14 +94,11 @@ public void testDownloadInvalidId() String TEXT = "test"; String id = putContent(TEXT).name().replace('f', 'z'); Path f = Paths.get(getTempDirectory().toString(), "download"); - try { - getContent(id, f); - fail("expected RuntimeException"); - } catch (RuntimeException e) { - String error = String.format("Invalid id: : %s", id); - assertEquals(formatErrorMessage(SC_UNPROCESSABLE_ENTITY, error), - e.getMessage()); - } + String error = String.format("Invalid id: : %s", id); + exception.expect(RuntimeException.class); + exception.expectMessage( + formatErrorMessage(SC_UNPROCESSABLE_ENTITY, error)); + getContent(id, f); } @Test @@ -109,14 +107,10 @@ public void testDownloadNotFound() String TEXT = "test"; AnyLongObjectId id = LongObjectIdTestUtils.hash(TEXT); Path f = Paths.get(getTempDirectory().toString(), "download"); - try { - getContent(id, f); - fail("expected RuntimeException"); - } catch (RuntimeException e) { - String error = String.format("Object '%s' not found", id.getName()); - assertEquals(formatErrorMessage(SC_NOT_FOUND, error), - e.getMessage()); - } + String error = String.format("Object '%s' not found", id.getName()); + exception.expect(RuntimeException.class); + exception.expectMessage(formatErrorMessage(SC_NOT_FOUND, error)); + getContent(id, f); } @SuppressWarnings("boxing")