Skip to content

Commit 36b9eae

Browse files
Copilothotlong
andcommitted
Fix code review issues: import path and validation assertion
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
1 parent ff9e1a8 commit 36b9eae

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

packages/spec/src/data/field-name-validation.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { describe, it, expect } from 'vitest';
2-
import { FieldSchema } from './src/data/field.zod';
2+
import { FieldSchema } from './field.zod';
33

44
describe('FieldSchema name validation', () => {
55
it('should reject camelCase name when provided', () => {

packages/spec/src/stack.test.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -427,8 +427,9 @@ describe('defineStack', () => {
427427
it('should validate config in default mode (strict by default)', () => {
428428
const config = { manifest: baseManifest, objects: [] };
429429
const result = defineStack(config);
430-
// Default is now strict=true, so result is validated and might be a different object reference
431-
expect(result).toEqual(config);
430+
// Default is now strict=true, so result is validated and is a different object reference
431+
expect(result).not.toBe(config); // Validation creates new object
432+
expect(result).toEqual(config); // But content is the same
432433
expect(result.manifest).toBeDefined();
433434
});
434435

0 commit comments

Comments
 (0)