fix unit test for OIDC

aliyun · Jul 22, 2024 · dbfd7fd · dbfd7fd
1 parent 0a35f47
commit dbfd7fd
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/aliyun-java-sdk-core/src/test/java/com/aliyuncs/auth/OIDCCredentialsProviderTest.java b/aliyun-java-sdk-core/src/test/java/com/aliyuncs/auth/OIDCCredentialsProviderTest.java
@@ -66,7 +66,7 @@ public void run() throws Throwable {
     @Test
     public void getCredentialsTest() throws ClientException {
         String filePath = OIDCCredentialsProviderTest.class.getClassLoader().getResource("oidctoken").getPath();
-        final AlibabaCloudCredentialsProvider provider = new OIDCCredentialsProvider("roleArn", "providerArn", filePath, null, null);
+        final AlibabaCloudCredentialsProvider provider = new OIDCCredentialsProvider("roleArn", "providerArn", filePath, null, "us-west-1");
         Assert.assertNotNull(provider);
         ClientException ex = Assert.assertThrows(ClientException.class, new ThrowingRunnable() {
             @Override

diff --git a/...sdk-core/src/test/java/com/aliyuncs/auth/STSAssumeRoleSessionCredentialsProviderTest.java b/...sdk-core/src/test/java/com/aliyuncs/auth/STSAssumeRoleSessionCredentialsProviderTest.java
@@ -124,7 +124,6 @@ public void withRoleSessionDurationSecondsSmallerThan900Seconds() {
         STSAssumeRoleSessionCredentialsProvider provider = new STSAssumeRoleSessionCredentialsProvider(credentials,
                 roleArn, clientProfile);
         provider.withRoleSessionDurationSeconds(duration);
-
     }
 
     @Test
@@ -137,7 +136,6 @@ public void withRoleSessionDurationSecondsLargerThan3600Seconds() {
         STSAssumeRoleSessionCredentialsProvider provider = new STSAssumeRoleSessionCredentialsProvider(credentials,
                 roleArn, clientProfile);
         provider.withRoleSessionDurationSeconds(duration);
-
     }
 
     @Test
@@ -193,6 +191,5 @@ public void getCredentialsReturnPreviousCredentials() throws ClientException {
 
         AlibabaCloudCredentials credentials3 = provider.getCredentials();
         Assert.assertEquals(credentials2, credentials3);
-
     }
 }
diff --git a/aliyun-java-sdk-core/src/test/java/com/aliyuncs/utils/EnvironmentUtilsTest.java b/aliyun-java-sdk-core/src/test/java/com/aliyuncs/utils/EnvironmentUtilsTest.java
@@ -15,19 +15,25 @@ public void testGetSetHttpProxy() {
         Assert.assertNull(EnvironmentUtils.getHttpProxy());
         EnvironmentUtils.setHttpProxy("http://www.aliyun.com");
         Assert.assertEquals("http://www.aliyun.com", EnvironmentUtils.getHttpProxy());
+        // reset http proxy
+        EnvironmentUtils.setHttpProxy(null);
     }
 
     @Test
     public void testGetSetHttpsProxy() {
         Assert.assertNull(EnvironmentUtils.getHttpsProxy());
         EnvironmentUtils.setHttpsProxy("https://www.aliyun.com");
         Assert.assertEquals("https://www.aliyun.com", EnvironmentUtils.getHttpsProxy());
+        // reset https proxy
+        EnvironmentUtils.setHttpsProxy(null);
     }
 
     @Test
     public void testGetSetNoProxy() {
         Assert.assertNull(EnvironmentUtils.getNoProxy());
         EnvironmentUtils.setNoProxy("https://www.aliyun.com");
         Assert.assertEquals("https://www.aliyun.com", EnvironmentUtils.getNoProxy());
+        // reset no proxy
+        EnvironmentUtils.setNoProxy(null);
     }
 }