Skip to content

MAX-5172 CRA unable to run tests after install #13

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

Merged
merged 1 commit into from
Oct 27, 2017
Merged

MAX-5172 CRA unable to run tests after install #13

merged 1 commit into from
Oct 27, 2017

Conversation

shinxi
Copy link
Collaborator

@shinxi shinxi commented Oct 26, 2017

  • Remove pacakge-lock.json which is the evil killer for npm test failure & @svmx/react-scripts double install.
  • Fix npm start compile issue. Update webpack configuration & public/index.html template to load webcomponent polyfills on need, application that don't use ui-component-predix don't need webcomponents polyfills.
  • There's a know issue at create-react-app since React 16. Add requestAnimationFrame polyfill to prevent test warnings.
  • Fix test case issue at Enzyme 3.
  • Upgrade to enzyme 3.1 and add its adapter.
  • Upgrade react-test-renderer to integrate with React 16.
  • Update jest setup file location.
  • Clean template dependencies.
  • Update eslint configuration to work align with prettier configuration.
  • Remove unused Link at ContainersList.js.

* Remove pacakge-lock.json
* Update jest setup file location.
* Upgrade to enzyme 3.1 and add its adapter.
* Update eslint configuration to work align with prettier configuration.
* Add a fixed react version(React 16) at template dependencies.
* Upgrade react-test-renderer to integrate with React 16.
* Load webcomponent polyfills on need. Update webpack configuration & public/index.html template to support application that is not using ui-component-predix, in other words, no need webcomponents polyfills.
* Remove unused Link at ContainersList.js
* Fix test case issue at Enzyme 3.
* Add requestAnimationFrame polyfill to prevent test warnings.
Copy link

@joshsylvester joshsylvester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so many of those looked just like my changes - it must have been the package.json killing me. thank you so much for the hard work!

@joshsylvester joshsylvester merged commit 4bc7ead into ServiceMax-Engineering:master Oct 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants