Fix up custom render + rerender samples

[RoystonS]

What:

This fixes the samples mentioned in issue #225.

Why:

The code samples in the README.md were incorrect.

How:

I built my own custom renderer, and once I was happy that it was working, I backported the changes to the README.md.

Checklist:

• Documentation
• Tests N/A
• Ready to be merged
• Added myself to contributors table

Note that there are a few blank lines added to the README.md, a couple of which change the formatting (e.g. around the issue links in the data-testid section). These were inserted by the repository's own pre-commit hook (presumably running prettier or some such?).

[kentcdodds]

This looks great! Thanks! Let's just see if we can improve the formatting a bit.

[kentcdodds]

Did prettier add these extra lines here? If so, could you put them all on a single line? Otherwise the badges will no longer be side-by-side and will instead take up their own lives which will not look good.

[RoystonS]

Indeed. Yes, I did flag exactly that up in the PR comment. I'm guessing that whoever previously modified the README.md previously skipped the prettier step. I'll go back and put them on a single line.

[RoystonS]

I'll need to remove the blank lines and add 'prettier ignore' markup. For the issue links, pre-commit/prettier automatically splits them onto separate lines, and then on a subsequent commit adds blank lines between them.

[RoystonS]

I've fought prettier and won. Revised PR with a cleaner set of changes.

[kentcdodds]

Thank you!

[kentcdodds]

🎉 This PR is included in version 5.3.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[mbest]

I was searching for how to customize render and rerender and this is only example I could find. It's now no longer in the Readme.

[kentcdodds]

Hi @mbest, this is what you want: https://testing-library.com/docs/react-testing-library/setup#custom-render

[mbest]

Thanks for the pointer. I did get the example from here to work, but the wrapper option is a lot cleaner.

I was specifically searching for how to get rerender to work, and unfortunately, that doc page doesn't mention rerender.

[MatanBobi]

@mbest, I think that rerender can be overriden within the response of the render function (as you can see in the doc page @kentcdodds attached).
What's the use case you're trying to solve when overriding the rerender function? Maybe if you'll explain we'll be able to show an example :)

[kentcdodds]

I don't think he was trying to override the rerender method. He just wanted re-render to work nicely without having to specify any wrapping components again.

[MatanBobi]

I don't think he was trying to override the rerender method. He just wanted re-render to work nicely without having to specify any wrapping components again.

Oh, my bad, didn't get that 👍 Sorry.