commit cbab6ca6e2867520452c894c2d20a4c8e75874ab
author Andreas Gampe <agampe@google.com> Tue Feb 20 18:51:41 2018 -0800
committer Andreas Gampe <agampe@google.com> Wed Feb 28 02:33:57 2018 +0000
tree 9f3fc855bf623a7933efe14f483f2b975da6cb08
parent 5bec64c5bc5e7928581c1bbccb7a803d9830df44

Wifi: Slightly refactor LogcatLogTest

Refactor so that there is no issue with CheckReturnValue warnings.

(cherry picked from commit bb5b6a2a1dbd68d582d032bf89f97ef2d65ee8ed)

Bug: 73094025
Test: m javac-check RUN_ERROR_PRONE=true
Merged-In: I83df7baf22c4329d1a8e24c17b1b8d656953d201
Change-Id: I83df7baf22c4329d1a8e24c17b1b8d656953d201

service/tests/wifitests/src/com/android/server/wifi/LogcatLogTest.java

diff --git a/service/tests/wifitests/src/com/android/server/wifi/LogcatLogTest.java b/service/tests/wifitests/src/com/android/server/wifi/LogcatLogTest.java
index ac5afe7..be183b2 100644
--- a/service/tests/wifitests/src/com/android/server/wifi/LogcatLogTest.java
+++ b/service/tests/wifitests/src/com/android/server/wifi/LogcatLogTest.java
@@ -131,9 +131,7 @@
     @Test
     public void logMessageSilentlyIgnoresExtraneousValues() {
         WifiLog.LogMessage logMessage = mLogger.err("%");
-        logMessage.c("hello world");
-        logMessage.c("more stuff");
-        logMessage.flush();
+        logMessage.c("hello world").c("more stuff").flush();
         assertEquals("hello world", logMessage.toString());
     }
 
@@ -144,9 +142,7 @@
     @Test
     public void logMessageSilentlyIgnoresExtraneousValuesEvenForEmptyFormat() {
         WifiLog.LogMessage logMessage = mLogger.err("");
-        logMessage.c("hello world");
-        logMessage.c("more stuff");
-        logMessage.flush();
+        logMessage.c("hello world").c("more stuff").flush();
         assertEquals("", logMessage.toString());
     }
 
@@ -157,9 +153,7 @@
     @Test
     public void logMessageSilentlyIgnoresExtraneousValuesEvenForFormatWithoutPlaceholders() {
         WifiLog.LogMessage logMessage = mLogger.err("literal format");
-        logMessage.c("hello world");
-        logMessage.c("more stuff");
-        logMessage.flush();
+        logMessage.c("hello world").c("more stuff").flush();
         assertEquals("literal format", logMessage.toString());
     }