Cleaned up project structure.

- Updated gems
- gemspec only has direct dependencies of the gem.
- Dev and Test dependencies are in Gemfile
- Removed tests (will write from scratch)

Author: hzalaz

.gitignore

@@ -1,4 +1,6 @@
 .ruby-version
 coverage
 Gemfile.lock
-*.gem
+*.gem
+
+.#*

.rspec

@@ -0,0 +1,3 @@
+--color
+--require spec_helper
+--format=progress

Gemfile

@@ -1,4 +1,16 @@
 source "http://rubygems.org"
 
 # Specify your gem's dependencies in omniauth-auth0.gemspec
-gemspec
+gemspec
+
+gem "gem-release"
+gem "rake"
+
+group :test do
+  gem "pry"
+  gem "rack-test"
+  gem "rspec", "~> 3.5"
+  gem "rubocop", ">= 0.30", :platforms => [:ruby_19, :ruby_20, :ruby_21, :ruby_22]
+  gem "simplecov"
+  gem "webmock"
+end

Rakefile

@@ -5,6 +5,15 @@ require 'rspec/core/rake_task'
 desc "Run specs"
 RSpec::Core::RakeTask.new
 
+begin
+  require 'rubocop/rake_task'
+  RuboCop::RakeTask.new
+rescue LoadError
+  task :rubocop do
+    $stderr.puts 'Rubocop is disabled'
+  end
+end
+
 desc 'Run specs'
-task :default => :spec
-task :test => :spec
+task :default => [:spec, :rubocop]
+task :test => :spec

examples/ruby-on-rails-webapp/config/initializers/auth0.rb

@@ -1,9 +1,9 @@
 Rails.application.config.middleware.use OmniAuth::Builder do
   provider(
     :auth0,
-    ENV["AUTH0_CLIENT_ID"],
-    ENV["AUTH0_CLIENT_SECRET"],
-    ENV["AUTH0_DOMAIN"],
+    client_id: ENV["AUTH0_CLIENT_ID"],
+    client_secret: ENV["AUTH0_CLIENT_SECRET"],
+    namespace: ENV["AUTH0_DOMAIN"],
     callback_path: "/auth/auth0/callback"
   )
 end

lib/omniauth/strategies/auth0.rb

@@ -1,6 +1,7 @@
-require "base64"
-require "omniauth-oauth2"
+require 'base64'
+require 'omniauth-oauth2'
 
+# Auth0 OmniAuth strategy
 module OmniAuth
   module Strategies
     class Auth0 < OmniAuth::Strategies::OAuth2
@@ -15,9 +16,9 @@ class Auth0 < OmniAuth::Strategies::OAuth2
       option :connection
 
       option :client_options, {
-        authorize_url: "/authorize",
-        token_url: "/oauth/token",
-        userinfo_url: "/userinfo"
+        authorize_url: '/authorize',
+        token_url: '/oauth/token',
+        userinfo_url: '/userinfo'
       }
 
       args [:client_id, :client_secret, :namespace, :provider_ignores_state, :connection]
@@ -35,8 +36,7 @@ def initialize(app, *args, &block)
             "https://#{options[:namespace]}/authorize?#{self.class.client_info_querystring}"
           @options.client_options.token_url =
             "https://#{options[:namespace]}/oauth/token?#{self.class.client_info_querystring}"
-          @options.client_options.userinfo_url =
-            "https://#{options[:namespace]}/userinfo"
+          @options.client_options.userinfo_url = "https://#{options[:namespace]}/userinfo"
         elsif !options[:setup]
           fail(ArgumentError.new("Received wrong number of arguments. #{args.inspect}"))
         end

omniauth-auth0.gemspec

@@ -5,8 +5,8 @@ require "omniauth-auth0/version"
 Gem::Specification.new do |s|
   s.name        = "omniauth-auth0"
   s.version     = OmniAuth::Auth0::VERSION
-  s.authors     = ["Auth0", "Ezequiel Morito", "Jose Romaniello"]
-  s.email       = ["support@auth0.com"]
+  s.authors     = ["Auth0"]
+  s.email       = ["info@auth0.com"]
   s.homepage    = "https://github.com/auth0/omniauth-auth0"
   s.summary     = %q{Omniauth OAuth2 strategy for the Auth0 platform.}
   s.description = %q{Auth0 is an authentication broker that supports social identity providers as well as enterprise identity providers such as Active Directory, LDAP, Google Apps, Salesforce.
@@ -23,14 +23,9 @@ omniauth-auth0 is the omniauth strategy for Auth0.
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
 
-  s.add_runtime_dependency 'omniauth-oauth2', '~> 1.1'
+  s.add_runtime_dependency 'omniauth-oauth2', '~> 1.4'
 
-  s.add_development_dependency 'rspec',     '~> 2.7'
-  s.add_development_dependency 'rack-test', '~> 0.6', '>= 0.6.3'
-  s.add_development_dependency 'simplecov', '~> 0.9', '>= 0.9.1'
-  s.add_development_dependency 'webmock', '~> 1.20', '>= 1.20.4'
-  s.add_development_dependency 'rake', '~> 10.3', '>= 10.3.2'
-  s.add_development_dependency 'gem-release', '~> 0.7'
+  s.add_development_dependency 'bundler', '~> 1.9'
 
   s.license = 'MIT'
 end

spec/omniauth/strategies/auth0_spec.rb

@@ -1,17 +1,19 @@
-$:.unshift File.expand_path('..', __FILE__)
-$:.unshift File.expand_path('../../lib', __FILE__)
-require 'simplecov'
-SimpleCov.start
+$LOAD_PATH.unshift File.expand_path("..", __FILE__)
+$LOAD_PATH.unshift File.expand_path("../../lib", __FILE__)
+require "simplecov"
+SimpleCov.start do
+  minimum_coverage(89.8)
+end if ENV['COVERAGE']
 require 'rspec'
 require 'rack/test'
 require 'webmock/rspec'
 require 'omniauth'
 require 'omniauth-auth0'
 
 RSpec.configure do |config|
-  config.include	WebMock::API
-  config.include	Rack::Test::Methods
-  config.extend		OmniAuth::Test::StrategyMacros, :type => :strategy
-  config.color = true
-  config.formatter = 'documentation'
-end
+  config.include WebMock::API
+  config.include Rack::Test::Methods
+  config.extend OmniAuth::Test::StrategyMacros, :type => :strategy
+end
+
+OmniAuth.config.logger = Logger.new("/dev/null")