Skip to content

style: remove redundant async in tests#12380

Merged
hansl merged 2 commits intoangular:masterfrom
alan-agius4:spec_files_schematics
Sep 27, 2018
Merged

style: remove redundant async in tests#12380
hansl merged 2 commits intoangular:masterfrom
alan-agius4:spec_files_schematics

Conversation

@alan-agius4
Copy link
Copy Markdown
Collaborator

@alan-agius4 alan-agius4 commented Sep 27, 2018

style: improve formatting of spec file

style: remove redundant async in tests
In component specs only the compileComponents is async thus the async on it are redundant

Closes #12161

In component specs only the `compileComponents` is async thus the `async` on `it` are redudnant

Closes #12161
@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Sep 27, 2018
@hansl hansl merged commit a97ccba into angular:master Sep 27, 2018
@hansl
Copy link
Copy Markdown
Contributor

hansl commented Sep 27, 2018

This is a breaking change so I'll not merge it in the patch branch.

@alan-agius4 alan-agius4 deleted the spec_files_schematics branch September 27, 2018 19:48
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Sep 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

__name@dasherize__.component.spec.ts don't use async in spec case

3 participants