chore(spdx): Give more context in require checks

Check the license ID first and also show the optional name to eventually give a hint which license is affected. For the extracted text, also show the (now non-blank) license ID. Signed-off-by: Sebastian Schuberth <[email protected]>
oss-review-toolkit · Dec 7, 2023 · b3c98bb · b3c98bb
1 parent f9d1124
commit b3c98bb
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/utils/spdx/src/main/kotlin/model/SpdxExtractedLicenseInfo.kt b/utils/spdx/src/main/kotlin/model/SpdxExtractedLicenseInfo.kt
@@ -60,8 +60,7 @@ data class SpdxExtractedLicenseInfo(
     val seeAlsos: List<String> = emptyList()
 ) {
     init {
-        require(extractedText.isNotBlank()) { "The extracted text must not be blank." }
-
-        require(licenseId.isNotBlank()) { "The license ID must not be blank." }
+        require(licenseId.isNotBlank()) { "The license ID must not be blank (the optional name is '$name')." }
+        require(extractedText.isNotBlank()) { "The extracted text must not be blank (the license ID is '$licenseId')." }
     }
 }