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

fix: remove ART #561

Merged
merged 1 commit into from
Feb 22, 2021
Merged

fix: remove ART #561

merged 1 commit into from
Feb 22, 2021

Conversation

mrdulin
Copy link
Contributor

@mrdulin mrdulin commented Feb 20, 2021

When running npm run test:rn on the master branch, the test will fail:

FAIL  components/button/__tests__/index.native.test.jsx
  ● Test suite failed to run

    Cannot find module 'art/core/class.js' from 'node_modules/react-native-mock-render/build/components/ART/Path.js'

    Require stack:
      node_modules/react-native-mock-render/build/components/ART/Path.js
      node_modules/react-native-mock-render/build/components/ART/index.js
      node_modules/react-native-mock-render/build/react-native.js
      scripts/jest/setup.js

      at Resolver.resolveModule (node_modules/jest-resolve/build/index.js:306:11)
      at Object.<anonymous> (node_modules/react-native-mock-render/build/components/ART/Path.js:1:104)

Related issues:
Root-App/react-native-mock-render#47
Root-App/react-native-mock-render#89

PR has not been merged, so fork a repaired version. This is a temporary solution, consider using jest's react-native preset instead, need to modify the jest configuration in the zarm-cli project.

@netlify
Copy link

netlify bot commented Feb 20, 2021

Deploy preview for zarm ready!

Built with commit 5af4ee0

https://deploy-preview-561--zarm.netlify.app

@mrdulin mrdulin mentioned this pull request Feb 20, 2021
@JeromeLin JeromeLin merged commit 23420eb into ZhongAnTech:master Feb 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants