Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Accept image names with library/ prefix as a valid substitute #6174

Merged
merged 11 commits into from
Dec 14, 2022
Merged

Accept image names with library/ prefix as a valid substitute #6174

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
9f694e1
Accept image names with `library/` prefix as a valid substitute
eddumelendez Nov 14, 2022
5117c55
Use `library/` as part of JDBC Container Providers
eddumelendez Nov 14, 2022
7b01bf3
fix
eddumelendez Nov 15, 2022
9fff180
Revert "Use `library/` as part of JDBC Container Providers"
eddumelendez Nov 16, 2022
65cf233
Revert "Accept image names with `library/` prefix as a valid substitute"
eddumelendez Nov 16, 2022
9c803a3
add prefix to alpine
eddumelendez Nov 16, 2022
6a008ef
Improve logic
eddumelendez Nov 17, 2022
3daf95d
Merge branch 'main' into docker_official_images
eddumelendez Dec 2, 2022
2fef0b0
move logic to isCompatibleWith
eddumelendez Dec 3, 2022
6d240ba
Update core/src/main/java/org/testcontainers/DockerClientFactory.java
eddumelendez Dec 13, 2022
e0807e7
Fix comment
eddumelendez Dec 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion core/src/main/java/org/testcontainers/DockerClientFactory.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ static Map<String, String> markerLabels() {
return Collections.unmodifiableMap(labels);
}

private static final DockerImageName TINY_IMAGE = DockerImageName.parse("alpine:3.16");
private static final DockerImageName TINY_IMAGE = DockerImageName.parse("library/alpine:3.16");
eddumelendez marked this conversation as resolved.
Show resolved Hide resolved

private static DockerClientFactory instance;

Expand Down
12 changes: 10 additions & 2 deletions core/src/main/java/org/testcontainers/utility/DockerImageName.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
import org.testcontainers.utility.Versioning.Sha256Versioning;
import org.testcontainers.utility.Versioning.TagVersioning;

import java.util.Arrays;
import java.util.regex.Pattern;

@EqualsAndHashCode(exclude = { "rawName", "compatibleSubstituteFor" })
Expand All @@ -26,6 +27,8 @@ public final class DockerImageName {

private static final Pattern REPO_NAME = Pattern.compile(REPO_NAME_PART + "(/" + REPO_NAME_PART + ")*");

private static final String LIBRARY_PREFIX = "library/";

private final String rawName;

@With
Expand Down Expand Up @@ -258,13 +261,18 @@ public void assertCompatibleWith(DockerImageName... anyOthers) {
throw new IllegalArgumentException("anyOthers parameter must be non-empty");
}

for (DockerImageName anyOther : anyOthers) {
String imageWithoutLibraryPrefix = this.repository.replace(LIBRARY_PREFIX, "");
DockerImageName[] dockerImageNames = Arrays.copyOf(anyOthers, anyOthers.length + 1);
dockerImageNames[dockerImageNames.length - 1] =
DockerImageName.parse(LIBRARY_PREFIX + imageWithoutLibraryPrefix);

for (DockerImageName anyOther : dockerImageNames) {
if (this.isCompatibleWith(anyOther)) {
return;
}
}

final DockerImageName exampleOther = anyOthers[0];
final DockerImageName exampleOther = dockerImageNames[0];

throw new IllegalStateException(
String.format(
Expand Down
6 changes: 6 additions & 0 deletions core/src/test/java/org/testcontainers/utility/DockerImageNameCompatibilityTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,12 @@ public void testAssertMethodAcceptsCompatible() {
subject.assertCompatibleWith(DockerImageName.parse("bar"));
}

@Test
public void testAssertMethodAcceptsCompatibleLibraryPrefix() {
eddumelendez marked this conversation as resolved.
Show resolved Hide resolved
DockerImageName subject = DockerImageName.parse("library/foo");
subject.assertCompatibleWith(DockerImageName.parse("foo"));
}

@Test
public void testAssertMethodRejectsIncompatible() {
DockerImageName subject = DockerImageName.parse("foo");
Expand Down