Skip to content

Commit 4e9549f

Browse files
authored
Merge TransferManager recipes to main recipe (#6392)
1 parent 5dac489 commit 4e9549f

File tree

11 files changed

+23
-238
lines changed

11 files changed

+23
-238
lines changed

test/v2-migration-tests/src/test/java/software/amazon/awssdk/v2migrationtests/MavenNoCompileTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ static void setUp() throws IOException {
5050
@Test
5151
@EnabledIf("versionAvailable")
5252
void mavenProject_shouldConvert() throws IOException {
53-
boolean experimental = true;
54-
verifyTransformation(experimental);
53+
verifyTransformation();
5554
}
5655
}

test/v2-migration-tests/src/test/java/software/amazon/awssdk/v2migrationtests/MavenProjectTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,7 @@ static void setUp() throws IOException {
4747
@Test
4848
@EnabledIf("versionAvailable")
4949
void mavenProject_shouldConvert() throws IOException {
50-
boolean experimental = false;
51-
verifyTransformation(experimental);
50+
verifyTransformation();
5251
verifyCompilation();
5352
}
5453
}

test/v2-migration-tests/src/test/java/software/amazon/awssdk/v2migrationtests/MavenTestBase.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -47,12 +47,8 @@ protected static void deleteTempDirectories() throws IOException {
4747
FileUtils.deleteDirectory(mavenExpected.toFile());
4848
}
4949

50-
protected static void verifyTransformation(boolean experimental) throws IOException {
50+
protected static void verifyTransformation() throws IOException {
5151
String recipeCmd = "-Drewrite.activeRecipes=software.amazon.awssdk.v2migration.AwsSdkJavaV1ToV2";
52-
if (experimental) {
53-
recipeCmd += "Experimental";
54-
}
55-
5652
List<String> rewriteArgs = new ArrayList<>();
5753
// pin version since updates have broken tests
5854
String rewriteMavenPluginVersion = "5.46.0";

test/v2-migration-tests/src/test/java/software/amazon/awssdk/v2migrationtests/MavenTransferManagerTest.java

Lines changed: 0 additions & 56 deletions
This file was deleted.

test/v2-migration-tests/src/test/resources/software/amazon/awssdk/v2migrationtests/maven-tm/after/pom.xml

Lines changed: 0 additions & 55 deletions
This file was deleted.

test/v2-migration-tests/src/test/resources/software/amazon/awssdk/v2migrationtests/maven-tm/before/pom.xml

Lines changed: 0 additions & 49 deletions
This file was deleted.

test/v2-migration-tests/src/test/resources/software/amazon/awssdk/v2migrationtests/maven/after/pom.xml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,11 @@
6262
<artifactId>s3-event-notifications</artifactId>
6363
<version>V2_VERSION</version>
6464
</dependency>
65+
<dependency>
66+
<groupId>software.amazon.awssdk</groupId>
67+
<artifactId>s3-transfer-manager</artifactId>
68+
<version>V2_VERSION</version>
69+
</dependency>
6570
<dependency>
6671
<groupId>software.amazon.awssdk</groupId>
6772
<artifactId>s3</artifactId>

test/v2-migration-tests/src/test/resources/software/amazon/awssdk/v2migrationtests/maven-tm/after/src/main/java/foo/bar/TransferManagerS3.java renamed to test/v2-migration-tests/src/test/resources/software/amazon/awssdk/v2migrationtests/maven/after/src/main/java/foo/bar/TransferManagerS3.java

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@
1515

1616
package foo.bar;
1717

18+
import java.io.File;
19+
import java.io.IOException;
20+
import java.io.OutputStream;
21+
import java.time.Duration;
1822
import software.amazon.awssdk.auth.credentials.AwsCredentials;
1923
import software.amazon.awssdk.auth.credentials.AwsCredentialsProvider;
2024
import software.amazon.awssdk.auth.credentials.StaticCredentialsProvider;
@@ -41,18 +45,13 @@
4145
import software.amazon.awssdk.transfer.s3.model.UploadRequest;
4246
import software.amazon.awssdk.transfer.s3.progress.TransferProgress;
4347

44-
import java.io.File;
45-
import java.io.IOException;
46-
import java.io.OutputStream;
47-
import java.time.Duration;
48-
4948
public class TransferManagerS3 {
5049

5150
File file = new File("path/to/file.txt");
5251

5352
void tmConstructor(AwsCredentials credentials, AwsCredentialsProvider credentialsProvider) {
5453
S3TransferManager tm = S3TransferManager.builder()
55-
.build();
54+
.build();
5655
S3TransferManager tmBuilderDefault = S3TransferManager.create();
5756
S3TransferManager tmBuilderWithS3 = S3TransferManager.builder().build();
5857
S3TransferManager tmConstructorWithCred = S3TransferManager.builder().s3Client(S3AsyncClient.builder().credentialsProvider(StaticCredentialsProvider.create(credentials)).build()).build();
@@ -66,7 +65,7 @@ void download(S3TransferManager tm, String bucket, String key) {
6665
FileDownload download2 = tm.downloadFile(DownloadFileRequest.builder().getObjectRequest(GetObjectRequest.builder().bucket(bucket).key(key).overrideConfiguration(AwsRequestOverrideConfiguration.builder().apiCallTimeout(Duration.ofMillis(timeout)).build()).build()).destination(file).build());
6766

6867
GetObjectRequest getObjectRequest = GetObjectRequest.builder().bucket(bucket).key(key)
69-
.build();
68+
.build();
7069

7170
FileDownload download3 = tm.downloadFile(DownloadFileRequest.builder().getObjectRequest(getObjectRequest).destination(file).build());
7271

@@ -78,19 +77,19 @@ void upload(S3TransferManager tm, String bucket, String key) {
7877

7978
File file = new File("file1.txt");
8079
PutObjectRequest requestWithFile = PutObjectRequest.builder().bucket(bucket).key(key)
81-
.build();
80+
.build();
8281
tm.uploadFile(UploadFileRequest.builder().putObjectRequest(requestWithFile).source(file).build());
8382

8483
PutObjectRequest requestWithoutPayload = PutObjectRequest.builder().bucket(bucket).key(key).websiteRedirectLocation("location")
85-
.build();
84+
.build();
8685
tm.upload(UploadRequest.builder().putObjectRequest(requestWithoutPayload).requestBody(AsyncRequestBody.empty()).build());
8786
}
8887

8988
void copy(S3TransferManager tm, String sourceBucket, String sourceKey, String destinationBucket, String destinationKey) {
9089
Copy copy = tm.copy(CopyRequest.builder().copyObjectRequest(CopyObjectRequest.builder().sourceBucket(sourceBucket).sourceKey(sourceKey).destinationBucket(destinationBucket).destinationKey(destinationKey).build()).build());
9190

9291
CopyObjectRequest copyRequest = CopyObjectRequest.builder().sourceBucket(sourceBucket).sourceKey(sourceKey).destinationBucket(destinationBucket).destinationKey(destinationKey)
93-
.build();
92+
.build();
9493
Copy copy2 = tm.copy(CopyRequest.builder().copyObjectRequest(copyRequest).build());
9594
}
9695

test/v2-migration-tests/src/test/resources/software/amazon/awssdk/v2migrationtests/maven-tm/before/src/main/java/foo/bar/TransferManagerS3.java renamed to test/v2-migration-tests/src/test/resources/software/amazon/awssdk/v2migrationtests/maven/before/src/main/java/foo/bar/TransferManagerS3.java

File renamed without changes.

v2-migration/src/main/resources/META-INF/rewrite/aws-sdk-java-v1-to-v2-with-tm.yml

Lines changed: 0 additions & 58 deletions
This file was deleted.

0 commit comments

Comments
 (0)