doorkeeper 4.2.6 → 5.5.4

data/spec/generators/migration_generator_spec.rb

@@ -1,20 +0,0 @@
-require 'spec_helper_integration'
-require 'generators/doorkeeper/migration_generator'
-
-describe 'Doorkeeper::MigrationGenerator' do
-  include GeneratorSpec::TestCase
-
-  tests Doorkeeper::MigrationGenerator
-  destination ::File.expand_path('../tmp/dummy', __FILE__)
-
-  describe 'after running the generator' do
-    before :each do
-      prepare_destination
-      run_generator
-    end
-
-    it 'creates a migration' do
-      assert_migration 'db/migrate/create_doorkeeper_tables.rb'
-    end
-  end
-end

data/spec/generators/templates/routes.rb

@@ -1,3 +0,0 @@
-Rails.application.routes.draw do
-
-end

data/spec/generators/views_generator_spec.rb

@@ -1,27 +0,0 @@
-require 'spec_helper_integration'
-require 'generators/doorkeeper/views_generator'
-
-describe Doorkeeper::Generators::ViewsGenerator do
-  include GeneratorSpec::TestCase
-
-  tests Doorkeeper::Generators::ViewsGenerator
-  destination File.expand_path('../tmp/dummy', __FILE__)
-
-  before :each do
-    prepare_destination
-  end
-
-  it 'create all views' do
-    run_generator
-    assert_file 'app/views/doorkeeper/applications/_form.html.erb'
-    assert_file 'app/views/doorkeeper/applications/edit.html.erb'
-    assert_file 'app/views/doorkeeper/applications/index.html.erb'
-    assert_file 'app/views/doorkeeper/applications/new.html.erb'
-    assert_file 'app/views/doorkeeper/applications/show.html.erb'
-
-    assert_file 'app/views/doorkeeper/authorizations/error.html.erb'
-    assert_file 'app/views/doorkeeper/authorizations/new.html.erb'
-
-    assert_file 'app/views/doorkeeper/authorized_applications/index.html.erb'
-  end
-end

data/spec/helpers/doorkeeper/dashboard_helper_spec.rb

@@ -1,24 +0,0 @@
-require 'spec_helper_integration'
-
-describe Doorkeeper::DashboardHelper do
-  describe '.doorkeeper_errors_for' do
-    let(:object) { double errors: { method: messages } }
-    let(:messages) { ['first message', 'second message'] }
-
-    context 'when object has errors' do
-      it 'returns error messages' do
-        messages.each do |message|
-          expect(helper.doorkeeper_errors_for(object, :method)).to include(
-            message.capitalize
-          )
-        end
-      end
-    end
-
-    context 'when object has no errors' do
-      it 'returns nil' do
-        expect(helper.doorkeeper_errors_for(object, :amonter_method)).to be_nil
-      end
-    end
-  end
-end

data/spec/lib/config_spec.rb

@@ -1,334 +0,0 @@
-require 'spec_helper_integration'
-
-describe Doorkeeper, 'configuration' do
-  subject { Doorkeeper.configuration }
-
-  describe 'resource_owner_authenticator' do
-    it 'sets the block that is accessible via authenticate_resource_owner' do
-      block = proc {}
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        resource_owner_authenticator(&block)
-      end
-      expect(subject.authenticate_resource_owner).to eq(block)
-    end
-  end
-
-  describe 'setup_orm_adapter' do
-    it 'adds specific error message to NameError exception' do
-      expect do
-        Doorkeeper.configure { orm 'hibernate' }
-      end.to raise_error(NameError, /ORM adapter not found \(hibernate\)/)
-    end
-
-    it 'does not change other exceptions' do
-      allow_any_instance_of(String).to receive(:classify) { raise NoMethodError }
-
-      expect do
-        Doorkeeper.configure { orm 'hibernate' }
-      end.to raise_error(NoMethodError, /ORM adapter not found \(hibernate\)/)
-    end
-  end
-
-  describe 'admin_authenticator' do
-    it 'sets the block that is accessible via authenticate_admin' do
-      block = proc {}
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        admin_authenticator(&block)
-      end
-      expect(subject.authenticate_admin).to eq(block)
-    end
-  end
-
-  describe 'access_token_expires_in' do
-    it 'has 2 hours by default' do
-      expect(subject.access_token_expires_in).to eq(2.hours)
-    end
-
-    it 'can change the value' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        access_token_expires_in 4.hours
-      end
-      expect(subject.access_token_expires_in).to eq(4.hours)
-    end
-
-    it 'can be set to nil' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        access_token_expires_in nil
-      end
-      expect(subject.access_token_expires_in).to be_nil
-    end
-  end
-
-  describe 'scopes' do
-    it 'has default scopes' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        default_scopes :public
-      end
-      expect(subject.default_scopes).to include('public')
-    end
-
-    it 'has optional scopes' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        optional_scopes :write, :update
-      end
-      expect(subject.optional_scopes).to include('write', 'update')
-    end
-
-    it 'has all scopes' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        default_scopes  :normal
-        optional_scopes :admin
-      end
-      expect(subject.scopes).to include('normal', 'admin')
-    end
-  end
-
-  describe 'use_refresh_token' do
-    it 'is false by default' do
-      expect(subject.refresh_token_enabled?).to be_falsey
-    end
-
-    it 'can change the value' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        use_refresh_token
-      end
-      expect(subject.refresh_token_enabled?).to be_truthy
-    end
-
-    it "does not includes 'refresh_token' in authorization_response_types" do
-      expect(subject.token_grant_types).not_to include 'refresh_token'
-    end
-
-    context "is enabled" do
-      before do
-        Doorkeeper.configure {
-          orm DOORKEEPER_ORM
-          use_refresh_token
-        }
-      end
-
-      it "includes 'refresh_token' in authorization_response_types" do
-        expect(subject.token_grant_types).to include 'refresh_token'
-      end
-    end
-  end
-
-  describe 'client_credentials' do
-    it 'has defaults order' do
-      expect(subject.client_credentials_methods).to eq([:from_basic, :from_params])
-    end
-
-    it 'can change the value' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        client_credentials :from_digest, :from_params
-      end
-      expect(subject.client_credentials_methods).to eq([:from_digest, :from_params])
-    end
-  end
-
-  describe 'force_ssl_in_redirect_uri' do
-    it 'is true by default in non-development environments' do
-      expect(subject.force_ssl_in_redirect_uri).to be_truthy
-    end
-
-    it 'can change the value' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        force_ssl_in_redirect_uri(false)
-      end
-      expect(subject.force_ssl_in_redirect_uri).to be_falsey
-    end
-  end
-
-  describe 'access_token_methods' do
-    it 'has defaults order' do
-      expect(subject.access_token_methods).to eq([:from_bearer_authorization, :from_access_token_param, :from_bearer_param])
-    end
-
-    it 'can change the value' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        access_token_methods :from_access_token_param, :from_bearer_param
-      end
-      expect(subject.access_token_methods).to eq([:from_access_token_param, :from_bearer_param])
-    end
-  end
-
-  describe 'enable_application_owner' do
-    it 'is disabled by default' do
-      expect(Doorkeeper.configuration.enable_application_owner?).not_to be_truthy
-    end
-
-    context 'when enabled without confirmation' do
-      before do
-        Doorkeeper.configure do
-          orm DOORKEEPER_ORM
-          enable_application_owner
-        end
-      end
-      it 'adds support for application owner' do
-        expect(Doorkeeper::Application.new).to respond_to :owner
-      end
-      it 'Doorkeeper.configuration.confirm_application_owner? returns false' do
-        expect(Doorkeeper.configuration.confirm_application_owner?).not_to be_truthy
-      end
-    end
-
-    context 'when enabled with confirmation set to true' do
-      before do
-        Doorkeeper.configure do
-          orm DOORKEEPER_ORM
-          enable_application_owner confirmation: true
-        end
-      end
-      it 'adds support for application owner' do
-        expect(Doorkeeper::Application.new).to respond_to :owner
-      end
-      it 'Doorkeeper.configuration.confirm_application_owner? returns true' do
-        expect(Doorkeeper.configuration.confirm_application_owner?).to be_truthy
-      end
-    end
-  end
-
-  describe 'realm' do
-    it 'is \'Doorkeeper\' by default' do
-      expect(Doorkeeper.configuration.realm).to eq('Doorkeeper')
-    end
-
-    it 'can change the value' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        realm 'Example'
-      end
-      expect(subject.realm).to eq('Example')
-    end
-  end
-
-  describe "grant_flows" do
-    it "is set to all grant flows by default" do
-      expect(Doorkeeper.configuration.grant_flows).
-        to eq(%w(authorization_code client_credentials))
-    end
-
-    it "can change the value" do
-      Doorkeeper.configure {
-        orm DOORKEEPER_ORM
-        grant_flows [ 'authorization_code', 'implicit' ]
-      }
-      expect(subject.grant_flows).to eq ['authorization_code', 'implicit']
-    end
-
-    context "when including 'authorization_code'" do
-      before do
-        Doorkeeper.configure {
-          orm DOORKEEPER_ORM
-          grant_flows ['authorization_code']
-        }
-      end
-
-      it "includes 'code' in authorization_response_types" do
-        expect(subject.authorization_response_types).to include 'code'
-      end
-
-      it "includes 'authorization_code' in token_grant_types" do
-        expect(subject.token_grant_types).to include 'authorization_code'
-      end
-    end
-
-    context "when including 'implicit'" do
-      before do
-        Doorkeeper.configure {
-          orm DOORKEEPER_ORM
-          grant_flows ['implicit']
-        }
-      end
-
-      it "includes 'token' in authorization_response_types" do
-        expect(subject.authorization_response_types).to include 'token'
-      end
-    end
-
-    context "when including 'password'" do
-      before do
-        Doorkeeper.configure {
-          orm DOORKEEPER_ORM
-          grant_flows ['password']
-        }
-      end
-
-      it "includes 'password' in token_grant_types" do
-        expect(subject.token_grant_types).to include 'password'
-      end
-    end
-
-    context "when including 'client_credentials'" do
-      before do
-        Doorkeeper.configure {
-          orm DOORKEEPER_ORM
-          grant_flows ['client_credentials']
-        }
-      end
-
-      it "includes 'client_credentials' in token_grant_types" do
-        expect(subject.token_grant_types).to include 'client_credentials'
-      end
-    end
-  end
-
-  it 'raises an exception when configuration is not set' do
-    old_config = Doorkeeper.configuration
-    Doorkeeper.module_eval do
-      @config = nil
-    end
-
-    expect do
-      Doorkeeper.configuration
-    end.to raise_error Doorkeeper::MissingConfiguration
-
-    Doorkeeper.module_eval do
-      @config = old_config
-    end
-  end
-
-  describe 'access_token_generator' do
-    it 'is \'Doorkeeper::OAuth::Helpers::UniqueToken\' by default' do
-      expect(Doorkeeper.configuration.access_token_generator).to(
-        eq('Doorkeeper::OAuth::Helpers::UniqueToken')
-      )
-    end
-
-    it 'can change the value' do
-      Doorkeeper.configure do
-        orm DOORKEEPER_ORM
-        access_token_generator 'Example'
-      end
-      expect(subject.access_token_generator).to eq('Example')
-    end
-  end
-
-  describe 'base_controller' do
-    context 'default' do
-      it { expect(Doorkeeper.configuration.base_controller).to eq('ActionController::Base') }
-    end
-
-    context 'custom' do
-      before do
-        Doorkeeper.configure do
-          orm DOORKEEPER_ORM
-          base_controller 'ApplicationController'
-        end
-      end
-
-      it { expect(Doorkeeper.configuration.base_controller).to eq('ApplicationController') }
-    end
-  end
-end

data/spec/lib/doorkeeper_spec.rb

@@ -1,150 +0,0 @@
-require 'spec_helper_integration'
-
-describe Doorkeeper do
-  describe "#authenticate" do
-    let(:request) { double }
-
-    it "calls OAuth::Token#authenticate" do
-      token_strategies = Doorkeeper.configuration.access_token_methods
-
-      expect(Doorkeeper::OAuth::Token).to receive(:authenticate).
-        with(request, *token_strategies)
-
-      Doorkeeper.authenticate(request)
-    end
-
-    it "accepts custom token strategies" do
-      token_strategies = [:first_way, :second_way]
-
-      expect(Doorkeeper::OAuth::Token).to receive(:authenticate).
-        with(request, *token_strategies)
-
-      Doorkeeper.authenticate(request, token_strategies)
-    end
-  end
-
-  describe "#configured?" do
-    after do
-      Doorkeeper.remove_instance_variable(:@config)
-    end
-
-    context "@config is set" do
-      it "returns true" do
-        Doorkeeper.instance_variable_set(:@config, "hi")
-
-        expect(Doorkeeper.configured?).to eq(true)
-      end
-    end
-
-    context "@config is not set" do
-      it "returns false" do
-        Doorkeeper.instance_variable_set(:@config, nil)
-
-        expect(Doorkeeper.configured?).to eq(false)
-      end
-    end
-
-    it "is deprecated" do
-      expect(ActiveSupport::Deprecation).to receive(:warn).
-        with("Method `Doorkeeper#configured?` has been deprecated without replacement.")
-
-      Doorkeeper.configured?
-    end
-  end
-
-  describe "#database_installed?" do
-    before do
-      ["AccessToken", "AccessGrant", "Application"].each do |klass|
-        @original_classes ||= {}
-        @original_classes[klass] = Doorkeeper.const_get(klass)
-        Doorkeeper.send(:remove_const, klass)
-      end
-    end
-
-    after do
-      ["AccessToken", "AccessGrant", "Application"].each do |klass|
-        Doorkeeper.send(:remove_const, klass)
-        Doorkeeper.const_set(klass, @original_classes[klass])
-      end
-    end
-
-    context "all tables exist" do
-      before do
-        klass = double table_exists?: true
-
-        Doorkeeper.const_set(:AccessToken, klass)
-        Doorkeeper.const_set(:AccessGrant, klass)
-        Doorkeeper.const_set(:Application, klass)
-      end
-
-      it "returns true" do
-        expect(Doorkeeper.database_installed?).to eq(true)
-      end
-
-      it "is deprecated" do
-        expect(ActiveSupport::Deprecation).to receive(:warn).
-          with("Method `Doorkeeper#database_installed?` has been deprecated without replacement.")
-
-        Doorkeeper.database_installed?
-      end
-    end
-
-    context "all tables do not exist" do
-      before do
-        klass = double table_exists?: false
-
-        Doorkeeper.const_set(:AccessToken, klass)
-        Doorkeeper.const_set(:AccessGrant, klass)
-        Doorkeeper.const_set(:Application, klass)
-      end
-
-      it "returns false" do
-        expect(Doorkeeper.database_installed?).to eq(false)
-      end
-
-      it "is deprecated" do
-        expect(ActiveSupport::Deprecation).to receive(:warn).
-          with("Method `Doorkeeper#database_installed?` has been deprecated without replacement.")
-
-        Doorkeeper.database_installed?
-      end
-    end
-  end
-
-  describe "#installed?" do
-    context "methods return true" do
-      before do
-        allow(Doorkeeper).to receive(:configured?).and_return(true).once
-        allow(Doorkeeper).to receive(:database_installed?).and_return(true).once
-      end
-
-      it "returns true" do
-        expect(Doorkeeper.installed?).to eq(true)
-      end
-    end
-
-    context "methods return false" do
-      before do
-        allow(Doorkeeper).to receive(:configured?).and_return(false).once
-        allow(Doorkeeper).to receive(:database_installed?).and_return(false).once
-      end
-
-      it "returns false" do
-        expect(Doorkeeper.installed?).to eq(false)
-      end
-    end
-
-    it "is deprecated" do
-      expect(ActiveSupport::Deprecation).to receive(:warn).
-        with("Method `Doorkeeper#configured?` has been deprecated without replacement.")
-
-      expect(ActiveSupport::Deprecation).to receive(:warn).
-        with("Method `Doorkeeper#database_installed?` has been deprecated without replacement.")
-
-      expect(ActiveSupport::Deprecation).to receive(:warn).
-        with("Method `Doorkeeper#installed?` has been deprecated without replacement.")
-
-      Doorkeeper.installed?
-    end
-  end
-end

data/spec/lib/models/expirable_spec.rb

@@ -1,50 +0,0 @@
-require 'spec_helper'
-require 'active_support/time'
-require 'doorkeeper/models/concerns/expirable'
-
-describe 'Expirable' do
-  subject do
-    Class.new do
-      include Doorkeeper::Models::Expirable
-    end.new
-  end
-
-  before do
-    allow(subject).to receive(:created_at).and_return(1.minute.ago)
-  end
-
-  describe :expired? do
-    it 'is not expired if time has not passed' do
-      allow(subject).to receive(:expires_in).and_return(2.minutes)
-      expect(subject).not_to be_expired
-    end
-
-    it 'is expired if time has passed' do
-      allow(subject).to receive(:expires_in).and_return(10.seconds)
-      expect(subject).to be_expired
-    end
-
-    it 'is not expired if expires_in is not set' do
-      allow(subject).to receive(:expires_in).and_return(nil)
-      expect(subject).not_to be_expired
-    end
-  end
-
-  describe :expires_in_seconds do
-    it 'should return the amount of time remaining until the token is expired' do
-      allow(subject).to receive(:expires_in).and_return(2.minutes)
-      expect(subject.expires_in_seconds).to eq(60)
-    end
-
-    it 'should return 0 when expired' do
-      allow(subject).to receive(:expires_in).and_return(30.seconds)
-      expect(subject.expires_in_seconds).to eq(0)
-    end
-
-    it 'should return nil when expires_in is nil' do
-      allow(subject).to receive(:expires_in).and_return(nil)
-      expect(subject.expires_in_seconds).to be_nil
-    end
-
-  end
-end

data/spec/lib/models/revocable_spec.rb

@@ -1,59 +0,0 @@
-require 'spec_helper'
-require 'active_support/core_ext/object/blank'
-require 'doorkeeper/models/concerns/revocable'
-
-describe 'Revocable' do
-  subject do
-    Class.new do
-      include Doorkeeper::Models::Revocable
-    end.new
-  end
-
-  describe :revoke do
-    it 'updates :revoked_at attribute with current time' do
-      utc = double utc: double
-      clock = double now: utc
-      expect(subject).to receive(:update_attribute).with(:revoked_at, clock.now.utc)
-      subject.revoke(clock)
-    end
-  end
-
-  describe :revoked? do
-    it 'is revoked if :revoked_at has passed' do
-      allow(subject).to receive(:revoked_at).and_return(Time.now.utc - 1000)
-      expect(subject).to be_revoked
-    end
-
-    it 'is not revoked if :revoked_at has not passed' do
-      allow(subject).to receive(:revoked_at).and_return(Time.now.utc + 1000)
-      expect(subject).not_to be_revoked
-    end
-
-    it 'is not revoked if :revoked_at is not set' do
-      allow(subject).to receive(:revoked_at).and_return(nil)
-      expect(subject).not_to be_revoked
-    end
-  end
-
-  describe :revoke_previous_refresh_token! do
-    it "revokes the previous token if existing, and resets the
-      `previous_refresh_token` attribute" do
-      previous_token = FactoryGirl.create(
-        :access_token,
-        refresh_token: "refresh_token"
-      )
-      current_token = FactoryGirl.create(
-        :access_token,
-        previous_refresh_token: previous_token.refresh_token
-      )
-
-      expect_any_instance_of(
-        Doorkeeper::AccessToken
-      ).to receive(:revoke).and_call_original
-      current_token.revoke_previous_refresh_token!
-
-      expect(current_token.previous_refresh_token).to be_empty
-      expect(previous_token.reload).to be_revoked
-    end
-  end
-end