Rename to "appendMetadata".

JAVA-5870

Author: vbabanin

driver-kotlin-coroutine/src/integrationTest/kotlin/com/mongodb/kotlin/client/coroutine/syncadapter/SyncMongoClient.kt

@@ -25,7 +25,7 @@ internal class SyncMongoClient(override val wrapped: MongoClient) : SyncMongoClu
 
     override fun getClusterDescription(): ClusterDescription = wrapped.getClusterDescription()
 
-    override fun updateMetadata(mongoDriverInformation: MongoDriverInformation) {
+    override fun appendMetadata(mongoDriverInformation: MongoDriverInformation) {
         throw UnsupportedOperationException("TODO-JAVA-5871")
     }
 }

driver-kotlin-coroutine/src/test/kotlin/com/mongodb/kotlin/client/coroutine/MongoClientTest.kt

@@ -48,8 +48,8 @@ class MongoClientTest {
             JMongoClient::class
                 .declaredFunctions
                 .map { it.name }
-                // TODO-JAVA-5871 remove .filterNot { it == "updateMetadata" }
-                .filterNot { it == "updateMetadata" }
+                // TODO-JAVA-5871 remove .filterNot { it == "appendMetadata" }
+                .filterNot { it == "appendMetadata" }
                 .toSet()
         val kMongoClientFunctions = MongoClient::class.declaredFunctions.map { it.name }.toSet()
 

driver-kotlin-sync/src/integrationTest/kotlin/com/mongodb/kotlin/client/syncadapter/SyncMongoClient.kt

@@ -24,7 +24,7 @@ internal class SyncMongoClient(override val wrapped: MongoClient) : SyncMongoClu
     override fun close(): Unit = wrapped.close()
 
     override fun getClusterDescription(): ClusterDescription = wrapped.clusterDescription
-    override fun updateMetadata(mongoDriverInformation: MongoDriverInformation) {
+    override fun appendMetadata(mongoDriverInformation: MongoDriverInformation) {
         throw UnsupportedOperationException("TODO-JAVA-5871")
     }
 }

driver-kotlin-sync/src/test/kotlin/com/mongodb/kotlin/client/MongoClientTest.kt

@@ -47,8 +47,8 @@ class MongoClientTest {
             JMongoClient::class
                 .declaredFunctions
                 .map { it.name }
-                // TODO-JAVA-5871 remove .filterNot { it == "updateMetadata" }
-                .filterNot { it == "updateMetadata" }
+                // TODO-JAVA-5871 remove .filterNot { it == "appendMetadata" }
+                .filterNot { it == "appendMetadata" }
                 .toSet()
 
         val kMongoClientFunctions =

driver-reactive-streams/src/main/com/mongodb/reactivestreams/client/MongoClient.java

@@ -61,8 +61,8 @@ public interface MongoClient extends MongoCluster, Closeable {
     ClusterDescription getClusterDescription();
 
     /**
-     * Updates the {@link MongoClient} metadata by appending the provided {@link MongoDriverInformation}
-     * to the existing metadata.
+     * Appends the provided {@link MongoDriverInformation} to the existing metadata.
+     *
      * <p>
      * This enables frameworks and libraries to include identifying metadata (e.g., name, version, platform) which might be visible in
      * the MongoD/MongoS logs. This can assist with diagnostics by making client identity visible to the server.
@@ -72,5 +72,5 @@ public interface MongoClient extends MongoCluster, Closeable {
      * @param mongoDriverInformation the driver information to append to the existing metadata
      * @since 5.6
      */
-    void updateMetadata(MongoDriverInformation mongoDriverInformation);
+    void appendMetadata(MongoDriverInformation mongoDriverInformation);
 }

driver-reactive-streams/src/main/com/mongodb/reactivestreams/client/internal/MongoClientImpl.java

@@ -327,7 +327,7 @@ public ClusterDescription getClusterDescription() {
     }
 
     @Override
-    public void updateMetadata(final MongoDriverInformation mongoDriverInformation) {
+    public void appendMetadata(final MongoDriverInformation mongoDriverInformation) {
         getCluster().getClientMetadata().append(mongoDriverInformation);
     }
 }

driver-reactive-streams/src/test/functional/com/mongodb/reactivestreams/client/syncadapter/SyncMongoClient.java

@@ -313,7 +313,7 @@ public ClusterDescription getClusterDescription() {
     }
 
     @Override
-    public void updateMetadata(final MongoDriverInformation mongoDriverInformation) {
-        wrapped.updateMetadata(mongoDriverInformation);
+    public void appendMetadata(final MongoDriverInformation mongoDriverInformation) {
+        wrapped.appendMetadata(mongoDriverInformation);
     }
 }

driver-scala/src/integrationTest/scala/org/mongodb/scala/syncadapter/SyncMongoClient.scala

@@ -1,7 +1,7 @@
 package org.mongodb.scala.syncadapter
 
 import com.mongodb.MongoDriverInformation
-import com.mongodb.client.{ MongoClient => JMongoClient }
+import com.mongodb.client.{MongoClient => JMongoClient}
 import org.mongodb.scala.MongoClient
 
 case class SyncMongoClient(wrapped: MongoClient) extends SyncMongoCluster(wrapped) with JMongoClient {
@@ -10,6 +10,6 @@ case class SyncMongoClient(wrapped: MongoClient) extends SyncMongoCluster(wrappe
 
   override def getClusterDescription = throw new UnsupportedOperationException
 
-  override def updateMetadata(mongoDriverInformation: MongoDriverInformation): Unit =
+  override def appendMetadata(mongoDriverInformation: MongoDriverInformation): Unit =
     throw new UnsupportedOperationException("TODO-JAVA-5871")
 }

driver-scala/src/test/scala/org/mongodb/scala/MongoClientSpec.scala

@@ -16,10 +16,10 @@
 
 package org.mongodb.scala
 
-import com.mongodb.reactivestreams.client.{ MongoClient => JMongoClient }
+import com.mongodb.reactivestreams.client.{MongoClient => JMongoClient}
 import org.bson.BsonDocument
 import org.mockito.Mockito.verify
-import org.mongodb.scala.model.bulk.{ ClientBulkWriteOptions, ClientNamespacedWriteModel }
+import org.mongodb.scala.model.bulk.{ClientBulkWriteOptions, ClientNamespacedWriteModel}
 import org.scalatestplus.mockito.MockitoSugar
 
 import scala.collection.JavaConverters._
@@ -37,7 +37,7 @@ class MongoClientSpec extends BaseSpec with MockitoSugar {
 
     wrapped.foreach((name: String) => {
       // TODO-JAVA-5871 remove this if statement, but leave the body.
-      if (!name.equals("updateMetadata")) {
+      if (!name.equals("appendMetadata")) {
         val cleanedName = name.stripPrefix("get")
         assert(local.contains(name) | local.contains(cleanedName.head.toLower + cleanedName.tail), s"Missing: $name")
       }

driver-sync/src/main/com/mongodb/client/MongoClient.java

@@ -64,8 +64,7 @@ public interface MongoClient extends MongoCluster, Closeable {
     ClusterDescription getClusterDescription();
 
     /**
-     * Updates the {@link MongoClient} metadata by appending the provided {@link MongoDriverInformation}
-     * to the existing metadata.
+     * Appends the provided {@link MongoDriverInformation} to the existing metadata.
      * <p>
      * This enables frameworks and libraries to include identifying metadata (e.g., name, version, platform) which might be visible in
      * the MongoD/MongoS logs. This can assist with diagnostics by making client identity visible to the server.
@@ -75,5 +74,5 @@ public interface MongoClient extends MongoCluster, Closeable {
      * @param mongoDriverInformation the driver information to append to the existing metadata
      * @since 5.6
      */
-    void updateMetadata(MongoDriverInformation mongoDriverInformation);
+    void appendMetadata(MongoDriverInformation mongoDriverInformation);
 }

driver-sync/src/main/com/mongodb/client/internal/MongoClientImpl.java

@@ -136,7 +136,7 @@ public ClusterDescription getClusterDescription() {
     }
 
     @Override
-    public void updateMetadata(final MongoDriverInformation mongoDriverInformation) {
+    public void appendMetadata(final MongoDriverInformation mongoDriverInformation) {
         delegate.getCluster().getClientMetadata().append(mongoDriverInformation);
     }
 

driver-sync/src/test/functional/com/mongodb/client/AbstractClientMetadataProseTest.java

@@ -169,7 +169,7 @@ void shouldAppendToDefaultClientMetadataWhenUpdatedAfterInitialization(@Nullable
     }
 
     @Test
-    void shouldNotCloseExistingConnectionsToUpdateMetadata() {
+    void shouldNotCloseExistingConnectionsToAppendMetadata() {
         //given
         MongoDriverInformation initialWrappingLibraryDriverInformation = MongoDriverInformation.builder()
                 .driverName("library")
@@ -186,7 +186,7 @@ void shouldNotCloseExistingConnectionsToUpdateMetadata() {
             assertNotNull(driverInformation);
 
             //when
-            mongoClient.updateMetadata(initialWrappingLibraryDriverInformation);
+            mongoClient.appendMetadata(initialWrappingLibraryDriverInformation);
 
             //then
             assertThat(executePingAndCaptureMetadataHandshake(mongoClient)).isEmpty();
@@ -260,7 +260,7 @@ private static void updateClientMetadata(@Nullable final String driverVersion,
         ofNullable(driverName).ifPresent(builder::driverName);
         ofNullable(driverVersion).ifPresent(builder::driverVersion);
         ofNullable(driverPlatform).ifPresent(builder::driverPlatform);
-        mongoClient.updateMetadata(builder.build());
+        mongoClient.appendMetadata(builder.build());
     }
 }