Commit 27651fb

mo <mo@mokhan.ca>
2017-11-30 16:14:05
move logout request to builder namespace.
1 parent 5f9919e
lib/saml/kit/builders/logout_request.rb
@@ -1,7 +1,7 @@
 module Saml
   module Kit
-    class LogoutRequest < Document
-      class Builder
+    module Builders
+      class LogoutRequest
         attr_accessor :id, :destination, :issuer, :name_id_format, :now
         attr_accessor :sign, :version
         attr_reader :user
spec/saml/bindings/http_post_spec.rb
@@ -22,7 +22,7 @@ RSpec.describe Saml::Kit::Bindings::HttpPost do
 
     it 'returns a SAMLRequest for a LogoutRequest' do
       user = double(:user, name_id_for: SecureRandom.uuid)
-      builder = Saml::Kit::LogoutRequest::Builder.new(user)
+      builder = Saml::Kit::Builders::LogoutRequest.new(user)
       url, saml_params = subject.serialize(builder, relay_state: relay_state)
 
       expect(url).to eql(location)
@@ -68,7 +68,7 @@ RSpec.describe Saml::Kit::Bindings::HttpPost do
 
     it 'deserializes to a LogoutRequest' do
       user = double(:user, name_id_for: SecureRandom.uuid)
-      builder = Saml::Kit::LogoutRequest::Builder.new(user)
+      builder = Saml::Kit::Builders::LogoutRequest.new(user)
       _, params = subject.serialize(builder)
       result = subject.deserialize(params)
       expect(result).to be_instance_of(Saml::Kit::LogoutRequest)
spec/saml/bindings/http_redirect_spec.rb
@@ -34,7 +34,7 @@ RSpec.describe Saml::Kit::Bindings::HttpRedirect do
 
     it 'deserializes the SAMLRequest to a LogoutRequest' do
       user = double(:user, name_id_for: SecureRandom.uuid)
-      url, _ = subject.serialize(Saml::Kit::LogoutRequest::Builder.new(user))
+      url, _ = subject.serialize(Saml::Kit::Builders::LogoutRequest.new(user))
       result = subject.deserialize(query_params_from(url))
       expect(result).to be_instance_of(Saml::Kit::LogoutRequest)
     end
spec/saml/builders/logout_request_spec.rb
@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-RSpec.describe Saml::Kit::LogoutRequest::Builder do
+RSpec.describe Saml::Kit::Builders::LogoutRequest do
   subject { described_class.new(user) }
   let(:user) { double(:user, name_id_for: name_id) }
   let(:name_id) { SecureRandom.uuid }
spec/saml/builders/logout_response_spec.rb
@@ -4,7 +4,7 @@ RSpec.describe Saml::Kit::LogoutResponse::Builder do
   subject { described_class.new(user, request, configuration: configuration) }
   let(:configuration) { double(issuer: issuer)  }
   let(:user) { double(:user, name_id_for: SecureRandom.uuid) }
-  let(:request) { Saml::Kit::LogoutRequest::Builder.new(user).build }
+  let(:request) { Saml::Kit::Builders::LogoutRequest.new(user).build }
   let(:issuer) { FFaker::Internet.http_url }
   let(:destination) { FFaker::Internet.http_url }
   let(:registry) { double(:registry) }
spec/saml/logout_request_spec.rb
@@ -2,7 +2,7 @@ require 'spec_helper'
 
 RSpec.describe Saml::Kit::LogoutRequest do
   subject { builder.build }
-  let(:builder) { described_class::Builder.new(user) }
+  let(:builder) { Saml::Kit::Builders::LogoutRequest.new(user) }
   let(:user) { double(:user, name_id_for: name_id) }
   let(:name_id) { SecureRandom.uuid }