Create --exts and Delete --ext option

We allow to pass a list of extensions instead of a regex and build the regex
from the options instead of the user. This is simpler and easier for the user.

Author: AlexB52

features/ruby-bare/retest/scenarios/custom_extensions.rb

@@ -1,6 +1,6 @@
 class CustomExtensionTest < Minitest::Test
   def setup
-    @command = %Q{retest "echo 'I captured a change'" --ext="\\.txt$"}
+    @command = %Q{retest "echo 'I captured a change'" --exts="txt"}
   end
 
   def teardown

lib/retest.rb

@@ -23,7 +23,7 @@ class Error < StandardError; end
   class FileNotFound < StandardError; end
 
   def self.listen(options, listener: Listen)
-    listener.to('.', only: options.extension, relative: true, force_polling: options.force_polling?) do |modified, added, removed|
+    listener.to('.', only: options.extensions, relative: true, force_polling: options.force_polling?) do |modified, added, removed|
       yield modified, added, removed
     end.start
   end

lib/retest/options.rb

@@ -63,10 +63,10 @@ class Options
       long "--diff=git-branch"
     end
 
-    option :ext do
-      desc "Regex of file extensions to listen to"
-      long "--ext=regex"
-      default "\\.rb$"
+    option :exts do
+      desc "Comma separated of filenames extensions to filter to"
+      long "--exts=<EXTENSIONS>"
+      default "rb"
     end
 
     flag :all do
@@ -155,8 +155,9 @@ def force_polling?
       params[:polling]
     end
 
-    def extension
-      Regexp.new(params[:ext])
+    def extensions
+      exts = params[:exts].split(',').map(&:strip)
+      Regexp.new("\\.(?:#{exts.join("|")})$")
     end
 
     def merge(options = [])

test/retest/options/help.txt

@@ -9,21 +9,21 @@ Arguments:
 
 
 Options:
-      --all              Run all the specs of a specificied ruby setup
-      --diff=git-branch  Pipes all matching tests from diffed branch to test
-                         command
-      --ext=regex        Regex of file extensions to listen to (default
-                         "\\.rb$")
-  -h, --help             Print usage
-      --notify           Play a sound when specs pass or fail (macOS only)
-      --polling          Use polling method when listening to file changes
-                         Some filesystems won't work without it
-                         VM/Vagrant Shared folders, NFS, Samba, sshfs...
-      --rails            Shortcut for a standard Rails setup
-      --rake             Shortcut for a standard Rake setup
-      --rspec            Shortcut for a standard RSpec setup
-      --ruby             Shortcut for a Ruby project
-  -v, --version          Print retest version
+      --all                Run all the specs of a specificied ruby setup
+      --diff=git-branch    Pipes all matching tests from diffed branch to test
+                           command
+      --exts=<EXTENSIONS>  Comma separated of filenames extensions to filter
+                           to (default "rb")
+  -h, --help               Print usage
+      --notify             Play a sound when specs pass or fail (macOS only)
+      --polling            Use polling method when listening to file changes
+                           Some filesystems won't work without it
+                           VM/Vagrant Shared folders, NFS, Samba, sshfs...
+      --rails              Shortcut for a standard Rails setup
+      --rake               Shortcut for a standard Rake setup
+      --rspec              Shortcut for a standard RSpec setup
+      --ruby               Shortcut for a Ruby project
+  -v, --version            Print retest version
 
 Examples:
   Runs a matching rails test after a file change

test/retest/options_test.rb

@@ -49,6 +49,24 @@ def test_notify?
       assert @subject.notify?
     end
 
+    def test_exts
+      @subject.args = ["--exts rb"]
+
+      assert @subject.extensions.match? "a/file/path.rb"
+      refute @subject.extensions.match? "a/file/path.js"
+      refute @subject.extensions.match? "a/file/path.ts"
+      refute @subject.extensions.match? "a/file/path.css"
+      refute @subject.extensions.match? "a/file/path.html"
+
+      @subject.args = ["--exts rb,js, html,css ,ts"]
+
+      assert @subject.extensions.match? "a/file/path.rb"
+      assert @subject.extensions.match? "a/file/path.js"
+      assert @subject.extensions.match? "a/file/path.ts"
+      assert @subject.extensions.match? "a/file/path.css"
+      assert @subject.extensions.match? "a/file/path.html"
+    end
+
     def test_all_version_copy
       @subject.args = %w[--notify --rake]
 

test/retest_test.rb

@@ -13,7 +13,7 @@ class ListenTests < MiniTest::Test
 
   def test_listen_default_behaviour
     listener = MiniTest::Mock.new
-    expected_options = {relative: true, only: Regexp.new('\\.rb$'), force_polling: false}
+    expected_options = {relative: true, only: Regexp.new('\\.(?:rb)$'), force_polling: false}
 
     listener.expect(:to, Struct.new(:start).new, ['.', expected_options])
 
@@ -24,7 +24,7 @@ def test_listen_default_behaviour
 
   def test_listen_when_polling
     listener = MiniTest::Mock.new
-    expected_options = {relative: true, only: Regexp.new('\\.rb$'), force_polling: true}
+    expected_options = {relative: true, only: Regexp.new('\\.(?:rb)$'), force_polling: true}
 
     listener.expect(:to, Struct.new(:start).new, ['.', expected_options])