Skip to content

[tests-only][full-ci] test(collaboration): cover WOPI lock modes - #12828

Open
anon-pradip wants to merge 3 commits into
owncloud:masterfrom
anon-pradip:test/200-ok-for-wopi
Open

[tests-only][full-ci] test(collaboration): cover WOPI lock modes#12828
anon-pradip wants to merge 3 commits into
owncloud:masterfrom
anon-pradip:test/200-ok-for-wopi

Conversation

@anon-pradip

@anon-pradip anon-pradip commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds acceptance test coverage for #12257:

  • Add acceptance coverage for WOPI LOCK requests after app-open.
  • Verify 200 OK for view/write modes, Viewer sharees, Space Viewers, and public-link Viewers.
  • Add public app-open support using the public-link token.

Related Issue

Part of: #12472

Motivation and Context

How Has This Been Tested?

  • test environment:
  • Locally and CI

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@kw-security

kw-security commented Aug 20, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@ishabaral ishabaral mentioned this pull request Aug 20, 2026
16 tasks
@anon-pradip
anon-pradip force-pushed the test/200-ok-for-wopi branch 2 times, most recently from 8ee8e46 to 14a0000 Compare August 20, 2026 11:35
@anon-pradip
anon-pradip force-pushed the test/200-ok-for-wopi branch from 14a0000 to ca37e9a Compare August 26, 2026 05:28
@anon-pradip
anon-pradip force-pushed the test/200-ok-for-wopi branch from ca37e9a to 3365ddb Compare August 26, 2026 06:04
| space | Personal |
| app | FakeOffice |
| view_mode | <mode> |
When user "Alice" sends a lock request with lock id "abcdef123" to the last opened file using wopi endpoint

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WOPI LOCK request: X-WOPI-Override selects the operation, X-WOPI-Lock carries the lock ID (any non-empty value works - only the status code is asserted).

@anon-pradip
anon-pradip marked this pull request as ready for review August 26, 2026 07:00
@anon-pradip
anon-pradip requested a review from a team as a code owner August 26, 2026 07:00
Comment thread tests/acceptance/features/apiCollaboration/wopi.feature Outdated
Comment thread tests/acceptance/features/apiCollaboration/wopi.feature Outdated
Comment thread tests/acceptance/features/apiCollaboration/wopi.feature Outdated
Comment thread tests/acceptance/features/apiCollaboration/wopi.feature
Comment thread tests/acceptance/features/apiCollaboration/wopi.feature Outdated
Comment thread tests/acceptance/features/apiCollaboration/wopi.feature
Comment thread tests/acceptance/features/apiCollaboration/wopi.feature
Comment thread tests/acceptance/features/apiCollaboration/wopi.feature
@anon-pradip
anon-pradip force-pushed the test/200-ok-for-wopi branch 2 times, most recently from 0dbc601 to ae45a04 Compare August 27, 2026 07:15
@anon-pradip
anon-pradip force-pushed the test/200-ok-for-wopi branch from ae45a04 to 0f61858 Compare August 27, 2026 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants