Feature request: accept async functions in beforeEach/afterEach

[ide]

I'm using Aphrodite with React and Aphrodite asynchronously runs document.querySelector("style[data-aphrodite]") (scheduled with asap, so it uses process.nextTick). Jest seems to clean up the jsdom environment after each test so this results in an error because the tick callback runs after document has been set to undefined.

So what I want to do is this:

it('renders snapshot', () => {
  // Render a React component
});

afterEach(() => new Promise(resolve => process.nextTick(resolve)));

That way each test flushes all tick callbacks and allows Aphrodite to run.

[cpojer]

duplicate of #1256.

This should be easy to implement: just overwrite beforeEach/afterEach the same way we overwrite it with jasmine-pit.

[cpojer]

please send a PR ;)

[ide]

I'll venmo/messenger 5 bucks to whomever does a good job :)

In the meantime I'm using fake timers and

  afterEach(() => {
    jest.runAllTicks();
  });

[cpojer]

afterEach already accepts done for async work, no?

[aaronabramov]

it does. http://jasmine.github.io/2.5/introduction.html#section-55
i had no idea jest doesn't support it. I can work on it this week

[ide]

This is already fixed in #1546 / 15.1.0.

[cpojer]

@DmitriiAbramov @ide ended up doing this himself, therefore paying 5 bucks to himself.

[aaronabramov]

oh. nevermind then.. still going through my jest emails from last week :)
thanks @ide for doing this!

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.