Changeset 16618 in josm for trunk/test/unit/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandlerTest.java

Timestamp:

2020-06-14T11:54:13+02:00 (6 years ago)

Author:

simon04

Message:

see #16567 - Fix deprecations related to JUnit 5 (patch by taylor.smock)

File:

• trunk/test/unit/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandlerTest.java (modified) (3 diffs)

trunk/test/unit/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandlerTest.java

@@ -2 +2 @@
 package org.openstreetmap.josm.io.remotecontrol.handler;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+
 import org.junit.Rule;
 import org.junit.Test;
-import org.junit.rules.ExpectedException;
 import org.openstreetmap.josm.io.remotecontrol.handler.RequestHandler.RequestHandlerBadRequestException;
 import org.openstreetmap.josm.testutils.JOSMTestRules;
@@ -14 +17 @@
  */
 public class LoadObjectHandlerTest {
-
-    /**
-     * Rule used for tests throwing exceptions.
-     */
-    @Rule
-    public ExpectedException thrown = ExpectedException.none();
-
     /**
      * Setup test.
@@ -37 +33 @@
     /**
      * Unit test for bad request - no param.
-     * @throws Exception if any error occurs
      */
     @Test
-    public void testBadRequestNoParam() throws Exception {
-        newHandler(null).handle();
+    public void testBadRequestNoParam() {
+        assertDoesNotThrow(() -> newHandler(null).handle());
     }
 
     /**
      * Unit test for bad request - invalid URL.
-     * @throws Exception if any error occurs
      */
     @Test
-    public void testBadRequestInvalidUrl() throws Exception {
-        thrown.expect(RequestHandlerBadRequestException.class);
-        thrown.expectMessage("The following keys are mandatory, but have not been provided: objects");
-        newHandler("invalid_url").handle();
+    public void testBadRequestInvalidUrl() {
+        Exception e = assertThrows(RequestHandlerBadRequestException.class, () -> newHandler("invalid_url").handle());
+        assertEquals("The following keys are mandatory, but have not been provided: objects", e.getMessage());
     }
 
     /**
      * Unit test for bad request - incomplete URL.
-     * @throws Exception if any error occurs
      */
     @Test
-    public void testBadRequestIncompleteUrl() throws Exception {
-        thrown.expect(RequestHandlerBadRequestException.class);
-        thrown.expectMessage("The following keys are mandatory, but have not been provided: objects");
-        newHandler("https://localhost").handle();
+    public void testBadRequestIncompleteUrl() {
+        Exception e = assertThrows(RequestHandlerBadRequestException.class, () -> newHandler("https://localhost").handle());
+        assertEquals("The following keys are mandatory, but have not been provided: objects", e.getMessage());
     }
 
     /**
      * Unit test for nominal request - local data file.
-     * @throws Exception if any error occurs
      */
     @Test
-    public void testNominalRequest() throws Exception {
-        newHandler("https://localhost?objects=foo,bar").handle();
+    public void testNominalRequest() {
+        assertDoesNotThrow(() -> newHandler("https://localhost?objects=foo,bar").handle());
     }
 }