Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using Jest mock timers and waitFor together causes tests to timeout #506

Closed
dcalhoun opened this issue Aug 14, 2020 · 16 comments · Fixed by #568
Closed

Using Jest mock timers and waitFor together causes tests to timeout #506

dcalhoun opened this issue Aug 14, 2020 · 16 comments · Fixed by #568

Comments

@dcalhoun
Copy link
Contributor

Describe the bug

Related to #391. Using jest.useFakeTimers() in combination with waitFor, causes the tests using waitFor to fail due to timeout error:

Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.Error: Timeout - Async callback was not invoked within the 5000 ms timeout specified by jest.setTimeout.

Relying upon jest.useFakeTimers("modern") instead causes the above failure for all tests if the file merely imports waitFor at all, regardless if the given test uses waitFor or not.

Expected behavior

All tests in the reproduction test case should pass.

Steps to Reproduce

  1. Clone the reproduction test case.
  2. yarn install
  3. yarn test

Versions

npmPackages:
    @testing-library/react-native: ^7.0.1 => 7.0.1 
    react: ^16.13.1 => 16.13.1 
    react-native: ^0.63.2 => 0.63.2 
    react-test-renderer: ^16.13.1 => 16.13.1
@mdjastrzebski
Copy link
Member

Not sure if I understood your issues correctly. AFAIK when using fake timers you should not use call waitFor with await.

Please compare how were are using fake timers with waitFor in our own test suit.

It seems that just this change (await waitFor(() => { -> waitFor(() => {) fixes your legacy-timers.test.js.

Not sure how to fix your failing tests using modern timers.

@dcalhoun
Copy link
Contributor Author

dcalhoun commented Sep 8, 2020

@mdjastrzebski thank you for the response. What you said about not awaiting the return of waitFor when using fake timers makes sense. I now understand the following statement from the waitFor documentation.

If you're using Jest's Timer Mocks, remember not to use async/await syntax as it will stall your tests.

The phrasing of that always confused me, but I now understand. I'll likely open a PR to improve that piece of documentation.

That said, it is still confusing as to why modern timers causes all of the tests to fail in my test case. I'd appreciate any guidance you are able to provide on that issue. Thanks!

@mdjastrzebski
Copy link
Member

@thymikee maybe you can with modern times here.

@thymikee
Copy link
Member

thymikee commented Sep 9, 2020

Running jest.runOnlyPendingTimers() or jest.runAllTimers() doesn't help? waitFor relies on setTimeout internally, so that may be a thing

@dcalhoun
Copy link
Contributor Author

dcalhoun commented Sep 9, 2020

@thymikee no, running jest.runOnlyPendingTimers() or jest.runAllTimers() does not appear to fix the issue. I could understand if waitFor and timer mocks were fundamentally incompatible, but I wanted to seek out if that is the case. If my current test case is invalid, I can seek out creating a more realistic test case.

That said, it is curious that "legacy" timers can work, but "modern" timers do not. It is particularly odd that enabling "modern" timers will break a test file if you merely import waitFor. One does not even need to invoke waitFor for tests in the given file to fail. Do you know why that would be the case?

Is it possible to use "modern" timers and waitFor together?

@thymikee
Copy link
Member

thymikee commented Sep 9, 2020

Not really sure where the incompatibility comes from. We already had fixed some issues around this topic here: #397, please take a look. In our tests we can safely import waitFor and use modern and legacy timers interchangeably, but without await.

@dcalhoun
Copy link
Contributor Author

dcalhoun commented Sep 9, 2020

@thymikee yes, I had reviewed #397 as well in hopes of finding an answer. To reduce the number of variables, I copied the provided tests from RNTL into my test case repository. The new branch (add-rntl-tests) still experiences the below failures. So, I'm thinking something must be a difference in the configuration or package versions?

Screen Shot 2020-09-09 at 13 55 53

@dcalhoun
Copy link
Contributor Author

dcalhoun commented Sep 9, 2020

@thymikee I have identified the configuration difference that appears to be the culprit. The RNTL repository babel.config.js does not include module:metro-react-native-babel-preset. My test case babel.config.js does include module:metro-react-native-babel-preset.

The inclusion of module:metro-react-native-babel-preset is a part of the default React Native template. Adding module:metro-react-native-babel-preset to the RNTL repository causes the tests to begin to fail as I have outlined in my original post.

At this point, I'm not sure if this is a RNTL issue, Jest issue, or a React Native issue. Do you know why module:metro-react-native-babel-preset is not a part of the RNTL repository? Any ideas as to why its inclusion would cause this issue with combining "modern" mock timers and waitFor?

Thanks!

@thymikee
Copy link
Member

We don't use Metro babel preset, because we're a Node.js library, not a JSC/Hermes app. You'd need to check on the compiled output to see what's the difference in waitFor. I have no immediate idea what might causing that. Maybe async/await is transpiled by Metro? Or they use custom promise implementation?

@dcalhoun
Copy link
Contributor Author

@thymikee makes sense. However, given that this library is intended to be used with a JSC/Hermes app, I would think testing in that environment would be ideal for this library.

I'll try to research further. I'm not sure how I'd go about comparing the compiled output Jest holds in-memory. If you have any guidance on that, it'd be appreciated. Thanks.

@thymikee
Copy link
Member

jest --showConfig

Then find "cacheDirectory" and you'll see the transformed output.

given that this library is intended to be used with a JSC/Hermes app, I would think testing in that environment would be ideal for this library

We may adjust our Babel config for testing to reflect that, PRs welcome :)

@dcalhoun
Copy link
Contributor Author

@thymikee I ran the waitFor tests within this repo with and without module:metro-react-native-babel-preset, but I'm not going to pretend to understand what the issue might be in the diff. 🙃 I am definitely not intimately familiar with Babel and how it works. Any assistance you are wiling to provide is appreciated.

It appears that when using module:metro-react-native-babel-preset regenerator is used to manage the async work. So, maybe the issue resides in its usage?

Without module:metro-react-native-babel-preset (Passes)

test('works with fake timers', async () => {
  jest.useFakeTimers('modern');
  const mockFn = jest.fn(() => {
    throw Error('test');
  });

  try {
    (0, _.waitFor)(() => mockFn(), {
      timeout: 400,
      interval: 200
    });
  } catch (e) {// suppress
  }

  jest.advanceTimersByTime(400);
  expect(mockFn).toHaveBeenCalledTimes(3);
});
Full Test File
ce53c85bde85f5c2506a2ba83e183d7f
"use strict";

var _react = _interopRequireDefault(require("react"));

var _reactNative = require("react-native");

var _ = require("..");

function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }

function _defineProperty(obj, key, value) { if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }

class Banana extends _react.default.Component {
  constructor(...args) {
    super(...args);

    _defineProperty(this, "changeFresh", () => {
      this.props.onChangeFresh();
    });
  }

  render() {
    return /*#__PURE__*/_react.default.createElement(_reactNative.View, null, this.props.fresh && /*#__PURE__*/_react.default.createElement(_reactNative.Text, null, "Fresh"), /*#__PURE__*/_react.default.createElement(_reactNative.TouchableOpacity, {
      onPress: this.changeFresh
    }, /*#__PURE__*/_react.default.createElement(_reactNative.Text, null, "Change freshness!")));
  }

}

class BananaContainer extends _react.default.Component {
  constructor(...args) {
    super(...args);

    _defineProperty(this, "state", {
      fresh: false
    });

    _defineProperty(this, "onChangeFresh", async () => {
      await new Promise(resolve => setTimeout(resolve, 300));
      this.setState({
        fresh: true
      });
    });
  }

  render() {
    return /*#__PURE__*/_react.default.createElement(Banana, {
      onChangeFresh: this.onChangeFresh,
      fresh: this.state.fresh
    });
  }

}

afterEach(() => {
  jest.useRealTimers();
});
test('waits for element until it stops throwing', async () => {
  const {
    getByText,
    queryByText
  } = (0, _.render)( /*#__PURE__*/_react.default.createElement(BananaContainer, null));

  _.fireEvent.press(getByText('Change freshness!'));

  expect(queryByText('Fresh')).toBeNull();
  const freshBananaText = await (0, _.waitFor)(() => getByText('Fresh'));
  expect(freshBananaText.props.children).toBe('Fresh');
});
test('waits for element until timeout is met', async () => {
  const {
    getByText
  } = (0, _.render)( /*#__PURE__*/_react.default.createElement(BananaContainer, null));

  _.fireEvent.press(getByText('Change freshness!'));

  await expect((0, _.waitFor)(() => getByText('Fresh'), {
    timeout: 100
  })).rejects.toThrow(); // Async action ends after 300ms and we only waited 100ms, so we need to wait
  // for the remaining async actions to finish

  await (0, _.waitFor)(() => getByText('Fresh'));
});
test('waits for element with custom interval', async () => {
  const mockFn = jest.fn(() => {
    throw Error('test');
  });

  try {
    await (0, _.waitFor)(() => mockFn(), {
      timeout: 400,
      interval: 200
    });
  } catch (e) {// suppress
  }

  expect(mockFn).toHaveBeenCalledTimes(3);
});
test('works with legacy fake timers', async () => {
  jest.useFakeTimers('legacy');
  const mockFn = jest.fn(() => {
    throw Error('test');
  });

  try {
    (0, _.waitFor)(() => mockFn(), {
      timeout: 400,
      interval: 200
    });
  } catch (e) {// suppress
  }

  jest.advanceTimersByTime(400);
  expect(mockFn).toHaveBeenCalledTimes(3);
});
test('works with fake timers', async () => {
  jest.useFakeTimers('modern');
  const mockFn = jest.fn(() => {
    throw Error('test');
  });

  try {
    (0, _.waitFor)(() => mockFn(), {
      timeout: 400,
      interval: 200
    });
  } catch (e) {// suppress
  }

  jest.advanceTimersByTime(400);
  expect(mockFn).toHaveBeenCalledTimes(3);
});
//# sourceMappingURL=data:application/json;charset=utf-8;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbIndhaXRGb3IudGVzdC5qcyJdLCJuYW1lcyI6WyJCYW5hbmEiLCJSZWFjdCIsIkNvbXBvbmVudCIsInByb3BzIiwib25DaGFuZ2VGcmVzaCIsInJlbmRlciIsImZyZXNoIiwiY2hhbmdlRnJlc2giLCJCYW5hbmFDb250YWluZXIiLCJQcm9taXNlIiwicmVzb2x2ZSIsInNldFRpbWVvdXQiLCJzZXRTdGF0ZSIsInN0YXRlIiwiYWZ0ZXJFYWNoIiwiamVzdCIsInVzZVJlYWxUaW1lcnMiLCJ0ZXN0IiwiZ2V0QnlUZXh0IiwicXVlcnlCeVRleHQiLCJmaXJlRXZlbnQiLCJwcmVzcyIsImV4cGVjdCIsInRvQmVOdWxsIiwiZnJlc2hCYW5hbmFUZXh0IiwiY2hpbGRyZW4iLCJ0b0JlIiwidGltZW91dCIsInJlamVjdHMiLCJ0b1Rocm93IiwibW9ja0ZuIiwiZm4iLCJFcnJvciIsImludGVydmFsIiwiZSIsInRvSGF2ZUJlZW5DYWxsZWRUaW1lcyIsInVzZUZha2VUaW1lcnMiLCJhZHZhbmNlVGltZXJzQnlUaW1lIl0sIm1hcHBpbmdzIjoiOztBQUNBOztBQUNBOztBQUNBOzs7Ozs7QUFFQSxNQUFNQSxNQUFOLFNBQXFCQyxlQUFNQyxTQUEzQixDQUEwQztBQUFBO0FBQUE7O0FBQUEseUNBQzFCLE1BQU07QUFDbEIsV0FBS0MsS0FBTCxDQUFXQyxhQUFYO0FBQ0QsS0FIdUM7QUFBQTs7QUFLeENDLEVBQUFBLE1BQU0sR0FBRztBQUNQLHdCQUNFLDZCQUFDLGlCQUFELFFBQ0csS0FBS0YsS0FBTCxDQUFXRyxLQUFYLGlCQUFvQiw2QkFBQyxpQkFBRCxnQkFEdkIsZUFFRSw2QkFBQyw2QkFBRDtBQUFrQixNQUFBLE9BQU8sRUFBRSxLQUFLQztBQUFoQyxvQkFDRSw2QkFBQyxpQkFBRCw0QkFERixDQUZGLENBREY7QUFRRDs7QUFkdUM7O0FBaUIxQyxNQUFNQyxlQUFOLFNBQThCUCxlQUFNQyxTQUFwQyxDQUF1RDtBQUFBO0FBQUE7O0FBQUEsbUNBQzdDO0FBQUVJLE1BQUFBLEtBQUssRUFBRTtBQUFULEtBRDZDOztBQUFBLDJDQUdyQyxZQUFZO0FBQzFCLFlBQU0sSUFBSUcsT0FBSixDQUFhQyxPQUFELElBQWFDLFVBQVUsQ0FBQ0QsT0FBRCxFQUFVLEdBQVYsQ0FBbkMsQ0FBTjtBQUNBLFdBQUtFLFFBQUwsQ0FBYztBQUFFTixRQUFBQSxLQUFLLEVBQUU7QUFBVCxPQUFkO0FBQ0QsS0FOb0Q7QUFBQTs7QUFRckRELEVBQUFBLE1BQU0sR0FBRztBQUNQLHdCQUNFLDZCQUFDLE1BQUQ7QUFBUSxNQUFBLGFBQWEsRUFBRSxLQUFLRCxhQUE1QjtBQUEyQyxNQUFBLEtBQUssRUFBRSxLQUFLUyxLQUFMLENBQVdQO0FBQTdELE1BREY7QUFHRDs7QUFab0Q7O0FBZXZEUSxTQUFTLENBQUMsTUFBTTtBQUNkQyxFQUFBQSxJQUFJLENBQUNDLGFBQUw7QUFDRCxDQUZRLENBQVQ7QUFJQUMsSUFBSSxDQUFDLDJDQUFELEVBQThDLFlBQVk7QUFDNUQsUUFBTTtBQUFFQyxJQUFBQSxTQUFGO0FBQWFDLElBQUFBO0FBQWIsTUFBNkIsNEJBQU8sNkJBQUMsZUFBRCxPQUFQLENBQW5DOztBQUVBQyxjQUFVQyxLQUFWLENBQWdCSCxTQUFTLENBQUMsbUJBQUQsQ0FBekI7O0FBRUFJLEVBQUFBLE1BQU0sQ0FBQ0gsV0FBVyxDQUFDLE9BQUQsQ0FBWixDQUFOLENBQTZCSSxRQUE3QjtBQUVBLFFBQU1DLGVBQWUsR0FBRyxNQUFNLGVBQVEsTUFBTU4sU0FBUyxDQUFDLE9BQUQsQ0FBdkIsQ0FBOUI7QUFFQUksRUFBQUEsTUFBTSxDQUFDRSxlQUFlLENBQUNyQixLQUFoQixDQUFzQnNCLFFBQXZCLENBQU4sQ0FBdUNDLElBQXZDLENBQTRDLE9BQTVDO0FBQ0QsQ0FWRyxDQUFKO0FBWUFULElBQUksQ0FBQyx3Q0FBRCxFQUEyQyxZQUFZO0FBQ3pELFFBQU07QUFBRUMsSUFBQUE7QUFBRixNQUFnQiw0QkFBTyw2QkFBQyxlQUFELE9BQVAsQ0FBdEI7O0FBRUFFLGNBQVVDLEtBQVYsQ0FBZ0JILFNBQVMsQ0FBQyxtQkFBRCxDQUF6Qjs7QUFFQSxRQUFNSSxNQUFNLENBQ1YsZUFBUSxNQUFNSixTQUFTLENBQUMsT0FBRCxDQUF2QixFQUFrQztBQUFFUyxJQUFBQSxPQUFPLEVBQUU7QUFBWCxHQUFsQyxDQURVLENBQU4sQ0FFSkMsT0FGSSxDQUVJQyxPQUZKLEVBQU4sQ0FMeUQsQ0FTekQ7QUFDQTs7QUFDQSxRQUFNLGVBQVEsTUFBTVgsU0FBUyxDQUFDLE9BQUQsQ0FBdkIsQ0FBTjtBQUNELENBWkcsQ0FBSjtBQWNBRCxJQUFJLENBQUMsd0NBQUQsRUFBMkMsWUFBWTtBQUN6RCxRQUFNYSxNQUFNLEdBQUdmLElBQUksQ0FBQ2dCLEVBQUwsQ0FBUSxNQUFNO0FBQzNCLFVBQU1DLEtBQUssQ0FBQyxNQUFELENBQVg7QUFDRCxHQUZjLENBQWY7O0FBSUEsTUFBSTtBQUNGLFVBQU0sZUFBUSxNQUFNRixNQUFNLEVBQXBCLEVBQXdCO0FBQUVILE1BQUFBLE9BQU8sRUFBRSxHQUFYO0FBQWdCTSxNQUFBQSxRQUFRLEVBQUU7QUFBMUIsS0FBeEIsQ0FBTjtBQUNELEdBRkQsQ0FFRSxPQUFPQyxDQUFQLEVBQVUsQ0FDVjtBQUNEOztBQUVEWixFQUFBQSxNQUFNLENBQUNRLE1BQUQsQ0FBTixDQUFlSyxxQkFBZixDQUFxQyxDQUFyQztBQUNELENBWkcsQ0FBSjtBQWNBbEIsSUFBSSxDQUFDLCtCQUFELEVBQWtDLFlBQVk7QUFDaERGLEVBQUFBLElBQUksQ0FBQ3FCLGFBQUwsQ0FBbUIsUUFBbkI7QUFFQSxRQUFNTixNQUFNLEdBQUdmLElBQUksQ0FBQ2dCLEVBQUwsQ0FBUSxNQUFNO0FBQzNCLFVBQU1DLEtBQUssQ0FBQyxNQUFELENBQVg7QUFDRCxHQUZjLENBQWY7O0FBSUEsTUFBSTtBQUNGLG1CQUFRLE1BQU1GLE1BQU0sRUFBcEIsRUFBd0I7QUFBRUgsTUFBQUEsT0FBTyxFQUFFLEdBQVg7QUFBZ0JNLE1BQUFBLFFBQVEsRUFBRTtBQUExQixLQUF4QjtBQUNELEdBRkQsQ0FFRSxPQUFPQyxDQUFQLEVBQVUsQ0FDVjtBQUNEOztBQUNEbkIsRUFBQUEsSUFBSSxDQUFDc0IsbUJBQUwsQ0FBeUIsR0FBekI7QUFFQWYsRUFBQUEsTUFBTSxDQUFDUSxNQUFELENBQU4sQ0FBZUsscUJBQWYsQ0FBcUMsQ0FBckM7QUFDRCxDQWZHLENBQUo7QUFpQkFsQixJQUFJLENBQUMsd0JBQUQsRUFBMkIsWUFBWTtBQUN6Q0YsRUFBQUEsSUFBSSxDQUFDcUIsYUFBTCxDQUFtQixRQUFuQjtBQUVBLFFBQU1OLE1BQU0sR0FBR2YsSUFBSSxDQUFDZ0IsRUFBTCxDQUFRLE1BQU07QUFDM0IsVUFBTUMsS0FBSyxDQUFDLE1BQUQsQ0FBWDtBQUNELEdBRmMsQ0FBZjs7QUFJQSxNQUFJO0FBQ0YsbUJBQVEsTUFBTUYsTUFBTSxFQUFwQixFQUF3QjtBQUFFSCxNQUFBQSxPQUFPLEVBQUUsR0FBWDtBQUFnQk0sTUFBQUEsUUFBUSxFQUFFO0FBQTFCLEtBQXhCO0FBQ0QsR0FGRCxDQUVFLE9BQU9DLENBQVAsRUFBVSxDQUNWO0FBQ0Q7O0FBQ0RuQixFQUFBQSxJQUFJLENBQUNzQixtQkFBTCxDQUF5QixHQUF6QjtBQUVBZixFQUFBQSxNQUFNLENBQUNRLE1BQUQsQ0FBTixDQUFlSyxxQkFBZixDQUFxQyxDQUFyQztBQUNELENBZkcsQ0FBSiIsInNvdXJjZXNDb250ZW50IjpbIi8vIEBmbG93XG5pbXBvcnQgUmVhY3QgZnJvbSAncmVhY3QnO1xuaW1wb3J0IHsgVmlldywgVGV4dCwgVG91Y2hhYmxlT3BhY2l0eSB9IGZyb20gJ3JlYWN0LW5hdGl2ZSc7XG5pbXBvcnQgeyByZW5kZXIsIGZpcmVFdmVudCwgd2FpdEZvciB9IGZyb20gJy4uJztcblxuY2xhc3MgQmFuYW5hIGV4dGVuZHMgUmVhY3QuQ29tcG9uZW50PGFueT4ge1xuICBjaGFuZ2VGcmVzaCA9ICgpID0+IHtcbiAgICB0aGlzLnByb3BzLm9uQ2hhbmdlRnJlc2goKTtcbiAgfTtcblxuICByZW5kZXIoKSB7XG4gICAgcmV0dXJuIChcbiAgICAgIDxWaWV3PlxuICAgICAgICB7dGhpcy5wcm9wcy5mcmVzaCAmJiA8VGV4dD5GcmVzaDwvVGV4dD59XG4gICAgICAgIDxUb3VjaGFibGVPcGFjaXR5IG9uUHJlc3M9e3RoaXMuY2hhbmdlRnJlc2h9PlxuICAgICAgICAgIDxUZXh0PkNoYW5nZSBmcmVzaG5lc3MhPC9UZXh0PlxuICAgICAgICA8L1RvdWNoYWJsZU9wYWNpdHk+XG4gICAgICA8L1ZpZXc+XG4gICAgKTtcbiAgfVxufVxuXG5jbGFzcyBCYW5hbmFDb250YWluZXIgZXh0ZW5kcyBSZWFjdC5Db21wb25lbnQ8e30sIGFueT4ge1xuICBzdGF0ZSA9IHsgZnJlc2g6IGZhbHNlIH07XG5cbiAgb25DaGFuZ2VGcmVzaCA9IGFzeW5jICgpID0+IHtcbiAgICBhd2FpdCBuZXcgUHJvbWlzZSgocmVzb2x2ZSkgPT4gc2V0VGltZW91dChyZXNvbHZlLCAzMDApKTtcbiAgICB0aGlzLnNldFN0YXRlKHsgZnJlc2g6IHRydWUgfSk7XG4gIH07XG5cbiAgcmVuZGVyKCkge1xuICAgIHJldHVybiAoXG4gICAgICA8QmFuYW5hIG9uQ2hhbmdlRnJlc2g9e3RoaXMub25DaGFuZ2VGcmVzaH0gZnJlc2g9e3RoaXMuc3RhdGUuZnJlc2h9IC8+XG4gICAgKTtcbiAgfVxufVxuXG5hZnRlckVhY2goKCkgPT4ge1xuICBqZXN0LnVzZVJlYWxUaW1lcnMoKTtcbn0pO1xuXG50ZXN0KCd3YWl0cyBmb3IgZWxlbWVudCB1bnRpbCBpdCBzdG9wcyB0aHJvd2luZycsIGFzeW5jICgpID0+IHtcbiAgY29uc3QgeyBnZXRCeVRleHQsIHF1ZXJ5QnlUZXh0IH0gPSByZW5kZXIoPEJhbmFuYUNvbnRhaW5lciAvPik7XG5cbiAgZmlyZUV2ZW50LnByZXNzKGdldEJ5VGV4dCgnQ2hhbmdlIGZyZXNobmVzcyEnKSk7XG5cbiAgZXhwZWN0KHF1ZXJ5QnlUZXh0KCdGcmVzaCcpKS50b0JlTnVsbCgpO1xuXG4gIGNvbnN0IGZyZXNoQmFuYW5hVGV4dCA9IGF3YWl0IHdhaXRGb3IoKCkgPT4gZ2V0QnlUZXh0KCdGcmVzaCcpKTtcblxuICBleHBlY3QoZnJlc2hCYW5hbmFUZXh0LnByb3BzLmNoaWxkcmVuKS50b0JlKCdGcmVzaCcpO1xufSk7XG5cbnRlc3QoJ3dhaXRzIGZvciBlbGVtZW50IHVudGlsIHRpbWVvdXQgaXMgbWV0JywgYXN5bmMgKCkgPT4ge1xuICBjb25zdCB7IGdldEJ5VGV4dCB9ID0gcmVuZGVyKDxCYW5hbmFDb250YWluZXIgLz4pO1xuXG4gIGZpcmVFdmVudC5wcmVzcyhnZXRCeVRleHQoJ0NoYW5nZSBmcmVzaG5lc3MhJykpO1xuXG4gIGF3YWl0IGV4cGVjdChcbiAgICB3YWl0Rm9yKCgpID0+IGdldEJ5VGV4dCgnRnJlc2gnKSwgeyB0aW1lb3V0OiAxMDAgfSlcbiAgKS5yZWplY3RzLnRvVGhyb3coKTtcblxuICAvLyBBc3luYyBhY3Rpb24gZW5kcyBhZnRlciAzMDBtcyBhbmQgd2Ugb25seSB3YWl0ZWQgMTAwbXMsIHNvIHdlIG5lZWQgdG8gd2FpdFxuICAvLyBmb3IgdGhlIHJlbWFpbmluZyBhc3luYyBhY3Rpb25zIHRvIGZpbmlzaFxuICBhd2FpdCB3YWl0Rm9yKCgpID0+IGdldEJ5VGV4dCgnRnJlc2gnKSk7XG59KTtcblxudGVzdCgnd2FpdHMgZm9yIGVsZW1lbnQgd2l0aCBjdXN0b20gaW50ZXJ2YWwnLCBhc3luYyAoKSA9PiB7XG4gIGNvbnN0IG1vY2tGbiA9IGplc3QuZm4oKCkgPT4ge1xuICAgIHRocm93IEVycm9yKCd0ZXN0Jyk7XG4gIH0pO1xuXG4gIHRyeSB7XG4gICAgYXdhaXQgd2FpdEZvcigoKSA9PiBtb2NrRm4oKSwgeyB0aW1lb3V0OiA0MDAsIGludGVydmFsOiAyMDAgfSk7XG4gIH0gY2F0Y2ggKGUpIHtcbiAgICAvLyBzdXBwcmVzc1xuICB9XG5cbiAgZXhwZWN0KG1vY2tGbikudG9IYXZlQmVlbkNhbGxlZFRpbWVzKDMpO1xufSk7XG5cbnRlc3QoJ3dvcmtzIHdpdGggbGVnYWN5IGZha2UgdGltZXJzJywgYXN5bmMgKCkgPT4ge1xuICBqZXN0LnVzZUZha2VUaW1lcnMoJ2xlZ2FjeScpO1xuXG4gIGNvbnN0IG1vY2tGbiA9IGplc3QuZm4oKCkgPT4ge1xuICAgIHRocm93IEVycm9yKCd0ZXN0Jyk7XG4gIH0pO1xuXG4gIHRyeSB7XG4gICAgd2FpdEZvcigoKSA9PiBtb2NrRm4oKSwgeyB0aW1lb3V0OiA0MDAsIGludGVydmFsOiAyMDAgfSk7XG4gIH0gY2F0Y2ggKGUpIHtcbiAgICAvLyBzdXBwcmVzc1xuICB9XG4gIGplc3QuYWR2YW5jZVRpbWVyc0J5VGltZSg0MDApO1xuXG4gIGV4cGVjdChtb2NrRm4pLnRvSGF2ZUJlZW5DYWxsZWRUaW1lcygzKTtcbn0pO1xuXG50ZXN0KCd3b3JrcyB3aXRoIGZha2UgdGltZXJzJywgYXN5bmMgKCkgPT4ge1xuICBqZXN0LnVzZUZha2VUaW1lcnMoJ21vZGVybicpO1xuXG4gIGNvbnN0IG1vY2tGbiA9IGplc3QuZm4oKCkgPT4ge1xuICAgIHRocm93IEVycm9yKCd0ZXN0Jyk7XG4gIH0pO1xuXG4gIHRyeSB7XG4gICAgd2FpdEZvcigoKSA9PiBtb2NrRm4oKSwgeyB0aW1lb3V0OiA0MDAsIGludGVydmFsOiAyMDAgfSk7XG4gIH0gY2F0Y2ggKGUpIHtcbiAgICAvLyBzdXBwcmVzc1xuICB9XG4gIGplc3QuYWR2YW5jZVRpbWVyc0J5VGltZSg0MDApO1xuXG4gIGV4cGVjdChtb2NrRm4pLnRvSGF2ZUJlZW5DYWxsZWRUaW1lcygzKTtcbn0pO1xuIl19

With module:metro-react-native-babel-preset (Fails)

test('works with fake timers', function _callee6() {
  var mockFn;
  return _regenerator.default.async(function _callee6$(_context6) {
    while (1) {
      switch (_context6.prev = _context6.next) {
        case 0:
          jest.useFakeTimers('modern');
          mockFn = jest.fn(function () {
            throw Error('test');
          });

          try {
            (0, _.waitFor)(function () {
              return mockFn();
            }, {
              timeout: 400,
              interval: 200
            });
          } catch (e) {}

          jest.advanceTimersByTime(400);
          expect(mockFn).toHaveBeenCalledTimes(3);

        case 5:
        case "end":
          return _context6.stop();
      }
    }
  }, null, null, null, Promise);
});
Full Test File
7fc2847285e7ee92e202d0101776c7b0
var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");

var _regenerator = _interopRequireDefault(require("@babel/runtime/regenerator"));

var _classCallCheck2 = _interopRequireDefault(require("@babel/runtime/helpers/classCallCheck"));

var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/createClass"));

var _assertThisInitialized2 = _interopRequireDefault(require("@babel/runtime/helpers/assertThisInitialized"));

var _inherits2 = _interopRequireDefault(require("@babel/runtime/helpers/inherits"));

var _possibleConstructorReturn2 = _interopRequireDefault(require("@babel/runtime/helpers/possibleConstructorReturn"));

var _getPrototypeOf2 = _interopRequireDefault(require("@babel/runtime/helpers/getPrototypeOf"));

var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));

var _react = _interopRequireDefault(require("react"));

var _reactNative = require("react-native");

var _ = require("..");

function _createSuper(Derived) { var hasNativeReflectConstruct = _isNativeReflectConstruct(); return function _createSuperInternal() { var Super = (0, _getPrototypeOf2.default)(Derived), result; if (hasNativeReflectConstruct) { var NewTarget = (0, _getPrototypeOf2.default)(this).constructor; result = Reflect.construct(Super, arguments, NewTarget); } else { result = Super.apply(this, arguments); } return (0, _possibleConstructorReturn2.default)(this, result); }; }

function _isNativeReflectConstruct() { if (typeof Reflect === "undefined" || !Reflect.construct) return false; if (Reflect.construct.sham) return false; if (typeof Proxy === "function") return true; try { Date.prototype.toString.call(Reflect.construct(Date, [], function () {})); return true; } catch (e) { return false; } }

var Banana = function (_React$Component) {
  (0, _inherits2.default)(Banana, _React$Component);

  var _super = _createSuper(Banana);

  function Banana() {
    var _this;

    (0, _classCallCheck2.default)(this, Banana);

    for (var _len = arguments.length, args = new Array(_len), _key = 0; _key < _len; _key++) {
      args[_key] = arguments[_key];
    }

    _this = _super.call.apply(_super, [this].concat(args));
    (0, _defineProperty2.default)((0, _assertThisInitialized2.default)(_this), "changeFresh", function () {
      _this.props.onChangeFresh();
    });
    return _this;
  }

  (0, _createClass2.default)(Banana, [{
    key: "render",
    value: function render() {
      return _react.default.createElement(_reactNative.View, null, this.props.fresh && _react.default.createElement(_reactNative.Text, null, "Fresh"), _react.default.createElement(_reactNative.TouchableOpacity, {
        onPress: this.changeFresh
      }, _react.default.createElement(_reactNative.Text, null, "Change freshness!")));
    }
  }]);
  return Banana;
}(_react.default.Component);

var BananaContainer = function (_React$Component2) {
  (0, _inherits2.default)(BananaContainer, _React$Component2);

  var _super2 = _createSuper(BananaContainer);

  function BananaContainer() {
    var _this2;

    (0, _classCallCheck2.default)(this, BananaContainer);

    for (var _len2 = arguments.length, args = new Array(_len2), _key2 = 0; _key2 < _len2; _key2++) {
      args[_key2] = arguments[_key2];
    }

    _this2 = _super2.call.apply(_super2, [this].concat(args));
    (0, _defineProperty2.default)((0, _assertThisInitialized2.default)(_this2), "state", {
      fresh: false
    });
    (0, _defineProperty2.default)((0, _assertThisInitialized2.default)(_this2), "onChangeFresh", function _callee() {
      return _regenerator.default.async(function _callee$(_context) {
        while (1) {
          switch (_context.prev = _context.next) {
            case 0:
              _context.next = 2;
              return _regenerator.default.awrap(new Promise(function (resolve) {
                return setTimeout(resolve, 300);
              }));

            case 2:
              _this2.setState({
                fresh: true
              });

            case 3:
            case "end":
              return _context.stop();
          }
        }
      }, null, null, null, Promise);
    });
    return _this2;
  }

  (0, _createClass2.default)(BananaContainer, [{
    key: "render",
    value: function render() {
      return _react.default.createElement(Banana, {
        onChangeFresh: this.onChangeFresh,
        fresh: this.state.fresh
      });
    }
  }]);
  return BananaContainer;
}(_react.default.Component);

afterEach(function () {
  jest.useRealTimers();
});
test('waits for element until it stops throwing', function _callee2() {
  var _render, getByText, queryByText, freshBananaText;

  return _regenerator.default.async(function _callee2$(_context2) {
    while (1) {
      switch (_context2.prev = _context2.next) {
        case 0:
          _render = (0, _.render)(_react.default.createElement(BananaContainer, null)), getByText = _render.getByText, queryByText = _render.queryByText;

          _.fireEvent.press(getByText('Change freshness!'));

          expect(queryByText('Fresh')).toBeNull();
          _context2.next = 5;
          return _regenerator.default.awrap((0, _.waitFor)(function () {
            return getByText('Fresh');
          }));

        case 5:
          freshBananaText = _context2.sent;
          expect(freshBananaText.props.children).toBe('Fresh');

        case 7:
        case "end":
          return _context2.stop();
      }
    }
  }, null, null, null, Promise);
});
test('waits for element until timeout is met', function _callee3() {
  var _render2, getByText;

  return _regenerator.default.async(function _callee3$(_context3) {
    while (1) {
      switch (_context3.prev = _context3.next) {
        case 0:
          _render2 = (0, _.render)(_react.default.createElement(BananaContainer, null)), getByText = _render2.getByText;

          _.fireEvent.press(getByText('Change freshness!'));

          _context3.next = 4;
          return _regenerator.default.awrap(expect((0, _.waitFor)(function () {
            return getByText('Fresh');
          }, {
            timeout: 100
          })).rejects.toThrow());

        case 4:
          _context3.next = 6;
          return _regenerator.default.awrap((0, _.waitFor)(function () {
            return getByText('Fresh');
          }));

        case 6:
        case "end":
          return _context3.stop();
      }
    }
  }, null, null, null, Promise);
});
test('waits for element with custom interval', function _callee4() {
  var mockFn;
  return _regenerator.default.async(function _callee4$(_context4) {
    while (1) {
      switch (_context4.prev = _context4.next) {
        case 0:
          mockFn = jest.fn(function () {
            throw Error('test');
          });
          _context4.prev = 1;
          _context4.next = 4;
          return _regenerator.default.awrap((0, _.waitFor)(function () {
            return mockFn();
          }, {
            timeout: 400,
            interval: 200
          }));

        case 4:
          _context4.next = 8;
          break;

        case 6:
          _context4.prev = 6;
          _context4.t0 = _context4["catch"](1);

        case 8:
          expect(mockFn).toHaveBeenCalledTimes(3);

        case 9:
        case "end":
          return _context4.stop();
      }
    }
  }, null, null, [[1, 6]], Promise);
});
test('works with legacy fake timers', function _callee5() {
  var mockFn;
  return _regenerator.default.async(function _callee5$(_context5) {
    while (1) {
      switch (_context5.prev = _context5.next) {
        case 0:
          jest.useFakeTimers('legacy');
          mockFn = jest.fn(function () {
            throw Error('test');
          });

          try {
            (0, _.waitFor)(function () {
              return mockFn();
            }, {
              timeout: 400,
              interval: 200
            });
          } catch (e) {}

          jest.advanceTimersByTime(400);
          expect(mockFn).toHaveBeenCalledTimes(3);

        case 5:
        case "end":
          return _context5.stop();
      }
    }
  }, null, null, null, Promise);
});
test('works with fake timers', function _callee6() {
  var mockFn;
  return _regenerator.default.async(function _callee6$(_context6) {
    while (1) {
      switch (_context6.prev = _context6.next) {
        case 0:
          jest.useFakeTimers('modern');
          mockFn = jest.fn(function () {
            throw Error('test');
          });

          try {
            (0, _.waitFor)(function () {
              return mockFn();
            }, {
              timeout: 400,
              interval: 200
            });
          } catch (e) {}

          jest.advanceTimersByTime(400);
          expect(mockFn).toHaveBeenCalledTimes(3);

        case 5:
        case "end":
          return _context6.stop();
      }
    }
  }, null, null, null, Promise);
});
//# sourceMappingURL=data:application/json;charset=utf-8;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbIndhaXRGb3IudGVzdC5qcyJdLCJuYW1lcyI6WyJCYW5hbmEiLCJwcm9wcyIsIm9uQ2hhbmdlRnJlc2giLCJmcmVzaCIsImNoYW5nZUZyZXNoIiwiUmVhY3QiLCJDb21wb25lbnQiLCJCYW5hbmFDb250YWluZXIiLCJQcm9taXNlIiwicmVzb2x2ZSIsInNldFRpbWVvdXQiLCJzZXRTdGF0ZSIsInN0YXRlIiwiYWZ0ZXJFYWNoIiwiamVzdCIsInVzZVJlYWxUaW1lcnMiLCJ0ZXN0IiwiZ2V0QnlUZXh0IiwicXVlcnlCeVRleHQiLCJmaXJlRXZlbnQiLCJwcmVzcyIsImV4cGVjdCIsInRvQmVOdWxsIiwiZnJlc2hCYW5hbmFUZXh0IiwiY2hpbGRyZW4iLCJ0b0JlIiwidGltZW91dCIsInJlamVjdHMiLCJ0b1Rocm93IiwibW9ja0ZuIiwiZm4iLCJFcnJvciIsImludGVydmFsIiwidG9IYXZlQmVlbkNhbGxlZFRpbWVzIiwidXNlRmFrZVRpbWVycyIsImUiLCJhZHZhbmNlVGltZXJzQnlUaW1lIl0sIm1hcHBpbmdzIjoiOzs7Ozs7Ozs7Ozs7Ozs7Ozs7QUFDQTs7QUFDQTs7QUFDQTs7Ozs7O0lBRU1BLE07Ozs7Ozs7Ozs7Ozs7Ozs4RkFDVSxZQUFNO0FBQ2xCLFlBQUtDLEtBQUwsQ0FBV0MsYUFBWDtBQUNELEs7Ozs7Ozs2QkFFUTtBQUNQLGFBQ0UsNkJBQUMsaUJBQUQsUUFDRyxLQUFLRCxLQUFMLENBQVdFLEtBQVgsSUFBb0IsNkJBQUMsaUJBQUQsZ0JBRHZCLEVBRUUsNkJBQUMsNkJBQUQ7QUFBa0IsUUFBQSxPQUFPLEVBQUUsS0FBS0M7QUFBaEMsU0FDRSw2QkFBQyxpQkFBRCw0QkFERixDQUZGLENBREY7QUFRRDs7O0VBZGtCQyxlQUFNQyxTOztJQWlCckJDLGU7Ozs7Ozs7Ozs7Ozs7Ozt5RkFDSTtBQUFFSixNQUFBQSxLQUFLLEVBQUU7QUFBVCxLO2lHQUVRO0FBQUE7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUFBLGdEQUNSLElBQUlLLE9BQUosQ0FBWSxVQUFDQyxPQUFEO0FBQUEsdUJBQWFDLFVBQVUsQ0FBQ0QsT0FBRCxFQUFVLEdBQVYsQ0FBdkI7QUFBQSxlQUFaLENBRFE7O0FBQUE7QUFFZCxxQkFBS0UsUUFBTCxDQUFjO0FBQUVSLGdCQUFBQSxLQUFLLEVBQUU7QUFBVCxlQUFkOztBQUZjO0FBQUE7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUFBLEs7Ozs7Ozs2QkFLUDtBQUNQLGFBQ0UsNkJBQUMsTUFBRDtBQUFRLFFBQUEsYUFBYSxFQUFFLEtBQUtELGFBQTVCO0FBQTJDLFFBQUEsS0FBSyxFQUFFLEtBQUtVLEtBQUwsQ0FBV1Q7QUFBN0QsUUFERjtBQUdEOzs7RUFaMkJFLGVBQU1DLFM7O0FBZXBDTyxTQUFTLENBQUMsWUFBTTtBQUNkQyxFQUFBQSxJQUFJLENBQUNDLGFBQUw7QUFDRCxDQUZRLENBQVQ7QUFJQUMsSUFBSSxDQUFDLDJDQUFELEVBQThDO0FBQUE7O0FBQUE7QUFBQTtBQUFBO0FBQUE7QUFBQSxvQkFDYixjQUFPLDZCQUFDLGVBQUQsT0FBUCxDQURhLEVBQ3hDQyxTQUR3QyxXQUN4Q0EsU0FEd0MsRUFDN0JDLFdBRDZCLFdBQzdCQSxXQUQ2Qjs7QUFHaERDLHNCQUFVQyxLQUFWLENBQWdCSCxTQUFTLENBQUMsbUJBQUQsQ0FBekI7O0FBRUFJLFVBQUFBLE1BQU0sQ0FBQ0gsV0FBVyxDQUFDLE9BQUQsQ0FBWixDQUFOLENBQTZCSSxRQUE3QjtBQUxnRDtBQUFBLDRDQU9sQixlQUFRO0FBQUEsbUJBQU1MLFNBQVMsQ0FBQyxPQUFELENBQWY7QUFBQSxXQUFSLENBUGtCOztBQUFBO0FBTzFDTSxVQUFBQSxlQVAwQztBQVNoREYsVUFBQUEsTUFBTSxDQUFDRSxlQUFlLENBQUN0QixLQUFoQixDQUFzQnVCLFFBQXZCLENBQU4sQ0FBdUNDLElBQXZDLENBQTRDLE9BQTVDOztBQVRnRDtBQUFBO0FBQUE7QUFBQTtBQUFBO0FBQUE7QUFBQSxDQUE5QyxDQUFKO0FBWUFULElBQUksQ0FBQyx3Q0FBRCxFQUEyQztBQUFBOztBQUFBO0FBQUE7QUFBQTtBQUFBO0FBQUEscUJBQ3ZCLGNBQU8sNkJBQUMsZUFBRCxPQUFQLENBRHVCLEVBQ3JDQyxTQURxQyxZQUNyQ0EsU0FEcUM7O0FBRzdDRSxzQkFBVUMsS0FBVixDQUFnQkgsU0FBUyxDQUFDLG1CQUFELENBQXpCOztBQUg2QztBQUFBLDRDQUt2Q0ksTUFBTSxDQUNWLGVBQVE7QUFBQSxtQkFBTUosU0FBUyxDQUFDLE9BQUQsQ0FBZjtBQUFBLFdBQVIsRUFBa0M7QUFBRVMsWUFBQUEsT0FBTyxFQUFFO0FBQVgsV0FBbEMsQ0FEVSxDQUFOLENBRUpDLE9BRkksQ0FFSUMsT0FGSixFQUx1Qzs7QUFBQTtBQUFBO0FBQUEsNENBV3ZDLGVBQVE7QUFBQSxtQkFBTVgsU0FBUyxDQUFDLE9BQUQsQ0FBZjtBQUFBLFdBQVIsQ0FYdUM7O0FBQUE7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUFBO0FBQUEsQ0FBM0MsQ0FBSjtBQWNBRCxJQUFJLENBQUMsd0NBQUQsRUFBMkM7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUFBO0FBQ3ZDYSxVQUFBQSxNQUR1QyxHQUM5QmYsSUFBSSxDQUFDZ0IsRUFBTCxDQUFRLFlBQU07QUFDM0Isa0JBQU1DLEtBQUssQ0FBQyxNQUFELENBQVg7QUFDRCxXQUZjLENBRDhCO0FBQUE7QUFBQTtBQUFBLDRDQU1yQyxlQUFRO0FBQUEsbUJBQU1GLE1BQU0sRUFBWjtBQUFBLFdBQVIsRUFBd0I7QUFBRUgsWUFBQUEsT0FBTyxFQUFFLEdBQVg7QUFBZ0JNLFlBQUFBLFFBQVEsRUFBRTtBQUExQixXQUF4QixDQU5xQzs7QUFBQTtBQUFBO0FBQUE7O0FBQUE7QUFBQTtBQUFBOztBQUFBO0FBVzdDWCxVQUFBQSxNQUFNLENBQUNRLE1BQUQsQ0FBTixDQUFlSSxxQkFBZixDQUFxQyxDQUFyQzs7QUFYNkM7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUFBO0FBQUEsQ0FBM0MsQ0FBSjtBQWNBakIsSUFBSSxDQUFDLCtCQUFELEVBQWtDO0FBQUE7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUNwQ0YsVUFBQUEsSUFBSSxDQUFDb0IsYUFBTCxDQUFtQixRQUFuQjtBQUVNTCxVQUFBQSxNQUg4QixHQUdyQmYsSUFBSSxDQUFDZ0IsRUFBTCxDQUFRLFlBQU07QUFDM0Isa0JBQU1DLEtBQUssQ0FBQyxNQUFELENBQVg7QUFDRCxXQUZjLENBSHFCOztBQU9wQyxjQUFJO0FBQ0YsMkJBQVE7QUFBQSxxQkFBTUYsTUFBTSxFQUFaO0FBQUEsYUFBUixFQUF3QjtBQUFFSCxjQUFBQSxPQUFPLEVBQUUsR0FBWDtBQUFnQk0sY0FBQUEsUUFBUSxFQUFFO0FBQTFCLGFBQXhCO0FBQ0QsV0FGRCxDQUVFLE9BQU9HLENBQVAsRUFBVSxDQUVYOztBQUNEckIsVUFBQUEsSUFBSSxDQUFDc0IsbUJBQUwsQ0FBeUIsR0FBekI7QUFFQWYsVUFBQUEsTUFBTSxDQUFDUSxNQUFELENBQU4sQ0FBZUkscUJBQWYsQ0FBcUMsQ0FBckM7O0FBZG9DO0FBQUE7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUFBLENBQWxDLENBQUo7QUFpQkFqQixJQUFJLENBQUMsd0JBQUQsRUFBMkI7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUFBO0FBQzdCRixVQUFBQSxJQUFJLENBQUNvQixhQUFMLENBQW1CLFFBQW5CO0FBRU1MLFVBQUFBLE1BSHVCLEdBR2RmLElBQUksQ0FBQ2dCLEVBQUwsQ0FBUSxZQUFNO0FBQzNCLGtCQUFNQyxLQUFLLENBQUMsTUFBRCxDQUFYO0FBQ0QsV0FGYyxDQUhjOztBQU83QixjQUFJO0FBQ0YsMkJBQVE7QUFBQSxxQkFBTUYsTUFBTSxFQUFaO0FBQUEsYUFBUixFQUF3QjtBQUFFSCxjQUFBQSxPQUFPLEVBQUUsR0FBWDtBQUFnQk0sY0FBQUEsUUFBUSxFQUFFO0FBQTFCLGFBQXhCO0FBQ0QsV0FGRCxDQUVFLE9BQU9HLENBQVAsRUFBVSxDQUVYOztBQUNEckIsVUFBQUEsSUFBSSxDQUFDc0IsbUJBQUwsQ0FBeUIsR0FBekI7QUFFQWYsVUFBQUEsTUFBTSxDQUFDUSxNQUFELENBQU4sQ0FBZUkscUJBQWYsQ0FBcUMsQ0FBckM7O0FBZDZCO0FBQUE7QUFBQTtBQUFBO0FBQUE7QUFBQTtBQUFBLENBQTNCLENBQUoiLCJzb3VyY2VzQ29udGVudCI6WyIvLyBAZmxvd1xuaW1wb3J0IFJlYWN0IGZyb20gJ3JlYWN0JztcbmltcG9ydCB7IFZpZXcsIFRleHQsIFRvdWNoYWJsZU9wYWNpdHkgfSBmcm9tICdyZWFjdC1uYXRpdmUnO1xuaW1wb3J0IHsgcmVuZGVyLCBmaXJlRXZlbnQsIHdhaXRGb3IgfSBmcm9tICcuLic7XG5cbmNsYXNzIEJhbmFuYSBleHRlbmRzIFJlYWN0LkNvbXBvbmVudDxhbnk+IHtcbiAgY2hhbmdlRnJlc2ggPSAoKSA9PiB7XG4gICAgdGhpcy5wcm9wcy5vbkNoYW5nZUZyZXNoKCk7XG4gIH07XG5cbiAgcmVuZGVyKCkge1xuICAgIHJldHVybiAoXG4gICAgICA8Vmlldz5cbiAgICAgICAge3RoaXMucHJvcHMuZnJlc2ggJiYgPFRleHQ+RnJlc2g8L1RleHQ+fVxuICAgICAgICA8VG91Y2hhYmxlT3BhY2l0eSBvblByZXNzPXt0aGlzLmNoYW5nZUZyZXNofT5cbiAgICAgICAgICA8VGV4dD5DaGFuZ2UgZnJlc2huZXNzITwvVGV4dD5cbiAgICAgICAgPC9Ub3VjaGFibGVPcGFjaXR5PlxuICAgICAgPC9WaWV3PlxuICAgICk7XG4gIH1cbn1cblxuY2xhc3MgQmFuYW5hQ29udGFpbmVyIGV4dGVuZHMgUmVhY3QuQ29tcG9uZW50PHt9LCBhbnk+IHtcbiAgc3RhdGUgPSB7IGZyZXNoOiBmYWxzZSB9O1xuXG4gIG9uQ2hhbmdlRnJlc2ggPSBhc3luYyAoKSA9PiB7XG4gICAgYXdhaXQgbmV3IFByb21pc2UoKHJlc29sdmUpID0+IHNldFRpbWVvdXQocmVzb2x2ZSwgMzAwKSk7XG4gICAgdGhpcy5zZXRTdGF0ZSh7IGZyZXNoOiB0cnVlIH0pO1xuICB9O1xuXG4gIHJlbmRlcigpIHtcbiAgICByZXR1cm4gKFxuICAgICAgPEJhbmFuYSBvbkNoYW5nZUZyZXNoPXt0aGlzLm9uQ2hhbmdlRnJlc2h9IGZyZXNoPXt0aGlzLnN0YXRlLmZyZXNofSAvPlxuICAgICk7XG4gIH1cbn1cblxuYWZ0ZXJFYWNoKCgpID0+IHtcbiAgamVzdC51c2VSZWFsVGltZXJzKCk7XG59KTtcblxudGVzdCgnd2FpdHMgZm9yIGVsZW1lbnQgdW50aWwgaXQgc3RvcHMgdGhyb3dpbmcnLCBhc3luYyAoKSA9PiB7XG4gIGNvbnN0IHsgZ2V0QnlUZXh0LCBxdWVyeUJ5VGV4dCB9ID0gcmVuZGVyKDxCYW5hbmFDb250YWluZXIgLz4pO1xuXG4gIGZpcmVFdmVudC5wcmVzcyhnZXRCeVRleHQoJ0NoYW5nZSBmcmVzaG5lc3MhJykpO1xuXG4gIGV4cGVjdChxdWVyeUJ5VGV4dCgnRnJlc2gnKSkudG9CZU51bGwoKTtcblxuICBjb25zdCBmcmVzaEJhbmFuYVRleHQgPSBhd2FpdCB3YWl0Rm9yKCgpID0+IGdldEJ5VGV4dCgnRnJlc2gnKSk7XG5cbiAgZXhwZWN0KGZyZXNoQmFuYW5hVGV4dC5wcm9wcy5jaGlsZHJlbikudG9CZSgnRnJlc2gnKTtcbn0pO1xuXG50ZXN0KCd3YWl0cyBmb3IgZWxlbWVudCB1bnRpbCB0aW1lb3V0IGlzIG1ldCcsIGFzeW5jICgpID0+IHtcbiAgY29uc3QgeyBnZXRCeVRleHQgfSA9IHJlbmRlcig8QmFuYW5hQ29udGFpbmVyIC8+KTtcblxuICBmaXJlRXZlbnQucHJlc3MoZ2V0QnlUZXh0KCdDaGFuZ2UgZnJlc2huZXNzIScpKTtcblxuICBhd2FpdCBleHBlY3QoXG4gICAgd2FpdEZvcigoKSA9PiBnZXRCeVRleHQoJ0ZyZXNoJyksIHsgdGltZW91dDogMTAwIH0pXG4gICkucmVqZWN0cy50b1Rocm93KCk7XG5cbiAgLy8gQXN5bmMgYWN0aW9uIGVuZHMgYWZ0ZXIgMzAwbXMgYW5kIHdlIG9ubHkgd2FpdGVkIDEwMG1zLCBzbyB3ZSBuZWVkIHRvIHdhaXRcbiAgLy8gZm9yIHRoZSByZW1haW5pbmcgYXN5bmMgYWN0aW9ucyB0byBmaW5pc2hcbiAgYXdhaXQgd2FpdEZvcigoKSA9PiBnZXRCeVRleHQoJ0ZyZXNoJykpO1xufSk7XG5cbnRlc3QoJ3dhaXRzIGZvciBlbGVtZW50IHdpdGggY3VzdG9tIGludGVydmFsJywgYXN5bmMgKCkgPT4ge1xuICBjb25zdCBtb2NrRm4gPSBqZXN0LmZuKCgpID0+IHtcbiAgICB0aHJvdyBFcnJvcigndGVzdCcpO1xuICB9KTtcblxuICB0cnkge1xuICAgIGF3YWl0IHdhaXRGb3IoKCkgPT4gbW9ja0ZuKCksIHsgdGltZW91dDogNDAwLCBpbnRlcnZhbDogMjAwIH0pO1xuICB9IGNhdGNoIChlKSB7XG4gICAgLy8gc3VwcHJlc3NcbiAgfVxuXG4gIGV4cGVjdChtb2NrRm4pLnRvSGF2ZUJlZW5DYWxsZWRUaW1lcygzKTtcbn0pO1xuXG50ZXN0KCd3b3JrcyB3aXRoIGxlZ2FjeSBmYWtlIHRpbWVycycsIGFzeW5jICgpID0+IHtcbiAgamVzdC51c2VGYWtlVGltZXJzKCdsZWdhY3knKTtcblxuICBjb25zdCBtb2NrRm4gPSBqZXN0LmZuKCgpID0+IHtcbiAgICB0aHJvdyBFcnJvcigndGVzdCcpO1xuICB9KTtcblxuICB0cnkge1xuICAgIHdhaXRGb3IoKCkgPT4gbW9ja0ZuKCksIHsgdGltZW91dDogNDAwLCBpbnRlcnZhbDogMjAwIH0pO1xuICB9IGNhdGNoIChlKSB7XG4gICAgLy8gc3VwcHJlc3NcbiAgfVxuICBqZXN0LmFkdmFuY2VUaW1lcnNCeVRpbWUoNDAwKTtcblxuICBleHBlY3QobW9ja0ZuKS50b0hhdmVCZWVuQ2FsbGVkVGltZXMoMyk7XG59KTtcblxudGVzdCgnd29ya3Mgd2l0aCBmYWtlIHRpbWVycycsIGFzeW5jICgpID0+IHtcbiAgamVzdC51c2VGYWtlVGltZXJzKCdtb2Rlcm4nKTtcblxuICBjb25zdCBtb2NrRm4gPSBqZXN0LmZuKCgpID0+IHtcbiAgICB0aHJvdyBFcnJvcigndGVzdCcpO1xuICB9KTtcblxuICB0cnkge1xuICAgIHdhaXRGb3IoKCkgPT4gbW9ja0ZuKCksIHsgdGltZW91dDogNDAwLCBpbnRlcnZhbDogMjAwIH0pO1xuICB9IGNhdGNoIChlKSB7XG4gICAgLy8gc3VwcHJlc3NcbiAgfVxuICBqZXN0LmFkdmFuY2VUaW1lcnNCeVRpbWUoNDAwKTtcblxuICBleHBlY3QobW9ja0ZuKS50b0hhdmVCZWVuQ2FsbGVkVGltZXMoMyk7XG59KTtcbiJdfQ==

@mikeduminy
Copy link
Contributor

mikeduminy commented Oct 7, 2020

Just hit this problem now as I was migrating our app to RN 0.63. I had a look at how other testing-librarys solve it and it seems like they check if jest fake timers are set and run different logic here, while also capturing the global timer functions before they are overridden and then use these in their waitFor implementation.

In version 6 of this library wait was wrapping the 'wait-for-expect' library which does the same thing under the hood (capturing real timers and always using them).

Not sure if this is a known and intended consequence of the deprecation of the previous repo and whatever rewriting took place, but it would be SUPER good to have it in this repo so we don't have to change tonnes of code. However, primarily I think it is unreasonable that using timer mocks in our test would affect the test library code and so I would strongly request that this library ensures it is unaffected by any user-land settings.

If the maintainers agree with this direction but don't have the time to do this any time soon then I can take over the implementation. Please let me know.

@thymikee
Copy link
Member

thymikee commented Oct 7, 2020

Would love to merge a PR fixing that for good 👍

@mikeduminy
Copy link
Contributor

Okay it looks like the general approach followed by wait-for-expect to capture the global timer funcs before they get mocked works, but it has highlighted a problem with the 'modern' timer mocks which is caused partially by the 'react-native' preset polyfilling global.promise and partially by the new timer mocks mocking process.nextTick. I've written most of the code for the first bit but to make it work with modern timers we need to patch a line in '@jest/fake-timers'.

Will send a PR tomorrow.

@mikeduminy
Copy link
Contributor

PR sent, please have a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants