Skip to content

Commit 5c3c2d5

Browse files
committed
TIKA-4525: set default values, add comments
1 parent 612d0ed commit 5c3c2d5

1 file changed

Lines changed: 9 additions & 2 deletions

File tree

tika-integration-tests/tika-pipes-s3-integration-tests/src/test/java/org/apache/tika/pipes/s3/tests/PipeIntegrationTests.java

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,15 +54,20 @@
5454
import org.apache.tika.pipes.core.pipesiterator.PipesIterator;
5555
import org.apache.tika.pipes.emitter.s3.S3Emitter;
5656

57+
// To enable these tests, fill OUTDIR and bucket, and adjust profile and region if needed.
5758
@Disabled("turn these into actual tests with mock s3")
5859
public class PipeIntegrationTests {
5960

6061
private static final Path OUTDIR = Paths.get("");
6162

63+
/**
64+
* This downloads files from a specific bucket.
65+
* @throws Exception
66+
*/
6267
@Test
6368
public void testBruteForce() throws Exception {
64-
String region = "";
65-
String profile = "";
69+
String region = "us-east-1";
70+
String profile = "default";
6671
String bucket = "";
6772
AwsCredentialsProvider provider = ProfileCredentialsProvider.builder().profileName(profile).build();
6873
S3Client s3Client = S3Client.builder().credentialsProvider(provider).region(Region.of(region)).build();
@@ -91,6 +96,7 @@ public void testBruteForce() throws Exception {
9196
System.out.println("iterated: " + cnt + " sz: " + sz);
9297
}
9398

99+
// to test this, files must be in the fetcher bucket
94100
@Test
95101
public void testS3ToFS() throws Exception {
96102
Fetcher fetcher = getFetcher("tika-config-s3ToFs.xml", "s3f");
@@ -121,6 +127,7 @@ public void testS3ToFS() throws Exception {
121127
}
122128
}
123129

130+
// to test this, files must be in the iterator bucket
124131
@Test
125132
public void testS3ToS3() throws Exception {
126133
Fetcher fetcher = getFetcher("tika-config-s3Tos3.xml", "s3f");

0 commit comments

Comments
 (0)