Merge pull request #20 from guard/fix_notifier_in_tests

prevent notifier from crashing in tests

Author: e2

Gemfile

@@ -1,6 +1,10 @@
 source "https://rubygems.org"
 
-gemspec development_group: :test
+gemspec development_group: :gem_build_tools
+
+group :gem_build_tools do
+  gem "rake"
+end
 
 # The development group will no be
 # installed on Travis CI.
@@ -15,6 +19,10 @@ group :development do
   gem "guard-compat", require: false
 end
 
+group :test do
+  gem "rspec", "~> 3.1"
+end
+
 platforms :rbx do
   gem "racc"
   gem "rubysl", "~> 2.0"

guard-cucumber.gemspec

@@ -17,15 +17,11 @@ Gem::Specification.new do |s|
 
   s.required_rubygems_version = ">= 1.3.6"
 
-  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"
-  s.add_development_dependency "rake"
-  s.add_development_dependency "rspec", "~> 3.1"
+  s.add_development_dependency "bundler", "~> 1.6"
 
   s.files = `git ls-files -z`.split("\x0").select do |f|
     /^lib\// =~ f

lib/guard/cucumber/notification_formatter.rb

@@ -1,6 +1,15 @@
 # Conditionally require this because it's run outside Guard
-# TODO: MOVE THIS OUTSIDE THE FORMATTER!!!!
-require "guard/notifier"
+#
+if Object.const_defined?(:Guard)
+  # TODO: MOVE THIS OUTSIDE THE FORMATTER!!!!
+  # TODO: (call notify() in Guard::Cucumber, not here in formatter
+
+  # If notifier is defined it's likely Guard::Compat::Plugin's stub
+  unless Guard.const_defined?(:Notifier)
+    require "guard"
+    require "guard/notifier"
+  end
+end
 
 require "cucumber"
 require "guard/compat/plugin"