[tests-only][full-ci] Add test for unlocking project space file #7698
Conversation
nabim777
force-pushed
the
add-tests-for-unlock-project-space
branch
from
1826350 to
e13b274
Compare
|
|
nabim777
force-pushed
the
add-tests-for-unlock-project-space
branch
4 times, most recently
from
55e084b to
ab6e487
Compare
nabim777
force-pushed
the
add-tests-for-unlock-project-space
branch
from
ab6e487 to
30f99bf
Compare
| And user "Alice" has uploaded a file inside space "project-space" with content "some data" to "textfile.txt" | ||
| And user "Alice" has locked file "textfile.txt" inside space "project-space" setting the following properties | ||
| | lockscope | <lock-scope> | | ||
| When user "Alice" unlocks the last created lock of file "textfile.txt" inside space "project-space" using the WebDAV API |
There was a problem hiding this comment.
do we already have coverage for locking project space file?
There was a problem hiding this comment.
I try to search but could not find so I think there is no coverage for locking and unlocking project spaces
There was a problem hiding this comment.
Then I would suggest adding locking and unlocking tests
There was a problem hiding this comment.
there is some coverage for locking the project space file
For the first batch of lock/unlock tests IMO we can shift the tests from core to ocis and provide the coverage for unlock as per there is coverage for locking and other edge cases can be added later or in other PR
| - [apiLocks/unlockFiles.feature:21](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiLocks/unlockFiles.feature#L21) | ||
| - [apiLocks/unlockFiles.feature:22](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiLocks/unlockFiles.feature#L22) |
There was a problem hiding this comment.
@ScharfViktor do we wait for fix or just keep it here for now?
There was a problem hiding this comment.
let keep it here. I ask fix it on the next week
nabim777
force-pushed
the
add-tests-for-unlock-project-space
branch
from
30f99bf to
a1eeb14
Compare
nabim777
force-pushed
the
add-tests-for-unlock-project-space
branch
4 times, most recently
from
1778207 to
aa6ef4a
Compare
nabim777
force-pushed
the
add-tests-for-unlock-project-space
branch
from
aa6ef4a to
1a52781
Compare
|
Kudos, SonarCloud Quality Gate passed!
|
delete-merged-branch
bot
deleted the
add-tests-for-unlock-project-space
branch
Description
This PR adds the test for unlocking project space file
Related Issue
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: