Angular 6 - Unit testing a subscribe function in constructor
I've been trying to unit test the subscribe function of this service. And looking at the code coverage report generated by istanbul, I can see that this code is not covered.
Code
layout.component.ts
import Component, HostListener, Input from '@angular/core';
import LayoutService from './layout.service';
import some from 'lodash';
@Component(
selector: 'cgm-layout',
templateUrl: './layout.component.html',
styleUrls: ['./layout.component.scss'],
providers: [LayoutService]
)
class LayoutComponent
message: any;
constructor(
private service: LayoutService
)
service.messagePublished$.subscribe(
message =>
this.setMessage(message);
);
setMessage(message): void
this.message = message;
setTimeout(() =>
this.message = null;
, 7000);
export
LayoutComponent
;
This is my Unit Test
layout.component.spec.ts
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let service;
beforeEach(async(() =>
service = new LayoutService();
mockLayoutService = jasmine.createSpyObj('LayoutService', ['messagePublished$']);
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
LayoutService
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
spyOn(service.messagePublished$, 'subscribe');
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
describe('setMessage', () =>
it('should set the Message', fakeAsync(() =>
component.setMessage('Message');
expect(component.message).toBe('Message');
tick(7000);
expect(component.message).toBeNull();
));
);
);
So the code never seems to go over the 'service.messagePublished$.subscribe' part. Here is the code coverage report.
The error I'm getting is 'Expected spy subscribe to have been called', which I'm guessing is the error you get when that code block is not covered.
javascript angular unit-testing karma-jasmine
add a comment |
I've been trying to unit test the subscribe function of this service. And looking at the code coverage report generated by istanbul, I can see that this code is not covered.
Code
layout.component.ts
import Component, HostListener, Input from '@angular/core';
import LayoutService from './layout.service';
import some from 'lodash';
@Component(
selector: 'cgm-layout',
templateUrl: './layout.component.html',
styleUrls: ['./layout.component.scss'],
providers: [LayoutService]
)
class LayoutComponent
message: any;
constructor(
private service: LayoutService
)
service.messagePublished$.subscribe(
message =>
this.setMessage(message);
);
setMessage(message): void
this.message = message;
setTimeout(() =>
this.message = null;
, 7000);
export
LayoutComponent
;
This is my Unit Test
layout.component.spec.ts
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let service;
beforeEach(async(() =>
service = new LayoutService();
mockLayoutService = jasmine.createSpyObj('LayoutService', ['messagePublished$']);
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
LayoutService
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
spyOn(service.messagePublished$, 'subscribe');
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
describe('setMessage', () =>
it('should set the Message', fakeAsync(() =>
component.setMessage('Message');
expect(component.message).toBe('Message');
tick(7000);
expect(component.message).toBeNull();
));
);
);
So the code never seems to go over the 'service.messagePublished$.subscribe' part. Here is the code coverage report.
The error I'm getting is 'Expected spy subscribe to have been called', which I'm guessing is the error you get when that code block is not covered.
javascript angular unit-testing karma-jasmine
add a comment |
I've been trying to unit test the subscribe function of this service. And looking at the code coverage report generated by istanbul, I can see that this code is not covered.
Code
layout.component.ts
import Component, HostListener, Input from '@angular/core';
import LayoutService from './layout.service';
import some from 'lodash';
@Component(
selector: 'cgm-layout',
templateUrl: './layout.component.html',
styleUrls: ['./layout.component.scss'],
providers: [LayoutService]
)
class LayoutComponent
message: any;
constructor(
private service: LayoutService
)
service.messagePublished$.subscribe(
message =>
this.setMessage(message);
);
setMessage(message): void
this.message = message;
setTimeout(() =>
this.message = null;
, 7000);
export
LayoutComponent
;
This is my Unit Test
layout.component.spec.ts
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let service;
beforeEach(async(() =>
service = new LayoutService();
mockLayoutService = jasmine.createSpyObj('LayoutService', ['messagePublished$']);
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
LayoutService
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
spyOn(service.messagePublished$, 'subscribe');
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
describe('setMessage', () =>
it('should set the Message', fakeAsync(() =>
component.setMessage('Message');
expect(component.message).toBe('Message');
tick(7000);
expect(component.message).toBeNull();
));
);
);
So the code never seems to go over the 'service.messagePublished$.subscribe' part. Here is the code coverage report.
The error I'm getting is 'Expected spy subscribe to have been called', which I'm guessing is the error you get when that code block is not covered.
javascript angular unit-testing karma-jasmine
I've been trying to unit test the subscribe function of this service. And looking at the code coverage report generated by istanbul, I can see that this code is not covered.
Code
layout.component.ts
import Component, HostListener, Input from '@angular/core';
import LayoutService from './layout.service';
import some from 'lodash';
@Component(
selector: 'cgm-layout',
templateUrl: './layout.component.html',
styleUrls: ['./layout.component.scss'],
providers: [LayoutService]
)
class LayoutComponent
message: any;
constructor(
private service: LayoutService
)
service.messagePublished$.subscribe(
message =>
this.setMessage(message);
);
setMessage(message): void
this.message = message;
setTimeout(() =>
this.message = null;
, 7000);
export
LayoutComponent
;
This is my Unit Test
layout.component.spec.ts
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let service;
beforeEach(async(() =>
service = new LayoutService();
mockLayoutService = jasmine.createSpyObj('LayoutService', ['messagePublished$']);
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
LayoutService
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
spyOn(service.messagePublished$, 'subscribe');
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
describe('setMessage', () =>
it('should set the Message', fakeAsync(() =>
component.setMessage('Message');
expect(component.message).toBe('Message');
tick(7000);
expect(component.message).toBeNull();
));
);
);
So the code never seems to go over the 'service.messagePublished$.subscribe' part. Here is the code coverage report.
The error I'm getting is 'Expected spy subscribe to have been called', which I'm guessing is the error you get when that code block is not covered.
javascript angular unit-testing karma-jasmine
javascript angular unit-testing karma-jasmine
edited Nov 13 '18 at 9:36
Simba3696
asked Nov 12 '18 at 10:34
Simba3696Simba3696
267
267
add a comment |
add a comment |
1 Answer
1
active
oldest
votes
I'd advise you to move your subscription from the constructor to an ngOnInit
. Angular created several lifecycle hooks which get called when a component get's created (ngOnInit) and other when data changes or when it gets destroyed - see Angular lifecycle hooks.
This way you can test your code by calling the ngOnInit()
method.
In case you cannot change the code you can try creating a component instance and check if your method was called like in pseudo-code below:
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let serviceSpy: jasmine.SpyObj<LayoutService>;;
beforeEach(async(() =>
const spy = spyOn(service.messagePublished$, 'subscribe')
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
provide: LayoutService, useValue: spy
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
serviceSpy = TestBed.get(ValueService);
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
);
That's a good idea but I'm only in charge of unit testing right now. So I can't really change the code. I have to figure out a way to access the code when it's in the constructor.
– Simba3696
Nov 12 '18 at 12:14
Updated the code for testing constructor tests. Hopefully it should give you an idea to how to check if the code has been run.
– Mac_W
Nov 12 '18 at 12:21
It is because you replace your LayoutService with a mock in this line:provide: LayoutService, useValue: mockLayoutService
and mockLayoutService doesn't have a property of messagePublished$. mockLayoutService needs to be an object with a property messagePublished$ which has a subscribe property.
– Mac_W
Nov 12 '18 at 14:44
Do you want to update your code? I'll try and help
– Mac_W
Nov 13 '18 at 8:34
Code updated. I hope it gives you a clearer picture.
– Simba3696
Nov 13 '18 at 9:08
|
show 6 more comments
Your Answer
StackExchange.ifUsing("editor", function ()
StackExchange.using("externalEditor", function ()
StackExchange.using("snippets", function ()
StackExchange.snippets.init();
);
);
, "code-snippets");
StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "1"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);
else
createEditor();
);
function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);
);
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53260326%2fangular-6-unit-testing-a-subscribe-function-in-constructor%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
1 Answer
1
active
oldest
votes
1 Answer
1
active
oldest
votes
active
oldest
votes
active
oldest
votes
I'd advise you to move your subscription from the constructor to an ngOnInit
. Angular created several lifecycle hooks which get called when a component get's created (ngOnInit) and other when data changes or when it gets destroyed - see Angular lifecycle hooks.
This way you can test your code by calling the ngOnInit()
method.
In case you cannot change the code you can try creating a component instance and check if your method was called like in pseudo-code below:
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let serviceSpy: jasmine.SpyObj<LayoutService>;;
beforeEach(async(() =>
const spy = spyOn(service.messagePublished$, 'subscribe')
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
provide: LayoutService, useValue: spy
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
serviceSpy = TestBed.get(ValueService);
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
);
That's a good idea but I'm only in charge of unit testing right now. So I can't really change the code. I have to figure out a way to access the code when it's in the constructor.
– Simba3696
Nov 12 '18 at 12:14
Updated the code for testing constructor tests. Hopefully it should give you an idea to how to check if the code has been run.
– Mac_W
Nov 12 '18 at 12:21
It is because you replace your LayoutService with a mock in this line:provide: LayoutService, useValue: mockLayoutService
and mockLayoutService doesn't have a property of messagePublished$. mockLayoutService needs to be an object with a property messagePublished$ which has a subscribe property.
– Mac_W
Nov 12 '18 at 14:44
Do you want to update your code? I'll try and help
– Mac_W
Nov 13 '18 at 8:34
Code updated. I hope it gives you a clearer picture.
– Simba3696
Nov 13 '18 at 9:08
|
show 6 more comments
I'd advise you to move your subscription from the constructor to an ngOnInit
. Angular created several lifecycle hooks which get called when a component get's created (ngOnInit) and other when data changes or when it gets destroyed - see Angular lifecycle hooks.
This way you can test your code by calling the ngOnInit()
method.
In case you cannot change the code you can try creating a component instance and check if your method was called like in pseudo-code below:
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let serviceSpy: jasmine.SpyObj<LayoutService>;;
beforeEach(async(() =>
const spy = spyOn(service.messagePublished$, 'subscribe')
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
provide: LayoutService, useValue: spy
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
serviceSpy = TestBed.get(ValueService);
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
);
That's a good idea but I'm only in charge of unit testing right now. So I can't really change the code. I have to figure out a way to access the code when it's in the constructor.
– Simba3696
Nov 12 '18 at 12:14
Updated the code for testing constructor tests. Hopefully it should give you an idea to how to check if the code has been run.
– Mac_W
Nov 12 '18 at 12:21
It is because you replace your LayoutService with a mock in this line:provide: LayoutService, useValue: mockLayoutService
and mockLayoutService doesn't have a property of messagePublished$. mockLayoutService needs to be an object with a property messagePublished$ which has a subscribe property.
– Mac_W
Nov 12 '18 at 14:44
Do you want to update your code? I'll try and help
– Mac_W
Nov 13 '18 at 8:34
Code updated. I hope it gives you a clearer picture.
– Simba3696
Nov 13 '18 at 9:08
|
show 6 more comments
I'd advise you to move your subscription from the constructor to an ngOnInit
. Angular created several lifecycle hooks which get called when a component get's created (ngOnInit) and other when data changes or when it gets destroyed - see Angular lifecycle hooks.
This way you can test your code by calling the ngOnInit()
method.
In case you cannot change the code you can try creating a component instance and check if your method was called like in pseudo-code below:
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let serviceSpy: jasmine.SpyObj<LayoutService>;;
beforeEach(async(() =>
const spy = spyOn(service.messagePublished$, 'subscribe')
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
provide: LayoutService, useValue: spy
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
serviceSpy = TestBed.get(ValueService);
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
);
I'd advise you to move your subscription from the constructor to an ngOnInit
. Angular created several lifecycle hooks which get called when a component get's created (ngOnInit) and other when data changes or when it gets destroyed - see Angular lifecycle hooks.
This way you can test your code by calling the ngOnInit()
method.
In case you cannot change the code you can try creating a component instance and check if your method was called like in pseudo-code below:
import CUSTOM_ELEMENTS_SCHEMA, NO_ERRORS_SCHEMA from '@angular/core';
import async, ComponentFixture, fakeAsync, TestBed, tick from '@angular/core/testing';
import of from 'rxjs';
import LayoutComponent from './layout.component';
import LayoutService from './layout.service';
describe('LayoutComponent', () =>
let component: LayoutComponent;
let fixture: ComponentFixture<LayoutComponent>;
let serviceSpy: jasmine.SpyObj<LayoutService>;;
beforeEach(async(() =>
const spy = spyOn(service.messagePublished$, 'subscribe')
TestBed.configureTestingModule(
declarations: [
LayoutComponent,
],
providers: [
provide: LayoutService, useValue: spy
],
schemas: [
NO_ERRORS_SCHEMA, CUSTOM_ELEMENTS_SCHEMA
]
)
.compileComponents();
serviceSpy = TestBed.get(ValueService);
));
beforeEach(() =>
fixture = TestBed.createComponent(LayoutComponent);
component = fixture.componentInstance;
fixture.detectChanges();
component.message = 'Garbage';
);
it('should call messagePublished', () =>
TestBed.createComponent(LayoutComponent);
expect(service.messagePublished$.subscribe).toHaveBeenCalled();
);
);
edited Nov 13 '18 at 10:06
answered Nov 12 '18 at 11:44
Mac_WMac_W
825415
825415
That's a good idea but I'm only in charge of unit testing right now. So I can't really change the code. I have to figure out a way to access the code when it's in the constructor.
– Simba3696
Nov 12 '18 at 12:14
Updated the code for testing constructor tests. Hopefully it should give you an idea to how to check if the code has been run.
– Mac_W
Nov 12 '18 at 12:21
It is because you replace your LayoutService with a mock in this line:provide: LayoutService, useValue: mockLayoutService
and mockLayoutService doesn't have a property of messagePublished$. mockLayoutService needs to be an object with a property messagePublished$ which has a subscribe property.
– Mac_W
Nov 12 '18 at 14:44
Do you want to update your code? I'll try and help
– Mac_W
Nov 13 '18 at 8:34
Code updated. I hope it gives you a clearer picture.
– Simba3696
Nov 13 '18 at 9:08
|
show 6 more comments
That's a good idea but I'm only in charge of unit testing right now. So I can't really change the code. I have to figure out a way to access the code when it's in the constructor.
– Simba3696
Nov 12 '18 at 12:14
Updated the code for testing constructor tests. Hopefully it should give you an idea to how to check if the code has been run.
– Mac_W
Nov 12 '18 at 12:21
It is because you replace your LayoutService with a mock in this line:provide: LayoutService, useValue: mockLayoutService
and mockLayoutService doesn't have a property of messagePublished$. mockLayoutService needs to be an object with a property messagePublished$ which has a subscribe property.
– Mac_W
Nov 12 '18 at 14:44
Do you want to update your code? I'll try and help
– Mac_W
Nov 13 '18 at 8:34
Code updated. I hope it gives you a clearer picture.
– Simba3696
Nov 13 '18 at 9:08
That's a good idea but I'm only in charge of unit testing right now. So I can't really change the code. I have to figure out a way to access the code when it's in the constructor.
– Simba3696
Nov 12 '18 at 12:14
That's a good idea but I'm only in charge of unit testing right now. So I can't really change the code. I have to figure out a way to access the code when it's in the constructor.
– Simba3696
Nov 12 '18 at 12:14
Updated the code for testing constructor tests. Hopefully it should give you an idea to how to check if the code has been run.
– Mac_W
Nov 12 '18 at 12:21
Updated the code for testing constructor tests. Hopefully it should give you an idea to how to check if the code has been run.
– Mac_W
Nov 12 '18 at 12:21
It is because you replace your LayoutService with a mock in this line:
provide: LayoutService, useValue: mockLayoutService
and mockLayoutService doesn't have a property of messagePublished$. mockLayoutService needs to be an object with a property messagePublished$ which has a subscribe property.– Mac_W
Nov 12 '18 at 14:44
It is because you replace your LayoutService with a mock in this line:
provide: LayoutService, useValue: mockLayoutService
and mockLayoutService doesn't have a property of messagePublished$. mockLayoutService needs to be an object with a property messagePublished$ which has a subscribe property.– Mac_W
Nov 12 '18 at 14:44
Do you want to update your code? I'll try and help
– Mac_W
Nov 13 '18 at 8:34
Do you want to update your code? I'll try and help
– Mac_W
Nov 13 '18 at 8:34
Code updated. I hope it gives you a clearer picture.
– Simba3696
Nov 13 '18 at 9:08
Code updated. I hope it gives you a clearer picture.
– Simba3696
Nov 13 '18 at 9:08
|
show 6 more comments
Thanks for contributing an answer to Stack Overflow!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53260326%2fangular-6-unit-testing-a-subscribe-function-in-constructor%23new-answer', 'question_page');
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown