Skip to content

api-test: fix removeAccessToDrive.feature:145 - #3179

Merged
v-scharf merged 3 commits into
mainfrom
fix-test-removeAccess0toDrive
Jul 28, 2026
Merged

api-test: fix removeAccessToDrive.feature:145#3179
v-scharf merged 3 commits into
mainfrom
fix-test-removeAccess0toDrive

Conversation

@v-scharf

Copy link
Copy Markdown
Contributor

@v-scharf
v-scharf requested review from rhafer and saw-jan July 27, 2026 14:47
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

Comment thread tests/acceptance/bootstrap/SharingNgContext.php Outdated
@v-scharf
v-scharf enabled auto-merge (squash) July 28, 2026 07:33
@v-scharf
v-scharf requested a review from saw-jan July 28, 2026 07:33
@v-scharf

Copy link
Copy Markdown
Contributor Author

apiSharingNg1/enableDisableShareSync.feature:422 test is flaky because Brian (a member of the group) occasionally receives a 404 when requesting the permissions of a project space.

The issue cannot be reproduced locally, but it periodically fails in CI. Retrying the permission request once should be enough to mitigate this failure.

### When user "Brian" tries to remove own group access from space "NewSpace" using root endpoint of the Graph API
		_______________________________________________________________________

		==> REQUEST [2026-07-28T10:03:54]
		GET /graph/v1.0/me
		X-Request-ID: apiSharingNg1/removeAccessToDrive.feature:145-156

		_______________________________________________________________________

		==> REQUEST [2026-07-28T10:03:54]
		GET /graph/v1beta1/drives/f3cd509c-0f5e-4d84-8b34-1d7224225df3$ec2bcb75-9683-489e-959f-a73fc44a3dcf/root/permissions
		X-Request-ID: apiSharingNg1/removeAccessToDrive.feature:145-156

		<== RESPONSE [2026-07-28T10:03:54]
		404 Not Found
		X-Xss-Protection: 1; mode=block
		<== RES BODY
		{"error":{"code":"itemNotFound","innererror":{"date":"2026-07-28T10:03:54Z","request-id":"apiSharingNg1/removeAccessToDrive.feature:145-156"},"message":"getting space"}}

@v-scharf
v-scharf merged commit a2bdbc8 into main Jul 28, 2026
53 of 54 checks passed
@v-scharf
v-scharf deleted the fix-test-removeAccess0toDrive branch July 28, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants