Commit dfd11e1

mo <mo@mokhan.ca>
2017-11-30 16:16:31
move logout response.
1 parent 27651fb
lib/saml/kit/builders/logout_response.rb
@@ -1,7 +1,7 @@
 module Saml
   module Kit
-    class LogoutResponse < Document
-      class Builder
+    module Builders
+      class LogoutResponse
         attr_accessor :id, :issuer, :version, :status_code, :sign, :now, :destination
         attr_reader :request
 
@@ -33,7 +33,7 @@ module Saml
         end
 
         def build
-          LogoutResponse.new(to_xml, request_id: request.id)
+          Saml::Kit::LogoutResponse.new(to_xml, request_id: request.id)
         end
 
         private
lib/saml/kit/logout_request.rb
@@ -19,7 +19,7 @@ module Saml
       end
 
       def response_for(user)
-        LogoutResponse::Builder.new(user, self)
+        Saml::Kit::Builders::LogoutResponse.new(user, self)
       end
     end
   end
spec/saml/bindings/http_post_spec.rb
@@ -37,7 +37,7 @@ RSpec.describe Saml::Kit::Bindings::HttpPost do
     it 'returns a SAMLResponse for a LogoutResponse' do
       user = double(:user, name_id_for: SecureRandom.uuid)
       request = instance_double(Saml::Kit::AuthenticationRequest, id: SecureRandom.uuid)
-      builder = Saml::Kit::LogoutResponse::Builder.new(user, request)
+      builder = Saml::Kit::Builders::LogoutResponse.new(user, request)
       url, saml_params = subject.serialize(builder, relay_state: relay_state)
 
       expect(url).to eql(location)
spec/saml/bindings/http_redirect_spec.rb
@@ -56,7 +56,7 @@ RSpec.describe Saml::Kit::Bindings::HttpRedirect do
     it 'deserializes the SAMLResponse to a LogoutResponse' do
       user = double(:user, name_id_for: SecureRandom.uuid, assertion_attributes_for: [])
       request = double(:request, id: SecureRandom.uuid, provider: provider, acs_url: FFaker::Internet.http_url, name_id_format: Saml::Kit::Namespaces::PERSISTENT, issuer: FFaker::Internet.http_url)
-      url, _ = subject.serialize(Saml::Kit::LogoutResponse::Builder.new(user, request))
+      url, _ = subject.serialize(Saml::Kit::Builders::LogoutResponse.new(user, request))
       result = subject.deserialize(query_params_from(url))
       expect(result).to be_instance_of(Saml::Kit::LogoutResponse)
     end
spec/saml/builders/logout_response_spec.rb
@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-RSpec.describe Saml::Kit::LogoutResponse::Builder do
+RSpec.describe Saml::Kit::Builders::LogoutResponse do
   subject { described_class.new(user, request, configuration: configuration) }
   let(:configuration) { double(issuer: issuer)  }
   let(:user) { double(:user, name_id_for: SecureRandom.uuid) }
spec/saml/logout_request_spec.rb
@@ -117,7 +117,7 @@ RSpec.describe Saml::Kit::LogoutRequest do
   describe "#response_for" do
     it 'returns a logout response for a particular user' do
       user = double(:user)
-      expect(subject.response_for(user)).to be_instance_of(Saml::Kit::LogoutResponse::Builder)
+      expect(subject.response_for(user)).to be_instance_of(Saml::Kit::Builders::LogoutResponse)
     end
   end
 end