Add toMatchNamedSnapshot matcher

[samuelfullerthomas]

Summary

Adds a toMatchNamedSnapshot matcher for use:

• in concurrent test mode
• for people who want to name their snapshots for whatever reason.

I figured that having a separate matcher made the most sense so that I didn't have to change the API for the original matcher, which would be most compatible for existing toMatchSnapshot api usage.

Addresses #2180 and #5801. I've got some test written but I was hoping that y'all might have some thoughts about:

• is this the right way forward? Or is there something else you were hoping for?
• should I just add the same unit tests as the toMatchSnapshot?

Test plan

• Updating unit tests in the jest-snapshot package
• Updating integration tests

[samuelfullerthomas]

thoughts @SimenB ?

[SimenB]

Oh, this is interesting, sorry I missed it! Could you rebase?

We also need an integration test, plus tests asserting that some error is thrown if two tests are named the same in the same test suite

[samuelfullerthomas]

Hi @SimenB - can rebase - will try to get this ready this week

[github-actions]

This pull request 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.