Merge pull request #24 from mitfik/master

Refresh the project, upgrade ruby version, replace debugger with byebug, fix pending rspec tests.

Author: Robert Mitwicki

.ruby-version

@@ -1 +1 @@
-2.0.0
+2.1.5

.travis.yml

@@ -3,4 +3,5 @@ bundler_args: --without development
 rvm:
   - rbx-19mode
   - jruby-19mode
-  - 1.9.3
+  - 2.1.5
+env: JRUBY_OPTS=--2.0

Gemfile

@@ -5,7 +5,7 @@ group :development do
   # for gems that are nice in development
   # but don't break the build when missing
   # Example: debugger
-  gem 'debugger'
+  gem 'byebug'
   gem "guard"
   gem "guard-rspec"
 

README.md

@@ -6,7 +6,7 @@ The core logic for handling CAS requests independent of any particular storage o
 
 ## Requirements
 
-* ruby 1.9.x
+* ruby 2.1.x
 
 ## Contributing
 

lib/rubycas-server-core/tickets/validations.rb

@@ -4,7 +4,7 @@ module Validations
 
     # Validate login ticket
     #
-    # Returned [succes, error_message]
+    # Returned [success, error_message]
     def validate_login_ticket(ticket)
       $LOG.debug "Validating login ticket '#{ticket}'"
       success = false
@@ -55,6 +55,8 @@ def validate_ticket_granting_ticket(ticket)
 
     def validate_service_ticket(service, ticket, allow_proxy_tickets = false)
       $LOG.debug "Validating service/proxy ticket '#{ticket}' for service '#{service}'"
+      success = false
+      error = nil
 
       if service.nil? or ticket.nil?
         error = Error.new(:INVALID_REQUEST, "Ticket or service parameter was missing in the request.")
@@ -74,19 +76,16 @@ def validate_service_ticket(service, ticket, allow_proxy_tickets = false)
             " but the requested service '#{service}' does not match the service '#{st.service}' associated with this ticket.")
           $LOG.warn "#{error.code} - #{error.message}"
         else
+          st.consume!
+          success = true
           $LOG.info("Ticket '#{ticket}' for service '#{service}' for user '#{st.username}' successfully validated.")
         end
       else
         error = Error.new(:INVALID_TICKET, "Ticket '#{ticket}' not recognized.")
         $LOG.warn("#{error.code} - #{error.message}")
       end
 
-      if st
-        st.consume!
-      end
-
-
-      [st, error]
+      return [success, error]
     end
 
     def validate_proxy_ticket(service, ticket)

spec/rubycas-server-core/tickets/validations_spec.rb

@@ -16,7 +16,7 @@
     it "should validate login ticket" do
       @lt = Tickets.generate_login_ticket(@client_hostname)
       success, error = @cas.validate_login_ticket(@lt.ticket)
-      success.should be_true
+      success.should be_truthy
       error.should be_nil
     end
   end
@@ -30,7 +30,7 @@
 
     it "should validate ticket granting ticket" do
       success, error = @cas.validate_ticket_granting_ticket(@tgt.ticket)
-      success.should be_true
+      success.should be_truthy
       error.should be_nil
     end
   end
@@ -46,7 +46,7 @@
 
     it "should validate service ticket" do
       success, error = @cas.validate_service_ticket(@service, @st.ticket)
-      success.should be_true
+      success.should be_truthy
       error.should be_nil
     end
   end

spec/rubycas-server-core/tickets_spec.rb

@@ -91,14 +91,17 @@ module RubyCAS::Server::Core
     describe ".generate_proxy_ticket(target_service, pgt)" do
       it "should return a ProxyGrantingTicket" do
         pending("Proxy ticket is not implemented yet")
+        fail
       end
 
       it "should not consume the generated ticket" do
         pending("Proxy ticket is not implemented yet")
+        fail
       end
 
       it "should start the ticket string with PT-" do
         pending("Proxy ticket is not implemented yet")
+        fail
       end
     end
   end