Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented May 13, 2019

All schematics can potentially be async. The synchronous test method variants (runSchematic/runExternalSchematic) will fail if the schematic, any of its rules, or any schematic it calls are async.

@clydin clydin added the target: patch This PR is targeted for the next patch release label May 13, 2019
@clydin clydin requested a review from alan-agius4 May 13, 2019 17:34
All schematics can potentially be async.  The synchronous test method variants (`runSchematic`/`runExternalSchematic`) will fail if the schematic, any of its rules, or any schematic it calls are async.
@clydin clydin force-pushed the deprecate-schematics-testing branch from 1a80072 to eb488f2 Compare May 14, 2019 15:43
@kyliau kyliau merged commit 2172db1 into angular:master May 15, 2019
@clydin clydin deleted the deprecate-schematics-testing branch May 15, 2019 21:20
crisbeto added a commit to crisbeto/angular that referenced this pull request May 30, 2019
Switches all of the tests away from the `runSchematic` method which was deprecated in 8.0 (see angular/angular-cli#14412). Along the same lines as angular/components#16150.
mhevery pushed a commit to angular/angular that referenced this pull request May 31, 2019
Switches all of the tests away from the `runSchematic` method which was deprecated in 8.0 (see angular/angular-cli#14412). Along the same lines as angular/components#16150.

PR Close #30747
@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 Sep 13, 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.

4 participants