Amends after code review

Author: dbssman

src/useFormHandler.ts

@@ -1,3 +1,4 @@
+import { DEFAULT_FIELD_VALUE } from './constants';
 import { ModifiedValues, TriggerValidation, Validations, FormHandler, ResetField, ResetForm, InitControl, SetError, ClearErrors, SetValue, ClearField, SetDirty, SetTouched, HandleBlur, HandleChange, GetInitValueForControl } from './types';
 import { FormState, HandleSubmit, Register } from './types';
 import { reactive, readonly, watch } from 'vue'
@@ -146,7 +147,7 @@ const useFormHandler: FormHandler = ({
       .filter(([name]) => formState.dirty[name]))
   }
 
-  const setValue: SetValue = async (name, value = '') => {
+  const setValue: SetValue = async (name, value = DEFAULT_FIELD_VALUE) => {
     if (!interceptor || await interceptor({ name, value, values, formState, clearErrors, modifiedValues, resetField, resetForm, setError, triggerValidation })) {
       values[name] = value
       setDirty(name, !isEqual(value, getInitValueForControl(name)))
@@ -158,14 +159,14 @@ const useFormHandler: FormHandler = ({
     triggerValidation(name)
   }
 
-  const handleChange: HandleChange = async (name, value = '') => {
+  const handleChange: HandleChange = async (name, value = DEFAULT_FIELD_VALUE) => {
     await setValue(name, value)
     setTouched(name, true)
     triggerValidation(name)
   }
 
   const clearField: ClearField = async (name) => {
-    await setValue(name, defaultValues[name] ?? '')
+    await setValue(name, getDefaultValueForControl(name))
   }
 
   const register: Register = (name, options = {}) => {

test/useFormHandler.test.ts

@@ -66,7 +66,7 @@ describe('Form handler testing', () => {
         await sleep()
         expect(formState.isValid).toBeFalsy()
         clearErrors('field')
-        expect(formState.errors.field).toStrictEqual({})
+        expect(formState.errors.field).toBeUndefined()
         await sleep()
         expect(formState.isValid).toBeTruthy()
     })
@@ -120,24 +120,20 @@ describe('Register function testing', () => {
         expect(field.isDirty).toBeUndefined()
         expect(field.isTouched).toBeUndefined()
         expect(field.onClear).toBeUndefined()
-        expect(field.ref).toBe(null)
         expect(field.onChange).toBeDefined()
         expect(field.modelValue).toBe(null)
         expect(field['onUpdate:modelValue']).toBeDefined()
         expect(values.field).toBe(null)
     })
-    it('Specified native field shouldn\'t have custom handlers', () => {
+    it('Specified native field should have native handlers', () => {
         const { register } = useFormHandler();
         const field = register('field', { native: true })
         expect(field.ref).toBeDefined()
         expect(field.onChange).toBeDefined()
-        expect(field.modelValue).toBeUndefined()
-        expect(field['onUpdate:modelValue']).toBeUndefined()
     })
     it('Specified custom field shouldn\'t have native handlers', () => {
         const { register } = useFormHandler();
         const field = register('field', { native: false })
-        expect(field.ref).toBeUndefined()
         expect(field.onChange).toBeUndefined()
         expect(field.modelValue).toBeDefined()
         expect(field['onUpdate:modelValue']).toBeDefined()