Skip to content

Commit

Permalink
CHANGE: clean warning by creating cucumber_opts with %w[] instead of ''
Browse files Browse the repository at this point in the history
- Declaring cucumber_opts with a string results in warnings when testing. Change that.
- Also lock minitest version to < 5.19.0 due to change in name of constant we rely on.
  • Loading branch information
faisal committed Sep 5, 2023
1 parent 82bcc56 commit 4193774
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
# main [(unreleased)](https://github.com/whitesmith/rubycritic/compare/v4.8.1...main)

* [CHANGE] Fix test warning related to `cucumber_opts` declaration (by [@faisal][])

# v4.8.1 / 2023-05-17 [(commits)](https://github.com/whitesmith/rubycritic/compare/v4.8.0...v4.8.1)

* [CHANGE] Update runtime dependencies to current versions of analysis gems (by [@faisal][])
Expand Down
2 changes: 1 addition & 1 deletion Rakefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ Rake::TestTask.new do |task|
end

Cucumber::Rake::Task.new(:features) do |t|
t.cucumber_opts = 'features --format progress --color'
t.cucumber_opts = %w[features --format progress --color]
end

RuboCop::RakeTask.new
Expand Down
2 changes: 1 addition & 1 deletion rubycritic.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ Gem::Specification.new do |spec|
spec.add_development_dependency 'diff-lcs', '~> 1.3'
spec.add_development_dependency 'fakefs', '~> 2.4.0'
spec.add_development_dependency 'mdl', '~> 0.12.0'
spec.add_development_dependency 'minitest', '>= 5.3.0'
spec.add_development_dependency 'minitest', '>= 5.3.0', '< 5.19.0'
spec.add_development_dependency 'minitest-around', '~> 0.5.0', '>= 0.4.0'
spec.add_development_dependency 'mocha', '~> 2.0.2'
spec.add_development_dependency 'rake', '~> 13.0.6', '>= 11.0.0'
Expand Down

0 comments on commit 4193774

Please sign in to comment.