Make function to package private

Signed-off-by: xiepuhuan <[email protected]>

Author: xiepuhuan

instrumentation/mongo/mongo-3.1/testing/src/main/java/io/opentelemetry/instrumentation/mongo/v3_1/AbstractMongo31ClientTest.java

@@ -35,15 +35,15 @@ public abstract class AbstractMongo31ClientTest
   private MongoClient client;
 
   @BeforeAll
-  public void setup() {
+  void setup() {
     MongoClientOptions.Builder options =
         MongoClientOptions.builder().description("some-description");
     configureMongoClientOptions(options);
     client = new MongoClient(new ServerAddress(host, port), options.build());
   }
 
   @AfterAll
-  public void cleanup() {
+  void cleanup() {
     if (client != null) {
       client.close();
     }
@@ -209,7 +209,7 @@ protected void error(String dbName, String collectionName) {
 
   @Test
   @DisplayName("test client failure")
-  public void testClientFailure() {
+  void testClientFailure() {
     MongoClientOptions options = MongoClientOptions.builder().serverSelectionTimeout(10).build();
     MongoClient client = new MongoClient(new ServerAddress(host, PortUtils.UNUSABLE_PORT), options);
 

instrumentation/mongo/mongo-3.7/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/mongo/v3_7/MongoClientTest.java

@@ -231,16 +231,13 @@ protected void error(String dbName, String collectionName) {
   @Test
   @DisplayName("test client failure")
   void testClientFailure() {
-    String dbName = "test_db";
-    String collectionName = createCollectionName();
-
     MongoClient client =
         MongoClients.create("mongodb://" + host + ":" + UNUSABLE_PORT + "/?connectTimeoutMS=10");
 
     assertThatThrownBy(
             () -> {
-              MongoDatabase db = client.getDatabase(dbName);
-              db.createCollection(collectionName);
+              MongoDatabase db = client.getDatabase("test_db");
+              db.createCollection(createCollectionName());
             })
         .isInstanceOf(MongoTimeoutException.class);
     // Unfortunately not caught by our instrumentation.

instrumentation/mongo/mongo-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/mongo/v4_0/Mongo4ReactiveClientTest.java

@@ -46,12 +46,12 @@ class Mongo4ReactiveClientTest extends AbstractMongoClientTest<MongoCollection<D
   private final List<AutoCloseable> cleanup = new ArrayList<>();
 
   @BeforeAll
-  public void setup() {
+  void setup() {
     client = MongoClients.create("mongodb://" + host + ":" + port);
   }
 
   @AfterAll
-  public void cleanup() throws Exception {
+  void cleanup() throws Exception {
     if (client != null) {
       client.close();
       client = null;

instrumentation/mongo/mongo-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/mongo/v4_0/MongoClientTest.java

@@ -36,12 +36,12 @@ class MongoClientTest extends AbstractMongoClientTest<MongoCollection<Document>>
   private MongoClient client;
 
   @BeforeAll
-  public void setup() {
+  void setup() {
     client = MongoClients.create("mongodb://" + host + ":" + port);
   }
 
   @AfterAll
-  public void cleanup() {
+  void cleanup() {
     if (client != null) {
       client.close();
       client = null;

instrumentation/mongo/mongo-async-3.3/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/mongoasync/v3_3/MongoAsyncClientTest.java

@@ -37,7 +37,7 @@ class MongoAsyncClientTest extends AbstractMongoClientTest<MongoCollection<Docum
   private MongoClient client;
 
   @BeforeAll
-  public void setup() {
+  void setup() {
     client =
         MongoClients.create(
             MongoClientSettings.builder()
@@ -51,7 +51,7 @@ public void setup() {
   }
 
   @AfterAll
-  public void cleanup() {
+  void cleanup() {
     if (client != null) {
       client.close();
       client = null;

instrumentation/mongo/mongo-common/testing/src/main/java/io/opentelemetry/instrumentation/mongo/testing/AbstractMongoClientTest.java

@@ -578,10 +578,7 @@ void testCollectionNameForGetMoreCommand() {
   @Test
   @DisplayName("test error")
   void testError() {
-    String dbName = "test_db";
-    String collectionName = createCollectionName();
-
-    assertThatIllegalArgumentException().isThrownBy(() -> error(dbName, collectionName));
+    assertThatIllegalArgumentException().isThrownBy(() -> error("test_db", createCollectionName()));
     // Unfortunately not caught by our instrumentation.
     testing().waitAndAssertTraces();
   }