Skip to content

Commit cf5ae74

Browse files
committed
cleanup(S3ClientProvider): Remove constructor used only in tests
1 parent 3db0e87 commit cf5ae74

File tree

4 files changed

+6
-9
lines changed

4 files changed

+6
-9
lines changed

src/main/java/software/amazon/nio/spi/s3/S3ClientProvider.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -105,10 +105,6 @@ public S3ClientProvider(S3NioSpiConfiguration c) {
105105
this.configuration = (c == null) ? new S3NioSpiConfiguration() : c;
106106
}
107107

108-
public S3ClientProvider() {
109-
this(null);
110-
}
111-
112108
public S3CrtAsyncClientBuilder asyncClientBuilder() {
113109
return asyncClientBuilder;
114110
}

src/test/java/software/amazon/nio/spi/s3/FixedS3ClientProvider.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ public class FixedS3ClientProvider extends S3ClientProvider {
1919
final public AwsClient client;
2020

2121
public FixedS3ClientProvider(S3AsyncClient client) {
22+
super(null);
2223
this.client = client;
2324
}
2425

src/test/java/software/amazon/nio/spi/s3/S3ClientProviderTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,13 @@ public class S3ClientProviderTest {
3737
S3ClientProvider provider;
3838

3939
@BeforeEach
40-
public void before() throws Exception {
41-
provider = new S3ClientProvider();
40+
public void before() {
41+
provider = new S3ClientProvider(null);
4242
}
4343

4444
@Test
4545
public void initialization() {
46-
final S3ClientProvider P = new S3ClientProvider();
46+
final S3ClientProvider P = new S3ClientProvider(null);
4747

4848
assertNotNull(P.configuration);
4949

src/test/java/software/amazon/nio/spi/s3/S3FileSystemTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,8 +88,8 @@ public void isReadOnly() {
8888

8989
@Test
9090
public void getAndSetClientProvider() {
91-
final S3ClientProvider P1 = new S3ClientProvider();
92-
final S3ClientProvider P2 = new S3ClientProvider();
91+
final S3ClientProvider P1 = new S3ClientProvider(null);
92+
final S3ClientProvider P2 = new S3ClientProvider(null);
9393
s3FileSystem.clientProvider(P1); then(s3FileSystem.clientProvider()).isSameAs(P1);
9494
s3FileSystem.clientProvider(P2); then(s3FileSystem.clientProvider()).isSameAs(P2);
9595
}

0 commit comments

Comments
 (0)