Skip to content

Commit 211d7d9

Browse files
authored
Merge pull request #2518 from rspec/update-cucumber
Update cucumber
2 parents 6a34d62 + 988395e commit 211d7d9

File tree

4 files changed

+26
-25
lines changed

4 files changed

+26
-25
lines changed

Rakefile

Lines changed: 1 addition & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -21,28 +21,7 @@ RSpec::Core::RakeTask.new(:spec) do |t|
2121
t.ruby_opts = %w[-w]
2222
end
2323

24-
Cucumber::Rake::Task.new(:cucumber) do |t|
25-
string_version = ENV.fetch("RAILS_VERSION", "~> 6.0.0")
26-
version =
27-
if string_version == "master" || string_version.nil?
28-
Float::INFINITY
29-
else
30-
string_version[/\d[\.-]\d/].tr('-', '.')
31-
end
32-
tags = []
33-
34-
if version.to_f >= 6.0
35-
tags << "~@rails_pre_6"
36-
end
37-
38-
if version.to_f < 6.0
39-
tags << "~@rails_post_6"
40-
end
41-
42-
cucumber_flag = tags.map { |tag| "--tag #{tag}" }
43-
44-
t.cucumber_opts = cucumber_flag
45-
end
24+
Cucumber::Rake::Task.new(:cucumber)
4625

4726
namespace :generate do
4827
desc "generate a fresh app with rspec installed"

cucumber.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
default: --require features --format progress --tags ~@wip
2-
pretty: --require features --format pretty --tags ~@wip
1+
default: --require features --format progress --tags 'not @wip'
2+
pretty: --require features --format pretty --tags 'not @wip'
33
wip: --require features --tags @wip

features/support/rails_versions.rb

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
def rails_version
2+
string_version = ENV.fetch("RAILS_VERSION", "~> 6.0.0")
3+
if string_version == "master" || string_version.nil?
4+
Float::INFINITY
5+
else
6+
string_version[/\d[\.-]\d/].tr('-', '.')
7+
end
8+
end
9+
10+
Before "@rails_pre_6" do |scenario|
11+
if rails_version.to_f >= 6.0
12+
warn "Skipping scenario #{scenario.name} on Rails v#{rails_version}"
13+
skip_this_scenario
14+
end
15+
end
16+
17+
Before "@rails_post_6" do |scenario|
18+
if rails_version.to_f < 6.0
19+
warn "Skipping scenario #{scenario.name} on Rails v#{rails_version}"
20+
skip_this_scenario
21+
end
22+
end

rspec-rails.gemspec

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,5 +58,5 @@ Gem::Specification.new do |s|
5858

5959
s.add_development_dependency 'ammeter', '~> 1.1.5'
6060
s.add_development_dependency 'aruba', '~> 0.14.12'
61-
s.add_development_dependency 'cucumber', '~> 1.3.5'
61+
s.add_development_dependency 'cucumber', '>= 3.2', '!= 4.0.0', '< 8.0.0'
6262
end

0 commit comments

Comments
 (0)