markrmiller commented on a change in pull request #585: URL: https://github.com/apache/solr/pull/585#discussion_r819245105
########## File path: solr/modules/s3-repository/src/test/org/apache/solr/s3/S3BackupRepositoryTest.java ########## @@ -320,17 +340,113 @@ protected URI getBaseUri() throws URISyntaxException { } private void pushObject(String path, String content) { - try (S3Client s3 = S3_MOCK_RULE.createS3ClientV2()) { + try (S3Client s3 = createS3ClientV2()) { s3.putObject(b -> b.bucket(BUCKET_NAME).key(path), RequestBody.fromString(content)); } } private File pullObject(String path) throws IOException { - try (S3Client s3 = S3_MOCK_RULE.createS3ClientV2()) { + try (S3Client s3 = createS3ClientV2()) { File file = temporaryFolder.newFile(); InputStream input = s3.getObject(b -> b.bucket(BUCKET_NAME).key(path)); FileUtils.copyInputStreamToFile(input, file); return file; } } + + public S3Client createS3ClientV2() { Review comment: Actually, I'm going to have check if that is still necessary or not. Was working on making some different version combo work properly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org