Fix test race condition with close event

[Copilot]

Test failures in CI due to race condition between Node.js child process exit and close events. The exit event fires before stdio streams flush, causing intermittent failures when output isn't fully captured.

Changes

• Use close event instead of exit: Waits for stdio stream closure before resolving, ensuring all output is captured
• Add handleCompletion guard: Prevents multiple event handlers from racing to resolve/reject the same promise
• Fix typo: "sucessfully" → "successfully"

// Before: exit event can fire before streams close
runTests.on("exit", (code) => {
  if (hasExited) return;
  hasExited = true;
  // ... process results
});

// After: close event guarantees streams are closed
runTests.on("close", (code) => {
  handleCompletion(() => {
    // ... process results
  });
});

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}