fix: Make props optional in rerender function returned from renderHookToSnapshotStream

[jerelmiller]

Fixes #13

Makes the props argument optional in the rerender function returned from renderHookToSnapshotStream. This aligns it with RTL's rerender function returned from renderHook.

[pkg-pr-new]

yarn@berry undefined https://pkg.pr.new/testing-library/react-render-stream-testing-library/@testing-library/react-render-stream@14

commit: ac0b317

[phryneas]

We can actually reach the same result, but more situational and "correct" by defaulting Props to void instead of the default unknown.

This way if you have an actual argument defined for the function, it will stay required, but if there is no argument it will be void, making it optional.

We do need a little bit more work to also correctly work if the function argument is optional from the start - the VoidOptionalArg takes care of that.

[phryneas]

I never really liked it that rerender was so loose, so I think this could solve it more correctly. Could you please give this a re-review from your side @jerelmiller?

If you're happy with this, merge the branch and it will immediately be released.

[jerelmiller]

@phryneas I would agree, I like the idea of making it optional only if its truly optional. RTL ended up changing my mind, but happy to go this route for this package. Thanks for the assist!