Skip to content

Conversation

@claneo
Copy link
Collaborator

@claneo claneo commented Dec 29, 2025

Summary

20251229-235944.mp4

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link

netlify bot commented Dec 29, 2025

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit aa118ce
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/6953358d94af8a00089722f1
😎 Deploy Preview https://deploy-preview-814--rstest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@claneo claneo marked this pull request as ready for review December 30, 2025 02:01
Copilot AI review requested due to automatic review settings December 30, 2025 02:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds continuous run/watch mode support to the VSCode extension, allowing tests to automatically re-run when files change. The implementation adds a new onTestRunStart lifecycle hook to the reporter interface and updates the VSCode extension to support continuous test execution.

  • Added onTestRunStart lifecycle hook to the Reporter interface for notifying when a test run begins
  • Implemented continuous/watch mode by passing command: 'watch' to the test worker and managing multiple test run cycles
  • Refactored test run management to create new TestRun objects for subsequent runs in watch mode

Reviewed changes

Copilot reviewed 17 out of 17 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
website/docs/zh/api/javascript-api/reporter.mdx Added onTestRunStart to the list of run-level hooks in Chinese documentation
website/docs/en/api/javascript-api/reporter.mdx Added onTestRunStart to the list of run-level hooks in English documentation
packages/vscode/tests/suite/progress.test.ts Refactored test suite to support testing continuous run mode; added new test case for continuous run behavior
packages/vscode/tests/fixtures/workspace-1/test/progress.test.ts Updated test fixture values to match test expectations
packages/vscode/src/worker/reporter.ts Added onTestRunStart and onTestRunEnd implementations; added onCoverageEnd support
packages/vscode/src/worker/index.ts Reordered reporters to ensure default reporter output is flushed before progress reporter
packages/vscode/src/types.ts Added 'watch' command type to support continuous run mode
packages/vscode/src/testRunReporter.ts Implemented logic to create new test runs for subsequent cycles in continuous mode
packages/vscode/src/master.ts Updated runTest to support continuous mode with promise-based completion tracking
packages/vscode/src/extension.ts Enabled continuous mode for all run profiles; updated to pass createTestRun function for creating subsequent runs
packages/core/src/types/reporter.ts Added onTestRunStart lifecycle hook to Reporter interface
packages/core/src/pool/forks.ts Removed SIGTERM handler cleanup code
packages/core/src/core/runTests.ts Added call to onTestRunStart for all reporters before running tests
packages/core/src/core/rstest.ts Simplified reporters type from union to just Reporter
packages/core/src/core/globalSetup.ts Removed SIGTERM handler cleanup code
e2e/reporter/rstest.customReporterConfig.ts Added onTestRunStart implementation to custom reporter
e2e/reporter/index.test.ts Added assertion to verify onTestRunStart is called

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@9aoy
Copy link
Collaborator

9aoy commented Jan 5, 2026

👍

@9aoy 9aoy merged commit 433438a into main Jan 5, 2026
16 checks passed
@9aoy 9aoy deleted the vscode-continuous-run branch January 5, 2026 06:39
@9aoy 9aoy mentioned this pull request Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants