[MOO-1924] Fix react native configs#136
Conversation
…ormer for native builds
weirdwater
left a comment
There was a problem hiding this comment.
Thanks for the PR. Considering that enzyme has been removed from the default jest configuration, do the templates in the generator need updating as well?
Hmm, didn't think about those. You're right, I'll have to do the changes there as well, also will need to add the RN testing library to the list of dependencies. |
|
When I generate a new widget with tests using this branch I get the following error when running My steps: # Prepare tools
cd ~/Projects/widgets-tools
pnpm install
cd ./packages/pluggable-widgets-tools
pnpm pack
# Generate widget
cd ~/Mendix/testApp/myPluggableWidgets
~/Projects/widgets-tools/packages/generator-widget/bin.js NativeTestApp
cd nativeTestApp
rm -rf node_modules
npm install ~/Projects/widgets-tools/packages/pluggable-widgets-tools/mendix-pluggable-widgets-tools-10.21.4.tgz
npm install
npm test |
|
Looking into it a bit deeper I believe we need to do the following to make the tests runnable after generating a widget: Generator Widget
Pluggable Widgets Tools
The version mismatch seems to be coming from the |
05df46f to
37281e6
Compare
|
@weirdwater can you please let me know if it works for you after my last update? |
|
Yes, using these latest changes I can generate a widget and run the tests. I will ask another team member to approve as well. |
|
One note: We will need a changelog entry for the generator-widget. |
…version, and testing library
|
@weirdwater I've updated the changelog so please re-review |
Checklist
[XX-000]: description)? ✅This PR contains
What is the purpose of this PR?
Update Jest configs for Native test to use recommended preset, as well as remove Enzyme support.
Relevant changes