Add update after simulateError

[jgzuke]

Fixes #1809

Note that

it('rerenders on an error during render', () => {
  const wrapper = mount(<ErrorBoundary spy={sinon.stub()} />);

  expect(wrapper.find({ children: 'HasThrown' })).to.have.lengthOf(0);
  expect(wrapper.find({ children: 'HasNotThrown' })).to.have.lengthOf(1);

  return new Promise((resolve) => {
    wrapper.setState({ throws: true }, () => {
      expect(wrapper.find({ children: 'HasThrown' })).to.have.lengthOf(1);
      expect(wrapper.find({ children: 'HasNotThrown' })).to.have.lengthOf(0);
      resolve();
    });
  });
});

Will still fail, I will fix that separately.

@ljharb

[jgzuke]

@ljharb I reverted the change and just left the tests, resulting in https://travis-ci.org/airbnb/enzyme/builds/427510199 (which is the expected failures). Im not really sure what the problem here might be

[jgzuke]

Are there any additional changes to be made here?

[ljharb]

@jgzuke this build: https://travis-ci.org/airbnb/enzyme/builds/429796257 still didn't fail (https://travis-ci.org/airbnb/enzyme/builds/429796341 is the one with the fix) :-/

basically, i want to see a failing build, that the fix fixes, to have confidence to merge this.

[jgzuke]

Im a bit confused why https://travis-ci.org/airbnb/enzyme/builds/427510199 fails but https://travis-ci.org/airbnb/enzyme/builds/429796257 passes given they contain the same changes, are you sure Jenkins is actually running on the displayed commit and not on the end of that branch

[ljharb]

hmm, i'm pretty sure. i'll try pushing just the first commit, and i won't push the second til the test is done.

[ljharb]

guess that was it! i'll merge this shortly.