Move scenario filtering to Cucumber hooks

Otherwise, running `cucumber` won't filter scenarios, only `rake
cucumber` will.

Author: pirj

Rakefile

@@ -22,28 +22,7 @@ RSpec::Core::RakeTask.new(:spec) do |t|
   t.rspec_opts = %w[--color]
 end
 
-Cucumber::Rake::Task.new(:cucumber) do |t|
-  string_version = ENV.fetch("RAILS_VERSION", "~> 6.0.0")
-  version =
-    if string_version == "master" || string_version.nil?
-      Float::INFINITY
-    else
-      string_version[/\d[\.-]\d/].tr('-', '.')
-    end
-  tags = []
-
-  if version.to_f >= 6.0
-    tags << "'not @rails_pre_6'"
-  end
-
-  if version.to_f < 6.0
-    tags << "'not @rails_post_6'"
-  end
-
-  cucumber_flag = tags.map { |tag| "--tag #{tag}" }
-
-  t.cucumber_opts = cucumber_flag
-end
+Cucumber::Rake::Task.new(:cucumber)
 
 namespace :generate do
   desc "generate a fresh app with rspec installed"

features/support/rails_versions.rb

@@ -0,0 +1,22 @@
+def rails_version
+  string_version = ENV.fetch("RAILS_VERSION", "~> 6.0.0")
+  if string_version == "master" || string_version.nil?
+    Float::INFINITY
+  else
+    string_version[/\d[\.-]\d/].tr('-', '.')
+  end
+end
+
+Before "@rails_pre_6" do |scenario|
+  if rails_version.to_f >= 6.0
+    warn "Skipping scenario #{scenario.name} on Rails v#{rails_version}"
+    skip_this_scenario
+  end
+end
+
+Before "@rails_post_6" do |scenario|
+  if rails_version.to_f < 6.0
+    warn "Skipping scenario #{scenario.name} on Rails v#{rails_version}"
+    skip_this_scenario
+  end
+end