Adopt SQLite project unit test harness#692
Open
brandonpayton wants to merge 4 commits into
Open
Conversation
93ef2b4 to
92cd505
Compare
Phase B-1 matrix build status —
|
| Package | Arch | Status | Sha |
|---|---|---|---|
| kandelo-sdk | wasm32 | built | bb590098 |
| lamp | wasm32 | built | 1bffff45 |
| rootfs | wasm32 | built | 992f6fe4 |
| wordpress | wasm32 | built | 6c61b817 |
| shell | wasm32 | built | 3192a681 |
| node-vfs | wasm32 | built | f0d638b2 |
Auto-generated; replaced on each push. Raw data in the publish-status workflow artifact.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Completes the SQLite project-unit testing epic handoff for
kad-wtbafter adopting useful work from external PR #5 and recording the current Node/browser validation state.Earlier child work for the harness and base validation has already reached
main. This PR now carries the four remaining unique integration commits overmain: browser SQLite memory-cap stability work, the SharedFS open-unlink/rename-over fix, browser fault/crash classification artifacts, and browser LIKE timing classification artifacts.Adopted brandonpayton/kandelo-agentic-dev PR #5 Scope
Across the
kad-wtbepic, the adopted scope includes:scripts/run-sqlite-project-unit-tests.sh, the combined SQLite project-unit wrapper around the official SQLite runner.test-runs/.fullon both hosts; the largerallpermutation is tracked separately bykad-29m.This PR and the child work avoid skipping or xfail-ing upstream SQLite tests as a substitute for runtime fixes.
Validation Status
PR branch:
integration/kad-wtb-sqlite-testingBase:
mainCurrent head:
92cd50563d8ad1110827032f9fd0fb7c0f7ad960GitHub mergeability after rebase: mergeable; review required; checks pending.
Node:
WebAssembly.compile().kad-36gis closed:examples/run-example.tsnow constrains exec fallback so arbitrary host files outsideKERNEL_CWDare not resolved as guest programs unless they are explicit.wasmcandidates.Browser:
testrunner.dbartifacts before page reload/navigation.kad-wtb.10is closed with browser memory/test harness stability work included in this PR.kad-wtb.12is closed: focused browsersavepoint6.testpassed after preserving unlinked SharedFS inodes until last close. The SharedFS fix is included in this PR.kad-wtb.13is closed: focused Node/browserlike.testreruns classify the originallike-14.2symptom as timing-threshold behavior, not a string/collation semantic mismatch. The classification artifacts are included in this PR.kad-wtb.11is closed as a classification: isolatedsysfault.testpasses;writecrash.testandwalfault.testare narrowed to browser process teardown / executable materialization andmunmaptrap behavior rather than the original generic crash snapshot. The classification report is included in this PR.Gate verification was recorded on the rebased child branches before merge:
cargo test -p kandelo --target aarch64-apple-darwin --lib,cd host && npx vitest run,scripts/run-libc-tests.sh,scripts/run-posix-tests.sh, andscripts/dev-shell.sh bash scripts/check-abi-version.shall passed in the closing notes for the latest child work. This PR-opening bead only created and updated the GitHub handoff; it did not add new source changes beyond rebasing the integration head onto currentmain.Known Remaining Blocker
kad-wtb.9remains open/hooked: focused browsersort4.teststill exposes the threaded-sorter crash/stall shape. The latest notes say the browser no longer traps OOB after positioned-I/O work, but the direct runner times out atsort4-2.3; the next focus is browser thread worker startup/channel handoff after late SQLite sorter clones.Artifacts
test-runs/gastown-sqlite-node-full-pr5/test-runs/gastown-sqlite-browser-full-pr5-snapshot/test-runs/gastown-sqlite-epic-synthesis/summary.mdtest-runs/kad-wtb13-like-*andtest-runs/gastown-sqlite-browser-full-pr5-snapshot/browser-fault-recheck.mdkad-wtb.12:test-runs/kad-wtb.12-savepoint6-focused-after-fixRelated
kad-wtbkad-wtb.8kad-wtb.9