More safe to commit changes

Author: johnnyshields

lib/ruby_saml/xml/document_signer.rb

@@ -53,7 +53,7 @@ def build_signature_element(noko, private_key, certificate, signature_method, di
             xml['ds'].SignedInfo do
               xml['ds'].CanonicalizationMethod(Algorithm: RubySaml::XML::C14N)
               xml['ds'].SignatureMethod(Algorithm: signature_method)
-              xml['ds'].Reference(URI: "##{noko.root.attr('ID')}") do
+              xml['ds'].Reference(URI: "##{noko.root['ID']}") do
                 xml['ds'].Transforms do
                   xml['ds'].Transform(Algorithm: RubySaml::XML::ENVELOPED_SIG)
                   xml['ds'].Transform(Algorithm: RubySaml::XML::C14N) do

test/logoutresponse_test.rb

@@ -24,7 +24,7 @@ class RubySamlTest < Minitest::Test
       end
 
       it "accept constructor-injected options" do
-        logoutresponse = RubySaml::Logoutresponse.new(valid_logout_response_document, nil, { :foo => :bar} )
+        logoutresponse = RubySaml::Logoutresponse.new(valid_logout_response_document, nil, { foo: :bar })
         refute logoutresponse.options.empty?
       end