Skip to content

Commit 542a563

Browse files
BrunoQuaresmaBrunoQuaresma
committed
fix(site): Fix template icon field validation
1 parent 2ea438c commit 542a563

File tree

19 files changed

+89
-78
lines changed

19 files changed

+89
-78
lines changed

‎site/src/components/AlertBanner/AlertBanner.stories.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import { Story } from "@storybook/react"
22
import { AlertBanner } from "./AlertBanner"
33
import Button from "@material-ui/core/Button"
4-
import { makeMockApiError } from "testHelpers/entities"
4+
import { makeMockValidationApiError } from "testHelpers/entities"
55
import { AlertBannerProps } from "./alertTypes"
66
import Link from "@material-ui/core/Link"
77

@@ -16,7 +16,7 @@ const ExampleAction = (
1616
</Button>
1717
)
1818

19-
const mockError = makeMockApiError({
19+
const mockError = makeMockValidationApiError({
2020
message: "Email or password was invalid",
2121
detail: "Password is invalid",
2222
})

‎site/src/components/SignInForm/SignInForm.stories.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { Story } from "@storybook/react"
2-
import { makeMockApiError } from "testHelpers/entities"
2+
import { makeMockValidationApiError } from "testHelpers/entities"
33
import { SignInForm, SignInFormProps } from "./SignInForm"
44

55
export default {
@@ -37,7 +37,7 @@ SigningIn.args = {
3737
export const WithError = Template.bind({})
3838
WithError.args = {
3939
...SignedOut.args,
40-
error: makeMockApiError({
40+
error: makeMockValidationApiError({
4141
message: "Email or password was invalid",
4242
validations: [
4343
{

‎site/src/components/Workspace/Workspace.stories.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ Failed.args = {
9595
...Running.args,
9696
workspace: Mocks.MockFailedWorkspace,
9797
workspaceErrors: {
98-
[WorkspaceErrors.BUILD_ERROR]: Mocks.makeMockApiError({
98+
[WorkspaceErrors.BUILD_ERROR]: Mocks.makeMockValidationApiError({
9999
message: "A workspace build is already active.",
100100
}),
101101
},
@@ -152,7 +152,7 @@ export const GetBuildsError = Template.bind({})
152152
GetBuildsError.args = {
153153
...Running.args,
154154
workspaceErrors: {
155-
[WorkspaceErrors.GET_BUILDS_ERROR]: Mocks.makeMockApiError({
155+
[WorkspaceErrors.GET_BUILDS_ERROR]: Mocks.makeMockValidationApiError({
156156
message: "There is a problem fetching builds.",
157157
}),
158158
},
@@ -162,7 +162,7 @@ export const CancellationError = Template.bind({})
162162
CancellationError.args = {
163163
...Failed.args,
164164
workspaceErrors: {
165-
[WorkspaceErrors.CANCELLATION_ERROR]: Mocks.makeMockApiError({
165+
[WorkspaceErrors.CANCELLATION_ERROR]: Mocks.makeMockValidationApiError({
166166
message: "Job could not be canceled.",
167167
}),
168168
},

‎site/src/components/WorkspaceScheduleForm/WorkspaceScheduleForm.stories.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import {
88
emptySchedule,
99
} from "pages/WorkspaceSettingsPage/WorkspaceSchedulePage/schedule"
1010
import { emptyTTL } from "pages/WorkspaceSettingsPage/WorkspaceSchedulePage/ttl"
11-
import { makeMockApiError } from "testHelpers/entities"
11+
import { makeMockValidationApiError } from "testHelpers/entities"
1212
import {
1313
WorkspaceScheduleForm,
1414
WorkspaceScheduleFormProps,
@@ -81,7 +81,7 @@ export const WithError = Template.bind({})
8181
WithError.args = {
8282
initialValues: { ...defaultInitialValues, ttl: 100 },
8383
initialTouched: { ttl: true },
84-
submitScheduleError: makeMockApiError({
84+
submitScheduleError: makeMockValidationApiError({
8585
message: "Something went wrong.",
8686
validations: [{ field: "ttl_ms", detail: "Invalid time until shutdown." }],
8787
}),

‎site/src/pages/CreateWorkspacePage/CreateWorkspacePageView.stories.tsx

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { ComponentMeta, Story } from "@storybook/react"
22
import {
3-
makeMockApiError,
3+
makeMockValidationApiError,
44
mockParameterSchema,
55
MockParameterSchemas,
66
MockTemplate,
@@ -85,7 +85,7 @@ export const GetTemplatesError = Template.bind({})
8585
GetTemplatesError.args = {
8686
...Parameters.args,
8787
createWorkspaceErrors: {
88-
[CreateWorkspaceErrors.GET_TEMPLATES_ERROR]: makeMockApiError({
88+
[CreateWorkspaceErrors.GET_TEMPLATES_ERROR]: makeMockValidationApiError({
8989
message: "Failed to fetch templates.",
9090
detail: "You do not have permission to access this resource.",
9191
}),
@@ -97,10 +97,11 @@ export const GetTemplateSchemaError = Template.bind({})
9797
GetTemplateSchemaError.args = {
9898
...Parameters.args,
9999
createWorkspaceErrors: {
100-
[CreateWorkspaceErrors.GET_TEMPLATE_SCHEMA_ERROR]: makeMockApiError({
101-
message: 'Failed to fetch template schema for "docker-amd64".',
102-
detail: "You do not have permission to access this resource.",
103-
}),
100+
[CreateWorkspaceErrors.GET_TEMPLATE_SCHEMA_ERROR]:
101+
makeMockValidationApiError({
102+
message: 'Failed to fetch template schema for "docker-amd64".',
103+
detail: "You do not have permission to access this resource.",
104+
}),
104105
},
105106
hasTemplateErrors: true,
106107
}
@@ -109,7 +110,7 @@ export const CreateWorkspaceError = Template.bind({})
109110
CreateWorkspaceError.args = {
110111
...Parameters.args,
111112
createWorkspaceErrors: {
112-
[CreateWorkspaceErrors.CREATE_WORKSPACE_ERROR]: makeMockApiError({
113+
[CreateWorkspaceErrors.CREATE_WORKSPACE_ERROR]: makeMockValidationApiError({
113114
message:
114115
'Workspace "test" already exists in the "docker-amd64" template.',
115116
validations: [

‎site/src/pages/DeploySettingsPage/GeneralSettingsPage/GeneralSettingsPageView.stories.tsx

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { ComponentMeta, Story } from "@storybook/react"
22
import {
3-
makeMockApiError,
3+
makeMockValidationApiError,
44
MockDeploymentDAUResponse,
55
} from "testHelpers/entities"
66
import {
@@ -43,5 +43,7 @@ NoDAUs.args = {
4343
export const DAUError = Template.bind({})
4444
DAUError.args = {
4545
deploymentDAUs: undefined,
46-
getDeploymentDAUsError: makeMockApiError({ message: "Error fetching DAUs." }),
46+
getDeploymentDAUsError: makeMockValidationApiError({
47+
message: "Error fetching DAUs.",
48+
}),
4749
}

‎site/src/pages/StarterTemplatePage/StarterTemplatePageView.stories.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { Story } from "@storybook/react"
22
import {
3-
makeMockApiError,
3+
makeMockValidationApiError,
44
MockOrganization,
55
MockTemplateExample,
66
} from "testHelpers/entities"
@@ -33,7 +33,7 @@ Error.args = {
3333
context: {
3434
exampleId: MockTemplateExample.id,
3535
organizationId: MockOrganization.id,
36-
error: makeMockApiError({
36+
error: makeMockValidationApiError({
3737
message: `Example ${MockTemplateExample.id} not found.`,
3838
}),
3939
starterTemplate: undefined,

‎site/src/pages/StarterTemplatesPage/StarterTemplatesPageView.stories.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { Story } from "@storybook/react"
22
import {
3-
makeMockApiError,
3+
makeMockValidationApiError,
44
MockOrganization,
55
MockTemplateExample,
66
MockTemplateExample2,
@@ -36,7 +36,7 @@ export const Error = Template.bind({})
3636
Error.args = {
3737
context: {
3838
organizationId: MockOrganization.id,
39-
error: makeMockApiError({
39+
error: makeMockValidationApiError({
4040
message: "Error on loading the template examples",
4141
}),
4242
starterTemplatesByTag: undefined,

‎site/src/pages/TemplateSettingsPage/TemplateGeneralSettingsPage/TemplateSettingsForm.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
nameValidator,
88
templateDisplayNameValidator,
99
onChangeTrimmed,
10+
iconValidator,
1011
} from "utils/formUtils"
1112
import * as Yup from "yup"
1213
import i18next from "i18next"
@@ -37,8 +38,8 @@ export const getValidationSchema = (): Yup.AnyObjectSchema =>
3738
MAX_DESCRIPTION_CHAR_LIMIT,
3839
i18next.t("descriptionMaxError", { ns: "templateSettingsPage" }),
3940
),
40-
4141
allow_user_cancel_workspace_jobs: Yup.boolean(),
42+
icon: iconValidator,
4243
})
4344

4445
export interface TemplateSettingsForm {
@@ -74,7 +75,7 @@ export const TemplateSettingsForm: FC<TemplateSettingsForm> = ({
7475
onSubmit,
7576
initialTouched,
7677
})
77-
const getFieldHelpers = getFormHelpers<UpdateTemplateMeta>(form, error)
78+
const getFieldHelpers = getFormHelpers(form, error)
7879
const { t } = useTranslation("templateSettingsPage")
7980
const styles = useStyles()
8081

‎site/src/pages/TemplateSettingsPage/TemplateGeneralSettingsPage/TemplateSettingsPageView.stories.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { action } from "@storybook/addon-actions"
22
import { Story } from "@storybook/react"
3-
import { makeMockApiError, MockTemplate } from "testHelpers/entities"
3+
import { makeMockValidationApiError, MockTemplate } from "testHelpers/entities"
44
import {
55
TemplateSettingsPageView,
66
TemplateSettingsPageViewProps,
@@ -25,7 +25,7 @@ Example.args = {}
2525

2626
export const SaveTemplateSettingsError = Template.bind({})
2727
SaveTemplateSettingsError.args = {
28-
submitError: makeMockApiError({
28+
submitError: makeMockValidationApiError({
2929
message: 'Template "test" already exists.',
3030
validations: [
3131
{

0 commit comments

Comments
 (0)