refactor: throw error when vine.object is constructed without properties

Closes: #17
vinejs · Nov 30, 2024 · 8a2186b · 8a2186b
1 parent 715e761
commit 8a2186b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/src/schema/object/main.ts b/src/schema/object/main.ts
@@ -97,6 +97,11 @@ export class VineObject<
   }
 
   constructor(properties: Properties, options?: FieldOptions, validations?: Validation<any>[]) {
+    if (!properties) {
+      throw new Error(
+        'Missing properties for "vine.object". Use an empty object if you do not want to validate any specific fields'
+      )
+    }
     super(options, validations)
     this.#properties = properties
   }

diff --git a/tests/integration/schema/object.spec.ts b/tests/integration/schema/object.spec.ts
@@ -11,6 +11,16 @@ import { test } from '@japa/runner'
 import vine from '../../../index.js'
 
 test.group('VineObject | flat object', () => {
+  test('fail to construct schema when object is instantiated without object', async ({
+    assert,
+  }) => {
+    assert.throws(
+      // @ts-expect-error
+      () => vine.object(),
+      'Missing properties for "vine.object". Use an empty object if you do not want to validate any specific fields'
+    )
+  })
+
   test('fail when value is not an object', async ({ assert }) => {
     const schema = vine.object({
       username: vine.string(),