omniauth-gplus 2.0.0.beta1 → 2.0.0.beta2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8acb5ae7c83c1728f717c4385e4217bcd8337621
-  data.tar.gz: 2b55f613f0eb0f415bd10c72a1bcdde02d18129a
+  metadata.gz: a54087e815dbb305001bfd818bc13b2e3581ce22
+  data.tar.gz: 269d6dd198735f2233632970ec2bca9260f89000
 SHA512:
-  metadata.gz: a23afce6c3d5f705b0ff975916f5b6fb2eb4270785e159214af5adfae0482efd234b85282ed6908f57de6672510a24140cca9eb3d3f45d07e71b8b47cad6f65e
-  data.tar.gz: ed04c2cc4276876e04d0b5d87120f56a0deea144d208b3decb28cd0b46e056090019a9cf93dc76b4201d739ac4bd4bf21ab5dbf1372930e38b0de97c14de11a7
+  metadata.gz: cc693f35739939f28022213a979ae99371ded6ab092f09dec0e13f5d68df565cd8bfaa87d75941d3a2c03e1d97d792515db7a3409f62e14c017cd93885506f42
+  data.tar.gz: c37d1242a31352fabb84b32d9c6206da16fa82790435fdd31d7d9761508528afe533a38fe1e05f23e74bb91bbc15aaecd4cc80b0096798035d0ca9f7e7e11f0a

data/.rubocop.yml

@@ -0,0 +1,3 @@
+# This is the configuration used to check the rubocop source code.
+
+inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -0,0 +1,78 @@
+AllCops:
+  Include:
+    - 'Gemfile'
+    - 'Rakefile'
+    - 'omniauth-gplus.gemspec'
+
+# Avoid long parameter lists
+ParameterLists:
+  Max: 5
+  CountKeywordArgs: true
+
+MethodLength:
+  CountComments: false
+  Max: 18
+
+# Avoid more than `Max` levels of nesting.
+BlockNesting:
+  Max: 2
+
+# Align with the style guide.
+CollectionMethods:
+  PreferredMethods:
+    map:      'collect'
+    reduce:   'inject'
+    find:     'detect'
+    find_all: 'select'
+
+# Do not force public/protected/private keyword to be indented at the same
+# level as the def keyword. My personal preference is to outdent these keywords
+# because I think when scanning code it makes it easier to identify the
+# sections of code and visually separate them. When the keyword is at the same
+# level I think it sort of blends in with the def keywords and makes it harder
+# to scan the code and see where the sections are.
+AccessModifierIndentation:
+  Enabled: false
+
+# Limit line length
+LineLength:
+  Enabled: false
+
+# Disable documentation checking until a class needs to be documented once
+Documentation:
+  Enabled: false
+
+# Enforce Ruby 1.8-compatible hash syntax
+HashSyntax:
+  EnforcedStyle: hash_rockets
+
+# No spaces inside hash literals
+SpaceInsideHashLiteralBraces:
+  EnforcedStyle: no_space
+
+# Allow dots at the end of lines
+DotPosition:
+  Enabled: false
+
+# Don't require magic comment at the top of every file
+Encoding:
+  Enabled: false
+
+EmptyLinesAroundAccessModifier:
+  Enabled: true
+
+# Align ends correctly
+EndAlignment:
+  AlignWith: variable
+
+# Indentation of when/else
+CaseIndentation:
+  IndentWhenRelativeTo: end
+  IndentOneStep: false
+
+Lambda:
+  Enabled: false
+
+FileName:
+  Exclude:
+    - 'lib/omniauth-gplus.rb'

data/.travis.yml

@@ -1,10 +1,26 @@
+before_install:
+  - gem update bundler
+  - bundle --version
+  - gem update --system 2.1.11
+  - gem --version
 language: ruby
 rvm:
+  - 1.9.2
   - 1.9.3
   - 2.0.0
-  - 2.1.0
-  - ruby-head
-  - rbx-19mode
-  - rbx-2
+  - 2.1
+  - jruby
   - rbx
-  - jruby-19mode
+  - rbx-2
+  - ruby-head
+matrix:
+  include:
+    - rvm: jruby-19mode
+      env: JRUBY_OPTS="$JRUBY_OPTS --debug"
+    - rvm: jruby-head
+      env: JRUBY_OPTS="$JRUBY_OPTS --debug"
+  allow_failures:
+    - rvm: jruby-head
+    - rvm: ruby-head
+    - rvm: rbx
+  fast_finish: true

data/README.md

@@ -17,6 +17,27 @@ omniauth-gplus
 It allows you to easily interact with the Google+ OAuth2 endpoint.
 
 
+Installing
+----------
+
+**Requirements**:
+  A GPlus API key/secret
+
+You can either install manually:
+
+``` bash
+$ gem install omniauth-gplus
+```
+
+Or put it in your `Gemfile`:
+
+``` ruby
+source 'https://rubygems.org'
+
+gem 'omniauth-gplus', '~> 2.0'
+```
+
+
 Examples
 --------
 
@@ -55,6 +76,11 @@ provider :gplus, ENV['GPLUS_KEY'], ENV['GPLUS_SECRET'], scope: 'plus.login', req
 
 To see and learn about all of the available App Activity types, visit: https://developers.google.com/+/api/moment-types/
 
+To get a refresh token please pass the following:
+
+```ruby
+provider :gplus, ENV['GPLUS_KEY'], ENV['GPLUS_SECRET'], scope: 'plus.login', request_visible_actions: 'AddActivity,BuyActivity', access_type: 'offline'
+```
 
 The information schema looks like this:
 
@@ -82,56 +108,15 @@ extra do
 end
 ```
 
-
-Installing
-----------
-
-**Requirements**:
-
-  1. Ruby 2.1, 2.0, 1.9 OR Rubinius 1.9 OR JRuby 1.9
-  2. A GPlus API key/secret
-
-You can either install manually:
-
-``` bash
-$ gem install omniauth-gplus
-```
-
-Or put it in your `Gemfile`:
-
-``` ruby
-source 'https://rubygems.org'
-
-gem 'omniauth-gplus', '~> 2.0'
-```
-
-
-Issues & Documentation
-----------------------
-
-* [Documentation](https://github.com/samdunne/omniauth-gplus/wiki)
-* [Issues](https://github.com/samdunne/omniauth-gplus/issues)
-
-Absolutely feel free to contribute, please!
-
-
-Contributing
-------------
-
-* **Fork** the repository
-* **Clone the repository** locally, or **edit via Github**
-* Create a **new branch** using the [Git Flow Standard](http://yakiloo.com/getting-started-git-flow/) conventions
-* Commit **often** and **when important**
-* **DO NOT CHANGE** ANY OF THESE (without making a new branch for *that* change):
-  * `*.gemspec`
-  * `Rakefile`
-  * `.ruby-version`
-  * `.ruby-gemset`
-  * `.gitignore`
-  * Any part of the git history
-* **Write tests** specifically for the changes you've made, if no test exist
-* **Push** your feature or hotfix branch to Github.
-* Make a **Pull Request**
+Supported Rubies
+----------------
+Actively tested with the following Ruby versions:
+- MRI 2.1.0
+- MRI 2.0.0
+- MRI 1.9.3
+- MRI 1.9.2
+- JRuby 1.7.9
+- Rubinius (latest stable)
 
 
 License

data/Rakefile

@@ -8,24 +8,31 @@ begin
   Bundler.setup :default, :development
 rescue Bundler::BundlerError => error
   $stderr.puts error.message
-  $stderr.puts "Run `bundle install` to install missing gems"
+  $stderr.puts 'Run `bundle install` to install missing gems.'
   exit error.status_code
 end
 
 Bundler::GemHelper.install_tasks
 
-desc "Run all of the tests"
+desc 'Run all of the tests'
 Rake::TestTask.new do |config|
   config.libs << 'test'
   config.pattern = 'test/**/*_test*'
-  # config.verbose = true
-  # config.warning = true
 end
 
-desc "Generate all of the docs"
+desc 'Generate all of the docs'
 YARD::Rake::YardocTask.new do |config|
   config.files = Dir['lib/**/*.rb']
 end
 
+begin
+  require 'rubocop/rake_task'
+  Rubocop::RakeTask.new
+rescue LoadError
+  task :rubocop do
+    $stderr.puts 'Rubocop is disabled'
+  end
+end
+
 desc 'Default: run tests, and generate docs'
-task :default => [ :test, :yard ]
+task :default => [:test, :yard, :rubocop]

data/lib/omniauth-gplus.rb

@@ -1,5 +1,2 @@
-require "omniauth-oauth2"
-require_relative "omniauth/strategies/gplus"
-require_relative "omniauth/gplus/version"
-
-OmniAuth.config.add_camelization("gplus", "GPlus")
+require 'omniauth/gplus'
+OmniAuth.config.add_camelization('gplus', 'GPlus')

data/lib/omniauth/gplus.rb

@@ -0,0 +1,2 @@
+require 'omniauth/gplus/version'
+require 'omniauth/strategies/gplus'

data/lib/omniauth/gplus/version.rb

@@ -1,5 +1,5 @@
 module OmniAuth
   module GPlus
-    VERSION = "2.0.0.beta1"
+    VERSION = '2.0.0.beta2'
   end
 end

data/lib/omniauth/strategies/gplus.rb

@@ -1,11 +1,15 @@
+require 'omniauth/strategies/oauth2'
+
 module OmniAuth
   module Strategies
-    class GPlus < OAuth2
-      option :client_options, {
-        site: 'https://www.googleapis.com/oauth2/v1',
-        authorize_url: 'https://www.google.com/accounts/o8/oauth2/authorization',
-        token_url: 'https://www.google.com/accounts/o8/oauth2/token'
-      }
+    class GPlus < OmniAuth::Strategies::OAuth2
+      class NoAuthorizationCodeError < StandardError; end
+      class UnknownSignatureAlgorithmError < NotImplementedError; end
+
+      option :client_options,
+             :site => 'https://www.googleapis.com/oauth2/v1',
+             :authorize_url => 'https://www.google.com/accounts/o8/oauth2/authorization',
+             :token_url => 'https://www.google.com/accounts/o8/oauth2/token'
 
       option :authorize_options, [:scope, :request_visible_actions]
 
@@ -15,9 +19,7 @@ module OmniAuth
 
       option :uid_field, :uid
 
-      uid do
-        raw_info['id']
-      end
+      uid { raw_info['id'] }
 
       info do
         {
@@ -27,7 +29,7 @@ module OmniAuth
           'last_name' => raw_info['family_name'],
           'image' => raw_info['picture'],
           'urls' => {
-            "Google+" => raw_info['link']
+            'Google+' => raw_info['link']
           }
         }
       end
@@ -44,9 +46,7 @@ module OmniAuth
       def authorize_params
         super.tap do |params|
           params['scope'] = format_scopes(params['scope'])
-          if (params['request_visible_actions'])
-            params['request_visible_actions'] = format_actions(params['request_visible_actions'])
-          end
+          params['request_visible_actions'] = format_actions(params['request_visible_actions']) if params['request_visible_actions']
           custom_parameters(params)
         end
       end
@@ -54,7 +54,7 @@ module OmniAuth
       private
 
       def format_actions(actions)
-        actions.split(/,\s*/).map(&method(:format_action)).join(" ")
+        actions.split(/,\s*/).collect(&method(:format_action)).join(' ')
       end
 
       def format_action(action)
@@ -62,7 +62,7 @@ module OmniAuth
       end
 
       def format_scopes(scopes)
-        scopes.split(/,\s*/).map(&method(:format_scope)).join(" ")
+        scopes.split(/,\s*/).collect(&method(:format_scope)).join(' ')
       end
 
       def format_scope(scope)
@@ -70,13 +70,11 @@ module OmniAuth
       end
 
       def custom_parameters(params)
-        ["scope", "client_options", "request_visible_actions"].each { |k| add_key_to_params(params, k) }
+        %w(scope client_options request_visible_actions access_type).each { |k| add_key_to_params(params, k) }
       end
 
       def add_key_to_params(params, key)
-        if request.params[key]
-          params[key] = request.params[key]
-        end
+        params[key] = request.params[key] if request.params[key]
       end
 
       def raw_info

data/omniauth-gplus.gemspec

@@ -1,34 +1,36 @@
-lib = File.expand_path('../lib', __FILE__)
-$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+# -*- encoding: utf-8 -*-
+$LOAD_PATH.push File.expand_path('../lib', __FILE__)
 require 'omniauth/gplus/version'
 
-Gem::Specification.new do |spec|
-  spec.name          = "omniauth-gplus"
-  spec.version       = OmniAuth::GPlus::VERSION
-  spec.authors       = ["Kurtis Rainbolt-Greene", "Sam Dunne"]
-  spec.email         = ["me@kurtisrainboltgreene.name", "sam@sam-dunne.com"]
-  spec.summary       = %q{A Google+ OAuth2 solution for Omniauth}
-  spec.description   = spec.summary
-  spec.homepage      = "http://samdunne.github.com/omniauth-gplus"
-  spec.license       = "MIT"
+Gem::Specification.new do |s|
+  s.name          = 'omniauth-gplus'
+  s.version       = OmniAuth::GPlus::VERSION
+  s.authors       = %w(Sam Dunne Kurtis Rainbolt-Greene )
+  s.email         = %w(sam@sam-dunne.com me@kurtisrainboltgreene.name)
+  s.summary       = 'A Google+ OAuth2 solution for Omniauth'
+  s.description   = s.summary
+  s.homepage      = 'http://samdunne.github.com/omniauth-gplus'
+  s.license       = 'MIT'
 
-  spec.files         = `git ls-files`.split($/)
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib"]
+  s.files         = `git ls-files`.split("\n")
+  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.executables   = `git ls-files -- bin/*`.split("\n").collect { |f| File.basename(f) }
+  s.require_paths = ['lib']
 
-  spec.add_runtime_dependency "omniauth-oauth2", '~> 1.1'
-  spec.add_development_dependency "bundler"
-  spec.add_development_dependency "rake"
-  spec.add_development_dependency "mocha"
-  spec.add_development_dependency 'yard'
-  spec.add_development_dependency 'kramdown'
-  spec.add_development_dependency 'pry'
-  spec.add_development_dependency 'coveralls'
+  s.add_runtime_dependency 'omniauth-oauth2', '~> 1.1'
+
+  s.add_development_dependency 'bundler'
+  s.add_development_dependency 'rake'
+  s.add_development_dependency 'mocha'
+  s.add_development_dependency 'yard'
+  s.add_development_dependency 'kramdown'
+  s.add_development_dependency 'pry'
+  s.add_development_dependency 'coveralls'
+  s.add_development_dependency 'rubocop'
 
   if RUBY_PLATFORM == 'rbx'
-    spec.add_development_dependency 'racc'
-    spec.add_development_dependency 'rubinius-coverage', github: 'rubinius/rubinius-coverage'
-    spec.add_development_dependency 'rubysl'
+    s.add_development_dependency 'racc'
+    s.add_development_dependency 'rubinius-coverage', :github => 'rubinius/rubinius-coverage'
+    s.add_development_dependency 'rubysl'
   end
 end

data/rubocop-todo.yml

@@ -0,0 +1,40 @@
+# This configuration was generated by `rubocop --auto-gen-config`
+# on 2014-05-11 15:31:15 +0100 using RuboCop version 0.21.0.
+# The point is for the user to remove these configuration records
+# one by one as the offenses are removed from the code base.
+# Note that changes in the inspected code, or installation of new
+# versions of RuboCop, may require this file to be generated again.
+
+# Offense count: 1
+Documentation:
+  Enabled: true
+
+# Offense count: 1
+# Configuration parameters: Exclude.
+FileName:
+  Enabled: true
+
+# Offense count: 1
+LineLength:
+  Max: 120
+
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: PreferredDelimiters.
+PercentLiteralDelimiters:
+  Enabled: false
+
+# Offense count: 2
+RegexpLiteral:
+  MaxSlashes: 0
+
+# Offense count: 1
+# Cop supports --auto-correct.
+SpecialGlobalVars:
+  Enabled: false
+
+# Offense count: 17
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+StringLiterals:
+  Enabled: true

data/test/helper.rb

@@ -3,12 +3,12 @@ Coveralls.wear!
 
 require 'omniauth-gplus'
 require 'minitest/autorun'
-require 'minitest/pride'
+# require 'minitest/pride'
 
 OmniAuth.config.test_mode = true
 
 def application
-  ->(env) { [200, {}, ["Hello World."]]}
+  ->(_env) { [200, {}, ['Hello World.']] }
 end
 
 def strategy

data/test/lib/omniauth/strategies/gplus_test.rb

@@ -1,6 +1,7 @@
 require 'helper'
 require 'mocha/setup'
 
+# Test class for omniauth-gplus
 class TestOmniAuthGPlus < MiniTest::Unit::TestCase
   def setup
     @request = mock('Request')
@@ -44,13 +45,13 @@ class TestOmniAuthGPlus < MiniTest::Unit::TestCase
 
   def test_state_state_is_not_included_in_request_params_when_present
     expected = 'some_state'
-    @request.stubs(:params).returns({ 'state' => expected })
+    @request.stubs(:params).returns('state' => expected)
     refute_equal expected, strategy.authorize_params[:state]
   end
 
   def test_do_not_store_state_in_the_session_when_present
     expected = 'some_state'
-    @request.stubs(:params).returns({ 'state' => expected })
+    @request.stubs(:params).returns('state' => expected)
     refute_empty strategy.authorize_params['state']
     refute_equal expected, strategy.authorize_params[:state]
     refute_empty strategy.session['omniauth.state']

metadata

@@ -1,15 +1,17 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-gplus
 version: !ruby/object:Gem::Version
-  version: 2.0.0.beta1
+  version: 2.0.0.beta2
 platform: ruby
 authors:
-- Kurtis Rainbolt-Greene
-- Sam Dunne
+- Sam
+- Dunne
+- Kurtis
+- Rainbolt-Greene
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-11 00:00:00.000000000 Z
+date: 2014-05-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth-oauth2
@@ -123,16 +125,32 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A Google+ OAuth2 solution for Omniauth
 email:
-- me@kurtisrainboltgreene.name
 - sam@sam-dunne.com
+- me@kurtisrainboltgreene.name
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
 - .coveralls.yml
 - .gitignore
+- .rubocop.yml
+- .rubocop_todo.yml
 - .ruby-gemset
 - .ruby-version
 - .travis.yml
@@ -142,9 +160,11 @@ files:
 - README.md
 - Rakefile
 - lib/omniauth-gplus.rb
+- lib/omniauth/gplus.rb
 - lib/omniauth/gplus/version.rb
 - lib/omniauth/strategies/gplus.rb
 - omniauth-gplus.gemspec
+- rubocop-todo.yml
 - test/helper.rb
 - test/lib/omniauth/strategies/gplus_test.rb
 homepage: http://samdunne.github.com/omniauth-gplus