Skip to content

Commit c2de74c

Browse files
RomanChtFameing
authored andcommitted
fix: Embedded Git container Network setup
1 parent a585234 commit c2de74c

File tree

2 files changed

+14
-4
lines changed

2 files changed

+14
-4
lines changed

embedded-git/src/main/java/com/playtika/testcontainer/git/EmbeddedGitBootstrapConfiguration.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -64,14 +64,12 @@ ToxiproxyContainer.ContainerProxy gitContainerProxy(ToxiproxyContainer toxiproxy
6464
public GenericContainer<?> embeddedGit(ConfigurableEnvironment environment,
6565
GitProperties properties,
6666
Optional<Network> network) {
67-
GenericContainer<?> gitContainer = configureCommonsAndStart(createContainer(properties), properties, log)
68-
.withNetworkAliases(GIT_NETWORK_ALIAS);
69-
network.ifPresent(gitContainer::withNetwork);
67+
GenericContainer<?> gitContainer = configureCommonsAndStart(createContainer(properties, network), properties, log);
7068
registerGitEnvironment(gitContainer, environment, properties);
7169
return gitContainer;
7270
}
7371

74-
private GenericContainer<?> createContainer(GitProperties properties) {
72+
private GenericContainer<?> createContainer(GitProperties properties, Optional<Network> network) {
7573
if (isEmpty(properties.getPathToRepositories())) {
7674
throw new RuntimeException("embedded.git.path-to-repositories is required");
7775
}
@@ -81,7 +79,9 @@ private GenericContainer<?> createContainer(GitProperties properties) {
8179
.withEnv("GIT_REPOSITORIES_PATH", "/projects")
8280
.withEnv("GIT_PASSWORD", properties.getPassword())
8381
.withExposedPorts(properties.getPort())
82+
.withNetworkAliases(GIT_NETWORK_ALIAS)
8483
.waitingFor(new HostPortWaitStrategy());
84+
network.ifPresent(container::withNetwork);
8585
if (isNotEmpty(properties.getPathToAuthorizedKeys())) {
8686
container.withFileSystemBind(properties.getPathToAuthorizedKeys(), "/home/git/.ssh/authorized_keys");
8787
}

embedded-git/src/test/java/EmbeddedGitBootstrapConfigurationTest.java

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
import org.springframework.context.annotation.Configuration;
1515
import org.springframework.core.env.ConfigurableEnvironment;
1616
import org.springframework.core.io.Resource;
17+
import org.testcontainers.containers.GenericContainer;
1718
import util.EncryptionUtils;
1819

1920
import java.io.File;
@@ -38,6 +39,10 @@
3839
}
3940
)
4041
class EmbeddedGitBootstrapConfigurationTest {
42+
43+
@Autowired
44+
GenericContainer embeddedGit;
45+
4146
@Autowired
4247
ConfigurableListableBeanFactory beanFactory;
4348

@@ -61,6 +66,11 @@ public void propertiesAreAvailable() {
6166
assertThat(environment.getProperty("embedded.git.path-to-repositories")).isEqualTo("src/test/resources/repository");
6267
}
6368

69+
@Test
70+
public void shouldHaveCorrectNetworkAlias() {
71+
assertThat(embeddedGit.getNetworkAliases()).contains("git.testcontainer.docker");
72+
}
73+
6474
@Test
6575
@SneakyThrows
6676
public void testPushViaSsh() {

0 commit comments

Comments
 (0)