re-enable integration test

aliyun · Sep 30, 2024 · 4c1d22b · 4c1d22b
1 parent f1d3dad
commit 4c1d22b
Show file tree

Hide file tree

Showing 3 changed files with 54 additions and 38 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -12,18 +12,30 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
+      with:
+        show-progress: false
     - name: Setup .NET Core
       uses: actions/setup-dotnet@v2
       with:
         dotnet-version: '2.x'
     - name: install altcover
       run: dotnet tool install --global altcover.visualizer --version 8.6.14
     - name: Install dependencies
-      run: cd aliyun-net-sdk-core.Tests/ && dotnet add package AltCover --version 8.6.14 && cd ../ && dotnet restore && dotnet build
-    - name: Test
+      run: cd aliyun-net-sdk-core.Tests/ && dotnet add package AltCover --version 8.6.14 && cd ../ && dotnet restore -v minimal && dotnet build
+    - name: Unit Test
       run: dotnet test aliyun-net-sdk-core.Tests/ /p:AltCover=true
     - name: Upload Coverage Report
       uses: codecov/codecov-action@v4
       with:
         token: ${{ secrets.CODECOV_TOKEN }} # required
+    - name: Setup proxy
+      run: |
+        npm install o_o -g
+        o_o &
+    - name: Integration Test
+      run: dotnet test aliyun-sdk-feature-test/
+      if: env.ACCESS_KEY_ID != ''
+      env:
+        ACCESS_KEY_ID: ${{ secrets.ACCESS_KEY_ID }}
+        ACCESS_KEY_SECRET: ${{ secrets.ACCESS_KEY_SECRET }}
diff --git a/aliyun-sdk-feature-test/ErrorHandler/ErrorHandlingTest.cs b/aliyun-sdk-feature-test/ErrorHandler/ErrorHandlingTest.cs
@@ -34,8 +34,10 @@ public class ErrorHandlingTest
         [Fact]
         public void BadFormatTypeTest()
         {
-            var request = new ActiveFlowLogRequest();
-            request.FlowLogId = "testFlow";
+            var request = new ActiveFlowLogRequest
+            {
+                FlowLogId = "testFlow"
+            };
 
             var profile = DefaultProfile.GetProfile("cn-hangzhou", FeatureTest.BasicAccessKeyId, FeatureTest.BasicAccessKeySecret);
             var client = new DefaultAcsClient(profile);
@@ -52,15 +54,17 @@ public void BadFormatTypeTest()
         [Fact]
         public void ParameterErrorTest()
         {
-            var request = new CommonRequest();
-            request.Domain = "ecs-cn-hangzhou.aliyuncs.com";
-            request.Version = "2014-05-26";
-            request.Action = "actionError";
+            var request = new CommonRequest
+            {
+                Domain = "ecs-cn-hangzhou.aliyuncs.com",
+                Version = "2014-05-26",
+                Action = "actionError"
+            };
 
             var response = FeatureTest.DefaultClient.GetCommonResponse(request);
             var content = Encoding.UTF8.GetString(response.HttpResponse.Content);
 
-            Assert.Contains("Keyword=InvalidParameter", content);
+            Assert.Contains("InvalidAction.NotFound", content);
         }
     }
 }
diff --git a/aliyun-sdk-feature-test/HttpProxy/HttpProxyTest.cs b/aliyun-sdk-feature-test/HttpProxy/HttpProxyTest.cs
@@ -32,22 +32,22 @@ public class HttpProxyTest
         public void HttpProxy()
         {
             var client = FeatureTest.DefaultClient;
-            try
-            {
-                var request = new DescribeAccessPointsRequest();
-                client.SetHttpProxy("http://localhost:8989");
-
-                var response = client.GetAcsResponse(request);
-                Assert.NotNull(response.HttpResponse.Content);
-
-                var expectValue = "HTTP/1.1 o_o";
-                string actualValue;
-                response.HttpResponse.Headers.TryGetValue("Via", out actualValue);
-                Assert.Equal(expectValue, actualValue);
+            try
+            {
+                var request = new DescribeAccessPointsRequest();
+                client.SetHttpProxy("http://localhost:8989");
+
+                var response = client.GetAcsResponse(request);
+                Assert.NotNull(response.HttpResponse.Content);
+
+                var expectValue = "HTTP/1.1 o_o";
+                string actualValue;
+                response.HttpResponse.Headers.TryGetValue("Via", out actualValue);
+                Assert.Equal(expectValue, actualValue);
             }
             finally
-            {
-                client.SetHttpProxy(null);
+            {
+                client.SetHttpProxy(null);
             }
 
         }
@@ -56,22 +56,22 @@ public void HttpProxy()
         public void HttpProxyWithCredential()
         {
             var client = FeatureTest.DefaultClient;
-            try
-            {
-                var request = new DescribeAccessPointsRequest();
-                client.SetHttpProxy("http://username:password@localhost:8989");
-                var response = client.GetAcsResponse(request);
-
-                var expectValue = "HTTP/1.1 o_o";
-                string actualValue;
-                response.HttpResponse.Headers.TryGetValue("Via", out actualValue);
-
-                Assert.Equal(expectValue, actualValue);
-                Assert.NotNull(response.HttpResponse.Content);
+            try
+            {
+                var request = new DescribeAccessPointsRequest();
+                client.SetHttpProxy("http://username:password@localhost:8989");
+                var response = client.GetAcsResponse(request);
+
+                var expectValue = "HTTP/1.1 o_o";
+                string actualValue;
+                response.HttpResponse.Headers.TryGetValue("Via", out actualValue);
+
+                Assert.Equal(expectValue, actualValue);
+                Assert.NotNull(response.HttpResponse.Content);
             }
             finally
-            {
-                client.SetHttpProxy(null);
+            {
+                client.SetHttpProxy(null);
             }
         }
     }