[Chip] Add chip classes

[pratikkarad]

• I have followed (at least) the PR section of the contributing guide.

I have tested this fix locally and also tested using a preview package on CodeSandbox.

Fixes #33246

[mui-bot]

Details of bundle changes

Generated by 🚫 dangerJS against f65bdb5

[mnajdova]

Let's add a test for this please.

[mnajdova]

Suggested change

	it('should render with the root and the info class', () => {
	const { container } = render(<Chip color="info" />);
	const chip = container.querySelector(`.${classes.root}`);
	expect(chip).to.have.class(classes.colorInfo);
	});
	it('should render with the root and the error class', () => {
	const { container } = render(<Chip color="error" />);
	const chip = container.querySelector(`.${classes.root}`);
	expect(chip).to.have.class(classes.colorError);
	});
	it('should render with the root and the warning class', () => {
	const { container } = render(<Chip color="warning" />);
	const chip = container.querySelector(`.${classes.root}`);
	expect(chip).to.have.class(classes.colorWarning);
	});
	it('should render with the root and the success class', () => {
	const { container } = render(<Chip color="success" />);
	const chip = container.querySelector(`.${classes.root}`);
	expect(chip).to.have.class(classes.colorSuccess);
	});
	it('should render with the color class name based on the color prop', () => {
	const capitalize = str => str.charAt(0).toUpperCase() + str.slice(1);
	['error', 'success', 'info', 'warning'].map((color) => {
	const { container } = render(<Chip color={color} />);
	const chip = container.querySelector(`.${classes.root}`);
	expect(chip).to.have.class(classes[`color${capitalize(color)}`]);
	});

Can we simplify maybe to something like this (I haven't run, there could be some issues with the code :))

[pratikkarad]

Yes, Thanks for the suggestion. I have updated the unit tests for color classes.

[mnajdova]

Looks good, left one final comment.

[pratikkarad]

PTAL, Thanks!

[pratikkarad]

Hi @mnajdova, any updates on this?

[pratikkarad]

Hey @mnajdova, any updates on this?

[mnajdova]

Looks good, thanks for the contribution. Sorry for the delay