Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
61 changes: 61 additions & 0 deletions packages/core/components/AutoField/index.spec.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
import { fireEvent, render, screen } from "@testing-library/react";
import { appStoreContext, createAppStore } from "../../store";
import { fieldContextStore } from "./store";
import { AutoFieldPrivate } from ".";

jest.mock("../Sortable", () => ({
Sortable: ({
children,
}: {
children: (props: {
isDragging: boolean;
ref: null;
handleRef: null;
}) => JSX.Element;
}) => children({ isDragging: false, ref: null, handleRef: null }),
SortableProvider: ({ children }: { children: JSX.Element }) => children,
}));

jest.mock("../../bundle", () => {
const { setDeep } = jest.requireActual("../../lib/data/set-deep");

return { setDeep };
});

describe("AutoField", () => {
it("updates controlled custom fields immediately", () => {
const appStore = createAppStore();
const onChange = jest.fn();

render(
<appStoreContext.Provider value={appStore}>
<fieldContextStore.Provider value={{ title: "Hello world" }}>
<AutoFieldPrivate
field={{
type: "custom",
render: ({ id, value, onChange }) => (
<input
id={id}
value={value ?? ""}
onChange={(e) => onChange(e.currentTarget.value)}
/>
),
}}
id="title"
name="title"
onChange={onChange}
/>
</fieldContextStore.Provider>
</appStoreContext.Provider>
);

fireEvent.change(screen.getByRole("textbox"), {
target: { value: "Hello brave world" },
});

expect(onChange).toHaveBeenCalledWith("Hello brave world", undefined);
expect((screen.getByRole("textbox") as HTMLInputElement).value).toBe(
"Hello brave world"
);
});
});
23 changes: 21 additions & 2 deletions packages/core/components/AutoField/index.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import getClassNameFactory from "../../lib/get-class-name-factory";
import { Field, FieldProps } from "../../types";
import { Field, FieldProps, UiState } from "../../types";

import styles from "./styles.module.css";
import {
Expand All @@ -25,6 +25,7 @@ import { useSafeId } from "../../lib/use-safe-id";
import { NestedFieldContext } from "./context";
import { useShallow } from "zustand/react/shallow";
import { getDeep } from "../../lib/data/get-deep";
import { setDeep } from "../../lib/data/set-deep";
import type {
FieldLabelPropsInternal,
FieldPropsInternalOptional,
Expand Down Expand Up @@ -78,6 +79,7 @@ function AutoFieldInternal<
const field = props.field as Field<ValueType>;
const label = field.label;
const labelIcon = field.labelIcon;
const fieldStore = useFieldStoreApi();

const defaultId = useSafeId();
const resolvedId = id || defaultId;
Expand Down Expand Up @@ -106,6 +108,22 @@ function AutoFieldInternal<
}
});

const shouldSyncLocalValue =
field.type === "custom" || !!overrides.fieldTypes?.[field.type];

const onChange = useCallback(
(value: any, uiState?: Partial<UiState>) => {
if (shouldSyncLocalValue) {
fieldStore.setState(
setDeep(fieldStore.getState(), props.name ?? resolvedId, value)
);
}

props.onChange(value, uiState);
},
[fieldStore, props.name, props.onChange, resolvedId, shouldSyncLocalValue]
);

const mergedProps = useMemo(
() => ({
...props,
Expand All @@ -115,8 +133,9 @@ function AutoFieldInternal<
Label,
id: resolvedId,
value: fieldValue,
onChange,
}),
[props, field, label, labelIcon, Label, resolvedId, fieldValue]
[props, field, label, labelIcon, Label, resolvedId, fieldValue, onChange]
);

const onFocus = useCallback(
Expand Down