Misc cleanup

-- Remove rack-test and webmock as development dependencies (they are not currently used)
-- Use gemspec for dependencies
-- Update gitignore
-- Change double quote to single quote for strings

Author: mattolson

.gitignore

@@ -1,6 +1,4 @@
-# Ignore lockfiles
-/Gemfile.lock
-
-# Ignore test output
-/spec/coverage
-/spec/reports
+.bundle
+Gemfile.lock
+coverage
+reports

.travis.yml

@@ -1,9 +1,9 @@
 language: ruby
 rvm:
   - ruby-head
-  - 2.3.0
-  - 2.2.4
-  - 2.1.8
+  - 2.3.1
+  - 2.2.5
+  - 2.1.10
 matrix:
   allow_failures:
     - rvm: ruby-head

Gemfile

@@ -1,13 +1,3 @@
 source 'https://rubygems.org'
 
-# Specify your gem's dependencies in omniauth-bigcommerce.gemspec
 gemspec
-
-group :development, :test do
-  gem 'rake'
-  gem 'guard'
-  gem 'guard-rspec'
-  gem 'guard-bundler'
-  gem 'rb-fsevent'
-  gem 'growl'
-end

lib/omniauth-bigcommerce.rb

@@ -1,2 +1,2 @@
-require "omniauth/bigcommerce/version"
+require 'omniauth/bigcommerce/version'
 require 'omniauth/strategies/bigcommerce'

lib/omniauth/bigcommerce/version.rb

@@ -1,5 +1,5 @@
 module OmniAuth
   module BigCommerce
-    VERSION = "0.2.0"
+    VERSION = '0.2.0'
   end
 end

lib/omniauth/strategies/bigcommerce.rb

@@ -3,16 +3,11 @@
 module OmniAuth
   module Strategies
     class BigCommerce < OmniAuth::Strategies::OAuth2
-      option :name, "bigcommerce"
-
+      option :name, 'bigcommerce'
       option :provider_ignores_state, true
-
-      option :scope, "users_basic_information"
-
+      option :scope, 'users_basic_information'
       option :authorize_options, [:scope, :context]
-
-      option :client_options,
-      {
+      option :client_options, {
         site: ENV['BC_AUTH_SERVICE'] || 'https://login.bigcommerce.com',
         authorize_url: '/oauth2/authorize',
         token_url: '/oauth2/token'
@@ -29,7 +24,7 @@ class BigCommerce < OmniAuth::Strategies::OAuth2
 
       credentials do
         {
-          :token => access_token
+          token: access_token
         }
       end
 
@@ -45,7 +40,7 @@ def raw_info
         @raw_info ||= access_token.params
       end
 
-      #Copied from OmniAuth Google OAuth2 (https://github.com/zquestz/omniauth-google-oauth2)
+      # Copied from OmniAuth Google OAuth2 (https://github.com/zquestz/omniauth-google-oauth2)
       def authorize_params
         super.tap do |params|
           options[:authorize_options].each do |k|

omniauth-bigcommerce.gemspec

@@ -20,7 +20,5 @@ Gem::Specification.new do |gem|
   gem.add_dependency 'omniauth-oauth2', '>= 1.1.1'
   gem.add_development_dependency 'rake'
   gem.add_development_dependency 'rspec'
-  gem.add_development_dependency 'rack-test'
   gem.add_development_dependency 'simplecov'
-  gem.add_development_dependency 'webmock'
 end

spec/omniauth/strategies/bigcommerce_spec.rb

@@ -4,11 +4,11 @@
   subject do
     OmniAuth::Strategies::BigCommerce.new({})
   end
-  
+
   before do
     OmniAuth.config.test_mode = true
   end
-  
+
   after do
     OmniAuth.config.test_mode = false
   end
@@ -46,7 +46,7 @@
       expect(subject.callback_path).to eq('/auth/bigcommerce/callback')
     end
   end
-  
+
   context 'authorize options' do
     describe 'context' do
       it 'should set the context parameter dynamically in the request' do

spec/spec_helper.rb

@@ -2,6 +2,7 @@
 
 require 'simplecov'
 SimpleCov.start
+
 require 'omniauth-bigcommerce'
 
 RSpec.configure do |config|