Merge pull request #111 from cucumber/equality-for-test-cases

Implement equality for test cases

lib/cucumber/core/test/case.rb

@@ -100,6 +100,18 @@ def feature
           source.first
         end
 
+        def hash
+          location.hash
+        end
+
+        def eql?(other)
+          other.hash == hash
+        end
+
+        def ==(other)
+          eql?(other)
+        end
+
         private
 
         def compose_around_hooks(visitor, *args, &block)

spec/cucumber/core/test/case_spec.rb

@@ -14,8 +14,8 @@ module Test
         include Core::Gherkin::Writer
 
         let(:test_case) { Test::Case.new(test_steps, [feature, scenario]) }
-        let(:feature) { double }
-        let(:scenario) { double }
+        let(:feature) { double(location: "features/test.feature:1") }
+        let(:scenario) { double(location: "features/test.feature:2") }
         let(:test_steps) { [double, double] }
 
         context 'describing itself' do
@@ -233,6 +233,29 @@ module Test
           end
         end
 
+        describe "equality" do
+          it "is equal to another test case at the same location" do
+            gherkin = gherkin('features/foo.feature') do
+              feature do
+                scenario do
+                  step
+                end
+              end
+            end
+            test_case_instances = []
+            receiver = double.as_null_object
+            allow(receiver).to receive(:test_case) do |test_case|
+              test_case_instances << test_case
+            end
+            2.times { compile([gherkin], receiver) }
+            expect(test_case_instances.length).to eq 2
+            expect(test_case_instances.uniq.length).to eq 1
+            expect(test_case_instances[0]).to be_eql test_case_instances[1]
+            expect(test_case_instances[0]).to eq test_case_instances[1]
+            expect(test_case_instances[0]).not_to equal test_case_instances[1]
+          end
+        end
+
       end
     end
   end

spec/cucumber/core/test/runner_spec.rb

@@ -220,9 +220,10 @@ module Cucumber::Core::Test
         let(:test_cases)      { [first_test_case, last_test_case] }
 
         it 'reports the results correctly for the following test case' do
-          expect(event_bus).to receive(:test_case_finished).with(last_test_case, anything) do |reported_test_case, result|
-            expect( result ).to be_passed
-          end
+          expect(event_bus).to receive(:test_case_finished) { |reported_test_case, result|
+            expect(result).to be_failed if reported_test_case.equal?(first_test_case)
+            expect(result).to be_passed if reported_test_case.equal?(last_test_case)
+          }.twice
 
           test_cases.each { |c| c.describe_to runner }
         end