Skip to content

WRN-20481: Fix screenshot test build fail #267

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 5 commits into from
Apr 7, 2022

Conversation

MikyungKim
Copy link
Contributor

@MikyungKim MikyungKim commented Apr 7, 2022

In this PR, we'd like to fix screenshot test build fail from sandstone.
With Webpack5, automatic Node.js polyfils are removed. (https://webpack.js.org/blog/2020-10-10-webpack-5-release/#automatic-nodejs-polyfills-removed)
But we need some modules from ui-test-utils to run screenshot tests.
So, I've added node-polyfill-webpack-plugin to resolve those modules.
Also, fixed lint error.

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])
Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])
Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])
Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])
@MikyungKim MikyungKim changed the title WRN-20481: Fix screenshot test build failed WRN-20481: Fix screenshot test build fail Apr 7, 2022
Copy link
Contributor

@hong6316 hong6316 left a comment

Choose a reason for hiding this comment

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

Test result checked(#872~#873)

LGTM

@hong6316 hong6316 merged commit 9fa7858 into feature/webpack5 Apr 7, 2022
@hong6316 hong6316 deleted the feature/WRN-20481 branch April 7, 2022 07:19
hong6316 added a commit that referenced this pull request Apr 11, 2022
* WRN-18582: Migrate to webpack5 (#264)

* WRN-18582: webpack5 initial commit

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-18582: Fixed serve issue

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-18582: Added webpack cache config

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* applied more changes

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* Added changelog

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* fix

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update `listen` to `startCallback` from webpack dev server

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* fix filename..ext issue

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-16762: Added support for tailwindcss and update dependencies (#263)

* initial modules update(before wp5 update)

* npm audit fix

* Update package.json

* fix lint errors

* update CHANGELOG.md

* Update package.json

* revert CHANGELOG.mx

* revert jest 27

* revert npm-shrinkwrap.json

* revert pack/serve.js

* for removing conflict with #262, revert versions

* Update react-refresh-webpack-plugin version

Co-authored-by: Mikyung Kim <[email protected]>

* Update babel-jest version

Co-authored-by: Mikyung Kim <[email protected]>

* Update babel-plugin-dev-expression version

Co-authored-by: Mikyung Kim <[email protected]>

* remove unnecessary code

* revert file permission

* update dependencies

* update package.json

* Update package.json

Co-authored-by: Mikyung Kim <[email protected]>

* Update package.json

* fix

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Co-authored-by: Mikyung Kim <[email protected]>

* update changelog and package.json

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Co-authored-by: taeyoung.hong <[email protected]>

* WRN-12124: Remove deprecated babel-eslint module (#261)

* use @babel/eslint-parser

* remove babel-preset-react-app

* remove @babel/eslint-parser

* Revert "remove @babel/eslint-parser"

This reverts commit 1b8b37e.

* Revert "remove babel-preset-react-app"

This reverts commit b8579cb.

* Revert "use @babel/eslint-parser"

This reverts commit b73146a.

* babel-eslint -> @babel/eslint-parser

* unpinning enact repos

* remove babel-eslint related word in README.md

* Update README.md

* modify changelog & readme

* Update package.json

Co-authored-by: Mikyung Kim <[email protected]>

* Update README.md

Co-authored-by: Mikyung Kim <[email protected]>

* update npm-shrinkwrap.json

* update @babel/core to the same version of dev-utils

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* Move some dependencies to dev dependencies

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Co-authored-by: Mikyung Kim <[email protected]>

* WRN-19252: Pass ProgressPlugin to VerboseLogPlugin (#265)

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-12748: Update eslint 8 related modules (#262)

* update eslint 8

* eslint 8 changes strict package exports(#14706), call another method

* revert js changes

* Update package.json

* restore eslintrc.js - 'prettier' (included 'prettier/babel', 'prettier/react')

* Update package.json

Co-authored-by: Mikyung Kim <[email protected]>

Co-authored-by: Mikyung Kim <[email protected]>

* WRN-19694: Update to React18 (#266)

* update react version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update testing library for react18

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update react version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update dependencies

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update dev-utils branch

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* revert dev-utils version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-20481: Fix screenshot test build fail (#267)

* Added node-polyfill-webpack-plugin

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* added some comments

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* link dev-utils

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* WRN-20481: Fix lint error and revert packages version

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update change log

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

* update dependencies

Enact-DCO-1.0-Signed-off-by: Mikyung Kim ([email protected])

Co-authored-by: taeyoung.hong <[email protected]>
hong6316 added a commit that referenced this pull request Sep 4, 2024
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