From 09a75266c6195777362e4765fc36813976c91cc2 Mon Sep 17 00:00:00 2001
From: Tomo Suzuki <suztomo@gmail.com>
Date: Wed, 28 Feb 2024 14:34:34 -0500
Subject: [PATCH] test: do not show unnecessary BatcherImplTest logging (#2519)

This is the follow-up of a pull request #2285, where we wanted to
catch any flaky test failure with logging, just in case the test
failed again.

https://github.com/googleapis/sdk-platform-java/pull/2285/files#r1416300479

It turned out that the fix worked well. We don't observe the test
failures any more. Therefore turning off the excessive logging.
---
 .../com/google/api/gax/batching/BatcherImplTest.java | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/gax-java/gax/src/test/java/com/google/api/gax/batching/BatcherImplTest.java b/gax-java/gax/src/test/java/com/google/api/gax/batching/BatcherImplTest.java
index 314fa81ae2..70340d77d6 100644
--- a/gax-java/gax/src/test/java/com/google/api/gax/batching/BatcherImplTest.java
+++ b/gax-java/gax/src/test/java/com/google/api/gax/batching/BatcherImplTest.java
@@ -873,7 +873,7 @@ public void testThrottlingBlocking() throws Exception {
                 public void run() {
                   batcherAddThreadHolder.add(Thread.currentThread());
                   batcher.add(1);
-                  logger.info("Called batcher.add(1)");
+                  logger.fine("Called batcher.add(1)");
                 }
               });
 
@@ -893,20 +893,20 @@ public void run() {
           () -> {
             try {
               Thread.sleep(throttledTime);
-              logger.info("Calling flowController.release");
+              logger.fine("Calling flowController.release");
               flowController.release(1, 1);
-              logger.info("Called flowController.release");
+              logger.fine("Called flowController.release");
             } catch (InterruptedException e) {
             }
           });
 
       try {
-        logger.info("Calling future.get(10 ms)");
+        logger.fine("Calling future.get(10 ms)");
         future.get(10, TimeUnit.MILLISECONDS);
         long afterGetCall = System.currentTimeMillis();
         long actualWaitTimeMs = afterGetCall - beforeGetCall;
 
-        logger.info("future.get(10 ms) unexpectedly returned. Wait time: " + actualWaitTimeMs);
+        logger.fine("future.get(10 ms) unexpectedly returned. Wait time: " + actualWaitTimeMs);
         // In a flaky test troubleshooting
         // (https://github.com/googleapis/sdk-platform-java/issues/1931), we observed that
         // "future.get" method did not throw TimeoutException in this multithreaded test.
@@ -919,7 +919,7 @@ public void run() {
             .isAtLeast(10);
       } catch (TimeoutException e) {
         // expected
-        logger.info("future.get(10 ms) timed out expectedly.");
+        logger.fine("future.get(10 ms) timed out expectedly.");
       }
 
       try {