fix(spdx-utils): Compare SPDX expressions case-insensitively

Fixes #3289.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>

Author: sschuberth

utils/spdx/src/main/kotlin/SpdxExpression.kt

@@ -524,7 +524,10 @@ class SpdxLicenseWithExceptionExpression(
 
     override fun equals(other: Any?) =
         when (other) {
-            is SpdxLicenseWithExceptionExpression -> license == other.license && exception == other.exception
+            is SpdxLicenseWithExceptionExpression -> {
+                license == other.license && exception.equals(other.exception, ignoreCase = true)
+            }
+
             else -> false
         }
 
@@ -581,7 +584,10 @@ class SpdxLicenseIdExpression(
 
     override fun equals(other: Any?) =
         when (other) {
-            is SpdxLicenseIdExpression -> id == other.id && orLaterVersion == other.orLaterVersion
+            is SpdxLicenseIdExpression -> {
+                id.equals(other.id, ignoreCase = true) && orLaterVersion == other.orLaterVersion
+            }
+
             else -> false
         }
 
@@ -622,7 +628,7 @@ data class SpdxLicenseReferenceExpression(
 
     override fun equals(other: Any?) =
         when (other) {
-            is SpdxLicenseReferenceExpression -> id == other.id
+            is SpdxLicenseReferenceExpression -> id.equals(other.id, ignoreCase = true)
             else -> false
         }
 

utils/spdx/src/test/kotlin/SpdxExpressionTest.kt

@@ -557,6 +557,11 @@ class SpdxExpressionTest : WordSpec({
             "a".toSpdx() shouldNotBe "a AND b".toSpdx()
             "a".toSpdx() shouldNotBe "a+".toSpdx()
         }
+
+        "compare expressions case-insensitively" {
+            "CDDL-1.1 OR GPL-2.0-only WITH Classpath-Exception-2.0".toSpdx() shouldBe
+                "Cddl-1.1 OR Gpl-2.0-only WITH Classpath-exception-2.0".toSpdx()
+        }
     }
 
     "hashCode()" should {