Refactor some exception tests

Author: garydgregory

src/test/java/org/apache/commons/lang3/concurrent/ConcurrentExceptionTest.java

@@ -0,0 +1,60 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.commons.lang3.concurrent;
+
+import static org.apache.commons.lang3.LangAssertions.assertIllegalArgumentException;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
+
+import org.junit.jupiter.api.Test;
+
+/**
+ * Tests {@link ConcurrentException}.
+ */
+class ConcurrentExceptionTest {
+
+    /**
+     * Tests creating a ConcurrentException with an error as cause.
+     */
+    @Test
+    void testCauseError() {
+        assertIllegalArgumentException(() -> new ConcurrentException("An error", new Error()));
+    }
+
+    /**
+     * Tests creating a ConcurrentException with null as cause.
+     */
+    @Test
+    void testCauseNull() {
+        assertIllegalArgumentException(() -> new ConcurrentException((Throwable) null));
+    }
+
+    @Test
+    void testCauseString() {
+        assertEquals("test", new ConcurrentException("test").getMessage());
+        assertNull(new ConcurrentException((String) null).getMessage());
+    }
+
+    /**
+     * Tests creating a ConcurrentException with a runtime exception as cause.
+     */
+    @Test
+    void testCauseUnchecked() {
+        assertIllegalArgumentException(() -> new ConcurrentException(new RuntimeException()));
+    }
+}

src/test/java/org/apache/commons/lang3/concurrent/ConcurrentRuntimeExceptionTest.java

@@ -0,0 +1,52 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.commons.lang3.concurrent;
+
+import static org.apache.commons.lang3.LangAssertions.assertIllegalArgumentException;
+
+import org.junit.jupiter.api.Test;
+
+/**
+ * Tests {@link ConcurrentRuntimeException}.
+ */
+class ConcurrentRuntimeExceptionTest {
+
+    /**
+     * Tries to create a ConcurrentRuntimeException with an error as cause.
+     */
+    @Test
+    void testCauseError() {
+        assertIllegalArgumentException(() -> new ConcurrentRuntimeException("An error", new Error()));
+    }
+
+    /**
+     * Tries to create a ConcurrentRuntimeException with null as cause.
+     */
+    @Test
+    void testCauseNull() {
+        assertIllegalArgumentException(() -> new ConcurrentRuntimeException(null));
+    }
+
+    /**
+     * Tries to create a ConcurrentRuntimeException with a runtime as cause.
+     */
+    @Test
+    void testCauseUnchecked() {
+        assertIllegalArgumentException(() -> new ConcurrentRuntimeException(new RuntimeException()));
+    }
+}

src/test/java/org/apache/commons/lang3/concurrent/ConcurrentUtilsTest.java

@@ -14,9 +14,9 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.apache.commons.lang3.concurrent;
 
-import static org.apache.commons.lang3.LangAssertions.assertIllegalArgumentException;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
@@ -39,60 +39,6 @@
  * Test class for {@link ConcurrentUtils}.
  */
 class ConcurrentUtilsTest extends AbstractLangTest {
-    /**
-     * Tests creating a ConcurrentException with an error as cause.
-     */
-    @Test
-    void testConcurrentExceptionCauseError() {
-        assertIllegalArgumentException(() -> new ConcurrentException("An error", new Error()));
-    }
-
-    /**
-     * Tests creating a ConcurrentException with null as cause.
-     */
-    @Test
-    void testConcurrentExceptionCauseNull() {
-        assertIllegalArgumentException(() -> new ConcurrentException((Throwable) null));
-    }
-
-    @Test
-    void testConcurrentExceptionCauseString() {
-        assertEquals("test", new ConcurrentException("test").getMessage());
-        assertNull(new ConcurrentException((String) null).getMessage());
-    }
-
-    /**
-     * Tests creating a ConcurrentException with a runtime exception as cause.
-     */
-    @Test
-    void testConcurrentExceptionCauseUnchecked() {
-        assertIllegalArgumentException(() -> new ConcurrentException(new RuntimeException()));
-    }
-
-    /**
-     * Tries to create a ConcurrentRuntimeException with an error as cause.
-     */
-    @Test
-    void testConcurrentRuntimeExceptionCauseError() {
-        assertIllegalArgumentException(() -> new ConcurrentRuntimeException("An error", new Error()));
-    }
-
-    /**
-     * Tries to create a ConcurrentRuntimeException with null as cause.
-     */
-    @Test
-    void testConcurrentRuntimeExceptionCauseNull() {
-        assertIllegalArgumentException(() -> new ConcurrentRuntimeException(null));
-    }
-
-    /**
-     * Tries to create a ConcurrentRuntimeException with a runtime as cause.
-     */
-    @Test
-    void testConcurrentRuntimeExceptionCauseUnchecked() {
-        assertIllegalArgumentException(() -> new ConcurrentRuntimeException(new RuntimeException()));
-    }
-
     /**
      * Tests constant future.
      *