Skip to content

Commit a60e108

Browse files
authored
Fix the S3 getUrl tests not being isolated from other tests (#2111)
1 parent 0b0240b commit a60e108

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

jetbrains-core/tst/software/aws/toolkits/jetbrains/services/s3/S3VirtualBucketTest.kt

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,11 @@ import software.amazon.awssdk.services.s3.model.ListObjectsV2Response
3535
import software.amazon.awssdk.services.s3.model.ObjectIdentifier
3636
import software.amazon.awssdk.services.s3.model.PutObjectRequest
3737
import software.amazon.awssdk.services.s3.model.PutObjectResponse
38+
import software.aws.toolkits.core.region.AwsRegion
3839
import software.aws.toolkits.core.utils.delegateMock
3940
import software.aws.toolkits.jetbrains.core.AwsClientManager
4041
import software.aws.toolkits.jetbrains.core.MockClientManagerRule
42+
import software.aws.toolkits.jetbrains.core.credentials.MockAwsConnectionManager.ProjectAccountSettingsManagerRule
4143
import software.aws.toolkits.jetbrains.services.s3.editor.S3VirtualBucket
4244
import java.io.ByteArrayInputStream
4345
import java.net.URL
@@ -53,6 +55,10 @@ class S3VirtualBucketTest {
5355
@Rule
5456
val mockClientManager = MockClientManagerRule(projectRule)
5557

58+
@Rule
59+
@JvmField
60+
val settingsManagerRule = ProjectAccountSettingsManagerRule(projectRule)
61+
5662
@Test
5763
fun deleteObjects() {
5864
val s3Client = mockClientManager.create<S3Client>()
@@ -203,16 +209,20 @@ class S3VirtualBucketTest {
203209

204210
@Test
205211
fun getUrl() {
212+
settingsManagerRule.settingsManager.changeRegionAndWait(AwsRegion("us-west-2", "US West (Oregon)", "aws"))
213+
206214
// Use real manager for this since it can affect the S3Configuration that goes into S3Utilities
207215
AwsClientManager(projectRule.project).getClient<S3Client>().use {
208216
val sut = S3VirtualBucket(Bucket.builder().name("test-bucket").build(), it)
209217

210-
assertThat(sut.generateUrl("prefix/key")).isEqualTo(URL("https://test-bucket.s3.amazonaws.com/prefix/key"))
218+
assertThat(sut.generateUrl("prefix/key")).isEqualTo(URL("https://test-bucket.s3.us-west-2.amazonaws.com/prefix/key"))
211219
}
212220
}
213221

214222
@Test
215223
fun getUrlError() {
224+
settingsManagerRule.settingsManager.changeRegionAndWait(AwsRegion("us-west-2", "US West (Oregon)", "aws"))
225+
216226
// Use real manager for this since it can affect the S3Configuration that goes into S3Utilities
217227
AwsClientManager(projectRule.project).getClient<S3Client>().use {
218228
val sut = S3VirtualBucket(Bucket.builder().name("test-bucket").build(), it)

0 commit comments

Comments
 (0)