Skip to content

fix(core/testing): move ComponentFixture to core - #9386

Merged
juliemr merged 1 commit into
angular:masterfrom
juliemr:component-fixture-move
Jun 24, 2016
Merged

juliemr merged 1 commit into
angular:masterfrom
juliemr:component-fixture-move

Conversation

@juliemr

@juliemr juliemr commented Jun 21, 2016

Copy link
Copy Markdown
Contributor
BREAKING CHANGE:

`ComponentFixture` will be moving out of `@angular/compiler/testing` to `@angular/core/testing` in
the next release.

@juliemr juliemr added this to the 2.0.0-rc.4 milestone Jun 21, 2016
@juliemr juliemr added area: testing Issues related to Angular testing features, such as TestBed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 21, 2016
@juliemr
juliemr force-pushed the component-fixture-move branch from 06a1614 to b8c832b Compare June 21, 2016 05:41
@vikerman

vikerman commented Jun 24, 2016 •

Copy link
Copy Markdown
Contributor

LGTM

The commit message can probably says "... will be moving out ... with this release" - because that's what will go into the Changelog.

@vikerman

Copy link
Copy Markdown
Contributor

I am not applying the labels before Igor has also reviewed it

@juliemr
juliemr force-pushed the component-fixture-move branch from b8c832b to 42e28b1 Compare June 24, 2016 04:57
@juliemr

juliemr commented Jun 24, 2016

Copy link
Copy Markdown
Contributor Author

I have rebased and changed the commit message to be fix instead of chore. @IgorMinar please take a look.

@juliemr juliemr changed the title chore(packaging): move component fixture to core fix(core/testing): move ComponentFixture to core Jun 24, 2016
@juliemr
juliemr force-pushed the component-fixture-move branch from 42e28b1 to 0dfc706 Compare June 24, 2016 05:06
@juliemr
juliemr force-pushed the component-fixture-move branch from 0dfc706 to 330f736 Compare June 24, 2016 05:44

@IgorMinar IgorMinar Jun 24, 2016 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use -> Import

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done.

BREAKING CHANGE:

`ComponentFixture` will be moving out of `@angular/compiler/testing` to `@angular/core/testing` in
this release. For now, it is deprecated from `@angular/compiler/testing`.
@juliemr
juliemr force-pushed the component-fixture-move branch from 330f736 to 3f129bc Compare June 24, 2016 19:27
@IgorMinar IgorMinar added pr_state: LGTM action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 24, 2016
@IgorMinar IgorMinar assigned juliemr and unassigned vikerman and IgorMinar Jun 24, 2016
@juliemr
juliemr merged commit 1143b03 into angular:master Jun 24, 2016
@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 8, 2019
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 area: testing Issues related to Angular testing features, such as TestBed cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants