can you please help me to test the code. I have not been able to test the catchError operator that is inside the pipe of the observable interval (1000) ...
I don't know much unit testing in angular for observables of an inteval (1000). I need to simulate the execution of an interval, and simulate the error. I did not do the code, the code was already done, I am a new employee and I was assigned the task of doing unit testing. I don't know if it is necessary to catchError on an interval.
My file ts is:
import { Injectable, EventEmitter } from '@angular/core';
import {Subscription, Observable, interval, BehaviorSubject, throwError} from 'rxjs';
import {take, map, catchError, finalize} from 'rxjs/operators';
import { environment } from 'environments/environment';
/**
* @Class CounterSessionProvider
* @description This class is a provider for control session time
* @author Andres Giraldo Londoño, Pragma S.A.
*/
@Injectable({
providedIn: 'root',
})
export class CounterSessionProvider {
// 4 minutes
public inactivityTime = 20; // environment.inactivityTime; // environment.inactivityTime -> 20 seconds
counterStateEmitter = new EventEmitter();
currentSubscription: Subscription;
public substracCounter$: BehaviorSubject<boolean>;
constructor() {
this.substracCounter$ = new BehaviorSubject(false);
}
public start() {
this.currentSubscription = this.initInterval().subscribe();
}
public initInterval(): Observable<number> {
return interval(1000).pipe(
take(this.inactivityTime),
map((index: number) => this.inactivityTime - (index + 1)),
catchError(err => {
this.counterStateEmitter.error(err);
return throwError(err);
})
);
}
public restarCounterTime(): void {
this.substracCounter$.next(true);
}
public stop() {
if (this.currentSubscription) {
this.currentSubscription.unsubscribe();
}
this.counterStateEmitter.emit('ABORTED');
}
}
My file spec.ts is:
import {fakeAsync, TestBed, tick} from '@angular/core/testing';
import { CounterSessionProvider } from './counter-session.provider';
import {interval, Observable, Subscription, throwError} from 'rxjs';
import {catchError, take} from 'rxjs/operators';
import { TestScheduler } from 'rxjs/testing';
describe('CounterSessionProvider', () => {
let counterSessionProvider: CounterSessionProvider;
let scheduler: TestScheduler;
beforeEach(() => {
counterSessionProvider = new CounterSessionProvider();
});
/*beforeEach(() => {
scheduler = new TestScheduler((actual, expected) => {
expect(actual).toEqual(expected);
});
});*/
/*it('should throw error when initInterval interval', fakeAsync(() => {
scheduler.run(helpers => {
spyOn(counterSessionProvider, 'initInterval').and.returnValue(throwError('ERROR'));
const actions$ = helpers.hot('--a', { a: 1000 });
const completion$ = helpers.cold('--c', { c: 1000 });
expect(counterSessionProvider.initInterval).toBe(completion$);
});
}));*/
it('should initialize currentSubscription from interval', fakeAsync(() => {
let currentVal = null;
const numberObservable = counterSessionProvider.initInterval();
counterSessionProvider.currentSubscription = numberObservable.subscribe((value: number) => {
currentVal = value;
});
tick(1000);
expect(currentVal).toEqual(19);
counterSessionProvider.currentSubscription.unsubscribe();
}));
it('should emit a String COMPLETE when start() is called', () => {
spyOn(counterSessionProvider, 'start');
counterSessionProvider.start();
counterSessionProvider.counterStateEmitter.subscribe(value => {
expect(value).toEqual('COMPLETE');
});
expect(counterSessionProvider.start).toHaveBeenCalled();
});
it('should call stop()', () => {
spyOn(counterSessionProvider, 'stop');
counterSessionProvider.currentSubscription = new Subscription();
counterSessionProvider.stop();
counterSessionProvider.counterStateEmitter.subscribe((value: string) => {
expect(value).toEqual('ABORTED');
});
expect(counterSessionProvider.stop).toHaveBeenCalled();
});
it('should call method start and initilize currentSubscription', fakeAsync(() => {
counterSessionProvider.start();
tick(1000);
counterSessionProvider.currentSubscription.unsubscribe();
expect(counterSessionProvider.currentSubscription).not.toBeNull();
}));
it('should call restarCounterTime() and emit a TRUE', () => {
counterSessionProvider.restarCounterTime();
counterSessionProvider.substracCounter$.subscribe(value => expect(value).toBeTruthy());
});
it('should call method stop and emit ABORTED', fakeAsync(() => {
counterSessionProvider.start();
tick(1000);
counterSessionProvider.stop();
}));
/*beforeEach(() => {
counterSessionProvider = TestBed.get(CounterSessionProvider);
});
it('should be created', () => {
expect(counterSessionProvider).toBeTruthy();
});
it('should return ABORTED when stop function is called', () => {
counterSessionProvider.start();
counterSessionProvider.counterState.subscribe((msg: string) => {
expect(msg).toEqual('ABORTED');
});
counterSessionProvider.stop();
});*/
});
But I need to cover all the code of your coverage. In the following image I show the coverage that I need.
Trying to apply the suggestions that Shashank Vivek gave me, it gives these results:
This is what I'll suggest, throw
something for catchError
, something like this demo otherwise that catchError
will not be called. I am not sure whether the catchError
should be removed, you should verify it with the team (because it seems to have some code inside it)
component.ts
public initInterval(): Observable<number> {
return interval(1000).pipe(
take(this.inactivityTime),
map((index: number) => {
// write some condition to "throw" otherwise "catchError" is of now use
if (this.inactivityTime > 30) {
throw new Error('inactivityTime greater than 30');
}
return this.inactivityTime - (index + 1);
}),
catchError((err) => {
this.counterStateEmitter.error(err);
return throwError(err);
}),
finalize(() => {
this.currentSubscription.unsubscribe();
this.counterStateEmitter.emit('COMPLETE');
})
);
}
and then spec.ts you set condition to throw error :
it('should handle error in initInterval', fakeAsync(() => {
let currentVal;
let someError;
spyOn(
counterSessionProvider.counterStateEmitter,
'error'
).and.callThrough();
counterSessionProvider.inactivityTime = 35; // just to simulate the error event as per the code I have added
const numberObservable = counterSessionProvider.initInterval();
counterSessionProvider.currentSubscription = numberObservable.subscribe(
(value: number) => {
currentVal = value;
},
(err) => {
someError = err;
}
);
tick(1000);
expect(currentVal).toBeUndefined();
expect(someError).toBeDefined(); // you can also use .toBe('inactivityTime greater than 30')
expect(counterSessionProvider.counterStateEmitter.error).toHaveBeenCalled();
counterSessionProvider.currentSubscription.unsubscribe();
}));
Give it a try and let me know if this works.
The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.