forst-lang / forst / 29646245273

75%

main: 76%

Build Type

Pull #146

github

Committed by haveyaseen

Commit Message

fix(executor): propagate context errors when subprocess is killed on cancel

When go run is killed after context cancellation or deadline, os/exec often
returns signal: killed instead of context.Canceled or DeadlineExceeded.
executeGoCode now returns ctx.Err() in that case so callers see the real
reason. Command runner tests assert on the returned error, not ctx.Err().

test: harden nodert, node-runtime, and sidecar tests from review

Sidecar client tests replace fragile try/catch with rejects.toThrow; the
timeout test now calls invokeFunction because discoverFunctions swallows
errors. paths tests clean up temp dirs, assert directory rejection via a
valid moduleId, and drop useless await on Bun rejects matchers. Minor nodert
stdout capture cleanup, exportAllowlistKey JSDoc, and reload transport test
rename to match mocked HTTP 500.

Pull Request Pull Request #146: test(ci): gate interop deploy paths and harden invoke coverage

Coverage Stats

101 of 137 new or added lines in 8 files covered. (73.72%)

10 existing lines in 3 files now uncovered.

35792 of 47446 relevant lines covered (75.44%)

1169.03 hits per line