Merge pull request #24 from oreoshake/gh-22-dup_tests_being_run

Individual scenarios are no longer run if the entire feature scheduled. (Fixes #22)

lib/guard/cucumber/inspector.rb

@@ -66,9 +66,9 @@ def clear_cucumber_files_list
         #
         def included_in_other_path?(path, paths)
           paths = paths.select { |p| p != path }
-          paths.any? { |p| path.include?(p) && (path.gsub(p, '')).include?('/') }
+          massaged = path[0...(path.index(":")||path.size)]
+          paths.any? { |p| (path.include?(p) && (path.gsub(p, '')).include?('/')) || massaged.include?(p) }
         end
-
       end
     end
   end

spec/guard/cucumber/inspector_spec.rb

@@ -25,6 +25,10 @@
     it 'removes duplicate paths' do
       inspector.clean(['features', 'features']).should == ['features']
     end
+
+    it 'removes individual feature tests if the path is already in paths to run' do
+      inspector.clean(['features/a.feature', 'features/a.feature:10']).should == ['features/a.feature']
+    end
 
     it 'removes feature folders included in other feature folders' do
       inspector.clean(['features/subfolder', 'features']).should == ['features']
@@ -34,4 +38,16 @@
       inspector.clean(['features/subfolder/b.feature', 'features']).should == ['features']
     end
   end
+
+  describe ".included_in_other_path?" do
+    it "detects dups when a scenario is specified, but the feature is already included" do
+      paths = Dir.glob
+
+      # dup, add a specific test
+      dup_path = paths.last + ":22"
+      paths << dup_path  
+
+      inspector.send(:included_in_other_path?, dup_path, paths).should be_true
+    end
+  end
 end