Visitar URL original
[tests-only][full-ci] test(e2e): verify permanent links copied in Vault mode open successfully by Asmitapaudel · Pull Request #13123 · owncloud/ocis · GitHub
Skip to content

[tests-only][full-ci] test(e2e): verify permanent links copied in Vault mode open successfully - #13123

Merged
PrajwolAmatya merged 1 commit into
owncloud:masterfrom
Asmitapaudel:test/vault-permanent-link
Nov 2, 2026
Merged

[tests-only][full-ci] test(e2e): verify permanent links copied in Vault mode open successfully#13123
PrajwolAmatya merged 1 commit into
owncloud:masterfrom
Asmitapaudel:test/vault-permanent-link

Conversation

@Asmitapaudel

@Asmitapaudel Asmitapaudel commented Oct 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Related to [owncloud/ocis#12881](#12881).

Add an E2E test to verify that a permanent link copied in Vault mode opens the file successfully.

The test verifies that the user can access the shared file through the permanent link and view its expected content without encountering a private link resolution error.

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
    • Locally
    • 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:

@update-docs

update-docs Bot commented Oct 12, 2026

Copy link
Copy Markdown

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@kw-security

kw-security commented Oct 12, 2026 •
edited
Loading

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.

@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from 1b3cd1a to 398306b Compare October 12, 2026 10:03
@Asmitapaudel Asmitapaudel changed the title test(e2e): verify Vault authentication when opening a permanent link from Drive mode [tests-only][full-ci] test(e2e): verify Vault authentication when opening a permanent link from Drive mode Oct 12, 2026
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from 398306b to 292b215 Compare October 12, 2026 10:06
@Asmitapaudel
Asmitapaudel marked this pull request as ready for review October 12, 2026 11:22
@Asmitapaudel
Asmitapaudel requested a review from a team as a code owner October 12, 2026 11:22
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from 292b215 to aab23b0 Compare October 13, 2026 11:11
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from aab23b0 to 0e39444 Compare October 13, 2026 11:17
@Asmitapaudel Asmitapaudel changed the title [tests-only][full-ci] test(e2e): verify Vault authentication when opening a permanent link from Drive mode [tests-only][full-ci] test(e2e): verify permanent links copied in Vault mode open successfully Oct 13, 2026
saw-jan
saw-jan reviewed Oct 14, 2026
View reviewed changes
Comment thread web/tests/e2e/specs/vault-storage/vault-access.spec.ts Resolved
saw-jan
saw-jan reviewed Oct 14, 2026
View reviewed changes
Comment thread web/tests/e2e/steps/ui/links.ts Resolved
saw-jan
saw-jan reviewed Oct 14, 2026
View reviewed changes
Comment thread web/tests/e2e/support/objects/app-files/resource/actions.ts Resolved
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from 0e39444 to 5ee158d Compare October 14, 2026 08:43
@Asmitapaudel
Asmitapaudel requested a review from saw-jan October 14, 2026 08:43
saw-jan
saw-jan reviewed Oct 14, 2026
View reviewed changes
Comment thread web/tests/e2e/support/objects/app-files/resource/actions.ts Resolved
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch 2 times, most recently from 5ee158d to b7c69cd Compare October 15, 2026 05:25
@Asmitapaudel
Asmitapaudel requested a review from saw-jan October 15, 2026 05:25
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from b7c69cd to 92ace09 Compare October 16, 2026 03:45
saw-jan
saw-jan reviewed Oct 16, 2026
View reviewed changes
Comment thread web/tests/e2e/steps/ui/links.ts Resolved
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from 92ace09 to 05207cf Compare October 16, 2026 06:44
@Asmitapaudel
Asmitapaudel requested a review from saw-jan October 16, 2026 06:46
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from 05207cf to fdf9384 Compare October 26, 2026 04:09
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from fdf9384 to cd5a0c1 Compare October 27, 2026 05:04
@Asmitapaudel
test(e2e): verify permanent links copied in Vault mode open successfully
a08cc05 
Signed-off-by: Asmitapaudel <asmeetapdl@gmail.com>
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-permanent-link branch from cd5a0c1 to a08cc05 Compare November 2, 2026 04:02
@PrajwolAmatya
PrajwolAmatya merged commit d0ece39 into owncloud:master Nov 2, 2026
80 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PrajwolAmatya PrajwolAmatya PrajwolAmatya approved these changes
@anon-pradip anon-pradip Awaiting requested review from anon-pradip
@kavitagautam kavitagautam Awaiting requested review from kavitagautam
@ishabaral ishabaral Awaiting requested review from ishabaral
@Ashim-Stha Ashim-Stha Awaiting requested review from Ashim-Stha
+1 more reviewer
@saw-jan saw-jan saw-jan approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Asmitapaudel @kw-security @saw-jan @PrajwolAmatya