refactor(model): Stop implementing Comparable with DependencyReference

The `AnalayzerResultBuilder` uses `copy()` which creates copies of `DependencyReference`s in its second test case. The assertion then checks the equality which succeeds with the `Comparable` interface implemented, but fails without. So, turn the `DependencyReference` into a data class to fix that. Signed-off-by: Frank Viernau <[email protected]>
oss-review-toolkit · May 27, 2024 · 4569367 · 4569367
1 parent cf8d8d0
commit 4569367
Showing 1 changed file with 2 additions and 12 deletions.
diff --git a/model/src/main/kotlin/DependencyGraph.kt b/model/src/main/kotlin/DependencyGraph.kt
@@ -295,7 +295,7 @@ data class RootDependencyIndex(
  * Note: This is by intention no data class. Equality is tested via references and not via the values contained.
  */
 @JsonInclude(JsonInclude.Include.NON_DEFAULT)
-class DependencyReference(
+data class DependencyReference(
     /**
      * Stores the numeric index of the package dependency referenced by this object. The package behind this index can
      * be resolved by evaluating the list of identifiers stored in [DependencyGraph] at this index.
@@ -326,17 +326,7 @@ class DependencyReference(
      * A list of [Issue]s that occurred handling this dependency.
      */
     val issues: List<Issue> = emptyList()
-) : Comparable<DependencyReference> {
-    /**
-     * Define an order on [DependencyReference] instances. Instances are ordered by their indices and fragment indices.
-     */
-    override fun compareTo(other: DependencyReference): Int =
-        if (pkg != other.pkg) {
-            pkg - other.pkg
-        } else {
-            fragment - other.fragment
-        }
-}
+)
 
 /**
  * A data class representing a node in the dependency graph.