(opendal) branch disable-aliyun_drive-for-binding-test-ci created (now cc5ae963ad)

[su...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

suyanhanx pushed a change to branch disable-aliyun_drive-for-binding-test-ci
in repository https://gitbox.apache.org/repos/asf/opendal.git


      at cc5ae963ad chore(ci): disable aliyun_drive for bindings test

This branch includes the following new commits:

     new cc5ae963ad chore(ci): disable aliyun_drive for bindings test

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

(opendal) 01/01: chore(ci): disable aliyun_drive for bindings test

[su...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

suyanhanx pushed a commit to branch disable-aliyun_drive-for-binding-test-ci
in repository https://gitbox.apache.org/repos/asf/opendal.git

commit cc5ae963addd676b62bc0628b7da59f003b3ff0b
Author: suyanhanx <su...@gmail.com>
AuthorDate: Wed Jun 19 14:30:39 2024 +0800

    chore(ci): disable aliyun_drive for bindings test
    
    Signed-off-by: suyanhanx <su...@gmail.com>
---
 .github/scripts/test_behavior/plan.py | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/.github/scripts/test_behavior/plan.py b/.github/scripts/test_behavior/plan.py
index e6d6001c37..db1ee988c8 100755
--- a/.github/scripts/test_behavior/plan.py
+++ b/.github/scripts/test_behavior/plan.py
@@ -86,7 +86,7 @@ class Hint:
     # Is bin ofs affected?
     bin_ofs: bool = field(default=False, init=False)
 
-    # Should we run all services test?
+    # Should we run all services tests?
     all_service: bool = field(default=False, init=False)
     # affected services set.
     services: set = field(default_factory=set, init=False)
@@ -95,7 +95,7 @@ class Hint:
 def calculate_hint(changed_files: list[str]) -> Hint:
     hint = Hint()
 
-    # Remove all files that ends with `.md`
+    # Remove all files that end with `.md`
     changed_files = [f for f in changed_files if not f.endswith(".md")]
 
     for p in changed_files:
@@ -179,9 +179,9 @@ def calculate_hint(changed_files: list[str]) -> Hint:
     return hint
 
 
-# unique_cases is used to only one setup for each service.
+# `unique_cases` is used to only one setup for each service.
 #
-# We need this because we have multiple setup for each service and they have already been
+# We need this because we have multiple setups for each service, and they have already been
 # tested by `core` workflow. So we can only test unique setup for each service for bindings.
 #
 # We make sure that we return the first setup for each service in alphabet order.
@@ -221,7 +221,10 @@ def generate_language_binding_cases(
 ) -> list[dict[str, str]]:
     cases = unique_cases(cases)
 
-    # Remove hdfs cases for java.
+    # Disable aliyun_drive case for every language.
+    cases = [v for v in cases if v["service"] != "aliyun_drive"]
+
+    # Remove hdfs cases for jav:a.
     if language == "java":
         cases = [v for v in cases if v["service"] != "hdfs"]
 
@@ -246,7 +249,7 @@ def generate_bin_cases(
     # Return empty if this bin is False
     if not getattr(hint, f"bin_{bin}"):
         return []
-    
+
     cases = unique_cases(cases)
 
     if bin == "ofs":