Skip to content

build: add option for running all component tests #18371

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

Merged

Conversation

devversion
Copy link
Member

Adds a --all option to the yarn test script. This option
can be used to run all component tests in the repository.

Flags like --no-watch and --firefox can be used in combination with
the new -all flag.

@devversion devversion added pr: merge safe target: patch This PR is targeted for the next patch release labels Feb 3, 2020
@devversion devversion requested a review from a team as a code owner February 3, 2020 09:19
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Feb 3, 2020
@devversion devversion added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Feb 3, 2020
@devversion devversion force-pushed the build/support-for-running-all-tests branch from 9a0d3f5 to d542d71 Compare February 3, 2020 19:58
@jelbourn
Copy link
Member

jelbourn commented Feb 3, 2020

Personally I like yarn test all but ¯\(ツ)

@devversion
Copy link
Member Author

Updated it to yarn test all since more people preferred that.

@mmalerba
Copy link
Contributor

mmalerba commented Feb 4, 2020

@devversion needs rebase

Adds a `--all` option to the `yarn test` script. This option
can be used to run _all_ component tests in the repository.

Flags like `--no-watch` and `--firefox` can be used in combination with
the new `-all` flag.
@devversion devversion force-pushed the build/support-for-running-all-tests branch from 86f42b0 to dfe15ad Compare February 5, 2020 12:57
@devversion
Copy link
Member Author

Rebased.

@mmalerba mmalerba merged commit 782f114 into angular:master Feb 7, 2020
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Mar 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants