Platform: Unit Tests Breaking : Uncaught TypeError: Cannot read property 'entities' of undefined

Created on 8 Feb 2019 · 11Comments · Source: ngrx/platform

I am getting this error randomly on different unit tests. It breaks on a different unit test each time tests are ran. However, sometimes all tests pass successfully.

Angular Version: 7.2.4
@ngrx Version: 7.0.0

Needs Reproduction Store

Source

cport1

Most helpful comment

I think what ultimately fixes the issue is destroying the feature after create component

  beforeEach(() => {
    fixture = TestBed.createComponent(ComponentNameComponent);
    component = fixture.componentInstance;
    fixture.detectChanges();
  });
  afterEach(() => {
    fixture.destroy();
  });

cport1 on 8 Feb 2019

👍13 😕4

All 11 comments

I think what ultimately fixes the issue is destroying the feature after create component

  beforeEach(() => {
    fixture = TestBed.createComponent(ComponentNameComponent);
    component = fixture.componentInstance;
    fixture.detectChanges();
  });
  afterEach(() => {
    fixture.destroy();
  });

cport1 on 8 Feb 2019

👍13 😕4

It's quite annoying to have to add the fixture.destroy() in each component I'm testing. I've never had to do this for anything else as a fresh instance is always created in each beforeEach. Why is this necessary when using provideMockStore()?

Can this issue please be re-opened @cport1.

BBlackwo on 16 Aug 2019

👍3

I reopened the issue.

cport1 on 26 Aug 2019

👍1

Really annoying it's breaking our verification system, even with fixture.destroy() it randomly fail. We have git pre hooks that run test on push ...

flaviuse on 28 Aug 2019

Really annoying it's breaking our verification system, even with fixture.destroy() it randomly fail. We have git pre hooks that run test on push ...

Does it fail on a different component each run?

cport1 on 29 Aug 2019

Does anyone have a small reproduction of this behavior?

timdeschryver on 30 Aug 2019

Closing this in favor of #2044, because I think it's a duplicate.
If it's not the case feel free to create a reproduction and create a new issue.

timdeschryver on 3 Sep 2019

happens for me as well, I tried to add afterEach(() => { fixture.destroy(); }); in one test and it helped for more tests to pass, but then another random test had failed...

noylev on 30 Jan 2020

i got the same issue

rameezcm on 10 Jun 2020

facing the same issue,

An error was thrown in afterAll error properties: Object({ longStack: 'TypeError: Cannot convert undefined or null to object

pras7430 on 9 Sep 2020

I had a similar error:
An error was thrown in afterAll error properties: Object({ longStack: 'TypeError: Cannot ...
and it turned out I just forgot to mock one selector, and the value which it was supposed to emit wasn't being checked in the given component tests, therefore sometimes everything would pass and sometimes throw an error.