HyunWooZZ commented on code in PR #46416:
URL: https://github.com/apache/arrow/pull/46416#discussion_r2094064471
##########
cpp/src/arrow/filesystem/gcsfs_test.cc:
##########
@@ -629,6 +629,38 @@ TEST_F(GcsIntegrationTest, GetFileInfoBucket) {
ASSERT_RAISES(Invalid, fs->GetFileInfo("gs://" + PreexistingBucketName()));
}
+// We intentionally do not test invalid permissions with storage-testbench,
+// because the testbench does not enforce any permission checks (ACL/IAM).
+// See:
+//
https://github.com/googleapis/storage-testbench?tab=readme-ov-file#what-is-this-testbench
+//
+// In real GCS environments, trying to access a bucket without permission
+// results in:
+// - Error code: 5
+// - Status : NOT_FOUND
+//
+// The following test depends on real GCS access and is not suitable for CI/CD
+// environments. To run this test manually, set the environment variable:
+// ARROW_RUN_REAL_GCS_TESTS=1
+//
+// Example:
+// ARROW_RUN_REAL_GCS_TESTS=1 ./debug/arrow-gcsfs-test
+TEST_F(GcsIntegrationTest, GetFileInfoWithoutPermission) {
+ if (!std::getenv("ARROW_RUN_REAL_GCS_TESTS")) {
Review Comment:
I wasn't aware of that convention. I appreciate you fixing it!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
