fix(validation): set input to invalid using error/errorMessages (#15269)

closes #15209

packages/vuetify/src/composables/__tests__/validation.spec.ts

@@ -109,4 +109,34 @@ describe('validation', () => {
     expect(wrapper.vm.isPristine).toBe(true)
     expect(wrapper.vm.isValid).toBeNull()
   })
+
+  it('should return valid if no rules are set', async () => {
+    const wrapper = mountFunction()
+
+    expect(wrapper.vm.isValid).toBe(true)
+
+    await wrapper.setProps({ rules: [] })
+
+    expect(wrapper.vm.isValid).toBe(true)
+
+    await wrapper.setProps({ error: true })
+
+    expect(wrapper.vm.isValid).toBe(false)
+  })
+
+  it('should return invalid if error is manually set', async () => {
+    const wrapper = mountFunction({
+      error: true,
+    })
+
+    expect(wrapper.vm.isValid).toBe(false)
+
+    await wrapper.setProps({ error: false, errorMessages: ['error'] })
+
+    expect(wrapper.vm.isValid).toBe(false)
+
+    await wrapper.setProps({ errorMessages: [] })
+
+    expect(wrapper.vm.isValid).toBe(true)
+  })
 })

packages/vuetify/src/composables/validation.ts

@@ -73,8 +73,8 @@ export function useValidation (
       : internalErrorMessages.value
   })
   const isValid = computed(() => {
-    if (!props.rules.length) return true
     if (props.error || errorMessages.value.length) return false
+    if (!props.rules.length) return true
 
     return isPristine.value ? null : true
   })