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

Tests for the color-picker alpha input box fail #6266

Closed
alisonailea opened this issue Jan 10, 2023 · 1 comment
Closed

Tests for the color-picker alpha input box fail #6266

alisonailea opened this issue Jan 10, 2023 · 1 comment
Labels
0 - new New issues that need assignment. p - high Issue should be addressed in the current milestone, impacts component or core functionality testing Issues related to automated or manual testing.

Comments

@alisonailea
Copy link
Contributor

Test type

unstable tests

Which Component(s)

color-picker

Unstable Tests

calcite-color-picker › color inputs › alpha enabled › color gets propagated to support inputs › valid color › allows nudging values

calcite-color-picker › color inputs › alpha enabled › color gets propagated to support inputs › valid color › allows modifying color via hex, RGB, HSV, opacity inputs

calcite-color-picker › color inputs › alpha enabled › color gets propagated to support inputs › when no-color › clearing color via supporting inputs › clears color via opacity input

calcite-color-picker › color inputs › alpha enabled › color gets propagated to support inputs › when no-color › clearing color via supporting inputs › restores previous color value when a nudge key is pressed

calcite-color-picker › color inputs › alpha enabled › color gets propagated to support inputs › when no-color › clearing color via supporting inputs › changes the value to the specified format after being empty

Test error, if applicable

Protocol error (Input.dispatchKeyEvent): Target closed.

PR skipped, if applicable

#6265

Additional Info

color-picker tests are heavily nested with many nested functions. This is making the tests hard to read and may be causing some of the instability.

@alisonailea alisonailea added testing Issues related to automated or manual testing. p - high Issue should be addressed in the current milestone, impacts component or core functionality 0 - new New issues that need assignment. labels Jan 10, 2023
@jcfranco
Copy link
Member

Tests were restored in #2841.

Closing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - new New issues that need assignment. p - high Issue should be addressed in the current milestone, impacts component or core functionality testing Issues related to automated or manual testing.
Projects
None yet
Development

No branches or pull requests

2 participants