Reduce some unnecessary Guava usage

gocd · Oct 2, 2023 · b5629d9 · b5629d9
1 parent e3b8a92
commit b5629d9
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 34 deletions.
diff --git a/common/src/test/java/com/thoughtworks/go/server/service/InstanceFactoryTest.java b/common/src/test/java/com/thoughtworks/go/server/service/InstanceFactoryTest.java
@@ -15,7 +15,6 @@
  */
 package com.thoughtworks.go.server.service;
 
-import com.google.common.collect.ImmutableMap;
 import com.thoughtworks.go.config.*;
 import com.thoughtworks.go.config.elastic.ClusterProfile;
 import com.thoughtworks.go.config.elastic.ElasticProfile;
@@ -30,10 +29,7 @@
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-import java.util.ArrayList;
-import java.util.Date;
-import java.util.List;
-import java.util.UUID;
+import java.util.*;
 
 import static com.thoughtworks.go.domain.JobResult.Passed;
 import static com.thoughtworks.go.domain.JobResult.Unknown;
@@ -321,7 +317,7 @@ void shouldCreateJobPlan() {
     @Test
     void shouldAddElasticProfileOnJobPlan() {
         ElasticProfile elasticProfile = new ElasticProfile("id", "prod-cluster");
-        DefaultSchedulingContext context = new DefaultSchedulingContext("foo", new Agents(), ImmutableMap.of("id", elasticProfile));
+        DefaultSchedulingContext context = new DefaultSchedulingContext("foo", new Agents(), Map.of("id", elasticProfile));
 
         ArtifactTypeConfigs artifactTypeConfigs = new ArtifactTypeConfigs();
         JobConfig jobConfig = new JobConfig(new CaseInsensitiveString("test"), null, artifactTypeConfigs);
@@ -336,7 +332,7 @@ void shouldAddElasticProfileOnJobPlan() {
     void shouldAddElasticProfileAndClusterProfileOnJobPlan() {
         ElasticProfile elasticProfile = new ElasticProfile("id", "clusterId");
         ClusterProfile clusterProfile = new ClusterProfile("clusterId", "pluginId");
-        DefaultSchedulingContext context = new DefaultSchedulingContext("foo", new Agents(), ImmutableMap.of("id", elasticProfile), ImmutableMap.of("clusterId", clusterProfile));
+        DefaultSchedulingContext context = new DefaultSchedulingContext("foo", new Agents(), Map.of("id", elasticProfile), Map.of("clusterId", clusterProfile));
 
         ArtifactTypeConfigs artifactTypeConfigs = new ArtifactTypeConfigs();
         JobConfig jobConfig = new JobConfig(new CaseInsensitiveString("test"), null, artifactTypeConfigs);

diff --git a/...r/src/test-fast/java/com/thoughtworks/go/server/service/EnvironmentConfigServiceTest.java b/...r/src/test-fast/java/com/thoughtworks/go/server/service/EnvironmentConfigServiceTest.java
@@ -15,7 +15,6 @@
  */
 package com.thoughtworks.go.server.service;
 
-import com.google.common.collect.ImmutableMap;
 import com.thoughtworks.go.config.*;
 import com.thoughtworks.go.config.commands.EntityConfigUpdateCommand;
 import com.thoughtworks.go.config.exceptions.RecordNotFoundException;
@@ -536,7 +535,7 @@ class EnvironmentConfigSyncTest {
 
         @Test
         void shouldSyncEnvironmentsFromAgentAssociationInDB1() {
-            shouldSyncEnvironmentsFromAgentAssociationInDB(ImmutableMap.of(
+            shouldSyncEnvironmentsFromAgentAssociationInDB(Map.of(
                     "dev", "uuid1",
                     "test", "uuid1",
                     "stage", "uuid2",
@@ -546,15 +545,15 @@ void shouldSyncEnvironmentsFromAgentAssociationInDB1() {
 
         @Test
         void shouldSyncEnvironmentsFromAgentAssociationInDB2() {
-            shouldSyncEnvironmentsFromAgentAssociationInDB(ImmutableMap.of(
+            shouldSyncEnvironmentsFromAgentAssociationInDB(Map.of(
                     "dev", "uuid1",
                     "stage", "uuid2"
             ));
         }
 
         @Test
         void shouldSyncEnvironmentsFromAgentAssociationInDB5() {
-            shouldSyncEnvironmentsFromAgentAssociationInDB(ImmutableMap.of(
+            shouldSyncEnvironmentsFromAgentAssociationInDB(Map.of(
                     "dev", "uuid3",
                     "test", "uuid3",
                     "stage", "uuid3",

diff --git a/spark/spark-spa/src/main/java/com/thoughtworks/go/spark/spa/LoginPageController.java b/spark/spark-spa/src/main/java/com/thoughtworks/go/spark/spa/LoginPageController.java
@@ -15,7 +15,6 @@
  */
 package com.thoughtworks.go.spark.spa;
 
-import com.google.common.collect.ImmutableMap;
 import com.thoughtworks.go.server.newsecurity.utils.SessionUtils;
 import com.thoughtworks.go.server.service.SecurityService;
 import com.thoughtworks.go.spark.Routes;
@@ -64,14 +63,7 @@ public ModelAndView show(Request request, Response response) {
             return null;
         }
 
-        Map<String, Object> meta = loginLogoutHelper.buildMeta(request);
-
-        Map<String, Object> object = ImmutableMap.<String, Object>builder()
-                .put("viewTitle", "Login")
-                .put("meta", meta)
-                .build();
-
-        return new ModelAndView(object, null);
+        return new ModelAndView(Map.of("viewTitle", "Login", "meta", loginLogoutHelper.buildMeta(request)), null);
     }
 
     private boolean securityIsDisabledOrAlreadyLoggedIn(HttpServletRequest request) {

diff --git a/spark/spark-spa/src/main/java/com/thoughtworks/go/spark/spa/LogoutPageController.java b/spark/spark-spa/src/main/java/com/thoughtworks/go/spark/spa/LogoutPageController.java
@@ -15,7 +15,6 @@
  */
 package com.thoughtworks.go.spark.spa;
 
-import com.google.common.collect.ImmutableMap;
 import com.thoughtworks.go.server.newsecurity.utils.SessionUtils;
 import com.thoughtworks.go.spark.Routes;
 import com.thoughtworks.go.spark.SparkController;
@@ -50,13 +49,6 @@ public void setupRoutes() {
     public ModelAndView show(Request request, Response response) {
         SessionUtils.recreateSessionWithoutCopyingOverSessionState(request.raw());
 
-        Map<String, Object> meta = loginLogoutHelper.buildMeta(request);
-
-        Map<String, Object> object = ImmutableMap.<String, Object>builder()
-                .put("viewTitle", "Logout")
-                .put("meta", meta)
-                .build();
-
-        return new ModelAndView(object, null);
+        return new ModelAndView(Map.of("viewTitle", "Logout", "meta", loginLogoutHelper.buildMeta(request)), null);
     }
 }
diff --git a/spark/spark-spa/src/main/java/com/thoughtworks/go/spark/spa/NewPreferencesController.java b/spark/spark-spa/src/main/java/com/thoughtworks/go/spark/spa/NewPreferencesController.java
@@ -16,7 +16,6 @@
 
 package com.thoughtworks.go.spark.spa;
 
-import com.google.common.collect.ImmutableMap;
 import com.thoughtworks.go.server.service.GoConfigService;
 import com.thoughtworks.go.spark.Routes;
 import com.thoughtworks.go.spark.SparkController;
@@ -55,12 +54,9 @@ public void setupRoutes() {
     }
 
     public ModelAndView index(Request request, Response response) {
-        Map<String, Object> meta = ImmutableMap.<String, Object>builder()
-                .put("smtp_configured", goConfigService.isSmtpEnabled())
-                .build();
         Map<String, Object> object = Map.of(
             "viewTitle", "Preferences",
-            "meta", meta
+            "meta", Map.of("smtp_configured", goConfigService.isSmtpEnabled())
         );
         return new ModelAndView(object, null);
     }