couchkeeper 0.6.7

data/spec/generators/install_generator_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper_integration'
+require 'generators/doorkeeper/install_generator'
+
+describe 'Doorkeeper::InstallGenerator' do
+  include GeneratorSpec::TestCase
+
+  tests Doorkeeper::InstallGenerator
+  destination ::File.expand_path("../tmp/dummy", __FILE__)
+
+  describe "after running the generator" do
+    before :each do
+      prepare_destination
+      FileUtils.mkdir(::File.expand_path("config", Pathname(destination_root)))
+      FileUtils.mkdir(::File.expand_path("db", Pathname(destination_root)))
+      FileUtils.copy_file(::File.expand_path("../templates/routes.rb", __FILE__), ::File.expand_path("config/routes.rb", Pathname.new(destination_root)))
+      run_generator
+    end
+
+    it "creates an initializer file" do
+      assert_file 'config/initializers/doorkeeper.rb'
+    end
+
+    it "copies the locale file" do
+      assert_file 'config/locales/doorkeeper.en.yml'
+    end
+
+    it "adds sample route" do
+      assert_file "config/routes.rb", /use_doorkeeper/
+    end
+  end
+end

data/spec/generators/migration_generator_spec.rb

@@ -0,0 +1,20 @@
+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

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

data/spec/generators/views_generator_spec.rb

@@ -0,0 +1,27 @@
+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/lib/config_spec.rb

@@ -0,0 +1,170 @@
+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 do end
+      Doorkeeper.configure do
+        orm DOORKEEPER_ORM
+        resource_owner_authenticator &block
+      end
+      subject.authenticate_resource_owner.should == block
+    end
+  end
+
+  describe "admin_authenticator" do
+    it "sets the block that is accessible via authenticate_admin" do
+      block = proc do end
+      Doorkeeper.configure do
+        orm DOORKEEPER_ORM
+        admin_authenticator &block
+      end
+      subject.authenticate_admin.should == block
+    end
+  end
+
+  describe "access_token_expires_in" do
+    it "has 2 hours by default" do
+      subject.access_token_expires_in.should == 2.hours
+    end
+
+    it "can change the value" do
+      Doorkeeper.configure do
+        orm DOORKEEPER_ORM
+        access_token_expires_in 4.hours
+      end
+      subject.access_token_expires_in.should == 4.hours
+    end
+
+    it "can be set to nil" do
+      Doorkeeper.configure do
+        orm DOORKEEPER_ORM
+        access_token_expires_in nil
+      end
+      subject.access_token_expires_in.should be_nil
+    end
+  end
+
+  describe "scopes" do
+    it "has default scopes" do
+      Doorkeeper.configure {
+        orm DOORKEEPER_ORM
+        default_scopes :public
+      }
+      subject.default_scopes.should include(:public)
+    end
+
+    it 'has optional scopes' do
+      Doorkeeper.configure {
+        orm DOORKEEPER_ORM
+        optional_scopes :write, :update
+      }
+      subject.optional_scopes.should include(:write, :update)
+    end
+
+    it 'has all scopes' do
+      Doorkeeper.configure do
+        orm DOORKEEPER_ORM
+        default_scopes  :normal
+        optional_scopes :admin
+      end
+      subject.scopes.should include(:normal, :admin)
+    end
+  end
+
+  describe "use_refresh_token" do
+    it "is false by default" do
+      subject.refresh_token_enabled?.should be_false
+    end
+
+    it "can change the value" do
+      Doorkeeper.configure {
+        orm DOORKEEPER_ORM
+        use_refresh_token
+      }
+      subject.refresh_token_enabled?.should be_true
+    end
+  end
+
+  describe 'client_credentials' do
+    it 'has defaults order' do
+      subject.client_credentials_methods.should == [:from_basic, :from_params]
+    end
+
+    it "can change the value" do
+      Doorkeeper.configure {
+        orm DOORKEEPER_ORM
+        client_credentials :from_digest, :from_params
+      }
+      subject.client_credentials_methods.should == [:from_digest, :from_params]
+    end
+  end
+
+  describe 'access_token_credentials' do
+    it 'has defaults order' do
+      subject.access_token_methods.should == [:from_bearer_authorization, :from_access_token_param, :from_bearer_param]
+    end
+
+    it "can change the value" do
+      Doorkeeper.configure {
+        orm DOORKEEPER_ORM
+        access_token_methods :from_access_token_param, :from_bearer_param
+      }
+      subject.access_token_methods.should == [:from_access_token_param, :from_bearer_param]
+    end
+  end
+
+  describe "enable_application_owner" do
+    it "is disabled by default" do
+      Doorkeeper.configuration.enable_application_owner?.should_not be_true
+    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
+        Doorkeeper::Application.new.should respond_to :owner
+      end
+      it "Doorkeeper.configuration.confirm_application_owner? returns false" do
+        Doorkeeper.configuration.confirm_application_owner?.should_not be_true
+      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
+        Doorkeeper::Application.new.should respond_to :owner
+      end
+      it "Doorkeeper.configuration.confirm_application_owner? returns true" do
+        Doorkeeper.configuration.confirm_application_owner?.should be_true
+      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
+end

data/spec/lib/models/expirable_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+require 'timecop'
+require 'active_support/time'
+require 'doorkeeper/models/expirable'
+
+describe 'Expirable' do
+  subject do
+    Class.new do
+      include Doorkeeper::Models::Expirable
+    end.new
+  end
+
+  before do
+    subject.stub :created_at => 1.minute.ago
+  end
+
+  describe :expired? do
+    it "is not expired if time has not passed" do
+      subject.stub :expires_in => 2.minutes
+      subject.should_not be_expired
+    end
+
+    it "is expired if time has passed" do
+      subject.stub :expires_in => 10.seconds
+      subject.should be_expired
+    end
+
+    it "is not expired if expires_in is not set" do
+      subject.stub :expires_in => nil
+      subject.should_not be_expired
+    end
+  end
+
+  describe :expires_in_seconds do
+    it "should return the amount of time remaining until the token is expired" do
+      subject.stub :expires_in => 2.minutes
+      subject.expires_in_seconds.should == 60 
+    end
+
+    it "should return 0 when expired" do
+      subject.stub :expires_in => 30.seconds
+      subject.expires_in_seconds.should == 0 
+    end
+
+    it "should return nil when expires_in is nil" do
+      subject.stub :expires_in => nil
+      subject.expires_in_seconds.should be_nil
+    end
+    
+  end
+end

data/spec/lib/models/revocable_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+require 'active_support/core_ext/object/blank'
+require 'doorkeeper/models/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
+      clock = double :now => stub
+      subject.should_receive(:update_column).with(:revoked_at, clock.now)
+      subject.revoke(clock)
+    end
+  end
+
+  describe :revoked? do
+    it "is revoked if :revoked_at is set" do
+      subject.stub :revoked_at => stub
+      subject.should be_revoked
+    end
+
+    it "is not revoked if :revoked_at is not set" do
+      subject.stub :revoked_at => nil
+      subject.should_not be_revoked
+    end
+  end
+end

data/spec/lib/models/scopes_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+require 'active_support/core_ext/module/delegation'
+require 'doorkeeper/oauth/scopes'
+require 'doorkeeper/models/scopes'
+
+describe 'Doorkeeper::Models::Scopes' do
+  subject do
+    Class.new(Hash) do
+      include Doorkeeper::Models::Scopes
+    end.new
+  end
+
+  before do
+    subject[:scopes] = 'public admin'
+  end
+
+  describe :scopes do
+    it 'is a `Scopes` class' do
+      subject.scopes.should be_a(Doorkeeper::OAuth::Scopes)
+    end
+
+    it 'includes scopes' do
+      subject.scopes.should include(:public)
+    end
+  end
+
+  describe :scopes_string do
+    it 'is a `Scopes` class' do
+      subject.scopes_string.should == 'public admin'
+    end
+  end
+end

data/spec/lib/oauth/authorization/uri_builder_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+require 'active_support/core_ext/string'
+require 'uri'
+require 'rack/utils'
+require 'doorkeeper/oauth/authorization/uri_builder'
+
+module Doorkeeper::OAuth::Authorization
+  describe URIBuilder do
+
+    subject { Object.new.class.send :include, URIBuilder }
+
+    describe :uri_with_query do
+      it 'returns the uri with query' do
+        uri = subject.uri_with_query 'http://example.com/', :parameter => 'value'
+        uri.should == 'http://example.com/?parameter=value'
+      end
+
+      it 'rejects nil values' do
+        uri = subject.uri_with_query 'http://example.com/', :parameter => ""
+        uri.should == 'http://example.com/?'
+      end
+
+      it 'preserves original query parameters' do
+        uri = subject.uri_with_query 'http://example.com/?query1=value', :parameter => 'value'
+        uri.should =~ /query1=value/
+        uri.should =~ /parameter=value/
+      end
+    end
+
+    describe :uri_with_fragment do
+      it 'returns uri with parameters as fragments' do
+        uri = subject.uri_with_fragment 'http://example.com/', :parameter => 'value'
+        uri.should == 'http://example.com/#parameter=value'
+      end
+    end
+  end
+end

data/spec/lib/oauth/authorization_code_request_spec.rb

@@ -0,0 +1,80 @@
+require 'spec_helper_integration'
+
+module Doorkeeper::OAuth
+  describe AuthorizationCodeRequest do
+    let(:server) { mock :server, :access_token_expires_in => 2.days, :refresh_token_enabled? => false }
+    let(:grant)  { FactoryGirl.create :access_grant }
+    let(:client) { grant.application }
+
+    subject do
+      AuthorizationCodeRequest.new server, grant, client, :redirect_uri => client.redirect_uri
+    end
+
+    it 'issues a new token for the client' do
+      expect do
+        subject.authorize
+      end.to change { client.access_tokens.count }.by(1)
+    end
+
+    it "issues the token with same grant's scopes" do
+      subject.authorize
+      Doorkeeper::AccessToken.last.scopes.should == grant.scopes
+    end
+
+    it 'revokes the grant' do
+      expect do
+        subject.authorize
+      end.to change { grant.reload.accessible? }
+    end
+
+    it 'requires the grant to be accessible' do
+      grant.revoke
+      subject.validate
+      subject.error.should == :invalid_grant
+    end
+
+    it 'requires the grant' do
+      subject.grant = nil
+      subject.validate
+      subject.error.should == :invalid_grant
+    end
+
+    it 'requires the client' do
+      subject.client = nil
+      subject.validate
+      subject.error.should == :invalid_client
+    end
+
+    it 'requires the redirect_uri' do
+      subject.redirect_uri = nil
+      subject.validate
+      subject.error.should == :invalid_request
+    end
+
+    it "matches the redirect_uri with grant's one" do
+      subject.redirect_uri = 'http://other.com'
+      subject.validate
+      subject.error.should == :invalid_grant
+    end
+
+    it "matches the client with grant's one" do
+      subject.client = FactoryGirl.create :application
+      subject.validate
+      subject.error.should == :invalid_grant
+    end
+
+    it 'skips token creation if there is a matching one' do
+      FactoryGirl.create(:access_token, :application_id => client.id, :resource_owner_id => grant.resource_owner_id, :scopes => "public write")
+      expect do
+        subject.authorize
+      end.to_not change { Doorkeeper::AccessToken.count }
+    end
+
+    it 'revokes matching token if expired' do
+      token = FactoryGirl.create(:access_token, :application_id => client.id, :resource_owner_id => grant.resource_owner_id, :scopes => "public write", :expires_in => -100)
+      expect do
+        subject.authorize
+      end.to change { token.reload.revoked? }
+    end
+  end
+end