Skip to content

test: reorganize tests into tests/ dir and add CI workflow#106

Merged
bhjsdev merged 1 commit into
mainfrom
test/reorg-and-ci
Jun 29, 2026
Merged

test: reorganize tests into tests/ dir and add CI workflow#106
bhjsdev merged 1 commit into
mainfrom
test/reorg-and-ci

Conversation

@ohxyz

@ohxyz ohxyz commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

What

  • Relocate tests — move all `*.test.js` from `src/` into a top-level `tests/` directory mirroring the `src/` layout. Sibling-relative imports rewritten to the `@/` alias (already configured in `vitest.config.js`).
  • Fix manager tests — the refactor: event-emitter for pane lifecycle + detached/windowless glass overhaul #102 refactor turned `detached-glass/manager.js` into a pure registry (open/close animation, `opening`/`closing` attrs, and Promise returns moved to the crud layer). Updated the stale tests to assert the registry-only contract.
  • Add CI — new `unit-tests` workflow runs the suite on PR open/synchronize/reopen.

Notes

  • All 74 tests pass.
  • `src/` source is unchanged; only test files moved + assertions updated.
  • The `unit-tests` job can be added as a required status check once it runs here.

- Relocate all *.test.js from src/ into tests/, mirroring the src layout;
  rewrite sibling-relative imports to the @/ alias.
- Update manager tests to match the post-refactor registry-only manager
  (animation/Promises now live in the crud layer).
- Add a unit-tests workflow that runs the suite on PR open/update.
@ohxyz ohxyz requested a review from bhjsdev as a code owner June 29, 2026 03:51
@bhjsdev bhjsdev merged commit bd61257 into main Jun 29, 2026
1 check passed
@bhjsdev bhjsdev deleted the test/reorg-and-ci branch June 29, 2026 03:55
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.

2 participants