-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Tests-Only] Acceptance test to assert the sharer can unshare and the receiver no longer sees the federated share #37090
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Codecov Report
@@ Coverage Diff @@
## master #37090 +/- ##
=========================================
Coverage 64.76% 64.76%
Complexity 19136 19136
=========================================
Files 1270 1270
Lines 74915 74915
Branches 1328 1328
=========================================
Hits 48516 48516
Misses 26009 26009
Partials 390 390
Continue to review full report at Codecov.
|
@jasson99 more conflicts. We can probably just get 1 of these PRs merged at a time... |
2d307b1
to
b99fe37
Compare
LGTM 👍 |
… receiver no longer sees the federated share
b99fe37
to
ec5d58f
Compare
Description
Acceptance test to assert the sharer can unshare and the receiver no longer sees the federated share
Related Issue
#34149
How Has This Been Tested?
CI
Types of changes
Checklist: