Visitar URL original
[tests-only][full-ci] test(e2e): Vault notifications are isolated from Drive notifications by Asmitapaudel · Pull Request #13148 · owncloud/ocis · GitHub
Skip to content

[tests-only][full-ci] test(e2e): Vault notifications are isolated from Drive notifications - #13148

Open
Asmitapaudel wants to merge 1 commit into
owncloud:masterfrom
Asmitapaudel:test/vault-notifications-isolated-from-drive
Open

[tests-only][full-ci] test(e2e): Vault notifications are isolated from Drive notifications#13148
Asmitapaudel wants to merge 1 commit into
owncloud:masterfrom
Asmitapaudel:test/vault-notifications-isolated-from-drive

Conversation

@Asmitapaudel

@Asmitapaudel Asmitapaudel commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Description

Add an E2E test to verify that Vault and Drive notifications are isolated, ensuring that notifications from one mode are not visible in the other while notifications are correctly displayed in their respective modes.

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:

@kw-security

kw-security commented Oct 14, 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.

PrajwolAmatya
PrajwolAmatya reviewed Oct 14, 2026
View reviewed changes
Comment thread web/tests/e2e/specs/vault-storage/notification.spec.ts
Comment thread web/tests/e2e/specs/vault-storage/notification.spec.ts Outdated
Comment thread web/tests/e2e/specs/vault-storage/notification.spec.ts Outdated
Comment thread web/tests/e2e/specs/vault-storage/notification.spec.ts Outdated
Comment thread web/tests/e2e/specs/vault-storage/notification.spec.ts
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-notifications-isolated-from-drive branch 2 times, most recently from b10fb79 to 2597b21 Compare October 14, 2026 09:13
@Asmitapaudel
Asmitapaudel marked this pull request as ready for review October 14, 2026 09:13
@Asmitapaudel
Asmitapaudel requested a review from a team as a code owner October 14, 2026 09:13
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-notifications-isolated-from-drive branch from 2597b21 to f7db075 Compare October 16, 2026 03:45
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-notifications-isolated-from-drive branch from f7db075 to 669a76a Compare October 26, 2026 04:07
@Asmitapaudel
Asmitapaudel force-pushed the test/vault-notifications-isolated-from-drive branch from 669a76a to c1293a6 Compare October 27, 2026 05:05

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PrajwolAmatya PrajwolAmatya Awaiting requested review from PrajwolAmatya

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Asmitapaudel @kw-security @PrajwolAmatya