refactor(evaluator)!: Turn licenseSource of violations to a Set

Prepare for an upcoming change.

Signed-off-by: Frank Viernau <frank.viernau@gmail.com>

Author: fviernau

evaluator/src/main/kotlin/Rule.kt

@@ -139,7 +139,7 @@ abstract class Rule(
             rule = name,
             pkg = pkgId,
             license = license,
-            licenseSource = licenseSource,
+            licenseSources = setOfNotNull(licenseSource),
             message = message,
             howToFix = howToFix
         )

evaluator/src/test/kotlin/EvaluatorTest.kt

@@ -102,7 +102,7 @@ class EvaluatorTest : WordSpec({
                 rule shouldBe "rule 1"
                 pkg shouldBe Identifier("type:namespace:name:1.0")
                 license shouldBe "license-1".toSpdx()
-                licenseSource shouldBe LicenseSource.DETECTED
+                licenseSources should containExactlyInAnyOrder(LicenseSource.DETECTED)
                 severity shouldBe Severity.ERROR
                 message shouldBe "message 1"
                 howToFix shouldBe "how to fix 1"
@@ -112,7 +112,7 @@ class EvaluatorTest : WordSpec({
                 rule shouldBe "rule 2"
                 pkg shouldBe Identifier("type:namespace:name:2.0")
                 license shouldBe "license-2".toSpdx()
-                licenseSource shouldBe LicenseSource.DECLARED
+                licenseSources should containExactlyInAnyOrder(LicenseSource.DECLARED)
                 severity shouldBe Severity.WARNING
                 message shouldBe "message 2"
                 howToFix shouldBe "how to fix 2"

evaluator/src/test/kotlin/RuleTest.kt

@@ -20,6 +20,7 @@
 package org.ossreviewtoolkit.evaluator
 
 import io.kotest.core.spec.style.WordSpec
+import io.kotest.matchers.collections.containExactlyInAnyOrder
 import io.kotest.matchers.collections.haveSize
 import io.kotest.matchers.should
 import io.kotest.matchers.shouldBe
@@ -55,7 +56,7 @@ class RuleTest : WordSpec() {
                     violation.rule shouldBe rule.name
                     violation.pkg shouldBe id
                     violation.license shouldBe license
-                    violation.licenseSource shouldBe licenseSource
+                    violation.licenseSources should containExactlyInAnyOrder(licenseSource)
                     violation.severity shouldBe Severity.HINT
                     violation.message shouldBe message
                     violation.howToFix shouldBe howToFix
@@ -74,7 +75,7 @@ class RuleTest : WordSpec() {
                     violation.rule shouldBe rule.name
                     violation.pkg shouldBe id
                     violation.license shouldBe license
-                    violation.licenseSource shouldBe licenseSource
+                    violation.licenseSources should containExactlyInAnyOrder(licenseSource)
                     violation.severity shouldBe Severity.WARNING
                     violation.message shouldBe message
                     violation.howToFix shouldBe howToFix
@@ -93,7 +94,7 @@ class RuleTest : WordSpec() {
                     violation.rule shouldBe rule.name
                     violation.pkg shouldBe id
                     violation.license shouldBe license
-                    violation.licenseSource shouldBe licenseSource
+                    violation.licenseSources should containExactlyInAnyOrder(licenseSource)
                     violation.severity shouldBe Severity.ERROR
                     violation.message shouldBe message
                     violation.howToFix shouldBe howToFix

model/src/main/kotlin/RuleViolation.kt

@@ -38,9 +38,9 @@ data class RuleViolation(
     val license: SpdxSingleLicenseExpression?,
 
     /**
-     * The [source][LicenseSource] of the [license]. Can be null if the rule does not work on licenses.
+     * The [license sources][licenseSources] of the [license]. Can be empty if the rule does not work on licenses.
      */
-    val licenseSource: LicenseSource?,
+    val licenseSources: Set<LicenseSource>,
 
     /**
      * The severity of the rule violation.

model/src/test/kotlin/OrtResultTest.kt

@@ -512,7 +512,7 @@ class OrtResultTest : WordSpec({
                             rule = "rule id",
                             pkg = Identifier("Maven", "org.ossreviewtoolkit", "resolved-violation", "0.8.15"),
                             license = null,
-                            licenseSource = null,
+                            licenseSources = emptySet(),
                             severity = Severity.HINT,
                             message = "Rule violation message to resolve",
                             howToFix = ""
@@ -534,7 +534,7 @@ class OrtResultTest : WordSpec({
                             rule = "Resolved rule violation",
                             pkg = Identifier("Maven", "org.ossreviewtoolkit", "resolved-violation", "0.8.15"),
                             license = null,
-                            licenseSource = null,
+                            licenseSources = emptySet(),
                             severity = Severity.ERROR,
                             message = "Rule violation message to resolve",
                             howToFix = ""
@@ -543,7 +543,7 @@ class OrtResultTest : WordSpec({
                             rule = "Rule violation without resolution",
                             pkg = Identifier("Maven", "com.example", "package-without-resolution", "1.0.0"),
                             license = null,
-                            licenseSource = null,
+                            licenseSources = emptySet(),
                             severity = Severity.WARNING,
                             message = "Message without any resolution",
                             howToFix = ""
@@ -552,7 +552,7 @@ class OrtResultTest : WordSpec({
                             rule = "Rule violation below minSeverity",
                             pkg = Identifier("Maven", "com.example", "violation-below-threshold", "3.14"),
                             license = null,
-                            licenseSource = null,
+                            licenseSources = emptySet(),
                             severity = Severity.HINT,
                             message = "Message without any resolution",
                             howToFix = ""

model/src/test/kotlin/config/RuleViolationResolutionTest.kt

@@ -66,7 +66,7 @@ private fun ruleViolation(message: String) =
         rule = "",
         pkg = null,
         license = null,
-        licenseSource = null,
+        licenseSources = emptySet(),
         severity = Severity.ERROR,
         message = message,
         howToFix = ""

plugins/commands/evaluator/src/main/kotlin/EvaluateCommand.kt

@@ -375,11 +375,7 @@ private fun RuleViolation.format() =
 
         license?.let { license ->
             append(license)
-            licenseSource?.let { source ->
-                append(" (")
-                append(source)
-                append(")")
-            }
+            licenseSources.takeUnless { it.isEmpty() }?.joinToString(prefix = "(", postfix = ")") { it.toString() }
 
             append(" - ")
         }

plugins/reporters/evaluated-model/src/main/kotlin/EvaluatedModelMapper.kt

@@ -393,7 +393,7 @@ internal class EvaluatedModelMapper(private val input: ReporterInput) {
             rule = ruleViolation.rule,
             pkg = pkg,
             license = license,
-            licenseSource = ruleViolation.licenseSource,
+            licenseSources = ruleViolation.licenseSources,
             severity = ruleViolation.severity,
             message = ruleViolation.message,
             howToFix = ruleViolation.howToFix,

plugins/reporters/evaluated-model/src/main/kotlin/EvaluatedRuleViolation.kt

@@ -36,7 +36,7 @@ data class EvaluatedRuleViolation(
     @JsonInclude(JsonInclude.Include.NON_NULL)
     val license: LicenseId?,
     @JsonInclude(JsonInclude.Include.NON_NULL)
-    val licenseSource: LicenseSource?,
+    val licenseSources: Set<LicenseSource>,
     val severity: Severity,
     @JsonInclude(JsonInclude.Include.NON_EMPTY)
     val message: String,

plugins/reporters/static-html/src/main/kotlin/StaticHtmlReporter.kt

@@ -320,7 +320,7 @@ class StaticHtmlReporter(override val descriptor: PluginDescriptor = StaticHtmlR
             td { +(ruleViolation.violation.pkg?.toCoordinates() ?: "-") }
             td {
                 +if (ruleViolation.violation.license != null) {
-                    "${ruleViolation.violation.licenseSource}: ${ruleViolation.violation.license}"
+                    "${ruleViolation.violation.licenseSources.joinToString()}: ${ruleViolation.violation.license}"
                 } else {
                     "-"
                 }