rails_sso 0.2.1 → 0.3.0

data/test/lib/rails_sso/helpers_test.rb

@@ -0,0 +1,74 @@
+require 'test_helper'
+
+class RailsSso::HelpersTest < ActiveSupport::TestCase
+  class DummyController
+    def self.helper_method(*list)
+    end
+
+    include RailsSso::Helpers
+
+    attr_reader :session
+
+    def initialize(session)
+      @session = session
+    end
+  end
+
+  AccessToken = Struct.new(:token, :refresh_token)
+
+  def setup
+    RailsSso.provider_name = 'OAuth2'
+    RailsSso.provider_key = 'provider_key'
+    RailsSso.provider_secret = 'provider_secret'
+
+    @controller = DummyController.new(session)
+  end
+
+  def session
+    @session ||= {}
+  end
+
+  test "#current_user_data will return nil if not authenticated" do
+    assert_nil @controller.current_user_data
+  end
+
+  test "#current_user_data will return user data if authenticated" do
+    # TODO: pending
+  end
+
+  test "#user_signed_in? returns false if not authenticated" do
+    refute @controller.user_signed_in?
+  end
+
+  test "#user_signed_in? returns true if authenticated" do
+    # TODO: pending
+  end
+
+  test "#access_token returns a new OAuth2::AccessToken object" do
+    session[:access_token] = 'abc'
+    session[:refresh_token] = 'def'
+
+    access_token = @controller.access_token
+
+    assert_instance_of OAuth2::AccessToken, access_token
+    assert_equal 'abc', access_token.token
+    assert_equal 'def', access_token.refresh_token
+  end
+
+  test "#invalidate_access_token! will call Single Sign-Out with access token" do
+    # @TODO: pending
+  end
+
+  test "save_access_token! will copy access and refresh token to session" do
+    access_token = AccessToken.new('abc', '1337')
+
+    @controller.save_access_token!(access_token)
+
+    assert_equal 'abc', session[:access_token]
+    assert_equal '1337', session[:refresh_token]
+  end
+
+  test "#refresh_access_token! will refresh token and copy new values" do
+    # @TODO: pending
+  end
+end

data/test/lib/rails_sso/response_error_test.rb

@@ -0,0 +1,21 @@
+require 'test_helper'
+
+class RailsSso::ResponseErrorTest < ActiveSupport::TestCase
+  test "assigns error code" do
+    err = RailsSso::ResponseError.new(:err_code)
+
+    assert_equal err.code, :err_code
+  end
+
+  test "assigns unauthenticated error message from locales" do
+    err = RailsSso::ResponseError.new(:unauthenticated)
+
+    assert_equal err.message, "You're not authenticated"
+  end
+
+  test "assigns unknown error message from locales" do
+    err = RailsSso::ResponseError.new(:unknown)
+
+    assert_equal err.message, "Something wrong happened"
+  end
+end

data/test/services/rails_sso/fetch_user_test.rb

@@ -1,39 +1,55 @@
 require 'test_helper'
 
 class RailsSso::FetchUserTest < ActiveSupport::TestCase
-  class AccessToken
-    def initialize(client)
-      @client = client
-    end
+  test "success call should fetch user with access token and return parsed data" do
+    data = RailsSso::FetchUser.new(success_client).call
 
-    def get(path)
-      OAuth2::Response.new(@client.get(path))
-    end
+    assert_equal data['name'], user_data['name']
+    assert_equal data['email'], user_data['email']
   end
 
-  def setup
-    @access_token = AccessToken.new(setup_access_token_client)
+  test "unauthenticated call should raise error" do
+    err = assert_raises(RailsSso::ResponseError) { RailsSso::FetchUser.new(unauthenticated_client).call }
+    assert_equal :unauthenticated, err.code
   end
 
-  test "call should fetch user with access token and return parsed data" do
-    data = RailsSso::FetchUser.new(@access_token).call
+  test "unknown call should raise error" do
+    err = assert_raises(RailsSso::ResponseError) { RailsSso::FetchUser.new(unknown_client).call }
+    assert_equal :unknown, err.code
+  end
 
-    assert_equal data['name'], user_data['name']
-    assert_equal data['email'], user_data['email']
+  def user_data
+    {
+      'name' => 'Kowalski',
+      'email' => 'jan@kowalski.pl'
+    }
   end
 
-  def setup_access_token_client
+  def response_headers
+    { 'Content-Type' => 'application/json' }
+  end
+
+  def success_client
     Faraday.new do |builder|
       builder.adapter :test do |stub|
-        stub.get('/api/v1/me') { |env| [200, { 'Content-Type' => 'application/json' }, user_data] }
+        stub.get('/api/v1/me') { |env| [200, response_headers, user_data] }
       end
     end
   end
 
-  def user_data
-    {
-      'name' => 'Kowalski',
-      'email' => 'jan@kowalski.pl'
-    }
+  def unauthenticated_client
+    Faraday.new do |builder|
+      builder.adapter :test do |stub|
+        stub.get('/api/v1/me') { |env| [401, response_headers, {}] }
+      end
+    end
+  end
+
+  def unknown_client
+    Faraday.new do |builder|
+      builder.adapter :test do |stub|
+        stub.get('/api/v1/me') { |env| [500, response_headers, {}] }
+      end
+    end
   end
 end

data/test/test_helper.rb

@@ -1,8 +1,9 @@
+require "codeclimate-test-reporter"
+CodeClimate::TestReporter.start
 # Configure Rails Environment
 ENV["RAILS_ENV"] = "test"
 
 require File.expand_path("../../test/dummy/config/environment.rb",  __FILE__)
-ActiveRecord::Migrator.migrations_paths = [File.expand_path("../../test/dummy/db/migrate", __FILE__)]
 require "rails/test_help"
 require "mocha/mini_test"
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails_sso
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Jan Dudulski
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-17 00:00:00.000000000 Z
+date: 2015-02-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -107,11 +107,15 @@ files:
 - app/controllers/rails_sso/sessions_controller.rb
 - app/services/rails_sso/fetch_user.rb
 - config/initializers/omniauth.rb
+- config/locales/en.yml
 - config/routes.rb
+- lib/generators/rails_sso_generator.rb
+- lib/generators/templates/sso.rb
 - lib/rails_sso.rb
-- lib/rails_sso/access_token.rb
+- lib/rails_sso/client.rb
 - lib/rails_sso/engine.rb
 - lib/rails_sso/helpers.rb
+- lib/rails_sso/response_error.rb
 - lib/rails_sso/version.rb
 - lib/tasks/rails_sso_tasks.rake
 - test/controllers/rails_sso/sessions_controller_test.rb
@@ -146,12 +150,14 @@ files:
 - test/dummy/config/locales/en.yml
 - test/dummy/config/routes.rb
 - test/dummy/config/secrets.yml
-- test/dummy/db/test.sqlite3
+- test/dummy/log/development.log
 - test/dummy/log/test.log
 - test/dummy/public/404.html
 - test/dummy/public/422.html
 - test/dummy/public/500.html
 - test/dummy/public/favicon.ico
+- test/lib/rails_sso/helpers_test.rb
+- test/lib/rails_sso/response_error_test.rb
 - test/rails_sso_test.rb
 - test/routes/sso_routes_test.rb
 - test/services/rails_sso/fetch_user_test.rb
@@ -183,10 +189,11 @@ summary: SSO Rails Engine
 test_files:
 - test/controllers/rails_sso/sessions_controller_test.rb
 - test/rails_sso_test.rb
+- test/lib/rails_sso/response_error_test.rb
+- test/lib/rails_sso/helpers_test.rb
 - test/test_helper.rb
 - test/services/rails_sso/fetch_user_test.rb
 - test/dummy/README.rdoc
-- test/dummy/db/test.sqlite3
 - test/dummy/public/favicon.ico
 - test/dummy/public/500.html
 - test/dummy/public/422.html
@@ -221,5 +228,6 @@ test_files:
 - test/dummy/app/assets/javascripts/application.js
 - test/dummy/app/assets/stylesheets/application.css
 - test/dummy/log/test.log
+- test/dummy/log/development.log
 - test/dummy/Rakefile
 - test/routes/sso_routes_test.rb

data/lib/rails_sso/access_token.rb

@@ -1,52 +0,0 @@
-require 'faraday-http-cache'
-
-module RailsSso
-  class AccessToken
-    attr_reader :token, :refresh_token
-
-    delegate :get, :patch, :post, :put, :delete, to: :access_token
-
-    def self.from_access_token(access_token)
-      new(access_token.token, access_token.refresh_token)
-    end
-
-    def initialize(token, refresh_token)
-      @token, @refresh_token = token, refresh_token
-    end
-
-    def refresh!
-      self.class.from_access_token(access_token.refresh!)
-    end
-
-    def access_token
-      @access_token ||= OAuth2::AccessToken.new(client, token, {
-        refresh_token: refresh_token
-      })
-    end
-
-    private
-
-    def client(cache = RailsSso.use_cache)
-      strategy.client.tap do |c|
-        if cache
-          c.options[:connection_build] = Proc.new do |conn|
-            conn.use :http_cache,
-              store: Rails.cache,
-              logger: Rails.logger,
-              shared_cache: false
-
-            conn.adapter Faraday.default_adapter
-          end
-        end
-      end
-    end
-
-    def strategy
-      @strategy ||= strategy_class.new(nil, RailsSso.provider_key, RailsSso.provider_secret)
-    end
-
-    def strategy_class
-      "OmniAuth::Strategies::#{RailsSso.provider_name.camelize}".constantize
-    end
-  end
-end