-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
test(e2e): add PublicAPI tests and use metadata for icon tests #5846
test(e2e): add PublicAPI tests and use metadata for icon tests #5846
Conversation
…test/add-exports-test-for-icons
Deploy preview for carbon-elements ready! Built with commit c7b0603 |
Deploy preview for carbon-components-react ready! Built with commit c7b0603 https://deploy-preview-5846--carbon-components-react.netlify.com |
if (Array.isArray(icon.namespace) && icon.namespace.length > 0) { | ||
entry.namespace = icon.namespace.join('/'); | ||
} | ||
entry.namespace = Array.isArray(icon.namespace) ? icon.namespace : []; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a quick heads up for metadata changes @vpicone just going to have this be an array instead of converting it 👍
Deploy preview for carbon-elements ready! Built with commit ec9cc37 |
Deploy preview for carbon-components-react ready! Built with commit e44d813 https://deploy-preview-5846--carbon-components-react.netlify.com |
Deploy preview for carbon-components-react ready! Built with commit ec9cc37 https://deploy-preview-5846--carbon-components-react.netlify.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests work as expected 👍 ✅
…n-design-system#5846) * chore(github): update CODEOWNERS syntax for PublicAPI files * test(e2e): add PublicAPI tests and use metadata for icon tests * fix(icons): add registry to exports and check for duplicate sizes * test(matchers): defer AAT module resolution until called * docs(icon-build-helpers): add jsdoc for load export * test(e2e): add ESM files to exports test * chore(test): update paths for registry * chore(github): update CODEOWNERS comments * fix(icon-build-helpers): set namespace for pictograms * fix(pictograms): add main and module fields to package.json * test(e2e): add e2e tests for pictograms and pictograms-react Co-authored-by: TJ Egan <tw15egan@gmail.com>
* chore(github): update CODEOWNERS syntax for PublicAPI files * test(e2e): add PublicAPI tests and use metadata for icon tests * fix(icons): add registry to exports and check for duplicate sizes * test(matchers): defer AAT module resolution until called * docs(icon-build-helpers): add jsdoc for load export * test(e2e): add ESM files to exports test * chore(test): update paths for registry * chore(github): update CODEOWNERS comments * fix(icon-build-helpers): set namespace for pictograms * fix(pictograms): add main and module fields to package.json * test(e2e): add e2e tests for pictograms and pictograms-react Co-authored-by: TJ Egan <tw15egan@gmail.com>
Adds updated e2e tests for icons and icons-react, including a Public API snapshot and tests that leverage metadata as a source of truth.
Changelog
New
load
helper toMetadata
to load metadata information in-memory instead of persisting to diske2e/icons
ande2e/icons-react
. These test the public exports of these packagesChanged
Removed