chore(util-registry): fix tests

This commit is contained in:
Sarah Vaupel 2019-12-04 14:59:44 +01:00 committed by Gregor Kleen
parent 86dd3a96d2
commit ea5351e483
2 changed files with 9 additions and 11 deletions

View File

@ -55,7 +55,7 @@ export class UtilRegistry {
setupInstances
.filter((instance) => instance && typeof instance.start === 'function')
.forEach((instance) => this.start(instance));
.forEach((instance) => instance.start());
if (DEBUG_MODE > 1) {
console.info('initialized js util instances:');
@ -100,10 +100,6 @@ export class UtilRegistry {
return instances.map((instance) => ({ scope: scope, util: util, ...instance }));
}
start(instance) {
instance.start();
}
find(name) {
return this._registeredUtils.find((util) => util.name === name);
}

View File

@ -153,18 +153,20 @@ describe('UtilRegistry', () => {
});
it('should start instances that provide a start function', () => {
spyOn(utilRegistry, 'start');
utilRegistry.register(TestUtil3);
utilRegistry.initAll(testScope);
expect(utilRegistry.start.calls.count()).toBe(2);
const initializedInstances = utilRegistry.initAll(testScope);
expect(initializedInstances.length).toBe(2);
expect(initializedInstances.map((instance) => instance.util)).toEqual([TestUtil3,TestUtil3]);
});
it('should not start instances that do not provide a start function', () => {
spyOn(utilRegistry, 'start');
utilRegistry.register(TestUtil1);
utilRegistry.register(TestUtil2);
utilRegistry.initAll(testScope);
expect(utilRegistry.start).not.toHaveBeenCalled();
const initializedInstances = utilRegistry.initAll(testScope);
expect(initializedInstances.length).toBe(2);
expect(initializedInstances.map((instance) => instance.util)).toEqual([TestUtil1,TestUtil2]);
});
});
});