Add tests for Radio and RadioGroup

kamebkj commented 3 years ago

Basic tests for the components but would love some guidance on a few parts:

Unsure what's the best assertion to use to test the size prop for both Radio and RadioGroup since the size is applied on the sibling element of the input[type=radio]. The size of the input element itself is always 1px
In RadioGroup the prop isRequired is adding the required state to all the options in its group. I have a test case to test their states but unsure why we have this prop in the first place
Not sure why we have defaultValue prop in RadioGroup so didn't write a test for it

Thanks!

netlify[bot] commented 3 years ago

✔️ Deploy Preview for evergreen-storybook ready!

🔨 Explore the source changes: 6fcee8d2230dd5c4bda4bb5acc881201d09c99b4

kamebkj commented 2 years ago

I forgot this PR is still opened. Thought I've merged it months ago 😅 . Thanks for the eyes on this!!

segmentio / evergreen