Skip to content

Commit 5fb4475

Browse files
authored
Clear env vars, sys props before test (#2659)
Avoids picking up values that were set by other tests using the same env variables and system properties. Also use AssertJ matchers in tests.
1 parent 6ceb543 commit 5fb4475

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

core/auth/src/test/java/software/amazon/awssdk/auth/credentials/internal/Ec2MetadataConfigProviderEndpointOverrideTest.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515

1616
package software.amazon.awssdk.auth.credentials.internal;
1717

18-
import static org.hamcrest.CoreMatchers.equalTo;
19-
import static org.junit.Assert.assertThat;
18+
import static org.assertj.core.api.Assertions.assertThat;
19+
2020
import java.util.Arrays;
2121
import org.junit.After;
2222
import org.junit.Before;
@@ -67,6 +67,9 @@ public static Iterable<Object> testCases() {
6767

6868
@Before
6969
public void setup() {
70+
ENVIRONMENT_VARIABLE_HELPER.reset();
71+
System.clearProperty(SdkSystemSetting.AWS_EC2_METADATA_SERVICE_ENDPOINT.property());
72+
7073
if (testCase.envEndpointOverride != null) {
7174
ENVIRONMENT_VARIABLE_HELPER.set(SdkSystemSetting.AWS_EC2_METADATA_SERVICE_ENDPOINT.environmentVariable(),
7275
testCase.envEndpointOverride);
@@ -97,7 +100,7 @@ public void teardown() {
97100
public void resolvesCorrectEndpointOverride() {
98101
String endpointOverride = Ec2MetadataConfigProvider.builder().build().getEndpointOverride();
99102

100-
assertThat(endpointOverride, equalTo(testCase.expectedEndpointOverride));
103+
assertThat(endpointOverride).isEqualTo(testCase.expectedEndpointOverride);
101104
}
102105

103106
private static String getTestFilePath(String testFile) {

core/auth/src/test/java/software/amazon/awssdk/auth/credentials/internal/Ec2MetadataConfigProviderEndpointResolutionTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@
1515

1616
package software.amazon.awssdk.auth.credentials.internal;
1717

18-
import static org.hamcrest.CoreMatchers.equalTo;
19-
import static org.junit.Assert.assertThat;
18+
import static org.assertj.core.api.Assertions.assertThat;
19+
2020
import java.util.Arrays;
2121
import org.junit.Before;
2222
import org.junit.Test;
@@ -63,7 +63,7 @@ public void setup() {
6363

6464
@Test
6565
public void resolvesCorrectEndpoint() {
66-
assertThat(Ec2MetadataConfigProvider.builder().build().getEndpoint(), equalTo(testCase.expectedEndpoint));
66+
assertThat(Ec2MetadataConfigProvider.builder().build().getEndpoint()).isEqualTo(testCase.expectedEndpoint);
6767
}
6868

6969
private static class TestCase {

0 commit comments

Comments
 (0)