cargdev/angular
1
0
Fork 0
Code Issues 15 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(core): Injector correctly honors the @Self flag (#24520)

Browse Source 
Injector was incorrectly returning instance from parent injector even
when `@Self` was specified.

PR Close #24520
This commit is contained in:
Miško Hevery
2018-06-14 10:36:20 -07:00
parent 3147a92ee9
commit f5b366147b
2 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
packages/core/test/view/ng_module_spec.ts
View File

@ -80,11 +80,14 @@ class FromChildWithOptionalDep {
}
class FromChildWithSkipSelfDep {
constructor(public depFromParent: ChildDep|null, public depFromChild: Bar|null) {}
constructor(
public skipSelfChildDep: ChildDep|null, public selfChildDep: ChildDep|null,
public optionalSelfBar: Bar|null) {}
static ngInjectableDef: InjectableDef<FromChildWithSkipSelfDep> = defineInjectable({
factory: () => new FromChildWithSkipSelfDep(
inject(ChildDep, InjectFlags.SkipSelf|InjectFlags.Optional),
inject(Bar, InjectFlags.Self|InjectFlags.Optional)),
inject(ChildDep, InjectFlags.Self),
inject(Bar, InjectFlags.Self|InjectFlags.Optional), ),
providedIn: MyChildModule,
});
}
@ -162,8 +165,9 @@ describe('NgModuleRef_ injector', () => {
it('injects skip-self and self deps across injectors properly', () => {
const instance = childRef.injector.get(FromChildWithSkipSelfDep);
expect(instance instanceof FromChildWithSkipSelfDep).toBeTruthy();
expect(instance.depFromParent).toBeNull();
expect(instance.depFromChild instanceof Bar).toBeTruthy();
expect(instance.skipSelfChildDep).toBeNull();
expect(instance.selfChildDep instanceof ChildDep).toBeTruthy();
expect(instance.optionalSelfBar).toBeNull();
});
it('does not inject something not scoped to the module',