Skip to content

Commit e332707

Browse files
authored
Merge branch 'main' into feat-tos-read-write-delete
2 parents 7293264 + a0c1d81 commit e332707

2 files changed

Lines changed: 30 additions & 0 deletions

File tree

.github/scripts/test_behavior/plan.py

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,14 @@
3535

3636
INTEGRATIONS = ["object_store"]
3737

38+
TEMPORARILY_DISABLED_CORE_SERVICES = {
39+
# FIXME: Re-enable gdrive behavior tests after #7384 is fixed.
40+
# Gdrive is currently not reliable enough for CI: #6684 tracks missing
41+
# freshly-created entries in listings, and existing behavior tests already
42+
# document redirect loops caused by Google Drive abuse detection.
43+
"gdrive",
44+
}
45+
3846

3947
def provided_cases() -> list[dict[str, str]]:
4048
root_dir = f"{GITHUB_DIR}/services"
@@ -59,6 +67,10 @@ def provided_cases() -> list[dict[str, str]]:
5967
if not os.getenv("GITHUB_HAS_SECRETS") == "true":
6068
cases[:] = [v for v in cases if "op://services" not in v["content"]]
6169

70+
cases[:] = [
71+
v for v in cases if v["service"] not in TEMPORARILY_DISABLED_CORE_SERVICES
72+
]
73+
6274
# Remove content from cases.
6375
cases = [
6476
{

.github/scripts/test_behavior/test_plan.py

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
# under the License.
1717

1818
import unittest
19+
from unittest.mock import patch
1920

2021
from plan import plan
2122

@@ -74,6 +75,23 @@ def test_integration_object_store(self):
7475
# Should contain fs
7576
self.assertTrue("fs" in cases)
7677

78+
@patch.dict("os.environ", {"GITHUB_HAS_SECRETS": "true"}, clear=False)
79+
def test_gdrive_is_temporarily_disabled(self):
80+
result = plan([".github/workflows/test_behavior.yml"])
81+
82+
self.assertTrue(result["components"]["core"])
83+
core_cases = [v["service"] for target in result["core"] for v in target["cases"]]
84+
self.assertFalse("gdrive" in core_cases)
85+
86+
for language in ["java", "python", "nodejs", "go", "c", "cpp", "dotnet"]:
87+
self.assertTrue(result["components"][f"binding_{language}"])
88+
binding_cases = [
89+
v["service"]
90+
for target in result[f"binding_{language}"]
91+
for v in target["cases"]
92+
]
93+
self.assertFalse("gdrive" in binding_cases)
94+
7795

7896
if __name__ == "__main__":
7997
unittest.main()

0 commit comments

Comments
 (0)