feat(unique_value): add == consistency

lib/convenient_service/support/unique_value.rb

@@ -23,9 +23,10 @@ def initialize(label = default_label)
       #
       # @internal
       #   NOTE: Every single Ruby object responds to `object_id`.
-      #   NOTE: NO `return unless other.instance_of?(self.class)` for performance.
       #
       def ==(other)
+        return unless other.instance_of?(self.class)
+
         object_id == other.object_id
       end
 

spec/lib/convenient_service/support/unique_value_spec.rb

@@ -61,6 +61,14 @@
       describe "#==" do
         let(:unique_value) { described_class.new(:foo) }
 
+        context "when `other` has different class" do
+          let(:other) { 42 }
+
+          it "returns `false`" do
+            expect(unique_value == other).to be_nil
+          end
+        end
+
         context "when `other` has different `object_id`" do
           let(:other) { described_class.new(:bar) }
 
@@ -81,6 +89,14 @@
       describe "#===" do
         let(:unique_value) { described_class.new(:foo) }
 
+        context "when `other` has different class" do
+          let(:other) { 42 }
+
+          it "returns `false`" do
+            expect(unique_value === other).to be_nil
+          end
+        end
+
         context "when `other` has different `object_id`" do
           let(:other) { described_class.new(:bar) }
 
@@ -101,6 +117,14 @@
       describe ".eql?" do
         let(:unique_value) { described_class.new(:foo) }
 
+        context "when `other` has different class" do
+          let(:other) { 42 }
+
+          it "returns `false`" do
+            expect(unique_value.eql?(other)).to be_nil
+          end
+        end
+
         context "when `other` has different `object_id`" do
           let(:other) { described_class.new(:bar) }