Skip to content

test(tools): use fast start seeds in generator tests - #12

Merged
utk09-NCL merged 1 commit into
utk09-NCL:mainfrom
princejain756:perf/generator-test-seeds
Nov 8, 2026
Merged

utk09-NCL merged 1 commit into
utk09-NCL:mainfrom
princejain756:perf/generator-test-seeds

Conversation

@princejain756

Copy link
Copy Markdown
Contributor

What changed

  • Set seed: 30_074 and maxTries: 200 in tools/latics-generator.test.ts.
  • Set seed: 2 and maxTries: 25 in tools/tilt-generator.test.ts.

Why

In tools/latics-generator.test.ts, seed: 1 tried 17,253 bad boards before it found the first good water:easy board at seed 17,254, and 4,486 more boards for the second one at seed 21,740. Each test run did this search 5 times (86,270 board evaluations, ~2,020 ms).

Seed 30,074 gives a good water:easy board on the first try, and the next good board is at seed 30,265 (191 tries later). This cuts tools/latics-generator.test.ts from ~2,020 ms to ~23 ms without making any test weaker.

How it was tested

  • npm run fix && npm run check passes.

AI tool used

None

Checklist

  • The change is small and has one purpose.
  • The change is UI only. It has no server, backend, auth or database code.
  • I did not weaken or skip tests.
  • Comments and UI text use simple, clear English.

@netlify

netlify Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for skardela ready!

Name Link
🔨 Latest commit 96369e0
🔍 Latest deploy log https://app.netlify.com/projects/skardela/deploys/6af0716747b2319bba1e2374
😎 Deploy Preview https://deploy-preview-12--skardela.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@princejain756
princejain756 force-pushed the perf/generator-test-seeds branch from 0f93c1d to 96369e0 Compare November 8, 2026 12:56
@utk09-NCL
utk09-NCL merged commit b14ed65 into utk09-NCL:main Nov 8, 2026
5 checks passed
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