Radio: isChecked` prop on a DOM element

Description

Rendering a Radio triggers some react warnings

Link to Reproduction

https://codesandbox.io/s/wispy-worker-frvdq

Steps to reproduce

Simply render a Radio:

<Radio value="Chakra UI">Chakra UI</Radio>

Chakra UI Version

@chakra-ui/react@1.6.10

Browser

Chrome 94.0.4606.81 (Official Build) (arm64)

Operating System

• macOS
• Windows
• Linux

Additional Information

I noticed that those errors are also printed when running the tests in chakra-ui project. Do you think it could be a good solution to fail a test when console.error is called? That could prevent this kind of errors now and in the future.

Something like:

// .../testSetup.ts

let consoleErrorSpy: jest.SpyInstance;

beforeEach(() => {
  consoleErrorSpy = jest.spyOn(console, 'error').mockName('console.error');
});

afterEach(() => {
  expect(consoleErrorSpy).not.toHaveBeenCalled();
});

Then if a call to console.error is expected in a test, you can do:

console.error('testing...');
expect(console.error).toHaveBeenCalled();
(console.error as jest.Mock).mockReset();

[futhr]

Sorry for being annoying but maybe the issue should be closed when there is a related release tag? I had to lock @chakra-ui/react to v1.6.9 for now due to this.

[segunadebayo]

I just pushed a fix for this. It was caused by a recent PR related to the radio component.

We’ll release an update shortly.