From fa7a47801ba03ce9c9ed7934eda9098940fba128 Mon Sep 17 00:00:00 2001
From: Faisal N Jawdat <faisal@faisal.com>
Date: Tue, 16 May 2023 08:27:21 -0400
Subject: [PATCH] CHANGE: clean warning by creating cucumber_opts with %w[]
 instead of ''

- Declare cucumber_opts with a %w[] instead of a string, as the latter produces a warning while testing, and may be unsafe.
---
 CHANGELOG.md | 1 +
 Rakefile     | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 81610dbd..e70f83f6 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,6 @@
 # main [(unreleased)](https://github.com/whitesmith/rubycritic/compare/v4.8.1...main)
 
+* [CHANGE] Fix test warning related to `cucumber_opts` declaration (by [@faisal][])
 * [BUGFIX] Stop using long-deprecated MiniTest module name, removed in 5.19.0 (by [@faisal][])
 
 # v4.8.1 / 2023-05-17 [(commits)](https://github.com/whitesmith/rubycritic/compare/v4.8.0...v4.8.1)
diff --git a/Rakefile b/Rakefile
index 2c3b91c6..01c8a375 100644
--- a/Rakefile
+++ b/Rakefile
@@ -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