Skip to content

feat: support rerun all tests via a CLI shortcut#454

Merged
9aoy merged 9 commits intomainfrom
shortcut-rerun-all
Aug 8, 2025
Merged

feat: support rerun all tests via a CLI shortcut#454
9aoy merged 9 commits intomainfrom
shortcut-rerun-all

Conversation

@9aoy
Copy link
Copy Markdown
Collaborator

@9aoy 9aoy commented Aug 7, 2025

Summary

support rerun all tests via a CLI shortcut.

image

Related Links

Checklist

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

Copilot AI review requested due to automatic review settings August 7, 2025 08:50
@netlify
Copy link
Copy Markdown

netlify bot commented Aug 7, 2025

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit 6f73471
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/689496ff64bf1400083e9669
😎 Deploy Preview https://deploy-preview-454--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.

Copy link
Copy Markdown
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 support for rerunning all tests via the a CLI shortcut in watch mode. The change refactors the server lifecycle management to enable on-demand test execution while maintaining the development server.

  • Restructured createRsbuildServer to return both getRsbuildStats and closeServer functions separately
  • Added runAll callback to CLI shortcuts for triggering test reruns
  • Updated watch mode to support interactive test execution

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
packages/core/src/core/runTests.ts Refactored server lifecycle and added watch mode CLI shortcuts with rerun capability
packages/core/src/core/rsbuild.ts Changed return type to separate stats getter from server closure
packages/core/src/core/cliShortcuts.ts Added 'a' shortcut for rerunning all tests and removed redundant help message

await close();
},
});
// TODO: no need `enter`
Copy link

Copilot AI Aug 7, 2025

Choose a reason for hiding this comment

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

The TODO comment contains a grammatical error. It should read '// TODO: no need for enter' or '// TODO: enter not needed'.

Suggested change
// TODO: no need `enter`
// TODO: no need for `enter`

Copilot uses AI. Check for mistakes.
@9aoy 9aoy merged commit e0258d8 into main Aug 8, 2025
16 checks passed
@9aoy 9aoy deleted the shortcut-rerun-all branch August 8, 2025 03:13
@9aoy 9aoy mentioned this pull request Aug 8, 2025
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