Revert "Improve user friendliness when class resolution fails."

3 files changed, 27 insertions, 45 deletions

diff --git a/jdisc_core/src/main/java/com/yahoo/jdisc/core/BsnVersion.java b/jdisc_core/src/main/java/com/yahoo/jdisc/core/BsnVersion.java
deleted file mode 100644
index ee9ecc9fc3e..00000000000
--- a/jdisc_core/src/main/java/com/yahoo/jdisc/core/BsnVersion.java
+++ /dev/null
@@ -1,21 +0,0 @@
-package com.yahoo.jdisc.core;
-
-import org.osgi.framework.Bundle;
-import org.osgi.framework.Version;
-
-/**
- * A bundle's symbolic name and version.
- *
- * @author gjoranv
- */
-public record BsnVersion(String symbolicName, Version version) {
-
-    public BsnVersion(Bundle bundle) {
-        this(bundle.getSymbolicName(), bundle.getVersion());
-    }
-
-    public String toReadableString() {
-        return symbolicName + " version:" + version;
-    }
-
-}
diff --git a/jdisc_core/src/main/java/com/yahoo/jdisc/core/BundleCollisionHook.java b/jdisc_core/src/main/java/com/yahoo/jdisc/core/BundleCollisionHook.java
index e78f3251890..3212bb4e6de 100644
--- a/jdisc_core/src/main/java/com/yahoo/jdisc/core/BundleCollisionHook.java
+++ b/jdisc_core/src/main/java/com/yahoo/jdisc/core/BundleCollisionHook.java
@@ -129,4 +129,31 @@ public class BundleCollisionHook implements CollisionHook, EventHook, FindHook {
         }
     }
 
+
+    static class BsnVersion {
+
+        private final String symbolicName;
+        private final Version version;
+
+        BsnVersion(Bundle bundle) {
+            this.symbolicName = bundle.getSymbolicName();
+            this.version = bundle.getVersion();
+        }
+
+        @Override
+        public boolean equals(Object o) {
+            if (this == o) return true;
+            if (o == null || getClass() != o.getClass()) return false;
+            BsnVersion that = (BsnVersion) o;
+            return Objects.equals(symbolicName, that.symbolicName) &&
+                    version.equals(that.version);
+        }
+
+        @Override
+        public int hashCode() {
+            return Objects.hash(symbolicName, version);
+        }
+
+    }
+
 }
diff --git a/jdisc_core/src/test/java/com/yahoo/jdisc/core/BsnVersionTest.java b/jdisc_core/src/test/java/com/yahoo/jdisc/core/BsnVersionTest.java
deleted file mode 100644
index baeb36d28a8..00000000000
--- a/jdisc_core/src/test/java/com/yahoo/jdisc/core/BsnVersionTest.java
+++ /dev/null
@@ -1,24 +0,0 @@
-package com.yahoo.jdisc.core;
-
-import org.junit.jupiter.api.Test;
-import org.osgi.framework.Version;
-
-import static org.junit.jupiter.api.Assertions.assertEquals;
-
-/**
- * @author gjoranv
- */
-public class BsnVersionTest {
-
-    @Test
-    void readable_string_can_be_retrieved() {
-        BsnVersion bsnVersion = new BsnVersion("com.yahoo.foo", new Version("1.0.0"));
-        assertEquals("com.yahoo.foo version:1.0.0", bsnVersion.toReadableString());
-    }
-
-    @Test
-    void version_qualifier_can_be_retrieved() {
-        BsnVersion bsnVersion = new BsnVersion("foo", new Version(1, 2, 3, "SNAPSHOT"));
-        assertEquals("SNAPSHOT", bsnVersion.version().getQualifier());
-    }
-}