From b57721aa45057e686046c32c7e7390be6d402ee2 Mon Sep 17 00:00:00 2001
From: Riley <rileyp.mckinley@gmail.com>
Date: Tue, 23 Apr 2024 16:23:38 -0400
Subject: [PATCH] #2898 modified parent pom

---
 pom.xml                                        |  1 +
 publish-subscribe/pom.xml                      | 18 +-----------------
 .../iluwatar/publishsubscribe/Borrower.java    |  3 +--
 .../publishsubscribe/BorrowerTest.java         | 13 -------------
 .../iluwatar/publishsubscribe/LenderTest.java  |  2 --
 5 files changed, 3 insertions(+), 34 deletions(-)

diff --git a/pom.xml b/pom.xml
index 7dc65f454b75..9f8766d3d119 100644
--- a/pom.xml
+++ b/pom.xml
@@ -58,6 +58,7 @@
     <sonar.projectName>Java Design Patterns</sonar.projectName>
   </properties>
   <modules>
+    <module>publish-subscribe</module>
     <module>abstract-factory</module>
     <module>collecting-parameter</module>
     <module>monitor</module>
diff --git a/publish-subscribe/pom.xml b/publish-subscribe/pom.xml
index 8b4d2806cd90..473c9362b661 100644
--- a/publish-subscribe/pom.xml
+++ b/publish-subscribe/pom.xml
@@ -35,7 +35,7 @@
         <artifactId>java-design-patterns</artifactId>
         <version>1.26.0-SNAPSHOT</version>
     </parent>
-    <artifactId>publishsubscribe</artifactId>
+    <artifactId>publish-subscribe</artifactId>
 
     <dependencies>
         <dependency>
@@ -43,27 +43,11 @@
             <artifactId>junit-jupiter-api</artifactId>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>org.mockito</groupId>
-            <artifactId>mockito-core</artifactId>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.apache.activemq</groupId>
             <artifactId>activemq-core</artifactId>
             <version>5.7.0</version>
         </dependency>
-        <dependency>
-            <groupId>org.apache.xbean</groupId>
-            <artifactId>xbean-spring</artifactId>
-            <version>4.24</version>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.activemq.tooling</groupId>
-            <artifactId>activemq-junit</artifactId>
-            <version>6.1.1</version>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
 </project>
\ No newline at end of file
diff --git a/publish-subscribe/src/main/java/com/iluwatar/publishsubscribe/Borrower.java b/publish-subscribe/src/main/java/com/iluwatar/publishsubscribe/Borrower.java
index a206a636af98..f94e4c31d102 100644
--- a/publish-subscribe/src/main/java/com/iluwatar/publishsubscribe/Borrower.java
+++ b/publish-subscribe/src/main/java/com/iluwatar/publishsubscribe/Borrower.java
@@ -1,7 +1,6 @@
 package com.iluwatar.publishsubscribe;
 
 import lombok.extern.slf4j.Slf4j;
-import org.slf4j.event.Level;
 import javax.jms.BytesMessage;
 import javax.jms.JMSException;
 import javax.jms.Message;
@@ -25,7 +24,7 @@ public class Borrower implements MessageListener {
   private TopicConnection tConnection;
   private TopicSession tSession;
   private Topic topic;
-  private double currentRate;
+  private final double currentRate;
   private static final String ERROR = "An error has occured!";
   private double newRate;
 
diff --git a/publish-subscribe/src/test/java/com/iluwatar/publishsubscribe/BorrowerTest.java b/publish-subscribe/src/test/java/com/iluwatar/publishsubscribe/BorrowerTest.java
index ded88e244087..cb33585a719f 100644
--- a/publish-subscribe/src/test/java/com/iluwatar/publishsubscribe/BorrowerTest.java
+++ b/publish-subscribe/src/test/java/com/iluwatar/publishsubscribe/BorrowerTest.java
@@ -1,25 +1,12 @@
 package com.iluwatar.publishsubscribe;
 
-import org.apache.activemq.Message;
-import org.apache.activemq.broker.Broker;
 import org.apache.activemq.broker.BrokerService;
-import org.apache.activemq.command.ActiveMQTextMessage;
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeAll;
-import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import javax.jms.BytesMessage;
 import javax.jms.JMSException;
-import javax.jms.TextMessage;
-import java.io.BufferedOutputStream;
-import java.io.ByteArrayOutputStream;
-import java.io.OutputStream;
-import java.io.PrintStream;
-import java.sql.SQLOutput;
-
-import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.when;
 
 public class BorrowerTest {
 
diff --git a/publish-subscribe/src/test/java/com/iluwatar/publishsubscribe/LenderTest.java b/publish-subscribe/src/test/java/com/iluwatar/publishsubscribe/LenderTest.java
index a103b0d9850f..e5420ee2603b 100644
--- a/publish-subscribe/src/test/java/com/iluwatar/publishsubscribe/LenderTest.java
+++ b/publish-subscribe/src/test/java/com/iluwatar/publishsubscribe/LenderTest.java
@@ -2,10 +2,8 @@
 
 import org.apache.activemq.broker.BrokerService;
 import org.junit.jupiter.api.AfterAll;
-import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeAll;
-import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
 public class LenderTest {