IBX-10693: Add tests for AltRadio components #49
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds test coverage for AltRadio components by implementing Storybook test stories that verify component behavior and user interactions.
- Creates comprehensive test coverage for AltRadiosListField and updates tests for AltRadioInput
- Implements interactive test scenarios using Storybook's play functions with user event simulation
- Tests radio button selection behavior, event handling, and state changes
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| AltRadiosListField.test.stories.tsx | Adds new test story covering radio list selection behavior and change events |
| AltRadioInput.test.stories.tsx | Updates existing test story with improved step descriptions and additional assertions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| await step('Click second item', async () => { | ||
| await userEvent.click(input2); | ||
|
|
||
| await expect(args.onChange).toHaveBeenCalledTimes(2); // eslint-disable-line no-magic-numbers |
There was a problem hiding this comment.
The magic number 2 should be extracted to a descriptive constant like EXPECTED_CALL_COUNT to improve code readability and maintainability.
Copilot uses AI. Check for mistakes.
| await step('Click second item once more', async () => { | ||
| await userEvent.click(input2); | ||
|
|
||
| await expect(args.onChange).toHaveBeenCalledTimes(2); // eslint-disable-line no-magic-numbers |
There was a problem hiding this comment.
The magic number 2 should be extracted to a descriptive constant like EXPECTED_CALL_COUNT to improve code readability and maintainability.
Copilot uses AI. Check for mistakes.
Description:
For QA:
Documentation: