SpotBugs - remove unnecessary code

apache · Oct 16, 2020 · e2d9d6a · e2d9d6a
1 parent 1b27eec
commit e2d9d6a
Show file tree

Hide file tree

Showing 8 changed files with 8 additions and 28 deletions.
diff --git a/modules/jdbc-pool/src/test/java/org/apache/tomcat/jdbc/test/TestGetConnection.java b/modules/jdbc-pool/src/test/java/org/apache/tomcat/jdbc/test/TestGetConnection.java
@@ -28,7 +28,6 @@ public class TestGetConnection extends DefaultTestCase {
     @Test
     public void testGetConnection() throws Exception {
         Connection con = this.datasource.getConnection();
-        Assert.assertTrue("Connection should implement javax.sql.PooledConnection",con instanceof PooledConnection);
         Connection actual = ((PooledConnection)con).getConnection();
         Assert.assertNotNull("Connection delegate should not be null.",actual);
         System.out.println("Actual connection:"+actual.getClass().getName());

diff --git a/test/jakarta/el/TestImportHandlerStandardPackages.java b/test/jakarta/el/TestImportHandlerStandardPackages.java
@@ -42,8 +42,6 @@ public void testClassListsAreComplete() throws Exception {
         f.setAccessible(true);
         Object obj = f.get(null);
 
-        Assert.assertTrue("Not Map", obj instanceof Map);
-
         @SuppressWarnings("unchecked")
         Map<String,Set<String>> standardPackageName = (Map<String, Set<String>>) obj;
 

diff --git a/test/jakarta/el/TestResourceBundleELResolver.java b/test/jakarta/el/TestResourceBundleELResolver.java
@@ -52,7 +52,6 @@ public void bug53001() {
 
         // Check that the method expression does return the keys
         Object result2 = me.invoke(context, null);
-        Assert.assertTrue(result2 instanceof Enumeration);
         @SuppressWarnings("unchecked")
         Enumeration<String> e = (Enumeration<String>) result2;
 

diff --git a/test/org/apache/catalina/loader/TestWebappClassLoader.java b/test/org/apache/catalina/loader/TestWebappClassLoader.java
@@ -48,8 +48,6 @@ public void testGetURLs() throws Exception {
 
         ClassLoader cl = ctx.getLoader().getClassLoader();
 
-        Assert.assertTrue(cl instanceof URLClassLoader);
-
         try (URLClassLoader ucl = (URLClassLoader) cl) {
             URL[] urls = ucl.getURLs();
             Assert.assertEquals(expected.length, urls.length);

diff --git a/test/org/apache/el/lang/TestELSupport.java b/test/org/apache/el/lang/TestELSupport.java
@@ -215,7 +215,6 @@ public void testCoerceToType14() {
         PropertyEditorManager.registerEditor(TesterType.class, TesterTypeEditorNoError.class);
         Object result = ELManager.getExpressionFactory().coerceToType(
                 "Foo", TesterType.class);
-        Assert.assertTrue(result instanceof TesterType);
         Assert.assertEquals("Foo", ((TesterType) result).getValue());
     }
 
@@ -224,7 +223,6 @@ public void testCoerceToType15() {
         PropertyEditorManager.registerEditor(TesterType.class, TesterTypeEditorError.class);
         Object result = ELManager.getExpressionFactory().coerceToType(
                 "Foo", TesterType.class);
-        Assert.assertTrue(result instanceof TesterType);
         Assert.assertEquals("Foo", ((TesterType) result).getValue());
     }
 

diff --git a/test/org/apache/el/stream/TestCollectionOperations.java b/test/org/apache/el/stream/TestCollectionOperations.java
@@ -123,8 +123,6 @@ public void testMap02() {
                 "beans.stream().filter(b->b.valLong > 1).map(b->[b.name, b.valLong]).toList()",
                 List.class);
 
-        Assert.assertTrue(result instanceof List);
-
         @SuppressWarnings("unchecked")
         List<List<Object>> list = (List<List<Object>>) result;
 

diff --git a/test/org/apache/jasper/compiler/TestTagPluginManager.java b/test/org/apache/jasper/compiler/TestTagPluginManager.java
@@ -56,15 +56,13 @@ public void testBug54240() throws Exception {
 
         Node n = nodes.getNode(0);
         Assert.assertNotNull(n);
-        Assert.assertTrue(n instanceof Node.CustomTag);
 
         Node.CustomTag t = (Node.CustomTag)n;
         Assert.assertNotNull(t.getAtSTag());
 
         Node.Nodes sTag = c.getAtSTag();
         Node scriptlet = sTag.getNode(0);
         Assert.assertNotNull(scriptlet);
-        Assert.assertTrue(scriptlet instanceof Node.Scriptlet);
         Node.Scriptlet s = (Node.Scriptlet)scriptlet;
         Assert.assertEquals("//Just a comment", s.getText());
     }

diff --git a/test/org/apache/tomcat/util/buf/TestUDecoder.java b/test/org/apache/tomcat/util/buf/TestUDecoder.java
@@ -25,25 +25,17 @@
 
 public class TestUDecoder {
 
-    @Test
-    public void testURLDecodeStringInvalid() {
+    @Test(expected = IllegalArgumentException.class)
+    public void testURLDecodeStringInvalid01() {
         // %n rather than %nn should throw an IAE according to the Javadoc
-        Exception exception = null;
-        try {
-            UDecoder.URLDecode("%5xxxxx", StandardCharsets.UTF_8);
-        } catch (Exception e) {
-            exception = e;
-        }
-        Assert.assertTrue(exception instanceof IllegalArgumentException);
+        UDecoder.URLDecode("%5xxxxx", StandardCharsets.UTF_8);
+    }
 
+
+    @Test(expected = IllegalArgumentException.class)
+    public void testURLDecodeStringInvalid02() {
         // Edge case trying to trigger ArrayIndexOutOfBoundsException
-        exception = null;
-        try {
-            UDecoder.URLDecode("%5", StandardCharsets.UTF_8);
-        } catch (Exception e) {
-            exception = e;
-        }
-        Assert.assertTrue(exception instanceof IllegalArgumentException);
+        UDecoder.URLDecode("%5", StandardCharsets.UTF_8);
     }