fix: Issue 2975 Refactored Visitor (#3001)

visitor/src/test/java/com/iluwatar/visitor/CommanderVisitorTest.java

@@ -38,9 +38,9 @@ class CommanderVisitorTest extends VisitorTest<CommanderVisitor> {
   public CommanderVisitorTest() {
     super(
         new CommanderVisitor(),
-        Optional.of("Good to see you commander"),
+        ("Good to see you commander"),
-        Optional.empty(),
+        null,
-        Optional.empty()
+         null
     );
   }
 

visitor/src/test/java/com/iluwatar/visitor/SergeantVisitorTest.java

@@ -38,9 +38,9 @@ class SergeantVisitorTest extends VisitorTest<SergeantVisitor> {
   public SergeantVisitorTest() {
     super(
         new SergeantVisitor(),
-        Optional.empty(),
+        null,
-        Optional.of("Hello sergeant"),
+        ("Hello sergeant"),
-        Optional.empty()
+        null
     );
   }
 

visitor/src/test/java/com/iluwatar/visitor/SoldierVisitorTest.java

@@ -38,9 +38,9 @@ class SoldierVisitorTest extends VisitorTest<SoldierVisitor> {
   public SoldierVisitorTest() {
     super(
         new SoldierVisitor(),
-        Optional.empty(),
+        null,
-        Optional.empty(),
+        null,
-        Optional.of("Greetings soldier")
+        ("Greetings soldier")
     );
   }
 

visitor/src/test/java/com/iluwatar/visitor/VisitorTest.java

@@ -31,7 +31,6 @@ import ch.qos.logback.classic.spi.ILoggingEvent;
 import ch.qos.logback.core.AppenderBase;
 import java.util.LinkedList;
 import java.util.List;
-import java.util.Optional;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
@@ -62,32 +61,32 @@ public abstract class VisitorTest<V extends UnitVisitor> {
   private final V visitor;
 
   /**
-   * The optional expected response when being visited by a commander.
+   * The expected response when being visited by a commander.
    */
-  private final Optional<String> commanderResponse;
+  private final String commanderResponse;
 
   /**
-   * The optional expected response when being visited by a sergeant.
+   * The expected response when being visited by a sergeant.
    */
-  private final Optional<String> sergeantResponse;
+  private final String sergeantResponse;
 
   /**
-   * The optional expected response when being visited by a soldier.
+   * The expected response when being visited by a soldier.
    */
-  private final Optional<String> soldierResponse;
+  private final String soldierResponse;
 
   /**
    * Create a new test instance for the given visitor.
    *
-   * @param commanderResponse The optional expected response when being visited by a commander
+   * @param commanderResponse The expected response when being visited by a commander
-   * @param sergeantResponse  The optional expected response when being visited by a sergeant
+   * @param sergeantResponse  The expected response when being visited by a sergeant
-   * @param soldierResponse   The optional expected response when being visited by a soldier
+   * @param soldierResponse   The expected response when being visited by a soldier
    */
   public VisitorTest(
       final V visitor,
-      final Optional<String> commanderResponse,
+      final String commanderResponse,
-      final Optional<String> sergeantResponse,
+      final String sergeantResponse,
-      final Optional<String> soldierResponse
+      final String soldierResponse
   ) {
     this.visitor = visitor;
     this.commanderResponse = commanderResponse;
@@ -98,8 +97,8 @@ public abstract class VisitorTest<V extends UnitVisitor> {
   @Test
   void testVisitCommander() {
     this.visitor.visit(new Commander());
-    if (this.commanderResponse.isPresent()) {
+    if (this.commanderResponse != null) {
-      assertEquals(this.commanderResponse.get(), appender.getLastMessage());
+      assertEquals(this.commanderResponse, appender.getLastMessage());
       assertEquals(1, appender.getLogSize());
     }
   }
@@ -107,8 +106,8 @@ public abstract class VisitorTest<V extends UnitVisitor> {
   @Test
   void testVisitSergeant() {
     this.visitor.visit(new Sergeant());
-    if (this.sergeantResponse.isPresent()) {
+    if (this.sergeantResponse != null) {
-      assertEquals(this.sergeantResponse.get(), appender.getLastMessage());
+      assertEquals(this.sergeantResponse, appender.getLastMessage());
       assertEquals(1, appender.getLogSize());
     }
   }
@@ -116,8 +115,8 @@ public abstract class VisitorTest<V extends UnitVisitor> {
   @Test
   void testVisitSoldier() {
     this.visitor.visit(new Soldier());
-    if (this.soldierResponse.isPresent()) {
+    if (this.soldierResponse != null) {
-      assertEquals(this.soldierResponse.get(), appender.getLastMessage());
+      assertEquals(this.soldierResponse, appender.getLastMessage());
       assertEquals(1, appender.getLogSize());
     }
   }