Commit 0ccff0a

@@ -66,12 +66,6 @@ module Saml
           Saml::Kit::ServiceProviderMetadata.new(to_xml)
         end
 
-        def self.build
-          builder = new
-          yield builder
-          builder.build
-        end
-
         private
 
         def entity_descriptor_options

@@ -17,6 +17,13 @@ module Saml
         attribute = document.find_by("/md:EntityDescriptor/md:#{name}").attribute("WantAssertionsSigned")
         attribute.text.downcase == "true"
       end
+
+      def self.build
+        builder = Saml::Kit::Builders::ServiceProviderMetadata.new
+        yield builder
+        builder.build
+      end
+
     end
   end
 end

@@ -36,17 +36,4 @@ RSpec.describe Saml::Kit::Builders::ServiceProviderMetadata do
       Saml::Kit.configuration.stripped_encryption_certificate,
     ])
   end
-
-  describe ".build" do
-    it 'provides a nice API for building metadata' do
-      result = described_class.build do |builder|
-        builder.entity_id = entity_id
-        builder.add_assertion_consumer_service(acs_url, binding: :http_post)
-      end
-
-      expect(result).to be_instance_of(Saml::Kit::ServiceProviderMetadata)
-      expect(result.entity_id).to eql(entity_id)
-      expect(result.assertion_consumer_service_for(binding: :http_post).location).to eql(acs_url)
-    end
-  end
 end

@@ -127,4 +127,19 @@ RSpec.describe Saml::Kit::ServiceProviderMetadata do
       expect(subject.matches?(fingerprint)).to be_falsey
     end
   end
+
+  describe ".build" do
+    let(:acs_url) { FFaker::Internet.uri("https") }
+
+    it 'provides a nice API for building metadata' do
+      result = described_class.build do |builder|
+        builder.entity_id = entity_id
+        builder.add_assertion_consumer_service(acs_url, binding: :http_post)
+      end
+
+      expect(result).to be_instance_of(Saml::Kit::ServiceProviderMetadata)
+      expect(result.entity_id).to eql(entity_id)
+      expect(result.assertion_consumer_service_for(binding: :http_post).location).to eql(acs_url)
+    end
+  end
 end