Skip to content

gap: Add runtime tests - #6016

Draft
mkoeppe wants to merge 2 commits into
emscripten-forge:mainfrom
passagemath:gap--add-node-tests
Draft

gap: Add runtime tests#6016
mkoeppe wants to merge 2 commits into
emscripten-forge:mainfrom
passagemath:gap--add-node-tests

Conversation

@mkoeppe

@mkoeppe mkoeppe commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Adding tests as suggested in #5965 (comment)

FYI @wangyenshu @ChrisJefferson

@wangyenshu

Copy link
Copy Markdown
Contributor

The documentation for gap wasm build is here https://github.com/gap-system/gap/tree/master/etc/emscripten. For a working website, the steps in https://github.com/gap-system/gap/blob/master/etc/emscripten/assemble-website.sh is needed.

@ChrisJefferson

Copy link
Copy Markdown

I don't know how much of a test you are happy to run. If a copy of GAP's test directory is available, Read("tst/testinstall.g");would be best, but it would take a while. If GASMAN is broken (as I suspect when ASYNCIFY was removed), these tests won't be enough to discover that as they aren't doing enough work to cause the GC to activate and garbage collection to not work correctly.

@mkoeppe

mkoeppe commented Jul 27, 2026

Copy link
Copy Markdown
Contributor Author

A specific test that exercises the GC (GASMAN) would be most valuable.

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