Skip to content

Commit 49ecd27

Browse files
committed
adjusting tests to satisfy more permutations
1 parent 798de72 commit 49ecd27

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

service/src/test/java/uk/nhs/adaptors/gp2gp/common/storage/S3StorageConnectorTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
import static org.junit.Assert.assertEquals;
1414
import static org.junit.Assert.assertThrows;
1515
import static org.junit.jupiter.api.Assertions.assertNotNull;
16-
import static org.mockito.ArgumentMatchers.any;
1716
import static org.mockito.Mockito.mock;
1817
import static org.mockito.Mockito.verify;
1918
import static org.mockito.Mockito.when;
@@ -39,9 +38,9 @@ void setUp() {
3938

4039
@Test
4140
void expectExceptionWhenS3ClientCantDeliverResponse() {
42-
s3StorageConnector = new S3StorageConnector(S3Client.builder().build(), config);
41+
S3StorageConnector storageConnector = new S3StorageConnector(S3Client.builder().build(), config);
4342
Exception exception = assertThrows(StorageConnectorException.class,
44-
() -> s3StorageConnector.downloadFromStorage("s3File"));
43+
() -> storageConnector.downloadFromStorage("s3File"));
4544

4645
assertEquals("Error occurred downloading from S3 Bucket", exception.getMessage());
4746
}
@@ -51,12 +50,13 @@ void downloadFromStorage() {
5150
var mockResponse = mock(GetObjectResponse.class);
5251
var mockInputStream = new ByteArrayInputStream("dummy-content".getBytes());
5352
var mockResponseInputStream = new ResponseInputStream<>(mockResponse, mockInputStream);
53+
final var request = GetObjectRequest.builder().bucket(config.getContainerName()).key(FILE_NAME).build();
5454

55-
when(mockS3Client.getObject(any(GetObjectRequest.class))).thenReturn(mockResponseInputStream);
55+
when(mockS3Client.getObject(request)).thenReturn(mockResponseInputStream);
5656

5757
var result = s3StorageConnector.downloadFromStorage(FILE_NAME);
5858

5959
assertNotNull(result);
60-
verify(mockS3Client).getObject(any(GetObjectRequest.class));
60+
verify(mockS3Client).getObject(request);
6161
}
6262
}

0 commit comments

Comments
 (0)