-
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
Do not use skeleton file when not required #35405
Conversation
Codecov Report
@@ Coverage Diff @@
## master #35405 +/- ##
=========================================
Coverage 65.66% 65.66%
Complexity 18666 18666
=========================================
Files 1221 1221
Lines 70597 70597
Branches 1288 1288
=========================================
Hits 46358 46358
Misses 23862 23862
Partials 377 377
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #35405 +/- ##
=========================================
Coverage 65.66% 65.66%
Complexity 18666 18666
=========================================
Files 1221 1221
Lines 70597 70597
Branches 1288 1288
=========================================
Hits 46358 46358
Misses 23862 23862
Partials 377 377
Continue to review full report at Codecov.
|
| username | | ||
| user1 | | ||
| user2 | | ||
# Given these users have been created with default attributes: |
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.
Oops, forgot to remove comments
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.
Please have a look at the comments.
| /simple-folder%20(2)/from_user1/ | | ||
| /simple-folder%20(3)/from_user2/ | | ||
| /simple-folder/from_user1/ | | ||
| /simple-folder%20(2)/from_user2/ | | ||
|
||
Scenario: receive shares with same name from different users |
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.
This scenario is the same as the scenario in line 33. Is there a special reason for this?
1f60a93
to
822563c
Compare
Backport |
822563c
to
5f82c4e
Compare
Rebased to get good CI |
It got a dumb composer error in drone https://drone.owncloud.com/owncloud/core/18013/830 |
5f82c4e
to
9904457
Compare
9904457
to
c8b5ae6
Compare
c8b5ae6
to
0138945
Compare
No code coverage result has come. I have rebased and force-pushed again. Maybe it will come next time? |
Description
Do not use skeleton folder for
webUIAcceptShare
andwebUIAdminSettings
suites to speed up CI.Related Issue
FixesPart of speed-up CI QA#621Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:
Open tasks: