Skip to content

Commit a2dc814

Browse files
Siddhantclaude
authored andcommitted
chore(sso-ui): swap SSO form chrome to Untitled UI primitives
Replace antd Card/Button/Typography/Upload.Dragger in SSOConfigurationForm with @openmetadata/ui-core-components Button + FileTrigger; keep RJSF and its widgets untouched. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 4d1f848 commit a2dc814

3 files changed

Lines changed: 162 additions & 124 deletions

File tree

openmetadata-ui/src/main/resources/ui/src/components/SettingsSso/SSOConfigurationForm/SSOConfigurationForm.test.tsx

Lines changed: 30 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,33 @@ jest.mock('@openmetadata/ui-core-components', () => ({
8080
{children}
8181
</div>
8282
),
83+
Button: ({
84+
children,
85+
onPress,
86+
isDisabled,
87+
isLoading,
88+
iconLeading: _iconLeading,
89+
color: _color,
90+
size: _size,
91+
...rest
92+
}: React.PropsWithChildren<{
93+
onPress?: () => void;
94+
isDisabled?: boolean;
95+
isLoading?: boolean;
96+
iconLeading?: unknown;
97+
color?: string;
98+
size?: string;
99+
}> &
100+
Record<string, unknown>) => (
101+
<button
102+
disabled={isDisabled || isLoading}
103+
type="button"
104+
{...rest}
105+
onClick={onPress}>
106+
{children}
107+
</button>
108+
),
109+
FileTrigger: ({ children }: { children: React.ReactNode }) => <>{children}</>,
83110
}));
84111

85112
// Mock SSOUtils - use actual implementations where needed
@@ -2080,9 +2107,9 @@ describe('SSOConfigurationForm', () => {
20802107
).toBeInTheDocument();
20812108
});
20822109

2083-
expect(
2084-
screen.getAllByTestId('sso-advanced-fields-toggle')
2085-
).toHaveLength(1);
2110+
expect(screen.getAllByTestId('sso-advanced-fields-toggle')).toHaveLength(
2111+
1
2112+
);
20862113
expect(
20872114
screen.getByTestId('sso-advanced-fields-panel')
20882115
).toBeInTheDocument();

0 commit comments

Comments
 (0)