Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add react-testing-library documentation/examples #4679

Merged
merged 3 commits into from
Jun 26, 2018

Conversation

kentcdodds
Copy link
Contributor

Rendered: https://github.com/kentcdodds/create-react-app/blob/1b3a028f0dea697322a2e85906c7642f357313c6/packages/react-scripts/template/README.md#use-react-testing-library

Considering the growing popularity of react-testing-library, it seems fitting to document and provide examples of using this library for folks looking for a testing utility that matches their preferences.

@Timer
Copy link
Contributor

Timer commented Jun 26, 2018

Thanks! Small nit, fix "assertsions".

@Timer Timer added this to the 2.0.0 milestone Jun 26, 2018
@kentcdodds
Copy link
Contributor Author

Done 👍

@Timer
Copy link
Contributor

Timer commented Jun 26, 2018

Awesome!

@Timer Timer merged commit ea7b374 into facebook:master Jun 26, 2018
@kentcdodds kentcdodds deleted the patch-1 branch June 26, 2018 16:46
Pavek pushed a commit to Pavek/create-react-app that referenced this pull request Jul 10, 2018
* add react-testing-library documentation/examples

* make react-testing-library a heading

* fix typo
zmitry pushed a commit to zmitry/create-react-app that referenced this pull request Aug 14, 2018
* add react-testing-library documentation/examples

* make react-testing-library a heading

* fix typo
cloud-walker pushed a commit to cloud-walker/create-react-app that referenced this pull request Aug 27, 2018
* Add modes to our Babel preset (1.x) (facebook#4668)

* [email protected]

* add react-testing-library documentation/examples (facebook#4679)

* add react-testing-library documentation/examples

* make react-testing-library a heading

* fix typo

* Fix link to the article about BEM (facebook#4858)

* Use file name whitelist to prevent RCE (facebook#4866)

* Use file name whitelist to prevent RCE

Use a whitelist to validate user-provided file names. This doesn't cover
the entire range of valid filenames but should cover almost all of them
in practice. Allows letters, numbers, periods, dashes, and underscores.
Opting to use a whitelist instead of a blacklist because getting this
wrong leaves us vulnerable to a RCE attack.

* Allow alphabet characters from all languages

Updated the whitelist to /^[\p{L}0-9/.\-_]+$/u, which matches
alphanumeric characters, periods, dashes, and underscores. Unicode
property support is stage 4 so I've inlined the transpiled version.

* Only use file name whitelist on Windows

* Log error message if file name does not pass whitelist

* Bump versions

* Bump release

* Add 1.1.5 release notes
@gnapse
Copy link
Contributor

gnapse commented Oct 4, 2018

This is lost somehow now. Not present in master anymore :(

@Timer
Copy link
Contributor

Timer commented Oct 4, 2018

No clue where it went. Can someone resubmit please?

@gnapse
Copy link
Contributor

gnapse commented Oct 4, 2018

I'm pretty sure it was in this merge commit: 4562ab6

I'll create a new PR to add it back.

@Timer
Copy link
Contributor

Timer commented Oct 4, 2018

Ah, I didn't realize this was merged into master and not next. Yes. We blew away all stuff in master in favor of next branch.

Timer pushed a commit that referenced this pull request Oct 4, 2018
* add react-testing-library documentation/examples

* make react-testing-library a heading

* fix typo
zmitry pushed a commit to zmitry/create-react-app that referenced this pull request Oct 29, 2018
…ebook#5290)

* add react-testing-library documentation/examples

* make react-testing-library a heading

* fix typo
@lock lock bot locked and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants