Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(vitest-angular): add UI and coverage options to test #1521

Merged
merged 2 commits into from
Dec 26, 2024

Conversation

jorgeandrespadilla
Copy link
Contributor

PR Checklist

Closes #1520

What is the new behavior?

This PR adds support for the --coverage and --ui flags in the @analogjs/vitest-angular builder CLI. Users can now dynamically enable these options without manually configuring them in angular.json.

Example usage:

ng test --coverage   # Display test coverage  
ng test --ui         # Enable UI mode  

This simplifies testing workflows by reducing unnecessary angular.json complexity.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

[optional] What gif best describes this PR or how it makes you feel?

Copy link

netlify bot commented Dec 24, 2024

Deploy Preview for analog-blog ready!

Name Link
🔨 Latest commit 5f4482d
🔍 Latest deploy log https://app.netlify.com/sites/analog-blog/deploys/676d97a11be42c0008173f18
😎 Deploy Preview https://deploy-preview-1521--analog-blog.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 site configuration.

Copy link

netlify bot commented Dec 24, 2024

Deploy Preview for analog-docs ready!

Name Link
🔨 Latest commit 5f4482d
🔍 Latest deploy log https://app.netlify.com/sites/analog-docs/deploys/676d97a116c15500087b4d6b
😎 Deploy Preview https://deploy-preview-1521--analog-docs.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 site configuration.

Copy link

netlify bot commented Dec 24, 2024

Deploy Preview for analog-app ready!

Name Link
🔨 Latest commit 5f4482d
🔍 Latest deploy log https://app.netlify.com/sites/analog-app/deploys/676d97a18d80300008f850af
😎 Deploy Preview https://deploy-preview-1521--analog-app.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 site configuration.

Copy link

netlify bot commented Dec 24, 2024

Deploy Preview for analog-ng-app ready!

Name Link
🔨 Latest commit 5f4482d
🔍 Latest deploy log https://app.netlify.com/sites/analog-ng-app/deploys/676d97a1a942bf00087a4138
😎 Deploy Preview https://deploy-preview-1521--analog-ng-app.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 site configuration.

@brandonroberts
Copy link
Member

Thanks @jorgeandrespadilla! Will you run pnpm run prettify to format the necessary files and push those changes?

@jorgeandrespadilla
Copy link
Contributor Author

Sure! Just pushed the changes.

@brandonroberts brandonroberts merged commit 026b3dc into analogjs:beta Dec 26, 2024
24 checks passed
@brandonroberts
Copy link
Member

Thanks! @allcontributors add @jorgeandrespadilla for code

Copy link
Contributor

@brandonroberts

We had trouble processing your request. Please try again later.

@jorgeandrespadilla
Copy link
Contributor Author

Thanks @brandonroberts! It seems the contributors bot ran into trouble yesterday. Could you give it another try when you get a chance?

@brandonroberts
Copy link
Member

Thanks! @allcontributors add @jorgeandrespadilla for code

Copy link
Contributor

@brandonroberts

@jorgeandrespadilla already contributed before to code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Vitest CLI --coverage and --ui flags
2 participants