Skip to content

Commit 1e95b48

Browse files
fix(ui): rename error text token to text-text-error-primary (#10285)
1 parent 5a062b1 commit 1e95b48

11 files changed

+79
-11
lines changed

ui/components/providers/radio-group-provider.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ export const RadioGroupProvider: FC<RadioGroupProviderProps> = ({
176176
</div>
177177

178178
{errorMessage && (
179-
<FormMessage className="text-text-error">
179+
<FormMessage className="text-text-error-primary">
180180
{errorMessage}
181181
</FormMessage>
182182
)}

ui/components/providers/workflow/forms/fields/wizard-input-field.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ export const WizardInputField = <T extends FieldValues>({
163163
)}
164164
</div>
165165
</FormControl>
166-
<FormMessage className="text-text-error max-w-full text-xs" />
166+
<FormMessage className="text-text-error-primary max-w-full text-xs" />
167167
</div>
168168
);
169169
}}

ui/components/providers/workflow/forms/fields/wizard-textarea-field.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ export const WizardTextareaField = <T extends FieldValues>({
8787
{description}
8888
</p>
8989
)}
90-
<FormMessage className="text-text-error max-w-full text-xs" />
90+
<FormMessage className="text-text-error-primary max-w-full text-xs" />
9191
</div>
9292
);
9393
}}

ui/components/providers/workflow/forms/select-credentials-type/alibabacloud/radio-group-alibabacloud-via-credentials-type-form.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ export const RadioGroupAlibabaCloudViaCredentialsTypeForm = <
5050
</WizardRadioCard>
5151
</RadioGroup>
5252
{errorMessage && (
53-
<FormMessage className="text-text-error">
53+
<FormMessage className="text-text-error-primary">
5454
{errorMessage}
5555
</FormMessage>
5656
)}

ui/components/providers/workflow/forms/select-credentials-type/aws/radio-group-aws-via-credentials-type-form.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ export const RadioGroupAWSViaCredentialsTypeForm = ({
4343
</WizardRadioCard>
4444
</RadioGroup>
4545
{errorMessage && (
46-
<FormMessage className="text-text-error">
46+
<FormMessage className="text-text-error-primary">
4747
{errorMessage}
4848
</FormMessage>
4949
)}

ui/components/providers/workflow/forms/select-credentials-type/cloudflare/radio-group-cloudflare-via-credentials-type-form.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ export const RadioGroupCloudflareViaCredentialsTypeForm = ({
4444
</WizardRadioCard>
4545
</RadioGroup>
4646
{errorMessage && (
47-
<FormMessage className="text-text-error">
47+
<FormMessage className="text-text-error-primary">
4848
{errorMessage}
4949
</FormMessage>
5050
)}

ui/components/providers/workflow/forms/select-credentials-type/gcp/radio-group-gcp-via-credentials-type-form.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ export const RadioGroupGCPViaCredentialsTypeForm = ({
4747
</WizardRadioCard>
4848
</RadioGroup>
4949
{errorMessage && (
50-
<FormMessage className="text-text-error">
50+
<FormMessage className="text-text-error-primary">
5151
{errorMessage}
5252
</FormMessage>
5353
)}

ui/components/providers/workflow/forms/select-credentials-type/github/radio-group-github-via-credentials-type-form.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ export const RadioGroupGitHubViaCredentialsTypeForm = ({
5454
</WizardRadioCard>
5555
</RadioGroup>
5656
{errorMessage && (
57-
<FormMessage className="text-text-error">
57+
<FormMessage className="text-text-error-primary">
5858
{errorMessage}
5959
</FormMessage>
6060
)}

ui/components/providers/workflow/forms/select-credentials-type/m365/radio-group-m365-via-credentials-type-form.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ export const RadioGroupM365ViaCredentialsTypeForm = ({
4444
</WizardRadioCard>
4545
</RadioGroup>
4646
{errorMessage && (
47-
<FormMessage className="text-text-error">
47+
<FormMessage className="text-text-error-primary">
4848
{errorMessage}
4949
</FormMessage>
5050
)}
Lines changed: 65 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,65 @@
1+
import { render, screen } from "@testing-library/react";
2+
import { useEffect } from "react";
3+
import { useForm } from "react-hook-form";
4+
import { describe, expect, it } from "vitest";
5+
6+
import {
7+
Form,
8+
FormControl,
9+
FormField,
10+
FormItem,
11+
FormLabel,
12+
FormMessage,
13+
} from "./Form";
14+
15+
interface TestValues {
16+
providerUid: string;
17+
}
18+
19+
function TestFormWithError() {
20+
const form = useForm<TestValues>({
21+
defaultValues: {
22+
providerUid: "",
23+
},
24+
});
25+
26+
useEffect(() => {
27+
form.setError("providerUid", {
28+
type: "manual",
29+
message: "Provider ID is required",
30+
});
31+
}, [form]);
32+
33+
return (
34+
<Form {...form}>
35+
<FormField
36+
control={form.control}
37+
name="providerUid"
38+
render={({ field }) => (
39+
<FormItem>
40+
<FormLabel>Provider UID</FormLabel>
41+
<FormControl>
42+
<input {...field} />
43+
</FormControl>
44+
<FormMessage />
45+
</FormItem>
46+
)}
47+
/>
48+
</Form>
49+
);
50+
}
51+
52+
describe("Form", () => {
53+
it("should use the existing error text token for labels and messages", async () => {
54+
// Given
55+
render(<TestFormWithError />);
56+
57+
// When
58+
const label = await screen.findByText("Provider UID");
59+
const message = await screen.findByText("Provider ID is required");
60+
61+
// Then
62+
expect(label).toHaveClass("text-text-error-primary");
63+
expect(message).toHaveClass("text-text-error-primary");
64+
});
65+
});

0 commit comments

Comments
 (0)