knopersikcuo/Planner
Watch
1
0
Fork
You've already forked Planner
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Optimize and reorganize the test suite #107

Merged
kleb merged 1 commit from perf/optimize-test-suite into main 2026-08-23 13:34:08 +02:00
Conversation 1 Commits 1 Files changed 37 +1027 -978
kleb commented 2026-08-23 13:33:03 +02:00
Collaborator
Copy link

Summary

  • remove obsolete tests for deleted source shapes and legacy limits
  • split the memory and persistence test collections by responsibility
  • use an API-only Vite configuration for server integration tests
  • run framework-sensitive tests serially and the isolated remainder on eight workers
  • add slow-file scheduling and private Vite caches for parallel workers

Performance

  • baseline: 336.11 seconds
  • optimized full suite: 40.74 seconds
  • improvement: 87.9%, about 8.25x faster

Verification

  • bun run test: 909 passed across 143 files
  • bun run typecheck
  • bun run lint
  • targeted Oxfmt check
  • git diff --check
## Summary - remove obsolete tests for deleted source shapes and legacy limits - split the memory and persistence test collections by responsibility - use an API-only Vite configuration for server integration tests - run framework-sensitive tests serially and the isolated remainder on eight workers - add slow-file scheduling and private Vite caches for parallel workers ## Performance - baseline: 336.11 seconds - optimized full suite: 40.74 seconds - improvement: 87.9%, about 8.25x faster ## Verification - `bun run test`: 909 passed across 143 files - `bun run typecheck` - `bun run lint` - targeted Oxfmt check - `git diff --check`
kAgent commented 2026-08-23 13:33:03 +02:00
Collaborator
Copy link

kReview review

Verdict: no findings

No findings to address in the reviewed diff.

The visible changes do not establish a concrete correctness or security regression.

Reviewed by kReview at 467ae5b6a8. This comment is conservative and based only on the PR diff, metadata, and supplied repository context.

Est. cost ~$0.32 (65.3k in / 290 out / gpt-5.6-sol).

<!-- codex-forgejo-review --> <!-- codex-forgejo-review-head:467ae5b6a82a2fd1698bf986aaad32523d206a1f --> ## kReview review **Verdict:** no findings No findings to address in the reviewed diff. The visible changes do not establish a concrete correctness or security regression. _Reviewed by kReview at `467ae5b6a8`. This comment is conservative and based only on the PR diff, metadata, and supplied repository context._ _Est. cost ~$0.32 (65.3k in / 290 out / gpt-5.6-sol)._ <!-- codex-forgejo-review-state:eyJoZWFkU2hhIjoiNDY3YWU1YjZhODJhMmZkMTY5OGJmOTg2YWFhZDMyNTIzZDIwNmExZiIsInN1bW1hcnkiOiJUaGUgdmlzaWJsZSBjaGFuZ2VzIGRvIG5vdCBlc3RhYmxpc2ggYSBjb25jcmV0ZSBjb3JyZWN0bmVzcyBvciBzZWN1cml0eSByZWdyZXNzaW9uLiIsImZpbmRpbmdzIjpbXSwiY3VtdWxhdGl2ZUNvc3QiOnsidXNkIjowLjMxODYxNiwiaW5wdXRUb2tlbnMiOjY1MzI0LCJvdXRwdXRUb2tlbnMiOjI5MCwibW9kZWwiOiJncHQtNS42LXNvbCJ9fQ== -->
kleb merged commit 5a80076235 into main 2026-08-23 13:34:08 +02:00
kleb deleted branch perf/optimize-test-suite 2026-08-23 13:34:08 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
knopersikcuo/Planner!107
No description provided.