guard-rspec 4.4.2 → 4.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 345b4747daab905ceacd2b760c6791e6fbd1926b
-  data.tar.gz: a5dbd439c07102bf2288b24485d58830f8ad71d1
+  metadata.gz: e488eab45fd07c9466727bcd31beec13cae327eb
+  data.tar.gz: b5ee0d60b588f120f02bccee664aa73532fb217c
 SHA512:
-  metadata.gz: 4c92905d2abffb7faeb0464bb8c40477f834dab44ffc13a12f493446ae1028daeae16552feb863695c3385c295fca02744fe52ac93b0c05ae7579ca7fede67e5
-  data.tar.gz: 1ca991fbcd0644d2aa12a1c137eef5ab3a07e2042c3bc2760eec4c736a115ca84619396f8cbaf6403d5b811273e0b4f4dab336a9ffd8cba0880cc0abc7b7c219
+  metadata.gz: f38b12805d768019e5956c51ce3689db31208ca5b1aa8e8a6f2bb3b076fc154415831f2bbbbbc9e24eda3f189b496896b905a1353e7129c567ba522e4e187819
+  data.tar.gz: 70e4e5af6118881f2893f9acfe3bbe00b070fe3cab363747605c1c4e3b86f8c1996a4d20cc1e18b416fd6e57624b975763ce137a5c4a5590eae8dbbad73aafed

data/.travis.yml

@@ -14,4 +14,3 @@ matrix:
   allow_failures:
     - rvm: jruby-19mode
     - rvm: rbx
-  

data/Guardfile

@@ -1,11 +1,27 @@
 group :specs, halt_on_fail: true do
   guard :rspec, cmd: "bundle exec rspec" do
-    watch(%r{^spec/.+_spec\.rb})
-    watch(%r{^lib/(.+)\.rb})     { |m| "spec/lib/#{m[1]}_spec.rb" }
-    watch("spec/spec_helper.rb") { "spec" }
+    require "guard/rspec/dsl"
+    dsl = Guard::RSpec::Dsl.new(self)
+
+    # RSpec files
+    rspec = dsl.rspec
+    watch(rspec.spec_files)
+    watch(rspec.spec_helper) { rspec.spec_dir }
+    watch(rspec.spec_support) { rspec.spec_dir }
+
+    # Ruby files
+    dsl.watch_spec_files_for(dsl.ruby.lib_files)
+
+    watch(%r{^(lib/guard/rspec/template)s/Guardfile$}) do
+      rspec.spec.("lib/guard/rspec/template")
+    end
+
+    watch(%r{^lib/guard/rspec/dsl.rb$}) do
+      rspec.spec.("lib/guard/rspec/template")
+    end
   end
 
-  guard :rubocop do
+  guard :rubocop, all_on_start: false do
     watch(%r{.+\.rb$})
     watch(%r{(?:.+/)?\.rubocop\.yml$}) { |m| File.dirname(m[0]) }
   end

data/guard-rspec.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |s|
   s.require_path = "lib"
 
   s.add_dependency "guard", "~> 2.1"
-  s.add_dependency "guard-compat", "~> 0.1"
+  s.add_dependency "guard-compat", "~> 1.1"
   s.add_dependency "rspec", ">= 2.99.0", "< 4.0"
 
   s.add_development_dependency "bundler", ">= 1.3.5", "< 2.0"

data/lib/guard/rspec.rb

@@ -25,7 +25,7 @@ module Guard
     end
 
     def start
-      ::Guard::UI.info "Guard::RSpec is running"
+      Guard::Compat::UI.info "Guard::RSpec is running"
       run_all if options[:all_on_start]
     end
 

data/lib/guard/rspec/deprecator.rb

@@ -24,7 +24,7 @@ module Guard
 
       def _spec_opts_env
         return if ENV["SPEC_OPTS"].nil?
-        UI.warning(
+        Compat::UI.warning(
           "The SPEC_OPTS environment variable is present." +
           " This can conflict with guard-rspec."
         )
@@ -77,7 +77,7 @@ module Guard
       end
 
       def _deprecated(message)
-        UI.warning %(Guard::RSpec DEPRECATION WARNING: #{message})
+        Compat::UI.warning %(Guard::RSpec DEPRECATION WARNING: #{message})
       end
     end
   end

data/lib/guard/rspec/dsl.rb

@@ -0,0 +1,52 @@
+require "ostruct"
+
+require "guard/rspec"
+
+module Guard
+  class RSpec < Plugin
+    class Dsl
+      def initialize(dsl)
+        @dsl = dsl
+      end
+
+      def watch_spec_files_for(expr)
+        @dsl.send(:watch, expr) { |m| rspec.spec.(m[1]) }
+      end
+
+      def rspec
+        @rspec ||= OpenStruct.new(to_s: "spec").tap do |rspec|
+          rspec.spec_dir = "spec"
+          rspec.spec = ->(m) { "#{rspec.spec_dir}/#{m}_spec.rb" }
+          rspec.spec_helper = "#{rspec.spec_dir}/spec_helper.rb"
+          rspec.spec_files = %r{^#{rspec.spec_dir}/.+_spec\.rb$}
+          rspec.spec_support = %r{^#{rspec.spec_dir}/support/(.+)\.rb$}
+        end
+      end
+
+      def ruby
+        # Ruby apps
+        @ruby || OpenStruct.new.tap do |ruby|
+          ruby.lib_files = %r{^(lib/.+)\.rb$}
+        end
+      end
+
+      def rails(options = {})
+        # Rails example
+        @rails ||= OpenStruct.new.tap do |rails|
+          exts = options.dup.delete(:view_extensions) || %w(erb haml slim)
+
+          rails.app_files = %r{^app/(.+)\.rb$}
+
+          rails.views = %r{^app/(views/.+/[^/]*\.(?:#{exts * "|"}))$}
+          rails.view_dirs = %r{^app/views/(.+)/[^/]*\.(?:#{exts * "|"})$}
+          rails.layouts = %r{^app/layouts/(.+)/.*\.("#{exts * "|"}")$}
+
+          rails.controllers = %r{^app/controllers/(.+)_controller\.rb$}
+          rails.routes = "config/routes.rb"
+          rails.app_controller = "app/controllers/application_controller.rb"
+          rails.spec_helper = "#{rspec.spec_dir}/rails_helper.rb"
+        end
+      end
+    end
+  end
+end

data/lib/guard/rspec/notifier.rb

@@ -12,7 +12,7 @@ module Guard
         failure_count, pending_count = _parse_summary(summary)
         image = _image(failure_count, pending_count)
         priority = _priority(image)
-        ::Guard::Notifier.notify(summary,
+        Guard::Compat::UI.notify(summary,
                                  title: @options[:title],
                                  image: image,
                                  priority: priority)
@@ -20,7 +20,7 @@ module Guard
 
       def notify_failure
         return unless options[:notification]
-        ::Guard::Notifier.notify("Failed",
+        Guard::Compat::UI.notify("Failed",
                                  title: @options[:title],
                                  image: :failed,
                                  priority: 2)

data/lib/guard/rspec/runner.rb

@@ -20,14 +20,14 @@ module Guard
         paths = options[:spec_paths]
         options = @options.merge(@options[:run_all])
         return true if paths.empty?
-        ::Guard::UI.info(options[:message], reset: true)
+        Compat::UI.info(options[:message], reset: true)
         _run(true, paths, options)
       end
 
       def run(paths)
         paths = inspector.paths(paths)
         return true if paths.empty?
-        ::Guard::UI.info("Running: #{paths.join(" ")}", reset: true)
+        Compat::UI.info("Running: #{paths.join(" ")}", reset: true)
         _run(false, paths, options)
       end
 
@@ -56,7 +56,7 @@ module Guard
 
       def _cmd_option_present(options)
         return true if options[:cmd]
-        ::Guard::UI.error("No cmd option specified, unable to run specs!")
+        Compat::UI.error("No cmd option specified, unable to run specs!")
         notifier.notify_failure
         false
       end

data/lib/guard/rspec/templates/Guardfile

@@ -8,31 +8,26 @@
 #  * 'just' rspec: 'rspec'
 
 guard :rspec, cmd: "bundle exec rspec" do
-  require "ostruct"
-
-  # Generic Ruby apps
-  rspec = OpenStruct.new
-  rspec.spec = ->(m) { "spec/#{m}_spec.rb" }
-  rspec.spec_dir = "spec"
-  rspec.spec_helper = "spec/spec_helper.rb"
-
-  watch(%r{^spec/.+_spec\.rb$})
-  watch(%r{^lib/(.+)\.rb$})     { |m| rspec.spec.("lib/#{m[1]}") }
-  watch(rspec.spec_helper)      { rspec.spec_dir }
-
-  # Rails example
-  rails = OpenStruct.new
-  rails.app = %r{^app/(.+)\.rb$}
-  rails.views_n_layouts = %r{^app/(.*)(\.erb|\.haml|\.slim)$}
-  rails.controllers = %r{^app/controllers/(.+)_controller\.rb$}
-  rails.routes = "config/routes.rb"
-  rails.app_controller = "app/controllers/application_controller.rb"
-  rails.spec_helper = "spec/rails_helper.rb"
-  rails.spec_support = %r{^spec/support/(.+)\.rb$}
-  rails.views = %r{^app/views/(.+)/.*\.(erb|haml|slim)$}
-
-  watch(rails.app) { |m| rspec.spec.(m[1]) }
-  watch(rails.views_n_layouts) { |m| rspec.spec.("#{m[1]}#{m[2]}") }
+  require "guard/rspec/dsl"
+  dsl = Guard::RSpec::Dsl.new(self)
+
+  # Feel free to open issues for suggestions and improvements
+
+  # RSpec files
+  rspec = dsl.rspec
+  watch(rspec.spec_helper) { rspec.spec_dir }
+  watch(rspec.spec_support) { rspec.spec_dir }
+  watch(rspec.spec_files)
+
+  # Ruby files
+  ruby = dsl.ruby
+  dsl.watch_spec_files_for(ruby.lib_files)
+
+  # Rails files
+  rails = dsl.rails(view_extensions: %w(erb haml slim))
+  dsl.watch_spec_files_for(rails.app_files)
+  dsl.watch_spec_files_for(rails.views)
+
   watch(rails.controllers) do |m|
     [
       rspec.spec.("routing/#{m[1]}_routing"),
@@ -41,18 +36,17 @@ guard :rspec, cmd: "bundle exec rspec" do
     ]
   end
 
-  watch(rails.spec_support)    { rspec.spec_dir }
+  # Rails config changes
   watch(rails.spec_helper)     { rspec.spec_dir }
-  watch(rails.routes)          { "spec/routing" }
-  watch(rails.app_controller)  { "spec/controllers" }
+  watch(rails.routes)          { "#{rspec.spec_dir}/routing" }
+  watch(rails.app_controller)  { "#{rspec.spec_dir}/controllers" }
 
   # Capybara features specs
-  watch(rails.views)     { |m| rspec.spec.("features/#{m[1]}") }
+  watch(rails.view_dirs)     { |m| rspec.spec.("features/#{m[1]}") }
 
   # Turnip features and steps
   watch(%r{^spec/acceptance/(.+)\.feature$})
   watch(%r{^spec/acceptance/steps/(.+)_steps\.rb$}) do |m|
     Dir[File.join("**/#{m[1]}.feature")][0] || "spec/acceptance"
   end
-
 end

data/lib/guard/rspec/version.rb

@@ -1,5 +1,5 @@
 module Guard
   module RSpecVersion
-    VERSION = "4.4.2"
+    VERSION = "4.5.0"
   end
 end

data/spec/lib/guard/rspec/deprecator_spec.rb

@@ -10,14 +10,14 @@ RSpec.describe Guard::RSpec::Deprecator do
     describe "handling of environment variable SPEC_OPTS" do
       it "shows warning if SPEC_OPTS is set" do
         ENV["SPEC_OPTS"] = "-f p"
-        expect(Guard::UI).to receive(:warning).with(
+        expect(Guard::Compat::UI).to receive(:warning).with(
           "The SPEC_OPTS environment variable is present." +
           " This can conflict with guard-rspec.")
         deprecator.warns_about_deprecated_options
         ENV["SPEC_OPTS"] = nil # otherwise other specs pick it up and fail
       end
       it "does not show warning if SPEC_OPTS is unset" do
-        expect(Guard::UI).to_not receive(:warning).with(
+        expect(Guard::Compat::UI).to_not receive(:warning).with(
           "The SPEC_OPTS environment variable is present." +
           " This can conflict with guard-rspec.")
         deprecator.warns_about_deprecated_options
@@ -28,7 +28,7 @@ RSpec.describe Guard::RSpec::Deprecator do
       let(:options) { { version: 1 } }
 
       it "shows deprecation warning" do
-        expect(Guard::UI).to receive(:warning).with(
+        expect(Guard::Compat::UI).to receive(:warning).with(
           "Guard::RSpec DEPRECATION WARNING:" +
           " The :version option is deprecated." +
           " Only RSpec ~> 2.14 is now supported.")
@@ -40,7 +40,7 @@ RSpec.describe Guard::RSpec::Deprecator do
       let(:options) { { exclude: "**" } }
 
       it "shows deprecation warning" do
-        expect(Guard::UI).to receive(:warning).with(
+        expect(Guard::Compat::UI).to receive(:warning).with(
           "Guard::RSpec DEPRECATION WARNING:" +
           " The :exclude option is deprecated." +
           " Please Guard ignore method instead." +
@@ -55,7 +55,7 @@ RSpec.describe Guard::RSpec::Deprecator do
         let(:options) { { option.to_sym => 1 } }
 
         it "shows deprecation warning" do
-          expect(Guard::UI).to receive(:warning).with(
+          expect(Guard::Compat::UI).to receive(:warning).with(
             "Guard::RSpec DEPRECATION WARNING: The :#{option} option is" +
             " deprecated. Please customize the new :cmd option to" +
             " fit your need.")
@@ -68,7 +68,7 @@ RSpec.describe Guard::RSpec::Deprecator do
       let(:options) { { keep_failed: true } }
 
       it "shows deprecation warning" do
-        expect(Guard::UI).to receive(:warning).with(
+        expect(Guard::Compat::UI).to receive(:warning).with(
           "Guard::RSpec DEPRECATION WARNING:" +
           " The :keep_failed option is deprecated." +
           " Please set new :failed_mode option value to" +
@@ -82,7 +82,7 @@ RSpec.describe Guard::RSpec::Deprecator do
       let(:options) { { focus_on_failed: true } }
 
       it "shows deprecation warning" do
-        expect(Guard::UI).to receive(:warning).with(
+        expect(Guard::Compat::UI).to receive(:warning).with(
           "Guard::RSpec DEPRECATION WARNING:" +
           " The :focus_on_failed option is deprecated." +
           " Focus mode is the default and can be changed using new" +

data/spec/lib/guard/rspec/notifier_spec.rb

@@ -7,8 +7,7 @@ RSpec.describe Guard::RSpec::Notifier do
   let(:notifier) { Guard::RSpec::Notifier.new(options) }
 
   def expect_notification(title = "RSpec results", message, image, priority)
-    expect(Guard::Notifier).
-      to receive(:notify).
+    expect(Guard::Compat::UI).to receive(:notify).
       with(message,  title: title, image: image, priority: priority)
   end
 
@@ -76,14 +75,14 @@ RSpec.describe Guard::RSpec::Notifier do
 
     describe "#notify_failure" do
       it "keeps quiet" do
-        expect(Guard::Notifier).not_to receive(:notify)
+        expect(Guard::Compat::UI).not_to receive(:notify)
         notifier.notify_failure
       end
     end
 
     describe "#notify" do
       it "keeps quiet" do
-        expect(Guard::Notifier).not_to receive(:notify)
+        expect(Guard::Compat::UI).not_to receive(:notify)
         notifier.notify("Summary")
       end
     end

data/spec/lib/guard/rspec/runner_spec.rb

@@ -11,7 +11,7 @@ RSpec.describe Guard::RSpec::Runner do
   let(:notifier) { double(Guard::RSpec::Notifier) }
 
   before do
-    allow(Guard::UI).to receive(:info)
+    allow(Guard::Compat::UI).to receive(:info)
     allow(Kernel).to receive(:system) { true }
     allow(Guard::RSpec::Inspectors::Factory).to receive(:create) { inspector }
     allow(Guard::RSpec::Notifier).to receive(:new) { notifier }
@@ -48,7 +48,7 @@ RSpec.describe Guard::RSpec::Runner do
 
   shared_examples "abort" do
     it "aborts" do
-      expect(Guard::UI).to_not receive(:info)
+      expect(Guard::Compat::UI).to_not receive(:info)
       subject
     end
 
@@ -78,7 +78,9 @@ RSpec.describe Guard::RSpec::Runner do
     end
 
     it "prints message" do
-      expect(Guard::UI).to receive(:info).with("Custom message", reset: true)
+      expect(Guard::Compat::UI).to receive(:info).
+        with("Custom message", reset: true)
+
       runner.run_all
     end
 
@@ -111,7 +113,7 @@ RSpec.describe Guard::RSpec::Runner do
       before do
         options[:cmd] = nil
         allow(Guard::RSpec::Command).to receive(:new)
-        allow(Guard::UI).to receive(:error).with(an_instance_of(String))
+        allow(Guard::Compat::UI).to receive(:error).with(an_instance_of(String))
         allow(notifier).to receive(:notify_failure)
         runner.run_all
       end
@@ -121,7 +123,8 @@ RSpec.describe Guard::RSpec::Runner do
       end
 
       it "issues a warning to the user" do
-        expect(Guard::UI).to have_received(:error).with(an_instance_of(String))
+        expect(Guard::Compat::UI).to have_received(:error).
+          with(an_instance_of(String))
       end
 
       it "notifies the notifer of failure" do
@@ -141,7 +144,7 @@ RSpec.describe Guard::RSpec::Runner do
     end
 
     it "prints running message" do
-      expect(Guard::UI).to receive(:info).
+      expect(Guard::Compat::UI).to receive(:info).
         with("Running: spec_path1 spec_path2", reset: true)
       runner.run(paths)
     end

data/spec/lib/guard/rspec/template_spec.rb

@@ -0,0 +1,56 @@
+require "guard/compat/test/template"
+
+# Do not require to simulate Guardfile loading more accurately
+# require 'guard/rspec'
+
+RSpec.describe "Guard::RSpec" do
+  describe "template" do
+    subject { Guard::Compat::Test::Template.new("Guard::RSpec") }
+
+    it "matches spec files by default" do
+      expect(subject.changed("spec/lib/foo_spec.rb")).
+        to eq(%w(spec/lib/foo_spec.rb))
+
+      expect(subject.changed("spec/spec_helper.rb")).to eq(%w(spec))
+    end
+
+    it "matches Ruby files by default" do
+      expect(subject.changed("lib/foo.rb")).to eq(%w(spec/lib/foo_spec.rb))
+    end
+
+    it "matches Rails files by default" do
+      expect(subject.changed("spec/rails_helper.rb")).to eq(%w(spec))
+
+      expect(subject.changed("app/models/foo.rb")).
+        to eq(%w(spec/models/foo_spec.rb))
+
+      expect(subject.changed("app/views/foo/bar.slim")).to eq(
+        %w(
+          spec/views/foo/bar.slim_spec.rb
+          spec/features/foo_spec.rb
+        )
+      )
+
+      expect(subject.changed("app/controllers/application_controller.rb")).
+        to eq(
+        %w(
+          spec/controllers/application_controller_spec.rb
+          spec/routing/application_routing_spec.rb
+          spec/acceptance/application_spec.rb
+          spec/controllers
+        )
+      )
+
+      expect(subject.changed("app/controllers/foo_controller.rb")).
+        to match_array(
+        %w(
+          spec/controllers/foo_controller_spec.rb
+          spec/routing/foo_routing_spec.rb
+          spec/acceptance/foo_spec.rb
+        )
+      )
+
+      expect(subject.changed("config/routes.rb")).to eq(%w(spec/routing))
+    end
+  end
+end

data/spec/lib/guard/rspec_spec.rb

@@ -8,7 +8,7 @@ RSpec.describe Guard::RSpec do
   let(:runner) { double(Guard::RSpec::Runner) }
 
   before do
-    allow(Guard::UI).to receive(:info)
+    allow(Guard::Compat::UI).to receive(:info)
     allow(Guard::RSpec::Deprecator).to receive(:warns_about_deprecated_options)
     allow(Guard::RSpec::Runner).to receive(:new) { runner }
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: guard-rspec
 version: !ruby/object:Gem::Version
-  version: 4.4.2
+  version: 4.5.0
 platform: ruby
 authors:
 - Thibaud Guillaume-Gentil
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-06 00:00:00.000000000 Z
+date: 2014-12-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: guard
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '1.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -131,6 +131,7 @@ files:
 - lib/guard/rspec.rb
 - lib/guard/rspec/command.rb
 - lib/guard/rspec/deprecator.rb
+- lib/guard/rspec/dsl.rb
 - lib/guard/rspec/inspectors/base_inspector.rb
 - lib/guard/rspec/inspectors/factory.rb
 - lib/guard/rspec/inspectors/focused_inspector.rb
@@ -152,6 +153,7 @@ files:
 - spec/lib/guard/rspec/inspectors/simple_inspector_spec.rb
 - spec/lib/guard/rspec/notifier_spec.rb
 - spec/lib/guard/rspec/runner_spec.rb
+- spec/lib/guard/rspec/template_spec.rb
 - spec/lib/guard/rspec_formatter_spec.rb
 - spec/lib/guard/rspec_spec.rb
 - spec/spec_helper.rb
@@ -190,6 +192,7 @@ test_files:
 - spec/lib/guard/rspec/inspectors/simple_inspector_spec.rb
 - spec/lib/guard/rspec/notifier_spec.rb
 - spec/lib/guard/rspec/runner_spec.rb
+- spec/lib/guard/rspec/template_spec.rb
 - spec/lib/guard/rspec_formatter_spec.rb
 - spec/lib/guard/rspec_spec.rb
 - spec/spec_helper.rb