---
number: 7415
title: 'Fix #7270 Migrated ManagerInstance test to playwright'
author: nabeel001
state: MERGED
createdAt: '2025-10-08T03:09:48Z'
updatedAt: '2025-10-08T11:53:38Z'
closedAt: '2025-10-08T11:53:38Z'
mergedAt: '2025-10-08T11:53:38Z'
head: ManagerInstance-convert-playwright
base: dev
url: 'https://github.com/neomjs/neo/pull/7415'
---
Please make sure to read the Contributing Guidelines:

https://github.com/neomjs/neo/blob/dev/CONTRIBUTING.md

<!-- PULL REQUEST TEMPLATE -->
<!-- (Update "[ ]" to "[x]" to check a box) -->

**What kind of change does this PR introduce?** (check at least one)

- [ ] Bugfix
- [ ] Feature
- [ ] Code style update
- [x] Refactor
- [ ] Build-related changes
- [ ] Other, please describe:

**Does this PR introduce a breaking change?** (check one)

- [ ] Yes
- [x] No

If yes, please describe the impact and migration path for existing applications:

**The PR fulfills these requirements:**

- [x] It's submitted to the `dev` branch, _not_ the `main` branch
- [x] When resolving a specific issue, it's referenced in the PR's title (e.g. `fix #xxx[,#xxx]`, where "xxx" is the issue number)

If adding a **new feature**, the PR's description includes:
- [ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

**Other information:**
Fixes #7270 
Migrated ManagerInstance test to playwright


## Comments

### `@tobiu` commented on 2025-10-08T11:53:31Z

input from gemini:

> ✦ Based on my review, the pull request looks good.
> 
>   The new Playwright test at test/playwright/unit/ManagerInstance.spec.mjs is a faithful and correct migration of the original Siesta
>   test. All tests and assertions have been translated properly.
> 
>   The contributor has also correctly followed all the guidelines outlined in the epic-enhance-workflow-with-mandatory-unit-testing.md
>   document:
>    - The necessary core imports (Neo, core) are present.
>    - A clear, intent-driven JSDoc comment with a @summary tag has been added.
>    - The test structure uses test.describe() and test() correctly.
> 
>   The implementation is clean and adheres to the project's testing standards. This PR is ready to be merged.

---

