[stable29] Apply group limit on remove from group

[backportbot]

Backport of #47180

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

• Review and resolve any conflicts
• Remove all the empty commits
• Amend HEAD commit to remove the line stating to skip CI

---

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

[come-nc]

/backport to stable28

[come-nc]

cypress failure is related I think

 Running: files_sharing/limit_to_same_group.cy.ts (3 of 6)
353
354
355
Limit to sharing to people in the same group
356
1) "before all" hook for "Alice can see the shared file"
357
358
359
0 passing (23s)
360
1 failing
361
362
1) Limit to sharing to people in the same group
363
"before all" hook for "Alice can see the shared file":
364
CypressError: `cy.request()` failed on:
365
366
http://172.17.0.2/index.php/apps/files
367
368
The response we received from your web server was:
369
370
> 401: Unauthorized
371
372
This was considered a failure because the status code was not `2xx` or `3xx`.
373
374
If you do not want status codes to cause failures pass the option: `failOnStatusCode: false`

[come-nc]

/backport to stable28

[come-nc]

But the node failure I do not understand 🤔

[come-nc]

cypress failure is related I think

 Running: files_sharing/limit_to_same_group.cy.ts (3 of 6)
353
354
355
Limit to sharing to people in the same group
356
1) "before all" hook for "Alice can see the shared file"
357
358
359
0 passing (23s)
360
1 failing
361
362
1) Limit to sharing to people in the same group
363
"before all" hook for "Alice can see the shared file":
364
CypressError: `cy.request()` failed on:
365
366
http://172.17.0.2/index.php/apps/files
367
368
The response we received from your web server was:
369
370
> 401: Unauthorized
371
372
This was considered a failure because the status code was not `2xx` or `3xx`.
373
374
If you do not want status codes to cause failures pass the option: `failOnStatusCode: false`

Maybe only flaky, it seems to pass now? (runner 2 still failed but on another test, files_sharing/limit_to_same_group.cy.ts passed.

[come-nc]

It was a flaky test, it’s now passing.
So, once the node issue is fixed (in stable29 as it’s failing there too), this can be merged and backported further.

[skjnldsv]

Manually approved and merged in RC1.
Discussed with team lead