From 774f3c27c8b6c18b411ff0ad4513576fe2accf49 Mon Sep 17 00:00:00 2001 From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com> Date: Tue, 21 Jan 2025 16:32:42 -0500 Subject: [PATCH] Bump build.buf:protovalidate from 0.4.2 to 0.5.0 (#329) * Bump build.buf:protovalidate from 0.4.2 to 0.5.0 Bumps [build.buf:protovalidate](https://github.com/bufbuild/protovalidate-java) from 0.4.2 to 0.5.0. - [Release notes](https://github.com/bufbuild/protovalidate-java/releases) - [Commits](https://github.com/bufbuild/protovalidate-java/compare/v0.4.2...v0.5.0) --- updated-dependencies: - dependency-name: build.buf:protovalidate dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] * fix --------- Signed-off-by: dependabot[bot] Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Andrew Parmet --- gradle/libs.versions.toml | 2 +- .../main/kotlin/protokt/v1/buf/validate/Validator.kt | 11 +++++++++-- .../protokt/v1/buf/validate/conformance/Main.kt | 6 +++++- 3 files changed, 15 insertions(+), 4 deletions(-) diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml index d0a6402e3..3f3310bc0 100644 --- a/gradle/libs.versions.toml +++ b/gradle/libs.versions.toml @@ -25,7 +25,7 @@ protobuf-java = "4.29.1" protobuf-js = "7.4.0" protobufGradlePlugin = "0.9.4" protovalidate = "0.9.0" -protovalidateJava = "0.4.2" +protovalidateJava = "0.5.0" slf4j = "2.0.16" # build diff --git a/protokt-protovalidate/src/main/kotlin/protokt/v1/buf/validate/Validator.kt b/protokt-protovalidate/src/main/kotlin/protokt/v1/buf/validate/Validator.kt index 774939ea7..056abad1e 100644 --- a/protokt-protovalidate/src/main/kotlin/protokt/v1/buf/validate/Validator.kt +++ b/protokt-protovalidate/src/main/kotlin/protokt/v1/buf/validate/Validator.kt @@ -58,8 +58,15 @@ class Validator @JvmOverloads constructor( descriptor.nestedTypes.forEach(::doLoad) } - fun validate(message: Message): ValidationResult = - evaluatorsByFullTypeName + fun validate(message: Message): ValidationResult { + val result = evaluatorsByFullTypeName .getValue(message::class.findAnnotation()!!.fullTypeName) .evaluate(MessageValue(message.toDynamicMessage(runtimeContext)), failFast) + + return if (result.isEmpty()) { + ValidationResult.EMPTY + } else { + ValidationResult(result.map { it.build() }) + } + } } diff --git a/testing/protovalidate-conformance/src/main/kotlin/protokt/v1/buf/validate/conformance/Main.kt b/testing/protovalidate-conformance/src/main/kotlin/protokt/v1/buf/validate/conformance/Main.kt index 9665cfcc8..b613ad8a2 100644 --- a/testing/protovalidate-conformance/src/main/kotlin/protokt/v1/buf/validate/conformance/Main.kt +++ b/testing/protovalidate-conformance/src/main/kotlin/protokt/v1/buf/validate/conformance/Main.kt @@ -94,7 +94,11 @@ object Main { TestResult.newBuilder().setSuccess(true).build() } else { TestResult.newBuilder() - .setValidationError(Violations.newBuilder().addAllViolations(result.violations).build()) + .setValidationError( + Violations.newBuilder() + .addAllViolations(result.toProto().violationsList) + .build() + ) .build() } } catch (e: ExecutionException) {