fraqtal/agentic-dev
Template
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

fix(core-shared): narrow Payload Field union before accessing name in tests

Browse Source 
Payload's Field type is a discriminated union; field.name only exists on
named variants. Tests now narrow on type (and 'name' in field for groups)
before reading variant-specific properties.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Danijel Martinek
2026-05-04 20:43:23 +02:00
parent 428346a36d
commit 884b0c01e8
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/core-shared/src/payload/fields/seo-fields.test.ts
View File

@@ -3,6 +3,9 @@ import { seoFields } from "./seo-fields";
describe("seoFields", () => { describe("seoFields", () => {
it("is a group field named 'seo'", () => { it("is a group field named 'seo'", () => {
if (seoFields.type !== "group" || !("name" in seoFields)) {
throw new Error("seoFields must be a named group");
}
expect(seoFields.name).toBe("seo"); expect(seoFields.name).toBe("seo");
expect(seoFields.type).toBe("group"); expect(seoFields.type).toBe("group");
}); });

3
packages/core-shared/src/payload/fields/slug-field.test.ts
View File

@@ -4,8 +4,8 @@ import { slugField } from "./slug-field";
describe("slugField", () => { describe("slugField", () => {
it("returns a Payload Field with default name 'slug'", () => { it("returns a Payload Field with default name 'slug'", () => {
const field = slugField(); const field = slugField();
if (field.type !== "text") throw new Error("expected text field");
expect(field.name).toBe("slug"); expect(field.name).toBe("slug");
expect(field.type).toBe("text");
expect(field.required).toBe(true); expect(field.required).toBe(true);
expect(field.unique).toBe(true); expect(field.unique).toBe(true);
expect(field.index).toBe(true); expect(field.index).toBe(true);
@@ -13,6 +13,7 @@ describe("slugField", () => {
it("accepts a custom field name", () => { it("accepts a custom field name", () => {
const field = slugField("permalink"); const field = slugField("permalink");
if (field.type !== "text") throw new Error("expected text field");
expect(field.name).toBe("permalink"); expect(field.name).toBe("permalink");
}); });
}); });