From 79188d7053222c7b1b5020523d577658935df43e Mon Sep 17 00:00:00 2001 From: yliuuuu <107505258+yliuuuu@users.noreply.github.com> Date: Thu, 11 Jan 2024 11:01:07 -0800 Subject: [PATCH] Include hints to find the failing tests in step summary. --- .../kotlin/org/partiql/runner/ConformanceComparison.kt | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/partiql-tests-runner/src/main/kotlin/org/partiql/runner/ConformanceComparison.kt b/test/partiql-tests-runner/src/main/kotlin/org/partiql/runner/ConformanceComparison.kt index 3d103ebe86..522c44848f 100644 --- a/test/partiql-tests-runner/src/main/kotlin/org/partiql/runner/ConformanceComparison.kt +++ b/test/partiql-tests-runner/src/main/kotlin/org/partiql/runner/ConformanceComparison.kt @@ -147,7 +147,7 @@ Number failing in Base (${first.commitId}) but now pass: ${failureFirstPassingSe this.appendLine("") } else { this.appendLine(":interrobang: CONFORMANCE REPORT REGRESSION DETECTED :interrobang:") - this.appendLine("Download Artifact from Summary to view the complete list") + this.appendLine("The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact.") } } @@ -162,7 +162,7 @@ Number failing in Base (${first.commitId}) but now pass: ${failureFirstPassingSe this.appendLine("") } else { this.appendLine("${failureFirstPassingSecond.size} test(s) were previously failing but now pass. Before merging, confirm they are intended to pass") - this.appendLine("Download Artifact from Summary to view the complete list") + this.appendLine("The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact.") } } } @@ -204,7 +204,7 @@ Number failing in ${first.engine} engine but pass in ${second.engine} engine: ${ this.appendLine("") } else { this.appendLine(":interrobang: CONFORMANCE REPORT REGRESSION DETECTED :interrobang:") - this.appendLine("Download Artifact from Summary to view the complete list") + this.appendLine("The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact.") } } @@ -219,7 +219,7 @@ Number failing in ${first.engine} engine but pass in ${second.engine} engine: ${ this.appendLine("") } else { this.appendLine("${failureFirstPassingSecond.size} test(s) were failing in ${first.engine} but now pass in ${second.engine}. Before merging, confirm they are intended to pass.") - this.appendLine("Download Artifact from Summary to view the complete list") + this.appendLine("The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact.") } } }