Merge pull request #309 from jrgriffiniii/fix-generator

Handle cases where the BrowseEverything configuration is not in place with a warning and empty configuration values

browse-everything.gemspec

@@ -26,16 +26,16 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'google_drive', '~> 2.1'
   spec.add_dependency 'googleauth', '0.6.6'
   spec.add_dependency 'puma', '~> 3.11'
-  spec.add_dependency 'rails', '>= 4.2'
+  spec.add_dependency 'rails', '>= 4.2', '< 6.0'
   spec.add_dependency 'ruby-box'
   spec.add_dependency 'signet', '~> 0.8'
+  spec.add_dependency 'sprockets', '~> 3.7'
   spec.add_dependency 'thor', '~> 0.19'
   spec.add_dependency 'typhoeus'
 
   spec.add_development_dependency 'bixby', '>= 1.0'
   spec.add_development_dependency 'bundler', '>= 1.3'
   spec.add_development_dependency 'capybara'
-  spec.add_development_dependency 'chromedriver-helper', '~> 2.1'
   spec.add_development_dependency 'coveralls'
   spec.add_development_dependency 'engine_cart', '~> 2.0'
   spec.add_development_dependency 'factory_bot_rails'
@@ -51,5 +51,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'selenium-webdriver'
   spec.add_development_dependency 'simplecov'
   spec.add_development_dependency 'sqlite3'
+  spec.add_development_dependency 'webdrivers'
   spec.add_development_dependency 'webmock'
 end

lib/browse_everything.rb

@@ -59,7 +59,8 @@ def configure(value)
           @config = ActiveSupport::HashWithIndifferentAccess.new config_values
           @config.deep_symbolize_keys
         rescue Errno::ENOENT
-          raise ConfigurationError, 'Missing browse_everything_providers.yml configuration file'
+          Rails.logger.warn 'Missing browse_everything_providers.yml configuration file'
+          @config = ActiveSupport::HashWithIndifferentAccess.new({})
         end
       else
         raise InitializationError, "Unrecognized configuration: #{value.inspect}"

lib/browse_everything/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BrowseEverything
-  VERSION = '1.0.1'
+  VERSION = '1.0.2'
 end

spec/spec_helper.rb

@@ -36,7 +36,7 @@ def coverage_needed?
 require 'rspec'
 require 'rspec/rails'
 require 'rspec/its'
-require 'chromedriver-helper'
+require 'webdrivers'
 
 # Requires supporting ruby files with custom matchers and macros, etc,
 # in spec/support/ and its subdirectories.
@@ -46,6 +46,12 @@ def coverage_needed?
 WebMock.disable_net_connect!(allow_localhost: true)
 
 RSpec.configure do |config|
+  config.before(:each, type: :feature) { WebMock.disable! }
+  config.after(:each, type: :feature) do
+    WebMock.enable!
+    WebMock.disable_net_connect!(allow_localhost: true)
+  end
+
   config.expect_with :rspec do |c|
     c.syntax = %i[should expect]
   end