Check for rendering of header, description, children, buttons
Verify colors for each variant in light and dark mode
Expandable variants
Analytics passthroughs
This PR also renames the testId prop in Alert to testID to be consistent the other components and fixes one of the Button tests to use queryByText instead of the async findByText.
Note: Moved accessibility tests to #279 since we'll want to configure jest matchers
Testing Packages
N/A
Screenshots/Video
Testing
N/A, they're unit tests
PR Checklist
Code reviewer validation:
General
[x] PR is linked to ticket(s)
[x] PR has changelog label applied if it's to be included in the changelog
[x] Acceptance criteria:
All satisfied or
Documented reason for not being performed or
Split to separate ticket and ticket is linked by relevant AC(s)
[x] Above PR sections adequately filled out
[x] If any breaking changes, in accordance with the pre-1.0.0 versioning guidelines: a CU ticket has been created for the VA Mobile App detailing necessary adjustments with the package version that will be published by this ticket
Code
[x] Tests are included if appropriate (or split to separate ticket)
Description of Change
Adds the following unit tests for Alert:
This PR also renames the
testId
prop in Alert totestID
to be consistent the other components and fixes one of the Button tests to use queryByText instead of the async findByText.Note: Moved accessibility tests to #279 since we'll want to configure jest matchers
Testing Packages
N/A
Screenshots/Video
Testing
N/A, they're unit tests
PR Checklist
Code reviewer validation:
changelog
label applied if it's to be included in the changelogPublish
If changes warrant a new version per the versioning guidelines and the PR is approved and ready to merge:
main
into branchmain