guard-rspec 4.7.1 → 4.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b6163bfd9a274c88eefd7a84f1e1f815d1191b33
-  data.tar.gz: 80deab415f449c74ec8b4f50589dae22f80ec021
+  metadata.gz: 918301c5591d32e57f968b46fe945cf4e5f6871d
+  data.tar.gz: 416877bc8284de1e6c44d200c2be36625dfc6829
 SHA512:
-  metadata.gz: 1618cbb9b313aff920902dfd34780c12b076b627a6f287cd9b8ab38451a38ba532e1d1d3dd1ca5a34a6d8fee07394a256431806a9a67a7e1b8581ea7899e2daa
-  data.tar.gz: dff4b7a5a92c25b42a5beda572ea707958fe2e2d173f2582838d4aa790d5fa8b64f1b330703738f8e7b301b75714655c2a59ec19d6e9dfc94e28107931e1c28d
+  metadata.gz: c7e36a93775573a76d509da8ed55307f0af64f8b6029bc04bbe8aff657d76731516f22fa1b371a3a5facadef0649181d661e0243bbb6de9d484eb8b03665de58
+  data.tar.gz: 5456241016e2fb4596f81bd17b26465e8e6457aa62aee18e227b47682b66e9c7ee79be30a23968fbd0979e53686e1ad32834f0ddf958ab7322a14018eccfa578

data/Gemfile

@@ -15,10 +15,7 @@ else
   gemspec development_group: :gem_build_tools
 end
 
-# bundler + rake - always included
 group :gem_build_tools do
-  gem "bundler", "~> 1.12", "< 2.0"
-  gem "rake", "~> 11.1"
 end
 
 group :development do
@@ -26,7 +23,3 @@ group :development do
   gem "guard-rubocop", require: false
   gem "guard-compat", ">= 0.0.2", require: false
 end
-
-group :tool do
-  gem "ruby_gntp", require: false
-end

data/README.md

@@ -120,6 +120,11 @@ run_all: { cmd: 'custom rspec command', message: 'custom message' } # Custom opt
 title: 'My project'    # Display a custom title for the notification, default: 'RSpec results'
 chdir: 'directory'     # run rspec from within a given subdirectory (useful if project has separate specs for submodules)
 results_file: 'some/path' # use the given file for storing results (instead of default relative path)
+bundler_env: :original_env # Specify which Bundler env to run the cmd under, default: :original_env
+                       # Available values:
+                       #  :clean_env - old behavior, uses Bundler environment with all bundler-related variables removed. This is deprecated in bundler 1.12.x.
+                       #  :original_env (default) - uses Bundler environment present before Bundler was activated
+                       #  :inherit - runs inside the current environment
 ```
 
 ### Using Launchy to view rspec results

data/lib/guard/rspec/options.rb

@@ -11,7 +11,8 @@ module Guard
         cmd_additional_args: nil,
         launchy:         nil,
         notification:    true,
-        title:           "RSpec results"
+        title:           "RSpec results",
+        bundler_env:     :original_env
       }.freeze
 
       class << self

data/lib/guard/rspec/rspec_process.rb

@@ -6,12 +6,13 @@ module Guard
       class Failure < RuntimeError
       end
 
-      attr_reader :results
+      attr_reader :results, :options
 
-      def initialize(command, formatter_tmp_file)
+      def initialize(command, formatter_tmp_file, options = {})
         @command = command
         @formatter_tmp_file = formatter_tmp_file
         @results = nil
+        @options = options
 
         @exit_code = _run
         @results = _read_results
@@ -24,7 +25,7 @@ module Guard
       private
 
       def _run
-        _without_bundler_env do
+        _with_desired_bundler_env do
           exit_code = _really_run
 
           msg = "Guard::RSpec: RSpec command %s exited with: %s"
@@ -65,11 +66,14 @@ module Guard
         File.delete(formatter_tmp_file) if File.exist?(formatter_tmp_file)
       end
 
-      def _without_bundler_env
-        if defined?(::Bundler)
+      def _with_desired_bundler_env
+        desired_bundler_env = options[:bundler_env]
+        if !defined?(::Bundler) || desired_bundler_env == :inherit
+          yield
+        elsif desired_bundler_env == :clean_env
           ::Bundler.with_clean_env { yield }
         else
-          yield
+          ::Bundler.with_original_env { yield }
         end
       end
 

data/lib/guard/rspec/runner.rb

@@ -62,7 +62,7 @@ module Guard
         # TODO: add option to specify the file
         file = _results_file(options[:results_file], options[:chdir])
 
-        process = RSpecProcess.new(cmd, file)
+        process = RSpecProcess.new(cmd, file, options)
         results = process.results
 
         inspector.failed(results.failed_paths)

data/lib/guard/rspec/version.rb

@@ -1,5 +1,5 @@
 module Guard
   module RSpecVersion
-    VERSION = "4.7.1".freeze
+    VERSION = "4.7.2".freeze
   end
 end

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

@@ -111,5 +111,33 @@ RSpec.describe Guard::RSpec::RSpecProcess do
         end
       end
     end
+
+    context "with bundler_env option" do
+      it "runs without Bunder changes when :inherit" do
+        expect(Bundler).to_not receive(:with_clean_env)
+        expect(Bundler).to_not receive(:with_original_env)
+
+        described_class.new(cmd, file, bundler_env: :inherit)
+      end
+
+      it "runs on clean Bunder changes when :clean_env" do
+        expect(Bundler).to receive(:with_clean_env)
+
+        described_class.new(cmd, file, bundler_env: :clean_env)
+      end
+
+      it "runs on original Bunder changes when :original_env" do
+        expect(Bundler).to receive(:with_original_env)
+
+        described_class.new(cmd, file, bundler_env: :original_env)
+      end
+    end
+
+    context "without bundler_env option" do
+      it "runs on original Bunder" do
+        expect(Bundler).to receive(:with_original_env)
+        subject
+      end
+    end
   end
 end

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

@@ -218,7 +218,7 @@ RSpec.describe Guard::RSpec::Runner do
           let(:results_file) { File.join(Dir.pwd, "foobar.txt") }
           it "uses the given file" do
             expect(Guard::RSpec::RSpecProcess).to receive(:new).
-              with(anything, results_file).and_return(process)
+              with(anything, results_file, options).and_return(process)
             runner.run(paths)
           end
         end
@@ -227,7 +227,7 @@ RSpec.describe Guard::RSpec::Runner do
           let(:results_file) { "/foo/foobar.txt" }
           it "uses the given path" do
             expect(Guard::RSpec::RSpecProcess).to receive(:new).
-              with(anything, results_file).and_return(process)
+              with(anything, results_file, options).and_return(process)
             runner.run(paths)
           end
         end
@@ -243,7 +243,7 @@ RSpec.describe Guard::RSpec::Runner do
             it "uses a path relative to chdir" do
               expected = "/foo/bar/moduleA/foobar.txt"
               expect(Guard::RSpec::RSpecProcess).to receive(:new).
-                with(anything, expected).and_return(process)
+                with(anything, expected, options).and_return(process)
               runner.run(paths)
             end
           end
@@ -252,7 +252,7 @@ RSpec.describe Guard::RSpec::Runner do
             let(:results_file) { "/foo/foobar.txt" }
             it "uses the full given path anyway" do
               expect(Guard::RSpec::RSpecProcess).to receive(:new).
-                with(anything, results_file).and_return(process)
+                with(anything, results_file, options).and_return(process)
               runner.run(paths)
             end
           end
@@ -270,7 +270,7 @@ RSpec.describe Guard::RSpec::Runner do
             it "uses a path relative to chdir" do
               expected = File.join(Dir.pwd, "moduleA/foobar.txt")
               expect(Guard::RSpec::RSpecProcess).to receive(:new).
-                with(anything, expected).and_return(process)
+                with(anything, expected, options).and_return(process)
               runner.run(paths)
             end
 
@@ -285,7 +285,7 @@ RSpec.describe Guard::RSpec::Runner do
             let(:results_file) { "/foo/foobar.txt" }
             it "uses the full given path anyway" do
               expect(Guard::RSpec::RSpecProcess).to receive(:new).
-                with(anything, results_file).and_return(process)
+                with(anything, results_file, options).and_return(process)
               runner.run(paths)
             end
           end
@@ -296,8 +296,9 @@ RSpec.describe Guard::RSpec::Runner do
     context "with no custom results file" do
       let(:options) { { cmd: "rspec" } }
       it "uses the default" do
+        expected_params = [anything, %r{/tmp/rspec_guard_result$}, options]
         expect(Guard::RSpec::RSpecProcess).to receive(:new).
-          with(anything, %r{/tmp/rspec_guard_result$}).and_return(process)
+          with(*expected_params).and_return(process)
         runner.run(paths)
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: guard-rspec
 version: !ruby/object:Gem::Version
-  version: 4.7.1
+  version: 4.7.2
 platform: ruby
 authors:
 - Thibaud Guillaume-Gentil
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-31 00:00:00.000000000 Z
+date: 2016-06-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: guard