konform icon indicating copy to clipboard operation
konform copied to clipboard

Subclassing errors from ValidationError

Open floatdrop opened this issue 3 years ago • 1 comments
trafficstars

Sometimes we need to attach additional meta-data to validation results (#48) and perform custom l10n for message (#2). Both tasks fits nicely into custom ValidationError types (also it is quite handy to get all possible error types for validation).

I have something like this in mind:

data class UserProfile(
    val fullName: String,
    val age: Int
)

sealed class UserValidationError: ValidationError

class AgeRestrictionValidationError(
    override val dataPath: String,
    userAge: Int,
    requiredAge: Int
) : UserValidationError {
    override val message = "User must be at least $requiredAge years, but found $userAge"
}

fun ValidationBuilder<User, UserValidationError>.minimumAge(ageRequirement: Int) {
    // Pass currentPath from constraint?
    addConstrains(AgeRestrictionValidationError(currentPath, it.age, ageRequirement)) { it.age > ageRequirement}
}

// Add returned errors superclass to generic parameter
val validator = Validation<User, UserValidationError> {
    UserProfile::age {
        minimumAge(18)
    }
}

Maybe there is a cleaner way to achieve this in DSL (for example - avoid overloading ValidationBuilder with custom validations).

floatdrop avatar Aug 03 '22 15:08 floatdrop

I have working implementation of this here - https://gist.github.com/floatdrop/11de8d9de827f8ac11c1a1ae8260ccbd

It contains solution for this issue, #8, #29 and #48, but changes behaviour significantly (for example – all checks are now failing fast and running in order).

Downside of defining error type is more added code to constraints. But it could be sorted with specialized overloads for ValidationBuilder<T, String> that provides default message.

floatdrop avatar Aug 06 '22 02:08 floatdrop

Interesting, I will check out your changes to see if there is anything worthwhile

all checks are now failing fast This is a non-going. If you want fail-fast you should just use an init { require(...) { "..." } } block in your data class instead (which I should add to the README as an alternative to this lib). An important use case for this lib is to construct the data without validation yes and then collecting all validation errors in one go.

dhoepelman avatar Mar 29 '24 13:03 dhoepelman