MetaMask / MetaMask/metamask-extension
Flaky test: `Test Snap Client Status can properly show client status locked state`
@seaona is already working on this.
Since May 15, 2025.
- Dominant language
- TypeScript
- Stars
- 13.2k
- Forks
- 5.6k
- Avg merge
- 2d 5h
- Merged PRs (30d)
- 451
Description
### What is this about?
[View logs](https://github.com/MetaMask/metamask-extension/actions/runs/14917149930/job/41906536469)
```js
UnexpectedAlertOpenError: Unexpected alert dialog detected. Performed handler "dismiss". Dialog text: The Snaps execution environment failed to start.
at Object.throwDecodedError (node_modules/selenium-webdriver/lib/error.js:523:15)
at parseHttpResponse (node_modules/selenium-webdriver/lib/http.js:524:13)
at Executor.execute (node_modules/selenium-webdriver/lib/http.js:456:28)
at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
at async thenableWebDriverProxy.execute (node_modules/selenium-webdriver/lib/webdriver.js:745:17)
at async WindowHandles.getCurrentWindowProperties (test/e2e/background-socket/window-handles.ts:71:22)
at async WindowHandles.switchToHandleAndCheckForProperty (test/e2e/background-socket/window-handles.ts:205:28)
at async WindowHandles.switchToWindowWithProperty (test/e2e/background-socket/window-handles.ts:161:31)
at async Driver.switchToWindowWithTitle (test/e2e/webdriver/driver.js:1168:12)
at async openTestSnapClickButtonAndInstall (test/e2e/page-objects/flows/install-test-snap.flow.ts:46:3)
at async (test/e2e/snaps/test-snap-clientstatus.spec.ts:23:9)
at async withFixtures (test/e2e/helpers.js:307:5)
at async Context. (test/e2e/snaps/test-snap-clientstatus.spec.ts:11:5)
```
### Scenario
_No response_
### Design
_No response_
### Technical Details
_No response_
### Threat Modeling Framework
_No response_
### Acceptance Criteria
_No response_
### Stakeholder review needed before the work gets merged
- [ ] Engineering (needed in most cases)
- [ ] Design
- [ ] Product
- [ ] QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
- [ ] Security
- [ ] Legal
- [ ] Marketing
- [ ] Management (please specify)
- [ ] Other (please specify)
### References
_No response_
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.