Fix docs, change MongoDb with MongoDB, remove logReplicaSetStatus

Author: silaev

docs/modules/databases/mongodb.md

@@ -3,20 +3,23 @@
 !!! note
     This module is INCUBATING. While it is ready for use and operational in the current version of Testcontainers, it is possible that it may receive breaking changes in the future. See [our contributing guidelines](/contributing/#incubating-modules) for more information on our incubating modules policy.
 
-# Java8 MongoDbContainer for constructing a single node MongoDB replica set. To construct a multi-node MongoDB cluster, consider the [mongodb-replica-set project](https://github.com/silaev/mongodb-replica-set/)              
-
 ## Usage example
 
-The following example shows how to create a MongoDbContainer
+The following example shows how to create a MongoDBContainer:
 
 <!--codeinclude-->
-[Creating a MongoDB container](../../../modules/mongodb/src/test/java/org/testcontainers/containers/MongoDbContainerTest.java) inside_block:creatingMongoDbContainer
+[Creating a MongoDB container](../../../modules/mongodb/src/test/java/org/testcontainers/containers/MongoDBContainerTest.java) inside_block:creatingMongoDBContainer
 <!--/codeinclude-->
 
+And how to start it:
+
 <!--codeinclude-->
-[Starting a MongoDB container](../../../modules/mongodb/src/test/java/org/testcontainers/containers/MongoDbContainerTest.java) inside_block:startingMongoDbContainer
+[Starting a MongoDB container](../../../modules/mongodb/src/test/java/org/testcontainers/containers/MongoDBContainerTest.java) inside_block:startingMongoDBContainer
 <!--/codeinclude-->
 
+!!! note
+    To construct a multi-node MongoDB cluster, consider the [mongodb-replica-set project](https://github.com/silaev/mongodb-replica-set/)     
+
 #### Motivation
 Implement a reusable, cross-platform, simple to install solution that doesn't depend on 
 fixed ports to test MongoDB transactions.  
@@ -31,7 +34,7 @@ For instance, to initialize a single node replica set on fixed ports via Docker,
 * Provide a special url for a user to employ with a MongoDB driver without specifying replicaSet
 
 As we can see, there is a lot of operations to execute and we even haven't touched a non-fixed port approach.
-That's where the MongoDbContainer might come in handy. 
+That's where the MongoDBContainer might come in handy. 
 
 ## Adding this module to your project dependencies
 

…tainers/containers/MongoDbContainer.java …tainers/containers/MongoDBContainer.javamodules/mongodb/src/main/java/org/testcontainers/containers/MongoDbContainer.java renamed to modules/mongodb/src/main/java/org/testcontainers/containers/MongoDBContainer.java modules/mongodb/src/main/java/org/testcontainers/containers/MongoDbContainer.java renamed to modules/mongodb/src/main/java/org/testcontainers/containers/MongoDBContainer.java

@@ -11,30 +11,28 @@
 /**
  * Constructs a single node MongoDB replica set for testing transactions.
  * <p>To construct a multi-node MongoDB cluster, consider the <a href="https://github.com/silaev/mongodb-replica-set/">mongodb-replica-set project on GitHub</a>
- * <p>Tested on a Mongo DB version 4.0.10+ (that is the default version if not specified).
- *
- * @author Konstantin Silaev on 9/30/2019
+ * <p>Tested on a MongoDB version 4.0.10+ (that is the default version if not specified).
  */
 @Slf4j
-public class MongoDbContainer extends GenericContainer<MongoDbContainer> {
+public class MongoDBContainer extends GenericContainer<MongoDBContainer> {
     private static final int CONTAINER_EXIT_CODE_OK = 0;
     private static final int MONGODB_INTERNAL_PORT = 27017;
     private static final int AWAIT_INIT_REPLICA_SET_ATTEMPTS = 60;
     private static final String MONGODB_VERSION_DEFAULT = "4.0.10";
     private static final String MONGODB_DATABASE_NAME_DEFAULT = "test";
 
-    public MongoDbContainer() {
+    public MongoDBContainer() {
         super("mongo:" + MONGODB_VERSION_DEFAULT);
-        configureMongoDbContainer();
+        configureMongoDBContainer();
 
     }
 
-    public MongoDbContainer(@NonNull final String dockerImageName) {
+    public MongoDBContainer(@NonNull final String dockerImageName) {
         super(dockerImageName);
-        configureMongoDbContainer();
+        configureMongoDBContainer();
     }
 
-    private void configureMongoDbContainer() {
+    private void configureMongoDBContainer() {
         withExposedPorts(MONGODB_INTERNAL_PORT);
         withCommand("--replSet", "docker-rs");
         waitingFor(
@@ -44,7 +42,7 @@ private void configureMongoDbContainer() {
 
     public String getReplicaSetUrl() {
         if (!isRunning()) {
-            throw new IllegalStateException("MongoDbContainer should be started first");
+            throw new IllegalStateException("MongoDBContainer should be started first");
         }
         return String.format(
             "mongodb://%s:%d/%s",
@@ -54,25 +52,11 @@ public String getReplicaSetUrl() {
         );
     }
 
-    @Override
-    public void start() {
-        super.start();
-        logReplicaSetStatus();
-    }
-
     @Override
     protected void containerIsStarted(InspectContainerResponse containerInfo) {
         initReplicaSet();
     }
 
-    @SneakyThrows(value = {IOException.class, InterruptedException.class})
-    private void logReplicaSetStatus() {
-        log.debug(
-            "REPLICA SET STATUS:\n{}",
-            execInContainer(buildMongoEvalCommand("rs.status()")).getStdout()
-        );
-    }
-
     private String[] buildMongoEvalCommand(final String command) {
         return new String[]{"mongo", "--eval", command};
     }

…ers/containers/MongoDbContainerTest.java …ers/containers/MongoDBContainerTest.javamodules/mongodb/src/test/java/org/testcontainers/containers/MongoDbContainerTest.java renamed to modules/mongodb/src/test/java/org/testcontainers/containers/MongoDBContainerTest.java modules/mongodb/src/test/java/org/testcontainers/containers/MongoDbContainerTest.java renamed to modules/mongodb/src/test/java/org/testcontainers/containers/MongoDBContainerTest.java

@@ -16,23 +16,23 @@
 import static org.junit.Assert.assertNotNull;
 
 
-public class MongoDbContainerTest {
+public class MongoDBContainerTest {
     /**
      * Taken from <a href="https://docs.mongodb.com/manual/core/transactions/">https://docs.mongodb.com</a>
      */
     @Test
     public void shouldExecuteTransactions() {
         try (
-            // creatingMongoDbContainer {
-            final MongoDbContainer mongoDbContainer = new MongoDbContainer()
+            // creatingMongoDBContainer {
+            final MongoDBContainer mongoDBContainer = new MongoDBContainer()
             // }
         ) {
 
-            // startingMongoDbContainer {
-            mongoDbContainer.start();
+            // startingMongoDBContainer {
+            mongoDBContainer.start();
             // }
 
-            final String mongoRsUrl = mongoDbContainer.getReplicaSetUrl();
+            final String mongoRsUrl = mongoDBContainer.getReplicaSetUrl();
             assertNotNull(mongoRsUrl);
             final MongoClient mongoSyncClient = MongoClients.create(mongoRsUrl);
             mongoSyncClient.getDatabase("mydb1").getCollection("foo")