Incorrect use for act in fireEvent.press

[rpopovici]

Describe the bug

At line

react-native-testing-library/src/fire-event.ts

Line 121 in 0f5cc2e

void act(() => {

you call act with void directive to silence the promise. Why? act is an async function which needs to be awaited.

Expected behavior

Return promise or wait for act and handler to finish

[mdjastrzebski]

act, as exported by React, can be both sync and async depending on the passed callback. Fire Event is a strictly sync API so this looks correct to me.

[rpopovici]

@mdjastrzebski according to the official docs the sync version is deprecated and unstable. To be removed in the future.

[mdjastrzebski]

@rpopovici thanks for linking the docs.

Closing as this issues is being addressed in #1787, and PR #1788.