Remove snapshots

[gnapse]

What:

Removing snapshots of error messages in the tests that are expected to throw.

Why:

Because the terminal output stored I these snapshots is not the same in terminals where colors are not supported, such as in Travis-CI.

Checklist:

• Tests
• Ready to be merged

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (master@48b11fb). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             master    #16   +/-   ##
=======================================
  Coverage          ?   100%           
=======================================
  Files             ?      8           
  Lines             ?     74           
  Branches          ?     15           
=======================================
  Hits              ?     74           
  Misses            ?      0           
  Partials          ?      0

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 48b11fb...95febd1. Read the comment docs.

[kentcdodds]

You could also strip the ansi characters with a snapshot serializer. I'm pretty sure there's a module that does this.

[microcood]

Also disable chalk before tests beforeAll(() => { chalk.enabled = false; }); will work.
More interesting question why CI does not work. How its different between PR and master

[gnapse]

There's a just issue I referenced in the PR that introduced this problem (jestjs/jest#4407) where they talk a bit more about this, and that it is expected. They suggest the serializer there, but I haven't found any yet. Suggestions of where to look are welcome. I already looked in jest-awesome with no luck.

I also should've detected this in the original PR, because the codecov report never appeared and I did not notice.

BTW, interesting the chalk.enabled fix. I guess that's the real fallback I should use if the serializer does not appear. At least it allows us to keep snapshots in place. Thanks!

[kentcdodds]

Try this one: https://github.com/joaogranado/jest-serializer-ansi

[gnapse]

Done in #17, but somehow build is still failing over there, even when they all pass locally :(

[kentcdodds]

Ugh.

[microcood]

Most likely that's the issue travis-ci/travis-ci#7967

[gnapse]

There doesn't seem to be a definite solution for this problem in that travis issue discussion :(

Given that #17 remains failing for some other reason even after serializing snapshots without the ansi stuff, I'm leaning towards merging this PR, because it nevertheless fixes the issue reported in testing-library/dom-testing-library#50.

I'll try to dig a bit more and wait until later today, to see if a this can be solved. Otherwise I'll merge this PR because we can always bring back the snapshots with proper serialization and/or travis settings fixed or whatever, in a later PR.

[gnapse]

🎉 This PR is included in version 1.3.1 🎉

The release is available on:

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

Your semantic-release bot 📦🚀