refactor(core): misc changes and integrate review feedback on #19996

closes #20224
This commit is contained in:
Victor Berchet 2017-11-06 11:20:45 -08:00
parent a460066972
commit 6e8e3bd248
5 changed files with 13 additions and 12 deletions

View File

@ -91,7 +91,7 @@ export class JitCompiler {
} else {
const summary = entry as CompileTypeSummary;
this._summaryResolver.addSummary(
{symbol: summary.type.reference, metadata: null, type: summary})
{symbol: summary.type.reference, metadata: null, type: summary});
}
}
}

View File

@ -229,10 +229,11 @@ export function main() {
const overwrittenValue = {};
TestBed.overrideProvider(SomeDep, {useFactory: () => overwrittenValue, deps: []});
const fixture = TestBed.configureTestingModule({providers: [SomeDep], imports: [SomeModule]})
const fixture =
TestBed.overrideProvider(SomeDep, {useFactory: () => overwrittenValue, deps: []})
.configureTestingModule({providers: [SomeDep], imports: [SomeModule]})
.createComponent(SomePublicComponent);
expect(fixture.componentInstance.dep).toBe(overwrittenValue);
});

View File

@ -163,8 +163,8 @@ export class TestBed implements Injector {
static overrideProvider(token: any, provider: {
useFactory: Function,
deps: any[],
}): void;
static overrideProvider(token: any, provider: {useValue: any;}): void;
}): typeof TestBed;
static overrideProvider(token: any, provider: {useValue: any;}): typeof TestBed;
static overrideProvider(token: any, provider: {
useFactory?: Function,
useValue?: any,
@ -503,7 +503,7 @@ export class TestBed implements Injector {
overrideTemplateUsingTestingModule(component: Type<any>, template: string) {
this._assertNotInstantiated('overrideTemplateUsingTestingModule', 'override template');
@Component({selector: 'empty', template: template})
@Component({selector: 'empty', template})
class OverrideComponent {
}

View File

@ -752,15 +752,15 @@ export function main() {
/Cannot override template when the test module has already been instantiated/);
});
it('should reset overrides when the testing modules is resetted', () => {
it('should reset overrides when the testing module is resetted', () => {
@Component({selector: 'comp', template: 'a'})
class MyComponent {
}
TestBed.overrideTemplateUsingTestingModule(MyComponent, 'b');
TestBed.resetTestingModule();
const fixture = TestBed.configureTestingModule({declarations: [MyComponent]})
const fixture = TestBed.resetTestingModule()
.configureTestingModule({declarations: [MyComponent]})
.createComponent(MyComponent);
expect(fixture.nativeElement).toHaveText('a');
});

View File

@ -118,10 +118,10 @@ export declare class TestBed implements Injector {
static overrideProvider(token: any, provider: {
useFactory: Function;
deps: any[];
}): void;
}): typeof TestBed;
static overrideProvider(token: any, provider: {
useValue: any;
}): void;
}): typeof TestBed;
static overrideTemplate(component: Type<any>, template: string): typeof TestBed;
static overrideTemplateUsingTestingModule(component: Type<any>, template: string): typeof TestBed;
/** @experimental */ static resetTestEnvironment(): void;