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

refactor(project): update AC config and add in AVT tests #10956

Merged
merged 4 commits into from
Mar 17, 2022

Conversation

joshblack
Copy link
Contributor

Related #9475

This PR updates our AC config to use a custom ruleset and adds in some example tests in different component test files.

Changelog

New

  • Add custom achecker.js config for AC

Changed

  • Update toHaveNoACViolations to use custom ruleset
  • Update test files to include AC and aXe tests

Removed

@joshblack joshblack requested a review from a team as a code owner March 10, 2022 21:38
@joshblack joshblack requested review from tw15egan and abbeyhrt March 10, 2022 21:38
@netlify
Copy link

netlify bot commented Mar 10, 2022

✔️ Deploy Preview for carbon-react-next ready!

🔨 Explore the source changes: 445a136

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-react-next/deploys/623379b157fc90000979a64a

😎 Browse the preview: https://deploy-preview-10956--carbon-react-next.netlify.app

@joshblack joshblack requested a review from dakahn March 10, 2022 21:38
@netlify
Copy link

netlify bot commented Mar 10, 2022

✔️ Deploy Preview for carbon-elements ready!

🔨 Explore the source changes: 445a136

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-elements/deploys/623379b1c6740c0009aef372

😎 Browse the preview: https://deploy-preview-10956--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Mar 10, 2022

✔️ Deploy Preview for carbon-components-react ready!
Built without sensitive environment variables

🔨 Explore the source changes: 445a136

🔍 Inspect the deploy log: https://app.netlify.com/sites/carbon-components-react/deploys/623379b19ff3d800088688c4

😎 Browse the preview: https://deploy-preview-10956--carbon-components-react.netlify.app

@kodiakhq kodiakhq bot merged commit ef46ea9 into carbon-design-system:main Mar 17, 2022
tay1orjones pushed a commit to tay1orjones/carbon that referenced this pull request Mar 23, 2022
…gn-system#10956)

* test(react): add AVT to breadcrumb

* feat(jest-config): update accessibility-checker config

* test(react): add accessibility-checker tests to ContentSwitcher

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
kodiakhq bot added a commit that referenced this pull request Mar 23, 2022
* refactor(project): update AC config and add in AVT tests (#10956)

* test(react): add AVT to breadcrumb

* feat(jest-config): update accessibility-checker config

* test(react): add accessibility-checker tests to ContentSwitcher

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>

* fix(Combobox): compose onStateChange event handlers (#10968)

* fix(Combobox): compose onStateChange event handlers

* fix(Combobox): remove unnecessary short circuiting

* fix(Combobox): get the optional chaining operator syntax right

* fix(Combobox): add conditional check to downshiftProps object as well

* chore(Carbon): update public api snapshot

* fix(deps): update dependency @carbon/telemetry to v0.1.0 (#10982)

Co-authored-by: Renovate Bot <[email protected]>
Co-authored-by: Josh Black <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>

* chore(DataTable): size prop audit (#10999)

* chore(DataTable): size prop audit first draft

* chore(DataTable): removed zombies

* chore(DataTable): removed v10 styles

* chore(DataTable): excluded v10 stories

* chore(DataTable): removed sm story from Toolbar

* chore(DataTable): included more dynamic and expansion stories

* refactor(react): update components to usePrefix or PrefixContent (#11015)

* refactor(react): update components to usePrefix or PrefixContent

* refactor(react): update components to usePrefix (2)

* fix(React): update to new prefix method

* refactor(react): update components to usePrefix (3)

* refactor(react): update components to usePrefix (4)

* chore(react): revert useNormalizedProps as it is used in classes

* chore(react): update tests and incorrect import paths

* test(react): update tests based on changes

* refactor(react): update TabContent usePrefix usage

Co-authored-by: D.A. Kahn <[email protected]>

* 10957 fix datepicker unknown prop warning (#10959)

* fix(Datepicker): add onOpen to destructured props

* fix(Datepicker): remove render call for onOpen

* fix(react): update data-table based on audit (#10966)

* chore: check-in work

* fix(styles): update search bar with data table

* docs(data-table): update batch actions > small story

Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>

* test: update snap

* chore(project): update action to no longer use v10 branch ref

Co-authored-by: Josh Black <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Co-authored-by: DAK <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Renovate Bot <[email protected]>
Co-authored-by: Scott Strubberg <[email protected]>
Co-authored-by: D.A. Kahn <[email protected]>
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.

3 participants