Skip to content

test: skip system tests blocked by PAP#2717

Merged
Pulkit0110 merged 1 commit intomainfrom
fix/skip-pap-system-sample-tests
Feb 5, 2026
Merged

test: skip system tests blocked by PAP#2717
Pulkit0110 merged 1 commit intomainfrom
fix/skip-pap-system-sample-tests

Conversation

@thiyaguk09
Copy link
Contributor

@thiyaguk09 thiyaguk09 commented Feb 4, 2026

Summary
This PR skips specific system tests that require public access permissions. Currently, the test environment restricts the assignment of allUsers or allAuthenticatedUsers roles, which causes these tests to fail with 403 Forbidden errors.

Technical Context
These tests validate flows involving public IAM/ACL overrides. Because the current test environment is configured to prevent public access, these tests are being skipped until the environment supports the necessary overrides for public bucket simulation.

Error Observed: 403 Forbidden (Access Restricted)

Changes

  • Skipped affected system tests requiring public access roles.
  • Added TODO comments to re-enable tests once environment support is available.

@thiyaguk09 thiyaguk09 requested review from a team as code owners February 4, 2026 08:26
@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: storage Issues related to the googleapis/nodejs-storage API. labels Feb 4, 2026
Skips tests that attempt to assign public or authenticated user roles
to IAM policies. These tests are currently failing with 403 errors due
to environment permission restrictions.
@thiyaguk09 thiyaguk09 force-pushed the fix/skip-pap-system-sample-tests branch from 5813b79 to 780a976 Compare February 5, 2026 05:29
@Pulkit0110 Pulkit0110 merged commit 4e3c328 into main Feb 5, 2026
20 checks passed
@Pulkit0110 Pulkit0110 deleted the fix/skip-pap-system-sample-tests branch February 5, 2026 05:42
thiyaguk09 added a commit to thiyaguk09/nodejs-storage-fork that referenced this pull request Feb 5, 2026
test: skip system tests requiring public access (googleapis#2717)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/nodejs-storage API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants