Skip to content

Commit 4b54554

Browse files
committed
ACC-2780 add S3MockUtils.s3mockContainer()
1 parent d72bc68 commit 4b54554

6 files changed

Lines changed: 11 additions & 17 deletions

File tree

contentgrid-appserver-contentstore-impl-s3/src/test/java/com/contentgrid/appserver/contentstore/impl/s3/S3ContentStoreWriteFailureTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
class S3ContentStoreWriteFailureTest {
3636

3737
@Container
38-
private static final S3MockContainer s3MockContainer = new S3MockContainer(S3MockUtils.S3_MOCK_VERSION);
38+
private static final S3MockContainer s3MockContainer = S3MockUtils.s3MockContainer();
3939

4040
private MinioAsyncClient client;
4141

contentgrid-appserver-contentstore-impl-s3/src/test/java/com/contentgrid/appserver/contentstore/impl/s3/S3MockContentStoreTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
class S3MockContentStoreTest extends AbstractS3ContentStoreTest {
1111

1212
@Container
13-
private static final S3MockContainer s3MockContainer = new S3MockContainer(S3MockUtils.S3_MOCK_VERSION);
13+
private static final S3MockContainer s3MockContainer = S3MockUtils.s3MockContainer();
1414

1515
@Override
1616
protected MinioAsyncClient createClient() {

contentgrid-appserver-contentstore-impl-utils/src/testFixtures/java/com/contentgrid/appserver/contentstore/impl/utils/testing/S3MockUtils.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
@UtilityClass
1212
public class S3MockUtils {
1313

14-
public static final String S3_MOCK_VERSION;
14+
private static final String S3_MOCK_VERSION;
1515

1616
static {
1717
// Read the version of com.adobe.testing:s3mock-testcontainers by looking at pom.properties
@@ -26,4 +26,8 @@ public class S3MockUtils {
2626
throw new RuntimeException(e);
2727
}
2828
}
29+
30+
public static S3MockContainer s3MockContainer() {
31+
return new S3MockContainer(S3_MOCK_VERSION);
32+
}
2933
}

contentgrid-appserver-infrastructure-impl-s3/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ dependencies {
1111

1212
testImplementation 'org.assertj:assertj-core'
1313
testImplementation testFixtures(project(':contentgrid-appserver-infrastructure-api'))
14+
testImplementation testFixtures(project(':contentgrid-appserver-contentstore-impl-utils'))
1415
testImplementation "org.testcontainers:junit-jupiter"
1516
testImplementation "com.adobe.testing:s3mock-testcontainers"
1617
}

contentgrid-appserver-infrastructure-impl-s3/src/test/java/com/contentgrid/appserver/impl/s3/S3ArtifactTest.java

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.contentgrid.appserver.impl.s3;
22

33
import com.adobe.testing.s3mock.testcontainers.S3MockContainer;
4+
import com.contentgrid.appserver.contentstore.impl.utils.testing.S3MockUtils;
45
import com.contentgrid.appserver.infrastructure.api.AbstractArtifactTest;
56
import com.contentgrid.appserver.infrastructure.api.Artifact;
67
import io.minio.MakeBucketArgs;
@@ -9,7 +10,6 @@
910
import java.io.ByteArrayInputStream;
1011
import java.io.ByteArrayOutputStream;
1112
import java.io.IOException;
12-
import java.util.Properties;
1313
import java.util.zip.ZipEntry;
1414
import java.util.zip.ZipOutputStream;
1515
import org.junit.jupiter.api.BeforeAll;
@@ -23,21 +23,10 @@ class S3ArtifactTest extends AbstractArtifactTest {
2323
private static final String OBJECT_KEY = "test.zip";
2424

2525
@Container
26-
private static final S3MockContainer S3_MOCK = new S3MockContainer(s3MockVersion());
26+
private static final S3MockContainer S3_MOCK = S3MockUtils.s3MockContainer();
2727

2828
private static S3Artifact artifact;
2929

30-
private static String s3MockVersion() {
31-
var props = new Properties();
32-
try (var is = S3MockContainer.class.getResourceAsStream(
33-
"/META-INF/maven/com.adobe.testing/s3mock-testcontainers/pom.properties")) {
34-
props.load(is);
35-
return props.getProperty("version");
36-
} catch (IOException e) {
37-
throw new RuntimeException(e);
38-
}
39-
}
40-
4130
@BeforeAll
4231
static void setup() throws Exception {
4332
var client = MinioAsyncClient.builder()

contentgrid-appserver-integration-test/src/test/java/com/contentgrid/appserver/integration/test/fixture/invoicing/InvoicingApiApplicationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ class InvoicingApiApplicationTest {
119119
PlatformTransactionManager transactionManager;
120120

121121
@Container
122-
static S3MockContainer s3MockContainer = new S3MockContainer(S3MockUtils.S3_MOCK_VERSION)
122+
static S3MockContainer s3MockContainer = S3MockUtils.s3MockContainer()
123123
.withInitialBuckets(BUCKET_NAME);
124124

125125
@DynamicPropertySource

0 commit comments

Comments
 (0)