chore: Add superficial code formatting fixes. (#2161)

This contains superficial changes to annotations. This tags specific fields labeled as deprecated in the javadoc
with @deprecated annotation. Also, this adds @RunWith(JUnit4.class) to several test cases to make
the test classes explicit.

Author: hessjcg

core/src/main/java/com/google/cloud/sql/core/CoreSocketFactory.java

@@ -23,6 +23,7 @@
  * @deprecated Use the official java API instead.
  * @see com.google.cloud.sql.ConnectorRegistry
  */
+@SuppressWarnings("InlineMeSuggester")
 @Deprecated
 public final class CoreSocketFactory {
 
@@ -80,6 +81,7 @@ public final class CoreSocketFactory {
    * @deprecated Use the official java API instead.
    * @see com.google.cloud.sql.ConnectorRegistry
    */
+  @Deprecated
   static void setApplicationName(String artifactId) {
     InternalConnectorRegistry.setApplicationName(artifactId);
   }

core/src/main/java/com/google/cloud/sql/core/InstanceCheckingTrustManger.java

@@ -170,7 +170,7 @@ private List<String> getSans(X509Certificate cert) throws CertificateException {
       return names;
     }
 
-    for (List item : sanAsn1Field) {
+    for (List<?> item : sanAsn1Field) {
       Integer type = (Integer) item.get(0);
       // RFC 5280 section 4.2.1.6.  "Subject Alternative Name"
       // describes the structure of subjectAlternativeName record.

core/src/main/java/com/google/cloud/sql/core/InternalConnectorRegistry.java

@@ -71,7 +71,7 @@ public final class InternalConnectorRegistry {
    * @deprecated Use {@link #setApplicationName(String)} to set the application name
    *     programmatically.
    */
-  static final String USER_TOKEN_PROPERTY_NAME = "_CLOUD_SQL_USER_TOKEN";
+  @Deprecated static final String USER_TOKEN_PROPERTY_NAME = "_CLOUD_SQL_USER_TOKEN";
 
   @VisibleForTesting
   InternalConnectorRegistry(

core/src/main/java/com/google/cloud/sql/core/JndiDnsResolver.java

@@ -61,7 +61,7 @@ public Collection<String> resolveTxt(String domainName)
       // See https://docs.oracle.com/javase/7/docs/technotes/guides/jndi/jndi-dns.html
 
       // Explicitly reference the JNDI DNS classes. This is required for GraalVM.
-      Hashtable contextProps = new Hashtable<>();
+      Hashtable<String, String> contextProps = new Hashtable<>();
       contextProps.put(Context.INITIAL_CONTEXT_FACTORY, "com.sun.jndi.dns.DnsContextFactory");
       contextProps.put(Context.OBJECT_FACTORIES, "com.sun.jndi.url.dns.dnsURLContextFactory");
       Attribute attr =

core/src/test/java/com/google/cloud/sql/ConnectorConfigTest.java

@@ -28,7 +28,10 @@
 import java.util.List;
 import java.util.function.Supplier;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
 
+@RunWith(JUnit4.class)
 public class ConnectorConfigTest {
   @Test
   public void testConfigFromBuilder() {

core/src/test/java/com/google/cloud/sql/CredentialFactoryTest.java

@@ -34,7 +34,10 @@
 import java.io.IOException;
 import java.util.Collections;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
 
+@RunWith(JUnit4.class)
 public class CredentialFactoryTest {
 
   @Test

core/src/test/java/com/google/cloud/sql/core/ApiClientRetryingCallableTest.java

@@ -23,7 +23,10 @@
 import java.util.concurrent.atomic.AtomicInteger;
 import org.junit.Assert;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
 
+@RunWith(JUnit4.class)
 public class ApiClientRetryingCallableTest {
   @Test
   public void testApiClientRetriesOn500ErrorAndSucceeds() throws Exception {

core/src/test/java/com/google/cloud/sql/core/ConnectionConfigTest.java

@@ -27,7 +27,10 @@
 import java.util.Properties;
 import java.util.stream.Collectors;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
 
+@RunWith(JUnit4.class)
 public class ConnectionConfigTest {
 
   @Test

core/src/test/java/com/google/cloud/sql/core/ConnectorTest.java

@@ -45,7 +45,10 @@
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
 
+@RunWith(JUnit4.class)
 public class ConnectorTest extends CloudSqlCoreTestingBase {
   ListeningScheduledExecutorService defaultExecutor;
   private final long TEST_MAX_REFRESH_MS = 5000L;

core/src/test/java/com/google/cloud/sql/core/ConstantCredentialsFactoryTest.java

@@ -20,7 +20,10 @@
 
 import com.google.auth.oauth2.GoogleCredentials;
 import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.JUnit4;
 
+@RunWith(JUnit4.class)
 public class ConstantCredentialsFactoryTest {
 
   @Test