From 18b04db1039ca52b82d617b9403dd73781769375 Mon Sep 17 00:00:00 2001
From: Robert Volkmann <20912167+robertvolkmann@users.noreply.github.com>
Date: Sun, 4 Dec 2022 09:55:33 +0100
Subject: [PATCH] dependencies: remove mongodb uber artifact (#2356)

---
 caching/pom.xml                               |  6 ++++-
 .../iluwatar/caching/database/MongoDb.java    |  3 +--
 .../caching/database/MongoDbTest.java         |  1 -
 hexagonal/pom.xml                             |  6 ++++-
 .../database/MongoTicketRepository.java       |  2 +-
 .../hexagonal/banking/MongoBankTest.java      |  2 +-
 .../database/MongoTicketRepositoryTest.java   |  8 +++----
 .../hexagonal/eventlog/MongoEventLogTest.java | 22 +++++++++----------
 pom.xml                                       |  6 -----
 9 files changed, 28 insertions(+), 28 deletions(-)

diff --git a/caching/pom.xml b/caching/pom.xml
index 695281bc2..d7470b5e4 100644
--- a/caching/pom.xml
+++ b/caching/pom.xml
@@ -46,7 +46,11 @@
     </dependency>
     <dependency>
       <groupId>org.mongodb</groupId>
-      <artifactId>mongo-java-driver</artifactId>
+      <artifactId>bson</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.mongodb</groupId>
+      <artifactId>mongodb-driver-legacy</artifactId>
     </dependency>
   </dependencies>
   <!--
diff --git a/caching/src/main/java/com/iluwatar/caching/database/MongoDb.java b/caching/src/main/java/com/iluwatar/caching/database/MongoDb.java
index be644d9b9..31f5fdc96 100644
--- a/caching/src/main/java/com/iluwatar/caching/database/MongoDb.java
+++ b/caching/src/main/java/com/iluwatar/caching/database/MongoDb.java
@@ -37,7 +37,6 @@ import com.mongodb.MongoCredential;
 import com.mongodb.ServerAddress;
 import com.mongodb.client.MongoDatabase;
 import com.mongodb.client.model.UpdateOptions;
-import java.util.List;
 import lombok.extern.slf4j.Slf4j;
 import org.bson.Document;
 
@@ -66,7 +65,7 @@ public class MongoDb implements DbManager {
                     DATABASE_NAME,
                     MONGO_PASSWORD.toCharArray());
     MongoClientOptions options = MongoClientOptions.builder().build();
-    client = new MongoClient(new ServerAddress(), List.of(mongoCredential), options);
+    client = new MongoClient(new ServerAddress(), mongoCredential, options);
     db = client.getDatabase(DATABASE_NAME);
   }
 
diff --git a/caching/src/test/java/com/iluwatar/caching/database/MongoDbTest.java b/caching/src/test/java/com/iluwatar/caching/database/MongoDbTest.java
index bf1aef70a..16d38f175 100644
--- a/caching/src/test/java/com/iluwatar/caching/database/MongoDbTest.java
+++ b/caching/src/test/java/com/iluwatar/caching/database/MongoDbTest.java
@@ -84,7 +84,6 @@ class MongoDbTest {
   void writeToDb() {
     MongoCollection<Document> mongoCollection = mock(MongoCollection.class);
     when(db.getCollection(CachingConstants.USER_ACCOUNT)).thenReturn(mongoCollection);
-    doNothing().when(mongoCollection).insertOne(any(Document.class));
     assertDoesNotThrow(()-> {mongoDb.writeToDb(userAccount);});
   }
 
diff --git a/hexagonal/pom.xml b/hexagonal/pom.xml
index 7788e9d52..0b570558e 100644
--- a/hexagonal/pom.xml
+++ b/hexagonal/pom.xml
@@ -45,7 +45,11 @@
     </dependency>
     <dependency>
       <groupId>org.mongodb</groupId>
-      <artifactId>mongo-java-driver</artifactId>
+      <artifactId>bson</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.mongodb</groupId>
+      <artifactId>mongodb-driver-legacy</artifactId>
     </dependency>
   </dependencies>
   <build>
diff --git a/hexagonal/src/main/java/com/iluwatar/hexagonal/database/MongoTicketRepository.java b/hexagonal/src/main/java/com/iluwatar/hexagonal/database/MongoTicketRepository.java
index d0d3d42fe..a8d7ecdb8 100644
--- a/hexagonal/src/main/java/com/iluwatar/hexagonal/database/MongoTicketRepository.java
+++ b/hexagonal/src/main/java/com/iluwatar/hexagonal/database/MongoTicketRepository.java
@@ -89,7 +89,7 @@ public class MongoTicketRepository implements LotteryTicketRepository {
     database = mongoClient.getDatabase(dbName);
     ticketsCollection = database.getCollection(ticketsCollectionName);
     countersCollection = database.getCollection(countersCollectionName);
-    if (countersCollection.count() <= 0) {
+    if (countersCollection.countDocuments() <= 0) {
       initCounters();
     }
   }
diff --git a/hexagonal/src/test/java/com/iluwatar/hexagonal/banking/MongoBankTest.java b/hexagonal/src/test/java/com/iluwatar/hexagonal/banking/MongoBankTest.java
index 0b6852f8f..9ed9c2e02 100644
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/banking/MongoBankTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/banking/MongoBankTest.java
@@ -55,7 +55,7 @@ class MongoBankTest {
 
   @Test
   void testSetup() {
-    assertEquals(0, mongoBank.getAccountsCollection().count());
+    assertEquals(0, mongoBank.getAccountsCollection().countDocuments());
   }
 
   @Test
diff --git a/hexagonal/src/test/java/com/iluwatar/hexagonal/database/MongoTicketRepositoryTest.java b/hexagonal/src/test/java/com/iluwatar/hexagonal/database/MongoTicketRepositoryTest.java
index e36ecec77..9a3785f50 100644
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/database/MongoTicketRepositoryTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/database/MongoTicketRepositoryTest.java
@@ -62,8 +62,8 @@ class MongoTicketRepositoryTest {
 
   @Test
   void testSetup() {
-    assertEquals(1, repository.getCountersCollection().count());
-    assertEquals(0, repository.getTicketsCollection().count());
+    assertEquals(1, repository.getCountersCollection().countDocuments());
+    assertEquals(0, repository.getTicketsCollection().countDocuments());
   }
 
   @Test
@@ -80,7 +80,7 @@ class MongoTicketRepositoryTest {
     var random = LotteryNumbers.createRandom();
     var original = new LotteryTicket(new LotteryTicketId(), details, random);
     var saved = repository.save(original);
-    assertEquals(1, repository.getTicketsCollection().count());
+    assertEquals(1, repository.getTicketsCollection().countDocuments());
     assertTrue(saved.isPresent());
     // fetch the saved lottery ticket from database and check its contents
     var found = repository.findById(saved.get());
@@ -92,6 +92,6 @@ class MongoTicketRepositoryTest {
     assertEquals(original.getLotteryNumbers(), ticket.getLotteryNumbers());
     // clear the collection
     repository.deleteAll();
-    assertEquals(0, repository.getTicketsCollection().count());
+    assertEquals(0, repository.getTicketsCollection().countDocuments());
   }
 }
diff --git a/hexagonal/src/test/java/com/iluwatar/hexagonal/eventlog/MongoEventLogTest.java b/hexagonal/src/test/java/com/iluwatar/hexagonal/eventlog/MongoEventLogTest.java
index 5189347d3..3216ac7eb 100644
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/eventlog/MongoEventLogTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/eventlog/MongoEventLogTest.java
@@ -56,31 +56,31 @@ class MongoEventLogTest {
 
   @Test
   void testSetup() {
-    assertEquals(0, mongoEventLog.getEventsCollection().count());
+    assertEquals(0, mongoEventLog.getEventsCollection().countDocuments());
   }
 
   @Test
   void testFundTransfers() {
     var playerDetails = new PlayerDetails("john@wayne.com", "000-000", "03432534543");
     mongoEventLog.prizeError(playerDetails, 1000);
-    assertEquals(1, mongoEventLog.getEventsCollection().count());
+    assertEquals(1, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.prizeError(playerDetails, 1000);
-    assertEquals(2, mongoEventLog.getEventsCollection().count());
+    assertEquals(2, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.ticketDidNotWin(playerDetails);
-    assertEquals(3, mongoEventLog.getEventsCollection().count());
+    assertEquals(3, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.ticketDidNotWin(playerDetails);
-    assertEquals(4, mongoEventLog.getEventsCollection().count());
+    assertEquals(4, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.ticketSubmitError(playerDetails);
-    assertEquals(5, mongoEventLog.getEventsCollection().count());
+    assertEquals(5, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.ticketSubmitError(playerDetails);
-    assertEquals(6, mongoEventLog.getEventsCollection().count());
+    assertEquals(6, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.ticketSubmitted(playerDetails);
-    assertEquals(7, mongoEventLog.getEventsCollection().count());
+    assertEquals(7, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.ticketSubmitted(playerDetails);
-    assertEquals(8, mongoEventLog.getEventsCollection().count());
+    assertEquals(8, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.ticketWon(playerDetails, 1000);
-    assertEquals(9, mongoEventLog.getEventsCollection().count());
+    assertEquals(9, mongoEventLog.getEventsCollection().countDocuments());
     mongoEventLog.ticketWon(playerDetails, 1000);
-    assertEquals(10, mongoEventLog.getEventsCollection().count());
+    assertEquals(10, mongoEventLog.getEventsCollection().countDocuments());
   }
 }
diff --git a/pom.xml b/pom.xml
index b75a4551f..419c004ec 100644
--- a/pom.xml
+++ b/pom.xml
@@ -43,7 +43,6 @@
     <guava.version>19.0</guava.version>
     <htmlunit.version>2.66.0</htmlunit.version>
     <guice.version>4.0</guice.version>
-    <mongo-java-driver.version>3.12.8</mongo-java-driver.version>
     <system-lambda.version>1.1.0</system-lambda.version>
     <urm.version>2.0.0</urm.version>
     <maven-surefire-plugin.version>3.0.0-M5</maven-surefire-plugin.version>
@@ -239,11 +238,6 @@
         <artifactId>guice</artifactId>
         <version>${guice.version}</version>
       </dependency>
-      <dependency>
-        <groupId>org.mongodb</groupId>
-        <artifactId>mongo-java-driver</artifactId>
-        <version>${mongo-java-driver.version}</version>
-      </dependency>
       <dependency>
         <groupId>com.github.stefanbirkner</groupId>
         <artifactId>system-lambda</artifactId>