chore(model): Improve the failure message when checking references

Use a more readable representation of IDs.

Signed-off-by: Sebastian Schuberth <[email protected]>

Author: sschuberth

model/src/main/kotlin/utils/DependencyGraphBuilder.kt

@@ -202,8 +202,9 @@ class DependencyGraphBuilder<D>(
 
     private fun checkReferences() {
         require(resolvedPackages.keys.containsAll(validPackageDependencies)) {
+            val danglingIds = validPackageDependencies - resolvedPackages.keys
             "The following references do not actually refer to packages: " +
-                "${validPackageDependencies - resolvedPackages.keys}."
+                danglingIds.joinToString(postfix = ".") { "'${it.toCoordinates()}'" }
         }
 
         val packageReferencesKeysWithMultipleDistinctPackageReferences = references.groupBy { it.key }.filter {

model/src/test/kotlin/utils/DependencyGraphBuilderTest.kt

@@ -223,7 +223,7 @@ class DependencyGraphBuilderTest : WordSpec({
                     .build()
             }
 
-            exception.message shouldContain depNoPkg.id.toString()
+            exception.message shouldContain depNoPkg.id.toCoordinates()
         }
 
         "take issues into account when checking for illegal references" {