Fix SARIF validation failure (#3507)

detekt · Mar 1, 2021 · 7cf1ab3 · 7cf1ab3
1 parent c90eafe
commit 7cf1ab3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/detekt-report-sarif/src/main/kotlin/io/github/detekt/report/sarif/SarifOutputReport.kt b/detekt-report-sarif/src/main/kotlin/io/github/detekt/report/sarif/SarifOutputReport.kt
@@ -36,7 +36,12 @@ class SarifOutputReport : OutputReport() {
     @OptIn(UnstableApi::class)
     override fun init(context: SetupContext) {
         this.config = context.config
-        this.basePath = context.getOrNull<Path>(DETEKT_OUTPUT_REPORT_BASE_PATH_KEY)?.toAbsolutePath()?.toUnifiedString()
+        this.basePath = context.getOrNull<Path>(DETEKT_OUTPUT_REPORT_BASE_PATH_KEY)
+            ?.toAbsolutePath()
+            ?.toUnifiedString()
+            ?.let {
+                if (!it.endsWith("/")) "$it/" else it
+            }
     }
 
     override fun render(detektion: Detektion): String {

diff --git a/detekt-report-sarif/src/test/resources/relative_path.sarif.json b/detekt-report-sarif/src/test/resources/relative_path.sarif.json
@@ -19,7 +19,7 @@
       },
       "originalUriBaseIds": {
         "%SRCROOT%": {
-          "uri": "file:///Users/tester/detekt"
+          "uri": "file:///Users/tester/detekt/"
         }
       },
       "results": [