rubycas-client 2.3.8 → 2.3.9.rc1

data/spec/database.yml

@@ -0,0 +1,7 @@
+test:
+  adapter: sqlite3
+  database: spec/test.sqlite3.db
+
+testjruby:
+  adapter: jdbcsqlite3
+  database: spec/test.sqlite3.db

data/spec/spec_helper.rb

@@ -1,16 +1,39 @@
 require 'bundler'
-
-Bundler.setup(:default, :test)
+Bundler.setup(:default, :development)
+require 'simplecov' unless ENV['TRAVIS']
 Bundler.require
 
+require 'rubycas-client'
+
+SPEC_TMP_DIR="spec/tmp"
+
+Dir["./spec/support/**/*.rb"].each do |f|
+  require f.gsub('.rb','') unless f.end_with? '_spec.rb'
+end
+
+require 'database_cleaner'
+
 RSpec.configure do |config|
-  #config.include Rack::Test::Methods
-  #config.include Webrat::Methods
-  #config.include Webrat::Matchers
-  #config.include TestHelpers
-  #config.include Helpers
   config.mock_with :rspec
   config.mock_framework = :rspec
+  config.include ActionControllerHelpers
+
+  config.before(:suite) do
+    ActiveRecordHelpers.setup_active_record
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with(:truncation)
+  end
+
+  config.after(:suite) do
+    ActiveRecordHelpers.teardown_active_record
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.start
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
 end
 
-require 'rubycas-client'

data/spec/support/action_controller_helpers.rb

@@ -0,0 +1,30 @@
+require 'action_pack'
+
+module ActionControllerHelpers
+
+  def mock_controller_with_session(request = nil, session={})
+
+    query_parameters = {:ticket => "bogusticket", :renew => false}
+    parameters = query_parameters.dup
+
+    #TODO this really need to be replaced with a "real" rails controller
+    request ||= mock_post_request
+    request.stub(:query_parameters) {query_parameters}
+    request.stub(:path_parameters) {{}}
+    controller = double("Controller")
+    controller.stub(:session) {session}
+    controller.stub(:request) {request}
+    controller.stub(:url_for) {"bogusurl"}
+    controller.stub(:query_parameters) {query_parameters}
+    controller.stub(:path_parameters) {{}}
+    controller.stub(:parameters) {parameters}
+    controller.stub(:params) {parameters}
+    controller
+  end
+
+  def mock_post_request
+      mock_request = ActionController::Request.new({})
+      mock_request.stub(:post?) {true}
+      mock_request
+  end
+end

data/spec/support/active_record_helpers.rb

@@ -0,0 +1,48 @@
+require 'active_record'
+
+module ActiveRecordHelpers
+
+  class << self
+    def setup_active_record
+      config_file = File.open("spec/database.yml")
+      db_config = HashWithIndifferentAccess.new(YAML.load(config_file))
+      ActiveRecord::Base.establish_connection(db_config[(RUBY_PLATFORM == "java") ? :testjruby : :test])
+      ActiveRecord::Migration.verbose = false
+      RubyCasTables.migrate(:up)
+    end
+
+    def teardown_active_record
+      ActiveRecord::Migration.verbose = false
+      RubyCasTables.migrate(:down)
+    end
+  end
+
+  class RubyCasTables < ActiveRecord::Migration
+    def self.up
+      #default rails sessions table
+      create_table :sessions do |t|
+        t.string :session_id, :null => false
+        t.text :data
+        t.timestamps
+      end
+      add_index :sessions, :session_id
+      add_index :sessions, :updated_at
+
+      #column added to sessions table by rubycas-client
+      add_column :sessions, :service_ticket, :string
+      add_index :sessions, :service_ticket
+
+      # pgtious table
+      create_table :cas_pgtious do |t|
+        t.string :pgt_iou, :null => false
+        t.string :pgt_id, :null => false
+        t.timestamps
+      end
+    end
+
+    def self.down
+      drop_table :sessions
+      drop_table :cas_pgtious
+    end
+  end
+end

data/spec/support/local_hash_ticket_store.rb

@@ -0,0 +1,48 @@
+require 'casclient/tickets/storage'
+
+class LocalHashTicketStore < CASClient::Tickets::Storage::AbstractTicketStore
+
+  attr_accessor :st_hash
+  attr_accessor :pgt_hash
+
+  def store_service_session_lookup(st, controller)
+    raise CASClient::CASException, "No service_ticket specified." if st.nil?
+    raise CASClient::CASException, "No controller specified." if controller.nil?
+    session_id = session_id_from_controller(controller)
+    st = st.ticket if st.kind_of? CASClient::ServiceTicket
+    st_hash[st] = session_id
+  end
+
+  def read_service_session_lookup(st)
+    raise CASClient::CASException, "No service_ticket specified." if st.nil?
+    st = st.ticket if st.kind_of? CASClient::ServiceTicket
+    st_hash[st]
+  end
+
+  def cleanup_service_session_lookup(st)
+    raise CASClient::CASException, "No service_ticket specified." if st.nil?
+    st = st.ticket if st.kind_of? CASClient::ServiceTicket
+    st_hash.delete(st)
+  end
+
+  def save_pgt_iou(pgt_iou, pgt)
+    raise CASClient::CASException.new("Invalid pgt_iou") if pgt_iou.nil?
+    raise CASClient::CASException.new("Invalid pgt") if pgt.nil?
+    pgt_hash[pgt_iou] = pgt
+  end
+
+  def retrieve_pgt(pgt_iou)
+    pgt = pgt_hash.delete(pgt_iou)
+    raise CASClient::CASException.new("Invalid pgt_iou") if pgt.nil?
+    pgt
+  end
+
+  def pgt_hash
+    @pgt_hash ||= {}
+  end
+
+  def st_hash
+    @pgt_hash ||= {}
+  end
+
+end

data/spec/support/local_hash_ticket_store_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe LocalHashTicketStore do
+  it_should_behave_like "a ticket store"
+end

data/spec/support/shared_examples_for_ticket_stores.rb

@@ -0,0 +1,137 @@
+shared_examples "a ticket store interacting with sessions" do
+  describe "#store_service_session_lookup" do
+    it "should raise CASException if the Service Ticket is nil" do
+      expect { subject.store_service_session_lookup(nil, "controller") }.to raise_exception(CASClient::CASException, /No service_ticket specified/)
+    end
+    it "should raise CASException if the controller is nil" do
+      expect { subject.store_service_session_lookup("service_ticket", nil) }.to raise_exception(CASClient::CASException, /No controller specified/)
+    end
+    it "should store the ticket without any errors" do
+      expect { subject.store_service_session_lookup(service_ticket, mock_controller_with_session(nil, session)) }.to_not raise_exception
+    end
+  end
+
+  describe "#get_session_for_service_ticket" do
+    context "the service ticket is nil" do
+      it "should raise CASException" do
+        expect { subject.get_session_for_service_ticket(nil) }.to raise_exception(CASClient::CASException, /No service_ticket specified/)
+      end
+    end
+    context "the service ticket is associated with a session" do
+      before do
+        subject.store_service_session_lookup(service_ticket, mock_controller_with_session(nil, session))
+        session.save!
+      end
+      it "should return the session_id and session for the given service ticket" do
+        result_session_id, result_session = subject.get_session_for_service_ticket(service_ticket)
+        result_session_id.should == session.session_id
+        result_session.session_id.should == session.session_id
+        result_session.data.should == session.data
+      end
+    end
+    context "the service ticket is not associated with a session" do
+      it "should return nils if there is no session for the given service ticket" do
+        subject.get_session_for_service_ticket(service_ticket).should == [nil, nil]
+      end
+    end
+  end
+
+  describe "#process_single_sign_out" do
+    context "the service ticket is nil" do
+      it "should raise CASException" do
+        expect { subject.process_single_sign_out(nil) }.to raise_exception(CASClient::CASException, /No service_ticket specified/)
+      end
+    end
+    context "the service ticket is associated with a session" do
+      before do
+        subject.store_service_session_lookup(service_ticket, mock_controller_with_session(nil, session))
+        session.save!
+        subject.process_single_sign_out(service_ticket)
+      end
+      context "the session" do
+        it "should be destroyed" do
+          ActiveRecord::SessionStore.session_class.find_by_session_id(session.session_id).should be_nil
+        end
+      end
+      it "should destroy session for the given service ticket" do
+        subject.process_single_sign_out(service_ticket)
+      end
+    end
+    context "the service ticket is not associated with a session" do
+      it "should run without error if there is no session for the given service ticket" do
+        expect { subject.process_single_sign_out(service_ticket) }.to_not raise_error
+      end
+    end
+  end
+
+  describe "#cleanup_service_session_lookup" do
+    context "the service ticket is nil" do
+      it "should raise CASException" do
+        expect { subject.cleanup_service_session_lookup(nil) }.to raise_exception(CASClient::CASException, /No service_ticket specified/)
+      end
+    end
+    it "should run without error" do
+      expect { subject.cleanup_service_session_lookup(service_ticket) }.to_not raise_exception
+    end
+  end
+end
+
+shared_examples "a ticket store" do
+  let(:ticket_store) { described_class.new }
+  let(:service_url) { "https://www.example.com/cas" }
+  let(:session) do
+    ActiveRecord::SessionStore::Session.create!(:session_id => "session#{rand(1000)}", :data => {})
+  end
+  subject { ticket_store }
+
+  context "when dealing with sessions, Service Tickets, and Single Sign Out" do
+    context "and the service ticket is a String" do
+      it_behaves_like "a ticket store interacting with sessions" do
+        let(:service_ticket) { "ST-ABC#{rand(1000)}" }
+      end
+    end
+    context "and the service ticket is a ServiceTicket" do
+      it_behaves_like "a ticket store interacting with sessions" do
+        let(:service_ticket) { CASClient::ServiceTicket.new("ST-ABC#{rand(1000)}", service_url) }
+      end
+    end
+    context "and the service ticket is a ProxyTicket" do
+      it_behaves_like "a ticket store interacting with sessions" do
+        let(:service_ticket) { CASClient::ProxyTicket.new("ST-ABC#{rand(1000)}", service_url) }
+      end
+    end
+  end
+
+  context "when dealing with Proxy Granting Tickets and their IOUs" do
+    let(:pgt) { "my_pgt_#{rand(1000)}" }
+    let(:pgt_iou) { "my_pgt_iou_#{rand(1000)}" }
+
+    describe "#save_pgt_iou" do
+      it "should raise CASClient::CASException if the pgt_iou is nil" do
+        expect { subject.save_pgt_iou(nil, pgt) }.to raise_exception(CASClient::CASException, /Invalid pgt_iou/)
+      end
+      it "should raise CASClient::CASException if the pgt is nil" do
+        expect { subject.save_pgt_iou(pgt_iou, nil) }.to raise_exception(CASClient::CASException, /Invalid pgt/)
+      end
+    end
+
+    describe "#retrieve_pgt" do
+      before do
+        subject.save_pgt_iou(pgt_iou, pgt)
+      end
+
+      it "should return the stored pgt" do
+        subject.retrieve_pgt(pgt_iou).should == pgt
+      end
+
+      it "should raise CASClient::CASException if the pgt_iou isn't in the store" do
+        expect { subject.retrieve_pgt("not_my"+pgt_iou) }.to raise_exception(CASClient::CASException, /Invalid pgt_iou/)
+      end
+
+      it "should not return the stored pgt a second time" do
+        subject.retrieve_pgt(pgt_iou).should == pgt
+        expect { subject.retrieve_pgt(pgt_iou) }.to raise_exception(CASClient::CASException, /Invalid pgt_iou/)
+      end
+    end
+  end
+end

metadata

@@ -1,8 +1,8 @@
 --- !ruby/object:Gem::Specification
 name: rubycas-client
 version: !ruby/object:Gem::Version
-  version: 2.3.8
-  prerelease: 
+  version: 2.3.9.rc1
+  prerelease: 6
 platform: ruby
 authors:
 - Matt Zukowski
@@ -11,11 +11,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-12-19 00:00:00.000000000Z
+date: 2012-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70206959748580 !ruby/object:Gem::Requirement
+  requirement: &70109785006580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -23,32 +23,32 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70206959748580
+  version_requirements: *70109785006580
 - !ruby/object:Gem::Dependency
   name: json
-  requirement: &70206959747260 !ruby/object:Gem::Requirement
+  requirement: &70109785005920 !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 1.6.1
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206959747260
+  version_requirements: *70109785005920
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70206959746080 !ruby/object:Gem::Requirement
+  requirement: &70109785005200 !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 2.7.0
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206959746080
+  version_requirements: *70109785005200
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70206959745140 !ruby/object:Gem::Requirement
+  requirement: &70109785004560 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -56,21 +56,32 @@ dependencies:
         version: '1.0'
   type: :development
   prerelease: false
-  version_requirements: *70206959745140
+  version_requirements: *70109785004560
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &70206959743740 !ruby/object:Gem::Requirement
+  requirement: &70109785003840 !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 1.6.2
+        version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206959743740
+  version_requirements: *70109785003840
 - !ruby/object:Gem::Dependency
   name: actionpack
-  requirement: &70206959742080 !ruby/object:Gem::Requirement
+  requirement: &70109785002580 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70109785002580
+- !ruby/object:Gem::Dependency
+  name: activerecord
+  requirement: &70109785001820 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -78,10 +89,76 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206959742080
+  version_requirements: *70109785001820
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70206959740920 !ruby/object:Gem::Requirement
+  requirement: &70109785001120 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70109785001120
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: &70109785000440 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70109785000440
+- !ruby/object:Gem::Dependency
+  name: guard
+  requirement: &70109784999740 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70109784999740
+- !ruby/object:Gem::Dependency
+  name: guard-rspec
+  requirement: &70109784998920 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70109784998920
+- !ruby/object:Gem::Dependency
+  name: database_cleaner
+  requirement: &70109784995060 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70109784995060
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: &70109784992260 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *70109784992260
+- !ruby/object:Gem::Dependency
+  name: jruby-openssl
+  requirement: &70109784990620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -89,10 +166,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206959740920
+  version_requirements: *70109784990620
 - !ruby/object:Gem::Dependency
-  name: rcov
-  requirement: &70206959740000 !ruby/object:Gem::Requirement
+  name: activerecord-jdbcsqlite3-adapter
+  requirement: &70109784753060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -100,7 +177,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70206959740000
+  version_requirements: *70109784753060
 description: 
 email: 
 executables: []
@@ -110,10 +187,12 @@ extra_rdoc_files:
 - README.rdoc
 files:
 - .rspec
+- .simplecov
 - .travis.yml
 - CHANGELOG.txt
 - Gemfile
 - Gemfile.lock
+- Guardfile
 - History.txt
 - LICENSE.txt
 - README.rdoc
@@ -157,9 +236,19 @@ files:
 - rails_generators/active_record_ticket_store/templates/README
 - rails_generators/active_record_ticket_store/templates/migration.rb
 - rubycas-client.gemspec
+- spec/.gitignore
+- spec/casclient/client_spec.rb
 - spec/casclient/frameworks/rails/filter_spec.rb
+- spec/casclient/tickets/storage/active_record_ticket_store_spec.rb
+- spec/casclient/tickets/storage_spec.rb
 - spec/casclient/validation_response_spec.rb
+- spec/database.yml
 - spec/spec_helper.rb
+- spec/support/action_controller_helpers.rb
+- spec/support/active_record_helpers.rb
+- spec/support/local_hash_ticket_store.rb
+- spec/support/local_hash_ticket_store_spec.rb
+- spec/support/shared_examples_for_ticket_stores.rb
 homepage: http://github.com/rubycas/rubycas-client
 licenses:
 - MIT
@@ -177,16 +266,16 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2445786591382746754
+      hash: -1272570906276872327
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
-  - - ! '>='
+  - - ! '>'
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 1.8.11
 signing_key: 
 specification_version: 3
 summary: Client library for the Central Authentication Service (CAS) protocol.