Closed
Bug 1474441
Opened 6 years ago
Closed 6 years ago
add support for testing the new React-based front end
Categories
(Socorro :: Webapp, task)
Socorro
Webapp
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: alexisdeschamps, Assigned: alexisdeschamps)
References
Details
We are in the process of adding scaffolding for development of React front-end components: https://github.com/mozilla-services/socorro/pull/4494. Part of the reason behind this endeavor is to improve Socorro's testing capabilities for the front-end parts of the application.
This bug implies adding the functionality that will allow for running tests against this new front end. :Osmose mentioned adding Enzyme (https://github.com/airbnb/enzyme) to test React components. To run Enzyme tests, I suggest using Jest (https://github.com/facebook/jest).
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → adeschamps
Status: NEW → ASSIGNED
Comment 1•6 years ago
|
||
Commits pushed to master at https://github.com/mozilla-services/socorro
https://github.com/mozilla-services/socorro/commit/267cfa74a29c5f60ea2fb27793dd370f476af3d6
fix bug 1474441: added Jest and Enzyme
why:
- we want to be able to run tests against the new React-based front end
in order to ensure quality
what:
- added Jest and Ezyme to the project
- added file to set up the Jest tests
- added documentation on how to run Jest tests
- added temporarily barebones ReportIndex React component
- added PageHeading React component
- added Enzyme tests for this component
https://github.com/mozilla-services/socorro/commit/26bd3cccd58b515e30424d74a9d86051c59c194e
fix bug 1474441: added review updates
- put jest config in separate file
- cleaned up run_tests.sh
- moved "node_modules/" to .eslintignore
- added resolve alias for staticfiles folder called "socorro"
- added babel-eslint as the staticfiles eslint parser
- this allowed for adding babel-plugin-transform-class-properties
- changed PageHeading propTypes to be a static property
- added license headers to JS files
https://github.com/mozilla-services/socorro/commit/560857b5369dbeaee2f3f6eed5f1e7eea6f51037
Merge pull request #4511 from mozilla-services/react-cupcake-testing
fix bug 1474441: added Jest and Enzyme
Updated•6 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•