fixup! Rework the gitconfig configmap configurator

Author: vinokurig

infrastructures/kubernetes/src/main/java/org/eclipse/che/workspace/infrastructure/kubernetes/KubernetesInfraModule.java

@@ -48,7 +48,7 @@
 import org.eclipse.che.workspace.infrastructure.kubernetes.environment.KubernetesEnvironmentFactory;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.RemoveNamespaceOnWorkspaceRemove;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.CredentialsSecretConfigurator;
-import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.GitconfigConfigmapConfigurator;
+import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.GitconfigConfigurator;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.NamespaceConfigurator;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.OAuthTokenSecretsConfigurator;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.PreferencesConfigMapConfigurator;
@@ -109,7 +109,7 @@ protected void configure() {
     namespaceConfigurators.addBinding().to(WorkspaceServiceAccountConfigurator.class);
     namespaceConfigurators.addBinding().to(UserProfileConfigurator.class);
     namespaceConfigurators.addBinding().to(UserPreferencesConfigurator.class);
-    namespaceConfigurators.addBinding().to(GitconfigConfigmapConfigurator.class);
+    namespaceConfigurators.addBinding().to(GitconfigConfigurator.class);
 
     bind(AuthorizationChecker.class).to(KubernetesAuthorizationCheckerImpl.class);
     bind(PermissionsCleaner.class).asEagerSingleton();

infrastructures/kubernetes/src/main/java/org/eclipse/che/workspace/infrastructure/kubernetes/namespace/configurator/GitconfigConfigmapConfigurator.java infrastructures/kubernetes/src/main/java/org/eclipse/che/workspace/infrastructure/kubernetes/namespace/configurator/GitconfigConfigurator.java

@@ -38,10 +38,10 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-public class GitconfigConfigmapConfigurator implements NamespaceConfigurator {
+public class GitconfigConfigurator implements NamespaceConfigurator {
   private final CheServerKubernetesClientFactory cheServerKubernetesClientFactory;
   private final Set<GitUserDataFetcher> gitUserDataFetchers;
-  private static final Logger LOG = LoggerFactory.getLogger(GitconfigConfigmapConfigurator.class);
+  private static final Logger LOG = LoggerFactory.getLogger(GitconfigConfigurator.class);
   private static final String CONFIGMAP_DATA_KEY = "gitconfig";
   // TODO: rename to a more generic name, since it is not only for user data
   private static final String GITCONFIG_CONFIGMAP_NAME = "workspace-userdata-gitconfig-configmap";
@@ -63,7 +63,7 @@ public class GitconfigConfigmapConfigurator implements NamespaceConfigurator {
       Pattern.compile("\\[(?<sectionName>[a-zA-Z0-9]+)](\\n\\s*\\S*\\s*=.*)*");
 
   @Inject
-  public GitconfigConfigmapConfigurator(
+  public GitconfigConfigurator(
       CheServerKubernetesClientFactory cheServerKubernetesClientFactory,
       Set<GitUserDataFetcher> gitUserDataFetchers) {
     this.cheServerKubernetesClientFactory = cheServerKubernetesClientFactory;

infrastructures/kubernetes/src/test/java/org/eclipse/che/workspace/infrastructure/kubernetes/namespace/configurator/GitconfigConfigmapConfiguratorTest.java infrastructures/kubernetes/src/test/java/org/eclipse/che/workspace/infrastructure/kubernetes/namespace/configurator/GitconfigConfiguratorTest.java

@@ -37,7 +37,7 @@
 import org.testng.annotations.Test;
 
 @Listeners(MockitoTestNGListener.class)
-public class GitconfigConfigmapConfiguratorTest {
+public class GitconfigConfiguratorTest {
 
   private NamespaceConfigurator configurator;
 
@@ -65,8 +65,7 @@ public class GitconfigConfigmapConfiguratorTest {
   public void setUp()
       throws InfrastructureException, ScmCommunicationException, ScmUnauthorizedException {
     configurator =
-        new GitconfigConfigmapConfigurator(
-            cheServerKubernetesClientFactory, Collections.emptySet());
+        new GitconfigConfigurator(cheServerKubernetesClientFactory, Collections.emptySet());
 
     serverMock = new KubernetesServer(true, true);
     serverMock.before();
@@ -83,8 +82,7 @@ public void shouldCreateGitconfigConfigmapWithUserSection() throws Exception {
     when(gitUserDataFetcher.fetchGitUserData())
         .thenReturn(new GitUserData("username", "userEmail"));
     configurator =
-        new GitconfigConfigmapConfigurator(
-            cheServerKubernetesClientFactory, Set.of(gitUserDataFetcher));
+        new GitconfigConfigurator(cheServerKubernetesClientFactory, Set.of(gitUserDataFetcher));
     // when
     configurator.configure(namespaceResolutionContext, TEST_NAMESPACE_NAME);
     // then
@@ -125,8 +123,7 @@ public void shouldUpdateGitconfigConfigmapWithUserSection()
         Collections.singletonMap("gitconfig", "[user]\n\tname = \"\"\n\temail= \"\""));
     serverMock.getClient().configMaps().inNamespace(TEST_NAMESPACE_NAME).create(gitconfigConfigmap);
     configurator =
-        new GitconfigConfigmapConfigurator(
-            cheServerKubernetesClientFactory, Set.of(gitUserDataFetcher));
+        new GitconfigConfigurator(cheServerKubernetesClientFactory, Set.of(gitUserDataFetcher));
     when(gitUserDataFetcher.fetchGitUserData())
         .thenReturn(new GitUserData("username", "userEmail"));
     // when
@@ -160,8 +157,7 @@ public void shouldUpdateGitconfigConfigmapWithStoredSectionsWithUserSection()
             "[other]\n\tkey = value\n[other1]\n\tkey = value\n[user]\n\tname = \"\"\n\temail= \"\""));
     serverMock.getClient().configMaps().inNamespace(TEST_NAMESPACE_NAME).create(gitconfigConfigmap);
     configurator =
-        new GitconfigConfigmapConfigurator(
-            cheServerKubernetesClientFactory, Set.of(gitUserDataFetcher));
+        new GitconfigConfigurator(cheServerKubernetesClientFactory, Set.of(gitUserDataFetcher));
     when(gitUserDataFetcher.fetchGitUserData())
         .thenReturn(new GitUserData("username", "userEmail"));
     // when
@@ -195,8 +191,7 @@ public void shouldNotUpdateGitconfigConfigmapWithUserSection()
             "gitconfig", "[user]\n\tname = gitconfig-username\n\temail = gitconfig-email"));
     serverMock.getClient().configMaps().inNamespace(TEST_NAMESPACE_NAME).create(gitconfigConfigmap);
     configurator =
-        new GitconfigConfigmapConfigurator(
-            cheServerKubernetesClientFactory, Set.of(gitUserDataFetcher));
+        new GitconfigConfigurator(cheServerKubernetesClientFactory, Set.of(gitUserDataFetcher));
     when(gitUserDataFetcher.fetchGitUserData())
         .thenReturn(new GitUserData("fetcher-username", "fetcher-userEmail"));
     // when

infrastructures/openshift/src/main/java/org/eclipse/che/workspace/infrastructure/openshift/OpenShiftInfraModule.java

@@ -51,7 +51,7 @@
 import org.eclipse.che.workspace.infrastructure.kubernetes.environment.KubernetesEnvironmentFactory;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.KubernetesNamespaceFactory;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.CredentialsSecretConfigurator;
-import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.GitconfigConfigmapConfigurator;
+import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.GitconfigConfigurator;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.NamespaceConfigurator;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.OAuthTokenSecretsConfigurator;
 import org.eclipse.che.workspace.infrastructure.kubernetes.namespace.configurator.PreferencesConfigMapConfigurator;
@@ -115,7 +115,7 @@ protected void configure() {
     namespaceConfigurators.addBinding().to(OAuthTokenSecretsConfigurator.class);
     namespaceConfigurators.addBinding().to(PreferencesConfigMapConfigurator.class);
     namespaceConfigurators.addBinding().to(OpenShiftWorkspaceServiceAccountConfigurator.class);
-    namespaceConfigurators.addBinding().to(GitconfigConfigmapConfigurator.class);
+    namespaceConfigurators.addBinding().to(GitconfigConfigurator.class);
 
     bind(AuthorizationChecker.class).to(OpenShiftAuthorizationCheckerImpl.class);
     bind(PermissionsCleaner.class).asEagerSingleton();