Merge pull request #18 from guard/fix_cucumber_requires

fix crashes due to missing/wrong requires

Rakefile

@@ -1,8 +1,13 @@
 require "bundler"
+
+require 'nenv'
 Bundler::GemHelper.install_tasks
 
 require "rspec/core/rake_task"
-RSpec::Core::RakeTask.new(:spec)
+RSpec::Core::RakeTask.new(:spec) do |t|
+  t.verbose = Nenv.ci?
+end
+
 task default: :spec
 
 require "rbconfig"

guard-cucumber.gemspec

@@ -20,6 +20,7 @@ Gem::Specification.new do |s|
   s.add_dependency "guard",       ">= 2.0.0"
   s.add_dependency "guard-compat",       "~> 1.0"
   s.add_dependency "cucumber",    ">= 1.3.0"
+  s.add_dependency "nenv",    "~> 0.1"
 
   # NOTE: this is the :test group since gemspec(development_group: :test) is used
   s.add_development_dependency "bundler", "~> 1.1"

lib/guard/cucumber/focuser.rb

@@ -1,3 +1,5 @@
+require "guard/compat/plugin"
+
 module Guard
   class Cucumber < Plugin
     # The Cucumber focuser updates cucumber feature paths to

lib/guard/cucumber/inspector.rb

@@ -1,3 +1,5 @@
+require "guard/compat/plugin"
+
 module Guard
   class Cucumber < Plugin
     # The inspector verifies of the changed paths are valid

lib/guard/cucumber/notification_formatter.rb

@@ -1,3 +1,10 @@
+# Conditionally require this because it's run outside Guard
+# TODO: MOVE THIS OUTSIDE THE FORMATTER!!!!
+require "guard/notifier"
+
+require "cucumber"
+require "guard/compat/plugin"
+
 require "cucumber/formatter/console"
 require "cucumber/formatter/io"
 
@@ -69,7 +76,7 @@ def after_feature_element(feature_element)
       def step_name(_keyword, step_match, status, _src_indent, _bckgnd, _loc)
         return unless [:failed, :pending, :undefined].index(status)
 
-        #TODO: "NO COVERATE HERE!!"
+        # TODO: NO COVERAGE HERE!
         @rerun = true
         step_name = step_match.format_args(lambda { |param| "*#{ param }*" })
 
@@ -83,6 +90,7 @@ def step_name(_keyword, step_match, status, _src_indent, _bckgnd, _loc)
       # result of the feature tests.
       #
       def notify_summary
+        # TODO: MOVE THIS OUTSIDE THE FORMATTER!!!!
         statuses = [:failed, :skipped, :undefined, :pending, :passed]
         statuses = statuses.reverse
         statuses.select! { |status| step_mother.steps(status).any? }

lib/guard/cucumber/runner.rb

@@ -1,3 +1,7 @@
+require "guard/compat/plugin"
+
+require "guard/cucumber/focuser"
+
 module Guard
   class Cucumber < Plugin
     # The Cucumber runner handles the execution of the cucumber binary.

spec/guard/cucumber/focuser_spec.rb

@@ -1,3 +1,5 @@
+require "guard/compat/test/helper"
+
 require "guard/cucumber/focuser"
 
 RSpec.describe Guard::Cucumber::Focuser do

spec/guard/cucumber_spec.rb

@@ -1,3 +1,6 @@
+require "guard/compat/test/helper"
+require "guard/cucumber"
+
 RSpec.describe Guard::Cucumber do
   subject { Guard::Cucumber.new(options) }
 

spec/spec_helper.rb

@@ -1,6 +1,3 @@
-require "guard/compat/test/helper"
-require "guard/cucumber"
-
 # This file was generated by the `rspec --init` command. Conventionally, all
 # specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
 # The generated `.rspec` file contains `--require spec_helper` which will cause