fix delete any in colorpicker

Author: SBIN2010

superset-frontend/src/explore/components/controls/ColorPickerControl.test.tsx

@@ -163,3 +163,80 @@ test('calls onChange with RGB object when resolveThemeTokens is false', async ()
 
   expect(onChange).toHaveBeenCalledWith({ r: 0, g: 150, b: 0, a: 0.2 });
 });
+
+test('resolves colorSuccess theme token correctly when matching color is selected', async () => {
+  const onChange = jest.fn();
+
+  jest
+    .spyOn(require('@apache-superset/core/theme'), 'useTheme')
+    .mockReturnValue({
+      colors: {
+        colorSuccess: 'rgba(82, 196, 26, 1)',
+      },
+    });
+
+  render(
+    <ColorPickerControl
+      {...defaultProps}
+      onChange={onChange}
+      resolveThemeTokens
+      presets={[{ label: 'Theme Tokens', colors: ['colorSuccess'] }]}
+    />,
+  );
+
+  const colorPickerTrigger = document.querySelector(
+    '.ant-color-picker-trigger',
+  );
+  expect(colorPickerTrigger).toBeInTheDocument();
+  await userEvent.click(colorPickerTrigger!);
+
+  await waitFor(() => {
+    expect(
+      document.querySelector('.ant-color-picker-presets-items'),
+    ).toBeInTheDocument();
+  });
+
+  const successPreset = document.querySelector(
+    '.ant-color-picker-presets-color [style*="82, 196, 26"]',
+  ) as HTMLElement | null;
+
+  expect(successPreset).toBeInTheDocument();
+
+  await userEvent.click(successPreset!);
+
+  expect(onChange).toHaveBeenCalledWith('colorSuccess');
+});
+
+test('handles theme with nested colors object', () => {
+  jest
+    .spyOn(require('@apache-superset/core/theme'), 'useTheme')
+    .mockReturnValue({
+      colors: { primary: '#007bff' },
+    });
+
+  const { container } = render(<ColorPickerControl {...defaultProps} />);
+  expect(
+    container.querySelector('.ant-color-picker-trigger'),
+  ).toBeInTheDocument();
+});
+
+test('handles theme without colors field', () => {
+  jest
+    .spyOn(require('@apache-superset/core/theme'), 'useTheme')
+    .mockReturnValue({
+      primary: '#007bff',
+    });
+
+  const { container } = render(<ColorPickerControl {...defaultProps} />);
+  expect(
+    container.querySelector('.ant-color-picker-trigger'),
+  ).toBeInTheDocument();
+});
+
+test('handles undefined theme gracefully', () => {
+  jest
+    .spyOn(require('@apache-superset/core/theme'), 'useTheme')
+    .mockReturnValue(undefined);
+
+  expect(() => render(<ColorPickerControl {...defaultProps} />)).not.toThrow();
+});

superset-frontend/src/explore/components/controls/ColorPickerControl.tsx

@@ -25,7 +25,7 @@ import {
   type ColorValue,
 } from '@superset-ui/core/components';
 import ControlHeader from '../ControlHeader';
-import { useTheme } from '@apache-superset/core/theme';
+import { useTheme, type SupersetTheme } from '@apache-superset/core/theme';
 
 const SPECIAL_COLORS = {
   Red: { r: 150, g: 0, b: 0, a: 0.2 },
@@ -83,6 +83,24 @@ function toDisplayHex(
   return rgbaToHex(value).toLowerCase();
 }
 
+const extractThemeColors = (
+  theme: SupersetTheme | undefined | null,
+): Record<string, string> => {
+  if (!theme || typeof theme !== 'object') {
+    return {};
+  }
+
+  if (
+    'colors' in theme &&
+    typeof theme.colors === 'object' &&
+    theme.colors !== null
+  ) {
+    return theme.colors as Record<string, string>;
+  }
+
+  return theme as unknown as Record<string, string>;
+};
+
 export default function ColorPickerControl({
   onChange,
   value,
@@ -96,7 +114,7 @@ export default function ColorPickerControl({
   const theme = useTheme();
 
   const themeColors = useMemo<Record<string, string>>(
-    () => (theme as any)?.colors || theme || {},
+    () => extractThemeColors(theme),
     [theme],
   );