This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-net.git


The following commit(s) were added to refs/heads/master by this push:
     new 94d46269 Use Assertions.assertInstanceOf()
94d46269 is described below

commit 94d46269db004f17afe8c7a454dbf4aca61ebb76
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sun Sep 1 19:03:12 2024 -0400

    Use Assertions.assertInstanceOf()
---
 .../DefaultFTPFileEntryParserFactoryTest.java      | 25 +++++++++++-----------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git 
a/src/test/java/org/apache/commons/net/ftp/parser/DefaultFTPFileEntryParserFactoryTest.java
 
b/src/test/java/org/apache/commons/net/ftp/parser/DefaultFTPFileEntryParserFactoryTest.java
index 8908f62c..591ed415 100644
--- 
a/src/test/java/org/apache/commons/net/ftp/parser/DefaultFTPFileEntryParserFactoryTest.java
+++ 
b/src/test/java/org/apache/commons/net/ftp/parser/DefaultFTPFileEntryParserFactoryTest.java
@@ -16,6 +16,7 @@
  */
 package org.apache.commons.net.ftp.parser;
 
+import static org.junit.jupiter.api.Assertions.assertInstanceOf;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 
 import junit.framework.TestCase;
@@ -35,26 +36,26 @@ public class DefaultFTPFileEntryParserFactoryTest extends 
TestCase {
         final DefaultFTPFileEntryParserFactory factory = new 
DefaultFTPFileEntryParserFactory();
 
         FTPFileEntryParser parser = factory.createFileEntryParser("unix");
-        assertTrue(parser instanceof UnixFTPEntryParser);
+        assertInstanceOf(UnixFTPEntryParser.class, parser);
 
         parser = factory.createFileEntryParser("UNIX");
-        assertTrue(parser instanceof UnixFTPEntryParser);
+        assertInstanceOf(UnixFTPEntryParser.class, parser);
         assertFalse(((UnixFTPEntryParser) parser).trimLeadingSpaces);
 
         parser = factory.createFileEntryParser("UNIX_LTRIM");
-        assertTrue(parser instanceof UnixFTPEntryParser);
+        assertInstanceOf(UnixFTPEntryParser.class, parser);
         assertTrue(((UnixFTPEntryParser) parser).trimLeadingSpaces);
 
         parser = factory.createFileEntryParser("Unix");
-        assertTrue(parser instanceof UnixFTPEntryParser);
+        assertInstanceOf(UnixFTPEntryParser.class, parser);
 
         parser = factory.createFileEntryParser("EnterpriseUnix");
-        assertTrue(parser instanceof UnixFTPEntryParser);
+        assertInstanceOf(UnixFTPEntryParser.class, parser);
         assertFalse(parser instanceof EnterpriseUnixFTPEntryParser);
 
         // works because contains the expression "Unix"
         parser = factory.createFileEntryParser("UnixFTPEntryParser");
-        assertTrue(parser instanceof UnixFTPEntryParser);
+        assertInstanceOf(UnixFTPEntryParser.class, parser);
 
         try {
             parser = factory.createFileEntryParser("NT");
@@ -65,19 +66,19 @@ public class DefaultFTPFileEntryParserFactoryTest extends 
TestCase {
         }
 
         parser = factory.createFileEntryParser("WindowsNT");
-        assertTrue(parser instanceof CompositeFileEntryParser);
+        assertInstanceOf(CompositeFileEntryParser.class, parser);
 
         parser = factory.createFileEntryParser("ThigaVMSaMaJig");
-        assertTrue(parser instanceof VMSFTPEntryParser);
+        assertInstanceOf(VMSFTPEntryParser.class, parser);
 
         parser = factory.createFileEntryParser("OS/2");
-        assertTrue(parser instanceof OS2FTPEntryParser);
+        assertInstanceOf(OS2FTPEntryParser.class, parser);
 
         parser = factory.createFileEntryParser("OS/400");
-        assertTrue(parser instanceof CompositeFileEntryParser);
+        assertInstanceOf(CompositeFileEntryParser.class, parser);
 
         parser = factory.createFileEntryParser("AS/400");
-        assertTrue(parser instanceof CompositeFileEntryParser);
+        assertInstanceOf(CompositeFileEntryParser.class, parser);
 
         // Added test to make sure it handles the UNIX systems that were
         // compiled with OS as "UNKNOWN". This test validates that the
@@ -92,7 +93,7 @@ public class DefaultFTPFileEntryParserFactoryTest extends 
TestCase {
         }
 
         parser = 
factory.createFileEntryParser("org.apache.commons.net.ftp.parser.OS2FTPEntryParser");
-        assertTrue(parser instanceof OS2FTPEntryParser);
+        assertInstanceOf(OS2FTPEntryParser.class, parser);
 
         try {
             
factory.createFileEntryParser("org.apache.commons.net.ftp.parser.DefaultFTPFileEntryParserFactory");

Reply via email to