CODEWITHSUNDEEP
angulartestingjasmine
Ravel Sbrissa Okada
Ravel Sbrissa Okada

Reputation: 129

Testing a method that uses observable on the constructor

I need to cover this method with Jasmine in my code

private subject = new Subject<any>();

getMessage(): Observable<any> {
    return this.subject.asObservable();
  }

Here is my constructor

fiscalDocument: FiscalDocumentData;
subscription: Subscription;

constructor(private _myFiscalDocumentsService: MyFiscalDocumentsService) {  
    this.subscription = this._myFiscalDocumentsService.getMessage().subscribe(message => {
      if (message) {
        this.fiscalDocument = message;

      } else {
        this.fiscalDocument = null;
      } 
    });

  }

I've already tried to do that in my test, I have the TestBed.confiugreTestingModule, but I won't post here because I believe that's my problem isn't there

let dummyOb: Observable<any>;

  beforeEach(() => {
    service = TestBed.get(MyFiscalDocumentsService);
    fixture = TestBed.createComponent(MyFiscalDocumentsDetailsComponent);
    component = fixture.componentInstance;
    fixture.detectChanges();
});

it('should getMessage', async () => {
    let generalSpy = spyOn(service, 'getMessage' ).and.returnValue(await Promise.resolve(dummyOb));
    expect(generalSpy).toHaveBeenCalled();
 });

The error message: Expected spy getMessage to have been called. EDIT: I got the if condition, now I'm trying the else

Upvotes: 2

Views: 1706

Answers (3)

Ravel Sbrissa Okada
Ravel Sbrissa Okada

Reputation: 129

I resolved in this way, an if-else condition wasn't necessary

Component

ngOnInit() {
  this.subscription = this._myFiscalDocumentsService.getMessage().subscribe(message => {
    this.fiscalDocument = message;
  });
}

Spec

it('should getMessage', async () => {
    service.subject.next(dummyMock)
    const spy = spyOn(service, 'getMessage').and.returnValue(of(dummyMock));
    component.ngOnInit();
    expect(component.subscription).toBeDefined();
    expect(spy).toHaveBeenCalled();
});

Upvotes: 0

Stanislav Dontsov
Stanislav Dontsov

Reputation: 1741

By the time you create a spy your constructor has already been called in beforeEach. Try this:

let dummyOb: Observable<any>;

it('should getMessage', async () => {
    let generalSpy = spyOn(service, 'getMessage' ).and.returnValue(await Promise.resolve(dummyOb));
    new MyFiscalDocumentsDetailsComponent(service);
    expect(generalSpy).toHaveBeenCalled();
 });

Upvotes: 1

Dmitriy Ivanko
Dmitriy Ivanko

Reputation: 1075

It is bad practice use some complex logic in constructor, use ngOnInit instead: Difference between Constructor and ngOnInit

Upvotes: 1

Related Questions