Skip to content

feat: add full test suite (C + JS) and test-suite.yml CI workflow - #2493

Draft
o4k741x with Copilot wants to merge 2 commits into
prodfrom
copilot/run-xaoex-workflows-again
Draft

feat: add full test suite (C + JS) and test-suite.yml CI workflow#2493
o4k741x with Copilot wants to merge 2 commits into
prodfrom
copilot/run-xaoex-workflows-again

Conversation

Copilot AI commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

No real tests existed — npm test was a no-op echo and CMake had only a smoke-run of the binary.

Test suite (tests/)

  • test_simsim.c — build sanity + exit-code contract (4 assertions)
  • test_maxopts.c — 9 unit tests for maxopts_everything_this_era(): nominal paths, NULL, zero/negative size, INT_MAX boundary
  • test_index.js — 15 tests for index.js: module shape, init()/info() return values, link format, idempotency

Build wiring

  • CMakeLists.txt: adds test_simsim and test_maxopts executables as UnitTest_* CTest targets
  • package.json: test script now runs node tests/test_index.js

Workflow (.github/workflows/test-suite.yml)

  • Triggers: push/PR to prod/main/production + workflow_dispatch
  • Jobs: JS tests (Node 18/20/22), CMake build+ctest (gcc + clang), direct C compile+run (gcc + clang), summary gate
  • All jobs scoped to permissions: contents: read

Copilot AI and others added 2 commits August 4, 2026 05:26
Co-authored-by: professoroakz <6593422+professoroakz@users.noreply.github.com>
Co-authored-by: professoroakz <6593422+professoroakz@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings August 4, 2026 05:28

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot AI changed the title feat: add full test suite (C + JS) and test-suite.yml workflow feat: add full test suite (C + JS) and test-suite.yml CI workflow Aug 4, 2026
Copilot AI requested a review from o4k741x August 4, 2026 05:30
@o4k741x
o4k741x force-pushed the copilot/run-xaoex-workflows-again branch from 4c2c6d4 to 621ff92 Compare August 9, 2026 08:23
@o4k741x
o4k741x force-pushed the prod branch 2 times, most recently from 125094f to 718be2e Compare August 9, 2026 09:08
@o4k741x
o4k741x force-pushed the copilot/run-xaoex-workflows-again branch from 621ff92 to 3c9ed98 Compare August 9, 2026 09:08
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.

3 participants