[kavitagautam]

Description

This PR adds test for vault resource isolation.

Related Issue

• [QA] Add e2e tests for Vault mode #13864

Motivation and Context

How Has This Been Tested?

• test environment:
• test case 1:
• test case 2:
• ...

Screenshots (if appropriate):

Open tasks:

• ...

[kw-security]

✅ 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.

[DeepDiver1975]

Verdict: changes requested. CI is red and the resource-isolation test as written cannot pass.

CI (build / build) fails at Check format (prettier); check-tests-passed is consequently red, e2e skipped. The diff has missing trailing newlines (steps/ui/index.ts, vault.spec.ts, etc.) and trailing whitespace (e.g. the blank line in the Resource Isolation beforeEach, the }) after the isolation test). Run pnpm format / prettier and push.

Resource Isolation test can't pass as written — tests/e2e/specs/vault-mode/vault.spec.ts:

• beforeEach creates drive/drive.txt, but the test asserts on drive-file.txt / vault-file.txt — names don't match the created file.
• userHasCreatedVaultFiles(... vault/vault.txt ...) is commented out, so vault-file.txt is never created; userShouldSeeResources(['vault-file.txt']) in Vault mode will fail.
• Net: the test that proves AC-3 (the PR's point) is not actually exercising isolation. Either finish the vault-file setup and align the filenames, or split it out until the userHasCreatedVaultFiles step exists.

The mode-switch test, the VaultPage POM, and generateOtpFromScreenshot (jimp is already a dep) look consistent with existing conventions; the issue is scoped to the isolation case + formatting.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[LukasHirt]

Web repo is going to be archived and will no longer accept new PRs. Please, use owncloud/ocis repository instead.