[RMBWEB-2780] Support for structured validation results

dumi/docs/concepts/state/index.md

@@ -29,7 +29,7 @@ export interface IState<V> {
   error: ValidationError
   /** The state's own error info, regardless of child states. */
   ownError: ValidationError
-  /** The state's validation result, is the same as the return value of the validator, regardless of child states. */
+  /** The state's validation result, regardless of child states. */
   rawError: ValidationResult
   /** Append validator(s). */
   withValidator(...validators: Array<Validator<V>>): this
@@ -101,7 +101,7 @@ States will not be auto-validated until it becomes **activated**. And they will
 
 ### Raw Error
 
-The state's validation result, is the same as the return value of the `validator`, regardless of child states. The difference compared to `ownError` is that it contains the type of `ValidationErrorObject`. You can check details about them in issue [#82](https://github.com/qiniu/formstate-x/issues/82).
+The state's validation result, regardless of child states. The difference compared to `ownError` is that it contains the type of `ValidationErrorObject`. You can check details about them in issue [#82](https://github.com/qiniu/formstate-x/issues/82).
 
 ### Disable State
 

src/adapter/v2.spec.ts

@@ -703,7 +703,6 @@ describe('toV2', () => {
       }
       value: V
       touched = false
-      ownError = undefined
       error = undefined
       rawError = undefined
       activated = false

src/adapter/v2.ts

@@ -1,4 +1,4 @@
-import { action, computed, makeObservable } from 'mobx'
+import { override, action, computed, makeObservable } from 'mobx'
 import * as v2 from 'formstate-x-v2'
 import { BaseState } from '../state'
 import * as v3 from '..'
@@ -25,7 +25,7 @@ class Upgrader<T extends v2.ComposibleValidatable<unknown, V>, V> extends BaseSt
 
   @computed get value() { return this.stateV2.value }
   @computed get touched() { return this.stateV2.dirty }
-  @computed get ownError() {
+  @override override get ownError() {
     return getV3OwnError(this.stateV2)
   }
   @computed get rawError() { return this.ownError }

src/state.ts

@@ -1,20 +1,22 @@
 import { action, autorun, computed, makeObservable, observable, when } from 'mobx'
-import { ValidationResult, ValidationError, IState, Validation, ValidateResult, ValidateStatus, Validator } from './types'
+import { ValidationResult, IState, Validation, ValidateResult, ValidateStatus, Validator } from './types'
 import Disposable from './disposable'
 import { applyValidators, isPromiseLike, isPassed, normalizeError } from './utils'
 
 /** Extraction for some basic features of State */
 export abstract class BaseState extends Disposable implements Pick<
   IState, 'rawError' | 'ownError' | 'hasOwnError' | 'hasError' | 'validateStatus' | 'validating' | 'validated'
-  > {
+> {
 
   abstract rawError: ValidationResult
 
   @computed get hasError() {
     return !isPassed(this.rawError)
   }
 
-  abstract ownError: ValidationError
+  @computed get ownError() {
+    return normalizeError(this.rawError)
+  }
 
   @computed get hasOwnError() {
     return !!this.ownError
@@ -56,14 +58,10 @@ export abstract class ValidatableState<V> extends BaseState implements IState<V>
   /**
    * The original return value of validation.
    */
-  @observable private _error: ValidationResult
+  @observable private validationResult: ValidationResult
 
   @computed get rawError() {
-    return this.disabled ? undefined : this._error
-  }
-
-  @computed get ownError() {
-    return normalizeError(this.rawError)
+    return this.disabled ? undefined : this.validationResult
   }
 
   @computed get error() {
@@ -74,7 +72,7 @@ export abstract class ValidatableState<V> extends BaseState implements IState<V>
    * Set validation result.
    */
   @action setError(error: ValidationResult) {
-    this._error = error
+    this.validationResult = error
   }
 
   /** List of validator functions. */
@@ -167,7 +165,7 @@ export abstract class ValidatableState<V> extends BaseState implements IState<V>
   @action reset() {
     this.activated = false
     this._validateStatus = ValidateStatus.NotValidated
-    this._error = undefined
+    this.validationResult = undefined
     this.validation = undefined
   }
 

src/transformedState.ts

@@ -1,4 +1,4 @@
-import { computed, override } from 'mobx'
+import { override, computed } from 'mobx'
 import { BaseState } from './state'
 import { IState, Validator, ValueOf } from './types'
 
@@ -23,10 +23,6 @@ export class TransformedState<S extends IState<$V>, V, $V = ValueOf<S>> extends
     return this.parseOriginalValue(this.$.value)
   }
 
-  @computed get ownError() {
-    return this.$.ownError
-  }
-
   @computed get rawError() {
     return this.$.rawError
   }

src/types.ts

@@ -40,7 +40,7 @@ export interface IState<V = unknown> {
   hasError: boolean
   /** The state's own error info, regardless of child states. */
   ownError: ValidationError
-  /** The state's validation result, is the same as the return value of the validator, regardless of child states. */
+  /** The state's validation result, regardless of child states. */
   rawError: ValidationResult
   /** If the state contains its own error info. */
   hasOwnError: boolean

src/utils.spec.ts

@@ -97,6 +97,7 @@ describe('isErrorObject', () => {
     expect(isErrorObject('foo')).toBe(false)
     expect(isErrorObject({})).toBe(false)
     expect(isErrorObject({ foo: 'foo' })).toBe(false)
+    expect(isErrorObject({ message: '' })).toBe(true)
     expect(isErrorObject({ message: 'msg' })).toBe(true)
     expect(isErrorObject({ message: 'msg', extra: 'ext' })).toBe(true)
     expect(isErrorObject(new Error('error msg'))).toBe(true)
@@ -106,7 +107,6 @@ describe('isErrorObject', () => {
 
     class Bar extends Foo {}
     expect(isErrorObject(new Bar())).toBe(true)
-    expect(() => isErrorObject({ message: '' })).toThrow(inValidErrorObjectMsg)
   })
 })
 
@@ -115,6 +115,7 @@ describe('normalizeValidationResult', () => {
     expect(normalizeError(undefined)).toBe(undefined)
     expect(normalizeError('')).toBe(undefined)
     expect(normalizeError('foo')).toBe('foo')
+    expect(() => normalizeError({ message: '' })).toThrow(inValidErrorObjectMsg)
     expect(normalizeError({ message: 'mewo' })).toBe('mewo')
     expect(normalizeError(Error('mewo2'))).toBe('mewo2')
 

src/utils.ts

@@ -1,9 +1,14 @@
 import { isObservableArray, IObservableArray } from 'mobx'
 import { Validator, ValidatorReturned, ValidationError, ValidationErrorObject, ValidationResult } from './types'
 
+export const inValidErrorObjectMsg = 'ValidationErrorObject message property cannot be empty'
+
 // ValidationResult -> ValidationError
 export function normalizeError(result: ValidationResult): ValidationError {
   if (isErrorObject(result)) {
+    if (!result.message) {
+      throw new Error(inValidErrorObjectMsg)
+    }
     return result.message
   }
 
@@ -14,13 +19,8 @@ export function normalizeError(result: ValidationResult): ValidationError {
   return result
 }
 
-export const inValidErrorObjectMsg = 'ValidationErrorObject message property cannot be empty'
-
 export function isErrorObject(err: any): err is ValidationErrorObject {
   if (err != null && typeof err === 'object' && 'message' in err) {
-    if (!err.message) {
-      throw new Error('ValidationErrorObject message property cannot be empty')
-    }
     return true
   }
   return false