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

Upgrade d3 to v7. #11282

Merged
merged 3 commits into from Nov 4, 2021
Merged

Upgrade d3 to v7. #11282

merged 3 commits into from Nov 4, 2021

Conversation

kialam
Copy link
Member

@kialam kialam commented Oct 26, 2021

SUMMARY

Upgrade D3 to v7.

COMPONENT NAME
  • UI

@@ -101,6 +101,9 @@
"coveragePathIgnorePatterns": [
"<rootDir>/src/locales",
"index.js"
],
"transformIgnorePatterns": [
"<rootDir>/node_modules/(?!d3)/"
Copy link
Member Author

Choose a reason for hiding this comment

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

It wasn't easy to track this down, however following a trail of release logs eventually led me to the solution on how to get jest to work again.

In short, d3 v7 introduces a new version of their sub packages (d3-array, d3-hierarchy, etc). v3 of these packages have adopted a new module type. These packages are now ESM modules instead of CommonJS. Jest ships with experimental support of ESM modules, and in order for the tests to properly import the packages when the tests are ran is to disable code transformation from within the d3 directory.

@nixocio nixocio added qe:e2e and removed qe:e2e labels Nov 1, 2021
@nixocio nixocio self-requested a review November 1, 2021 20:36
@cypress
Copy link

cypress bot commented Nov 1, 2021



Test summary

429 2 1927 0Flakiness 2


Run details

Project AWX - Functional
Status Failed
Commit a11d5cc
Started Nov 4, 2021 5:51 PM
Ended Nov 4, 2021 7:07 PM
Duration 15:56 💡
OS Linux Debian - 10.10
Browser Chrome 94

View run in Cypress Dashboard ➡️


Failures

job-templates/job-template-tabs.spec.js Failed
1 Job Templates Tab- Details > container group detail is visible and correct
activity-stream/activity-stream.spec.js Failed
1 Activity stream - List operations > activity link is visible and takes the user to activity stream page for each resource type

Flakiness

jobs/job-list.spec.js Flakiness
1 Jobs List > cancel button is disabled if one or more completed jobs are selected
dashboard/dashboard.spec.js Flakiness
1 Recently Used Templates > Can view Recent Templates

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@nixocio
Copy link
Contributor

nixocio commented Nov 3, 2021

Good job @kialam, the e2e test failures are also present on devel. Not related to this change.

@nixocio
Copy link
Contributor

nixocio commented Nov 3, 2021

See: #10594

@tiagodread tiagodread added qe:e2e and removed qe:e2e labels Nov 4, 2021
@tiagodread tiagodread self-requested a review November 4, 2021 17:13
Copy link
Contributor

@tiagodread tiagodread left a comment

Choose a reason for hiding this comment

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

I compared the broken tests with the broken tests in devel and they are the same, they are also not failing on downstream build.

@kialam kialam merged commit a7be25c into ansible:devel Nov 4, 2021
@kialam kialam deleted the upgrade-d3-to-v7 branch November 4, 2021 21:06
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.

None yet

4 participants