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 Test Application #4206

Open
2 of 79 tasks
Westbrook opened this issue Mar 22, 2024 · 2 comments
Open
2 of 79 tasks

Refactor Test Application #4206

Westbrook opened this issue Mar 22, 2024 · 2 comments

Comments

@Westbrook
Copy link
Contributor

Westbrook commented Mar 22, 2024

Code of conduct

  • I agree to follow this project's code of conduct.

Description of issue

Tests have been added to the project in a fairly ad hoc manner throughout its history. Refactoring tests to be delivered in via the following follows would make for component to component consistency and for a more clear destination for additional test contexts over time:

  • declarative.test.ts: test all declarative usage of the element in question
  • imperative.test.ts: all imperative interactions
  • keyboard.test.ts: all keyboard interactions
  • a11y-tree.test.ts: the delivered a11y tree across various states
  • controlled.test.ts: that the element can be "controlled" by the parent application
  • etc.

To Dos:

  • accordion
  • action-bar
  • action-button
  • action-group
  • action-menu
  • alert-dialog
  • asset
  • avatar
  • badge
  • banner
  • button
  • button-group
  • card
  • checkbox
  • clear-button
  • close-button
  • coachmark
  • color-area
  • color-field
  • color-handle
  • color-loupe
  • color-slider
  • color-wheel
  • combobox
  • dialog
  • divider
  • dropzone
  • field-group
  • field-label
  • help-text
  • icon
  • icons
  • icons-ui
  • icons-workflow
  • iconset
  • illustrated-message
  • infield-button
  • link
  • menu
  • meter
  • modal
  • number-field
  • opacity-checkerboard
  • overlay
  • picker
  • picker-button
  • popover
  • progress-bar
  • progress-circle
  • quick-actions
  • radio
  • search
  • sidenav
  • slider
  • split-button
  • split-view
  • status-light
  • swatch
  • switch
  • table
  • tabs
  • tags
  • textfield
  • thumbnail
  • toast
  • tooltip
  • top-nav
  • tray
  • underlay
  • base
  • bundle
  • grid
  • opacity-checkerboard
  • reactive-controllers
  • shared
  • styles
  • theme
  • truncated
@Westbrook Westbrook pinned this issue Mar 22, 2024
@najikahalsema
Copy link
Collaborator

The meat of this work will be addign tests to the controlled test group.

@nikkimk
Copy link
Contributor

nikkimk commented Jul 22, 2024

Instead of a11y-tree.testing.ts being separated into its own file, a11y tree testing should be a part of all of the other test files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants