Skip to content

Testing: MiniWebAppJitProjectTest PATH_INFO hello + contact POST (#1759) - #1841

Merged
PurHur merged 1 commit into
masterfrom
issue-1759-miniwebapp-jit-pathinfo-post
May 25, 2026
Merged

Testing: MiniWebAppJitProjectTest PATH_INFO hello + contact POST (#1759)#1841
PurHur merged 1 commit into
masterfrom
issue-1759-miniwebapp-jit-pathinfo-post

Conversation

@PurHur

@PurHur PurHur commented May 25, 2026

Copy link
Copy Markdown
Owner

Summary

Closes #1759.

Extends MiniWebAppJitProjectTest (phase 2 of #587) with two JIT project smokes that mirror existing VM/AOT coverage:

  • PATH_INFO helloMiniWebAppCgiEnv::pathInfoHello('Dev'), asserts Hello Dev
  • Contact POSTMiniWebAppCgiEnv::postQueryRouteContact(), asserts Thank you, PostDev

Reuses shared CGI helpers (no duplicated env strings). On JIT compile failure, runIndex() skips with an explicit not jittable message (#475, #58) instead of a bare non-zero exit.

Test plan

MINIWEBAPP_JIT_PROJECT_GATE=1 ./script/docker-ci-local.sh --filter MiniWebAppJitProjectTest
# or targeted (tar bind-mount harness):
MINIWEBAPP_JIT_PROJECT_GATE=1 ./script/docker-ci-local.sh  # ci-local @group miniwebapp-jit-project

On this Runforge harness, jit-runtime-probe.php segfaults (bin/jit.php -l exit 139), so all three tests skip in setUp with the existing MCJIT probe message — acceptable per #1759 acceptance (pass or explicit skip). On hosts with a working LLVM MCJIT probe, all three routes should assert.

Made with Cursor

Add JIT project smoke for hello PATH_INFO and contact POST, mirroring
MiniWebAppAotExecuteTest and MiniWebAppVmCliTest helpers. Skip with an
explicit message when JIT reports not jittable (#475, #58).

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit 6d2dd63 into master May 25, 2026
@PurHur
PurHur deleted the issue-1759-miniwebapp-jit-pathinfo-post branch May 25, 2026 01:12
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.

Testing: MiniWebAppJitProjectTest — PATH_INFO hello + contact POST (#587 phase 2)

1 participant