Positions now look identical in hashes when the Position configuration

is identical, too!

lib/trailblazer/workflow/collaboration.rb

@@ -68,6 +68,18 @@ def collect(&block)
             .collect { |position| position.to_a }
             .collect(&block)
         end
+
+        def ==(b)
+          eql?(b)
+        end
+
+        def eql?(b)
+          hash == b.hash
+        end
+
+        def hash
+          @positions.collect { |position| position.hash }.sort.join("").to_i
+        end
       end
 
 

test/structures_test.rb

@@ -25,4 +25,35 @@ class StructuresTest < Minitest::Spec
     # DISCUSS: check how the interal order is now different!
     assert_equal positions.collect { |activity, task| [activity, task] }.inspect, %([[\"B\", \"suspend:b\"], [\"A\", \"suspend:c\"]])
   end
+
+  it "Positions#==" do
+    activity_a = "A"
+    suspend_a  = "suspend:a"
+    activity_b = "B"
+    suspend_b = "suspend:b"
+
+    position_a = Trailblazer::Workflow::Collaboration::Position.new(activity_a, suspend_a)
+    position_b = Trailblazer::Workflow::Collaboration::Position.new(activity_b, suspend_b)
+    positions = Trailblazer::Workflow::Collaboration::Positions.new([position_a, position_b])
+
+    position_a_2 = Trailblazer::Workflow::Collaboration::Position.new(activity_a, suspend_a)
+    position_b_2 = Trailblazer::Workflow::Collaboration::Position.new(activity_b, suspend_b)
+    positions_2 = Trailblazer::Workflow::Collaboration::Positions.new([position_a_2, position_b_2])
+
+    #@ Position#==
+    assert position_a == position_a_2
+    #@ Position#hash for keying in hashes
+    assert_equal position_a.hash, position_a_2.hash
+
+    #@ Positions#==
+    assert positions == positions_2
+
+    #@ changed position order # DISCUSS: should we allow that in the first place?
+    assert positions == Trailblazer::Workflow::Collaboration::Positions.new([position_b, position_a])
+
+    #@ Positions with same Position configuration look identical in hashes.
+    some_hash = {positions => true}
+    assert_equal some_hash[positions], true
+    assert_equal some_hash[positions_2], true
+  end
 end