Added console warning

Author: thatblindgeye

packages/react-core/src/components/Wizard/Wizard.tsx

@@ -61,7 +61,7 @@ export interface WizardProps extends React.HTMLProps<HTMLDivElement> {
   shouldFocusContent?: boolean;
   /** Adds plain styling to the wizard. */
   isPlain?: boolean;
-  /** @beta Prevents the wizard from automatically applying plain styling when glass theme is enabled. */
+  /** @beta Prevents the wizard from automatically applying plain styling when glass theme is enabled. When both this and isPlain are true, isPlain takes precedence. */
   isNoPlainOnGlass?: boolean;
 }
 
@@ -85,6 +85,13 @@ export const Wizard = ({
   isNoPlainOnGlass = false,
   ...wrapperProps
 }: WizardProps) => {
+  if (isPlain && isNoPlainOnGlass) {
+    // eslint-disable-next-line no-console
+    console.warn(
+      `Wizard: When both isPlain and isNoPlainOnGlass are true, isPlain will take precedence and isNoPlainOnGlass will have no effect. It's recommended to pass only one prop according to the current theme.`
+    );
+  }
+
   const [activeStepIndex, setActiveStepIndex] = useState(startIndex);
   const initialSteps = buildSteps(children);
   const firstStepRef = useRef(initialSteps[startIndex - 1]);

packages/react-core/src/components/Wizard/__tests__/Wizard.test.tsx

@@ -687,3 +687,41 @@ test(`Renders with ${styles.modifiers.noPlainOnGlass} class when isNoPlainOnGlas
 
   expect(screen.getByTestId('wizard-no-plain')).toHaveClass(styles.modifiers.noPlainOnGlass);
 });
+
+test('Does not log a warning when only isPlain is passed', () => {
+  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
+
+  render(
+    <Wizard isPlain>
+      <WizardStep id="test-step" name="Test step" />
+    </Wizard>
+  );
+
+  expect(consoleWarning).not.toHaveBeenCalled();
+});
+
+test('Does not log a warning when only isNoPlainOnGlass is passed', () => {
+  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
+
+  render(
+    <Wizard isNoPlainOnGlass>
+      <WizardStep id="test-step" name="Test step" />
+    </Wizard>
+  );
+
+  expect(consoleWarning).not.toHaveBeenCalled();
+});
+
+test('Logs a warning when both isPlain and isNoPlainOnGlass are passed', () => {
+  const consoleWarning = jest.spyOn(console, 'warn').mockImplementation();
+
+  render(
+    <Wizard isPlain isNoPlainOnGlass>
+      <WizardStep id="test-step" name="Test step" />
+    </Wizard>
+  );
+
+  expect(consoleWarning).toHaveBeenCalledWith(
+    `Wizard: When both isPlain and isNoPlainOnGlass are true, isPlain will take precedence and isNoPlainOnGlass will have no effect. It's recommended to pass only one prop according to the current theme.`
+  );
+});