Skip to content

Comments

test(checkbox): refine testing suite#7785

Draft
dariusbercea-sage wants to merge 1 commit intomasterfrom
FE-6454
Draft

test(checkbox): refine testing suite#7785
dariusbercea-sage wants to merge 1 commit intomasterfrom
FE-6454

Conversation

@dariusbercea-sage
Copy link
Contributor

Proposed behaviour

Removes a number of Playwright tests that duplicate either Jest or Chromatic ones for the Checkbox Component

Current behaviour

There are a number of tests that exist in different forms in all the testing suites we do(Jest, Playwright, Chromatic) which cause a prolonged time of execution for no additional value.

Checklist

  • Commits follow our style guide
  • Related issues linked in commit messages if required
  • Screenshots are included in the PR if useful
  • All themes are supported if required
  • Unit tests added or updated if required
  • Playwright automation tests added or updated if required
  • Storybook added or updated if required
  • Translations added or updated (including creating or amending translation keys table in storybook) if required
  • Typescript d.ts file added or updated if required
  • Related docs have been updated if required

QA

  • Tested in provided StackBlitz sandbox/Storybook
  • Add new Playwright test coverage if required
  • Carbon implementation matches Design System/designs
  • UI Tests GitHub check reviewed if required

Additional context

This should decrease the time of execution without compromising testing and coverage

Testing instructions

Make sure the Playwright tests pass

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

Development

Successfully merging this pull request may close these issues.

1 participant