Merge pull request #375 from guard/misc_build_fixes

Misc fixes related to HoundCi/RubCop/Travis

Author: e2

Diff for: .hound.yml

@@ -0,0 +1,3 @@
+ruby:
+  enabled: true
+  config_file: .rubocop.yml

Diff for: .rubocop.yml

@@ -1,2 +1,5 @@
 inherit_from:
   .rubocop_todo.yml
+
+Style/StringLiterals:
+  Enabled: false

Diff for: .travis.yml

@@ -1,14 +1,14 @@
 language: ruby
-bundler_args: --without tool
+bundler_args: --without=tool development
 rvm:
   - 2.2.5
   - 2.3.1
-  - jruby-9.0.5.0
+  - jruby-9.1.1.0
 gemfile:
   - gemfiles/Gemfile.rspec-2.99
   - gemfiles/Gemfile.rspec-3.4
 matrix:
   allow_failures:
-    - rvm: jruby-9.0.50
+    - rvm: jruby-9.1.1.0
 sudo: false
 cache: bundler

Diff for: Gemfile

@@ -2,17 +2,23 @@ source "https://rubygems.org"
 
 if ENV["USE_INSTALLED_GUARD_RSPEC"] == "1"
   gem "guard-rspec"
-  gem "launchy"
 else
-  gemspec
+  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 :test do
   gem "coveralls", require: false
+  gem "rspec", "~> 3.4"
+  gem "launchy", "~> 2.4"
 end
 
 group :development do
-  gem "rspec", "~> 3.4"
   gem "rubocop", require: false
   gem "guard-rubocop", require: false
   gem "guard-compat", ">= 0.0.2", require: false

Diff for: gemfiles/Gemfile.rspec-2.99

@@ -1,14 +1,4 @@
-source "https://rubygems.org"
-
-gemspec path: '../'
+filename = "gemfiles/common"
+instance_eval(IO.read(filename), filename, 1)
 
 gem 'rspec', '~> 2.99'
-
-group :test do
-  gem 'coveralls', require: false
-  gem "guard-compat", ">= 0.0.2", require: false
-end
-
-group :tool do
-  gem 'ruby_gntp', require: false
-end

Diff for: gemfiles/Gemfile.rspec-3.4

@@ -1,14 +1,4 @@
-source "https://rubygems.org"
-
-gemspec path: '../'
+filename = "gemfiles/common"
+instance_eval(IO.read(filename), filename, 1)
 
 gem 'rspec', '~> 3.4.0'
-
-group :test do
-  gem 'coveralls', require: false
-  gem "guard-compat", ">= 0.0.2", require: false
-end
-
-group :tool do
-  gem 'ruby_gntp', require: false
-end

Diff for: gemfiles/common

@@ -0,0 +1,7 @@
+source "https://rubygems.org"
+gemspec path: '../'
+
+gem "bundler", "~> 1.12", "< 2.0"
+gem "rake", "~> 11.1"
+gem 'coveralls', require: false
+gem "launchy", "~> 2.4"

Diff for: guard-rspec.gemspec

@@ -22,8 +22,4 @@ Gem::Specification.new do |s|
   s.add_dependency "guard", "~> 2.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"
-  s.add_development_dependency "rake", "~> 10.1"
-  s.add_development_dependency "launchy", "~> 2.4"
 end

Diff for: spec/lib/guard/rspec/rspec_process_spec.rb

@@ -3,12 +3,6 @@
 require "guard/rspec/rspec_process"
 
 RSpec.describe Guard::RSpec::RSpecProcess do
-  before do
-    allow(Kernel).to receive(:spawn) do |*args|
-      raise "Not stubbed: Kernel.spawn(#{args.map(&:inspect) * ','})"
-    end
-  end
-
   let(:results) { instance_double(Guard::RSpec::Results) }
 
   let(:cmd) { "foo" }

Diff for: spec/spec_helper.rb

@@ -136,7 +136,7 @@ def instance_double(*args)
     end
 
     %w(spawn system).each do |meth|
-      allow(File).to receive(meth.to_sym) do |*args|
+      allow(Kernel).to receive(meth.to_sym) do |*args|
         abort "stub me: Kernel.#{meth}(#{args.map(&:inspect) * ','})!"
       end
     end