Make logging consistent in tests

Author: Avery-Dunn

msal4j-sdk/src/integrationtest/java/com.microsoft.aad.msal4j/AcquireTokenInteractiveIT.java

@@ -27,7 +27,7 @@
 
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 class AcquireTokenInteractiveIT extends SeleniumTest {
-    private final static Logger LOG = LoggerFactory.getLogger(AuthorizationCodeIT.class);
+    private static final Logger LOG = LoggerFactory.getLogger(AcquireTokenInteractiveIT.class);
 
     private Config cfg;
 
@@ -145,7 +145,7 @@ void acquireTokenInteractive_Ciam() {
             result = pca.acquireToken(parameters).get();
 
         } catch (Exception e) {
-            LOG.error("Error acquiring token with authCode: " + e.getMessage());
+            LOG.error("Error acquiring token with authCode: {}", e.getMessage());
             throw new RuntimeException("Error acquiring token with authCode: " + e.getMessage());
         }
 
@@ -236,7 +236,7 @@ private IAuthenticationResult acquireTokenInteractive(
             result = pca.acquireToken(parameters).get();
 
         } catch (Exception e) {
-            LOG.error("Error acquiring token with authCode: " + e.getMessage());
+            LOG.error("Error acquiring token with authCode: {}", e.getMessage());
             throw new RuntimeException("Error acquiring token with authCode: " + e.getMessage());
         }
         return result;
@@ -266,7 +266,7 @@ private IAuthenticationResult acquireTokenInteractive_instanceAware(
             result = pca.acquireToken(parameters).get();
 
         } catch (Exception e) {
-            LOG.error("Error acquiring token with authCode: " + e.getMessage());
+            LOG.error("Error acquiring token with authCode: {}", e.getMessage());
             throw new RuntimeException("Error acquiring token with authCode: " + e.getMessage());
         }
         return result;

msal4j-sdk/src/integrationtest/java/com.microsoft.aad.msal4j/AuthorizationCodeIT.java

@@ -28,7 +28,7 @@
 
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 class AuthorizationCodeIT extends SeleniumTest {
-    private final static Logger LOG = LoggerFactory.getLogger(AuthorizationCodeIT.class);
+    private static final Logger LOG = LoggerFactory.getLogger(AuthorizationCodeIT.class);
 
     private Config cfg;
 
@@ -202,7 +202,7 @@ private IAuthenticationResult acquireTokenAuthorizationCodeFlow(
                     .get();
 
         } catch (Exception e) {
-            LOG.error("Error acquiring token with authCode: " + e.getMessage());
+            LOG.error("Error acquiring token with authCode: {}", e.getMessage());
             throw new RuntimeException("Error acquiring token with authCode: " + e.getMessage());
         }
         return result;
@@ -219,7 +219,7 @@ private IAuthenticationResult acquireTokenInteractiveB2C(ConfidentialClientAppli
                     .build())
                     .get();
         } catch (Exception e) {
-            LOG.error("Error acquiring token with authCode: " + e.getMessage());
+            LOG.error("Error acquiring token with authCode: {}", e.getMessage());
             throw new RuntimeException("Error acquiring token with authCode: " + e.getMessage());
         }
         return result;

msal4j-sdk/src/integrationtest/java/com.microsoft.aad.msal4j/DeviceCodeIT.java

@@ -23,7 +23,7 @@
 
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 class DeviceCodeIT {
-    private final static Logger LOG = LoggerFactory.getLogger(DeviceCodeIT.class);
+    private static final Logger LOG = LoggerFactory.getLogger(DeviceCodeIT.class);
 
     private LabUserProvider labUserProvider;
     private WebDriver seleniumDriver;
@@ -152,7 +152,7 @@ private void runAutomatedDeviceCodeFlow(DeviceCode deviceCode, User user) {
             if (!isRunningLocally) {
                 SeleniumExtensions.takeScreenShot(seleniumDriver);
             }
-            LOG.error("Browser automation failed: " + e.getMessage());
+            LOG.error("Browser automation failed: {}", e.getMessage());
             throw new RuntimeException("Browser automation failed: " + e.getMessage());
         }
     }