Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore: Fix vault case failure and add one more test for vault key #8378

Merged
merged 4 commits into from
Feb 19, 2025

Conversation

cwangsmv
Copy link
Contributor

@cwangsmv cwangsmv commented Feb 19, 2025

Changes:

  • Fix the flaky case of the newly added vault related smoke add
  • Add one more test case for vault key check
  • Move check enterprise logic for one-line-editor only when subscription key matches the editor id

@cwangsmv cwangsmv marked this pull request as draft February 19, 2025 02:29
@cwangsmv cwangsmv force-pushed the fix/secret-item-switch-issue branch from 67e0dd4 to ffad65d Compare February 19, 2025 05:11
@cwangsmv cwangsmv changed the title Try to fix vault test case failure Chore: Fix vault case failure and add one more test for vault key Feb 19, 2025
@cwangsmv cwangsmv marked this pull request as ready for review February 19, 2025 08:07
@cwangsmv cwangsmv requested a review from ihexxa February 19, 2025 08:09
const expectedVaultKeyValue = 'eyJhbGciOiJBMjU2R0NNIiwiZXh0Ijp0cnVlLCJrIjoia';
const vaultKeyValue = await page.getByTestId('VaultKeyDisplayPanel').innerText();
await expect(vaultKeyValue).toContain(expectedVaultKeyValue);
await page.getByTitle('Copy Vault Key').click();
Copy link
Contributor

Choose a reason for hiding this comment

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

Are there other ways for this check without using the clipboard api?

@gatzjames gatzjames merged commit ea6ca10 into develop Feb 19, 2025
9 checks passed
@gatzjames gatzjames deleted the fix/secret-item-switch-issue branch February 19, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants