-
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
Add acceptance tests for federated resharing #33506
Conversation
ab1b64b
to
d9ee5c2
Compare
Codecov Report
@@ Coverage Diff @@
## master #33506 +/- ##
=========================================
Coverage 64.76% 64.76%
Complexity 18351 18351
=========================================
Files 1198 1198
Lines 69467 69467
Branches 1281 1281
=========================================
Hits 44993 44993
Misses 24101 24101
Partials 373 373
Continue to review full report at Codecov.
|
964fb00
to
b37601d
Compare
0a96918
to
b8c312a
Compare
c71a8d7
to
41ffbbd
Compare
Only CI gods know what is happening here...
|
aebe894
to
bdba793
Compare
e1c574f
to
b03b58b
Compare
e368450
to
b0b49a7
Compare
84eb86c
to
42a5178
Compare
42a5178
to
f55aacd
Compare
b9a1e4a
to
5064de1
Compare
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.
looks good, just minor change requests
I wrote some general explanations about how we usually write the tests
Also, if you don't care particularly to test the UI but the sharing in general, its might be better to write pure API tests, they are much faster
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/webUISharingExternal/federationSharing.feature
Outdated
Show resolved
Hide resolved
5064de1
to
d8a91a5
Compare
a0a2c3c
to
9e29c5a
Compare
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.
Looks good.
There are lots of other combinations that could also be tested some day, but this is a good start.
9e29c5a
to
d104b81
Compare
Stable10: #34172 |
Description
More acceptance tests for federation
Related Issue
#33422
Motivation and Context
Tests
How Has This Been Tested?
By CI
Types of changes
Checklist:
Open tasks: