react-boilerplate / react-boilerplate / 4037

100%

master: 100%

push

travis-ci

Migrating to react-testing-library (#2592)

* Rewritten the generate-templates-for-linting script (#2473)

## React Boilerplate

### A problem

In order to test the generator functionality the `test-generators-without-a-trace` is executed on Travis. This runs all generators for components/containers and language.
Then the linter is scheduled to run so it inlcudes these generated components etc.

But any, snapshot, test for components wrapped in `<LanguageProvider />` fail because of the added language by the language generator.

### A solution

Therefore I've rewritten the `test-generators-without-a-trace` script. It runs all generators as before but checks for linting issues right after and on the component, container or modified file, if the generation was successful.
When the generator actions were successful and no linting problems were found the component, container or any added files are removed and modified files are restored.
In case any error occured the component, container or any added files are NOT removed and modified files are NOT restored so issues can be traced.

* Replacing Enzyme with react-testing-library (updated) (#2589)

* [Fix] Fixing issue were not all attributes are passed as required by 'containers/RepoListItem/IssueIcon' that wraps this component, according to test case 'should adopt any attribute'

* Replacing 'enzyme' as testing utility with 'react-testing-library'.

* Adapting the generator test templates for components and containers to use react-testing-library.

* Adapting the testing documentation for use with react-testing-library

* [Doc] Adding comments to the test templates to make clear that the standard snapshot test is skipped by default.

* [Doc] Fixing a typing mistake.

* Adding cleanup() from react-testing-library to tests that were missing it.

* Adding some words to point out that the implemented tests in the example app deliberately show implementation variations in order... (continued)

106 of 108 branches covered (98.15%)

277 of 278 relevant lines covered (99.64%)

43.37 hits per line