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

Platform: Update usage of TestBed.get to TestBed.inject #2240

Closed
3 tasks
brandonroberts opened this issue Nov 11, 2019 · 5 comments · Fixed by #2311 or TypescriptID/platform#290
Closed
3 tasks

Platform: Update usage of TestBed.get to TestBed.inject #2240

brandonroberts opened this issue Nov 11, 2019 · 5 comments · Fixed by #2311 or TypescriptID/platform#290
Assignees
Labels
Accepting PRs community watch Someone from the community is working this issue/PR

Comments

@brandonroberts
Copy link
Member

TestBed.inject replaces TestBed.get<Type>. We need to clean up:

  • Library code
  • Docs
  • Schematics

Blocked by #2234

@sumitparakh
Copy link
Contributor

Hey, I wanna do this.

@brandonroberts brandonroberts added the community watch Someone from the community is working this issue/PR label Nov 11, 2019
@brandonroberts
Copy link
Member Author

@sumitparakh It's yours. Let us know if you need any assistance

@sumitparakh
Copy link
Contributor

Hi @brandonroberts , This update requires angular 9 and i see that currently ngrx is not upgraded to Angular 9..

Is there any other issue regarding Angular upgrade or should i upgrade to Angular 9 in this PR itself?

@santoshyadavdev
Copy link
Contributor

Angular 9 update is in progress @timdeschryver is already on it.

@sumitparakh
Copy link
Contributor

Alright. So I'll wait for that update and then continue with this PR

@sumitparakh sumitparakh mentioned this issue Dec 6, 2019
3 tasks
sumitparakh added a commit to sumitparakh/platform that referenced this issue Jan 8, 2020
angular 9 compatibillity changes for TestBed

Closes ngrx#2240
sumitparakh added a commit to sumitparakh/platform that referenced this issue Jan 8, 2020
TestBed.get replaced by TestBed.inject

Closes #ngrx#2240
sumitparakh added a commit to sumitparakh/platform that referenced this issue Jan 10, 2020
# This is the 1st commit message:

test(component): update usage of TestBed.get to TestBed.inject

angular 9 compatibillity changes for TestBed

Closes ngrx#2240

# This is the commit message ngrx#2:

test(docs): test cases updated in docs

TestBed.get replaced by TestBed.inject

Closes #ngrx#2240

# This is the commit message ngrx#3:

test(component): example tests type fixed

TestBed.get replace by TestBed.inject

done

# This is the commit message ngrx#4:

test(component): test issue fixed

angular 9 test upgrade changes

done

# This is the commit message ngrx#5:

test(component): changes under docs folder is reverted

angular 9 test upgrade

changes

# This is the commit message ngrx#6:

test(component): src tests changes reverted

angular 9 test upgrade

changes
sumitparakh added a commit to sumitparakh/platform that referenced this issue Jan 10, 2020
angular 9 compatibillity changes for TestBed

Closes ngrx#2240

test(docs): test cases updated in docs

TestBed.get replaced by TestBed.inject

Closes #ngrx#2240

test(component): example tests type fixed

TestBed.get replace by TestBed.inject

done

test(component): test issue fixed

angular 9 test upgrade changes

done

test(component): changes under docs folder is reverted

angular 9 test upgrade

changes

test(component): src tests changes reverted

angular 9 test upgrade

changes

test(component): reverted unnecessary testing changes

content folder changes are kept

changes

test(component): update usage of TestBed.get to TestBed.inject

angular 9 compatibillity changes for TestBed

Closes ngrx#2240

test(docs): test cases updated in docs

TestBed.get replaced by TestBed.inject

Closes #ngrx#2240

test(component): example tests type fixed

TestBed.get replace by TestBed.inject

done

test(component): test issue fixed

angular 9 test upgrade changes

done

test(component): changes under docs folder is reverted

angular 9 test upgrade

changes

test(component): src tests changes reverted

angular 9 test upgrade

changes

test(component): reverted unnecessary testing changes

content folder changes are kept

changes
sumitparakh added a commit to sumitparakh/platform that referenced this issue Jan 10, 2020
angular 9 compatibillity changes for TestBed

Closes ngrx#2240
sumitparakh added a commit to sumitparakh/platform that referenced this issue Jan 11, 2020
test cases updated in content folder only

Closes ngrx#2240
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepting PRs community watch Someone from the community is working this issue/PR
Projects
None yet
3 participants