diff --git a/lib/diaspora_federation/discovery/xrd_document.rb b/lib/diaspora_federation/discovery/xrd_document.rb
index 55c1b4f..09b0721 100644
--- a/lib/diaspora_federation/discovery/xrd_document.rb
+++ b/lib/diaspora_federation/discovery/xrd_document.rb
@@ -135,7 +135,7 @@ module DiasporaFederation
private_class_method def self.parse_xrd_document(xrd_doc)
raise ArgumentError unless xrd_doc.instance_of?(String)
- doc = Nokogiri::XML::Document.parse(xrd_doc)
+ doc = Nokogiri::XML(xrd_doc)
raise InvalidDocument, "Not an XRD document" if !doc.root || doc.root.name != "XRD"
doc
end
diff --git a/lib/diaspora_federation/federation/fetcher.rb b/lib/diaspora_federation/federation/fetcher.rb
index 83cc08f..bce97df 100644
--- a/lib/diaspora_federation/federation/fetcher.rb
+++ b/lib/diaspora_federation/federation/fetcher.rb
@@ -13,7 +13,7 @@ module DiasporaFederation
response = HttpClient.get(url)
raise "Failed to fetch #{url}: #{response.status}" unless response.success?
- magic_env_xml = Nokogiri::XML::Document.parse(response.body).root
+ magic_env_xml = Nokogiri::XML(response.body).root
magic_env = Salmon::MagicEnvelope.unenvelop(magic_env_xml)
Receiver::Public.new(magic_env).receive
rescue => e
diff --git a/lib/diaspora_federation/federation/receiver.rb b/lib/diaspora_federation/federation/receiver.rb
index cc4277f..bfe9436 100644
--- a/lib/diaspora_federation/federation/receiver.rb
+++ b/lib/diaspora_federation/federation/receiver.rb
@@ -11,7 +11,7 @@ module DiasporaFederation
magic_env = if legacy
Salmon::Slap.from_xml(data)
else
- magic_env_xml = Nokogiri::XML::Document.parse(data).root
+ magic_env_xml = Nokogiri::XML(data).root
Salmon::MagicEnvelope.unenvelop(magic_env_xml)
end
Public.new(magic_env).receive
diff --git a/lib/diaspora_federation/salmon/encrypted_magic_envelope.rb b/lib/diaspora_federation/salmon/encrypted_magic_envelope.rb
index de593ca..a883474 100644
--- a/lib/diaspora_federation/salmon/encrypted_magic_envelope.rb
+++ b/lib/diaspora_federation/salmon/encrypted_magic_envelope.rb
@@ -54,7 +54,7 @@ module DiasporaFederation
key = encoded_key.map {|k, v| [k, Base64.decode64(v)] }.to_h
xml = AES.decrypt(encrypted_json["encrypted_magic_envelope"], key["key"], key["iv"])
- Nokogiri::XML::Document.parse(xml).root
+ Nokogiri::XML(xml).root
end
end
end
diff --git a/lib/diaspora_federation/salmon/encrypted_slap.rb b/lib/diaspora_federation/salmon/encrypted_slap.rb
index 42dba8b..20953eb 100644
--- a/lib/diaspora_federation/salmon/encrypted_slap.rb
+++ b/lib/diaspora_federation/salmon/encrypted_slap.rb
@@ -87,7 +87,7 @@ module DiasporaFederation
# @raise [MissingMagicEnvelope] if the +me:env+ element is missing in the XML
def self.from_xml(slap_xml, privkey)
raise ArgumentError unless slap_xml.instance_of?(String) && privkey.instance_of?(OpenSSL::PKey::RSA)
- doc = Nokogiri::XML::Document.parse(slap_xml)
+ doc = Nokogiri::XML(slap_xml)
header_elem = doc.at_xpath("d:diaspora/d:encrypted_header", Slap::NS)
raise MissingHeader if header_elem.nil?
@@ -160,7 +160,7 @@ module DiasporaFederation
key = JSON.parse(privkey.private_decrypt(Base64.decode64(cipher_header["aes_key"])))
xml = AES.decrypt(cipher_header["ciphertext"], Base64.decode64(key["key"]), Base64.decode64(key["iv"]))
- Nokogiri::XML::Document.parse(xml).root
+ Nokogiri::XML(xml).root
end
# Encrypt the header xml with an AES cipher and encrypt the cipher params
diff --git a/lib/diaspora_federation/salmon/magic_envelope.rb b/lib/diaspora_federation/salmon/magic_envelope.rb
index 1c8acaa..d30a093 100644
--- a/lib/diaspora_federation/salmon/magic_envelope.rb
+++ b/lib/diaspora_federation/salmon/magic_envelope.rb
@@ -128,7 +128,7 @@ module DiasporaFederation
logger.debug "unenvelop message from #{sender}:\n#{data}"
- new(XmlPayload.unpack(Nokogiri::XML::Document.parse(data).root), sender)
+ new(XmlPayload.unpack(Nokogiri::XML(data).root), sender)
end
private
diff --git a/lib/diaspora_federation/salmon/slap.rb b/lib/diaspora_federation/salmon/slap.rb
index 3e65b54..8a9c7f1 100644
--- a/lib/diaspora_federation/salmon/slap.rb
+++ b/lib/diaspora_federation/salmon/slap.rb
@@ -40,7 +40,7 @@ module DiasporaFederation
# @raise [MissingMagicEnvelope] if the +me:env+ element is missing from the XML
def self.from_xml(slap_xml)
raise ArgumentError unless slap_xml.instance_of?(String)
- doc = Nokogiri::XML::Document.parse(slap_xml)
+ doc = Nokogiri::XML(slap_xml)
author_elem = doc.at_xpath("d:diaspora/d:header/d:author_id", Slap::NS)
raise MissingAuthor if author_elem.nil? || author_elem.content.empty?
diff --git a/spec/controllers/diaspora_federation/fetch_controller_spec.rb b/spec/controllers/diaspora_federation/fetch_controller_spec.rb
index 8ad8548..235783b 100644
--- a/spec/controllers/diaspora_federation/fetch_controller_spec.rb
+++ b/spec/controllers/diaspora_federation/fetch_controller_spec.rb
@@ -14,7 +14,7 @@ module DiasporaFederation
expect_callback(:fetch_public_key, alice.diaspora_id).and_return(alice.public_key)
- magic_env_xml = Nokogiri::XML::Document.parse(response.body).root
+ magic_env_xml = Nokogiri::XML(response.body).root
magic_env = Salmon::MagicEnvelope.unenvelop(magic_env_xml)
entity = magic_env.payload
@@ -33,7 +33,7 @@ module DiasporaFederation
expect_callback(:fetch_public_key, alice.diaspora_id).and_return(alice.public_key)
- magic_env_xml = Nokogiri::XML::Document.parse(response.body).root
+ magic_env_xml = Nokogiri::XML(response.body).root
magic_env = Salmon::MagicEnvelope.unenvelop(magic_env_xml)
entity = magic_env.payload
diff --git a/spec/integration/comment_integration_spec.rb b/spec/integration/comment_integration_spec.rb
index 1dcdae2..9060852 100644
--- a/spec/integration/comment_integration_spec.rb
+++ b/spec/integration/comment_integration_spec.rb
@@ -188,7 +188,7 @@ XML
expect_callback(:fetch_private_key, parent.author).and_return(parent_key)
expect_callback(:fetch_related_entity, "Post", parent_guid).and_return(parent)
- xml = Nokogiri::XML::Document.parse(new_data_comment_xml_alice).root
+ xml = Nokogiri::XML(new_data_comment_xml_alice).root
Salmon::XmlPayload.unpack(xml).to_xml
end
end
@@ -201,19 +201,19 @@ XML
end
it "relays legacy signatures and xml" do
- xml = Nokogiri::XML::Document.parse(legacy_comment_xml_alice).root
+ xml = Nokogiri::XML(legacy_comment_xml_alice).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.to_xml.to_xml).to eq(legacy_signature_comment_xml_bob.strip)
end
it "relays new signatures and xml" do
- xml = Nokogiri::XML::Document.parse(new_signature_comment_xml_alice).root
+ xml = Nokogiri::XML(new_signature_comment_xml_alice).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.to_xml.to_xml).to eq(new_signature_comment_xml_bob.strip)
end
it "relays new signatures with new data" do
- xml = Nokogiri::XML::Document.parse(new_data_comment_xml_alice).root
+ xml = Nokogiri::XML(new_data_comment_xml_alice).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.to_xml.to_xml).to eq(new_data_comment_xml_bob.strip)
end
@@ -229,7 +229,7 @@ XML
end
it "parses legacy signatures and xml" do
- xml = Nokogiri::XML::Document.parse(legacy_comment_xml_bob).root
+ xml = Nokogiri::XML(legacy_comment_xml_bob).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.author).to eq(author)
@@ -237,7 +237,7 @@ XML
end
it "parses legacy signatures and with new xml" do
- xml = Nokogiri::XML::Document.parse(legacy_signature_comment_xml_bob).root
+ xml = Nokogiri::XML(legacy_signature_comment_xml_bob).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.author).to eq(author)
@@ -245,7 +245,7 @@ XML
end
it "parses new signatures with legacy xml" do
- xml = Nokogiri::XML::Document.parse(legacy_new_signature_comment_xml_bob).root
+ xml = Nokogiri::XML(legacy_new_signature_comment_xml_bob).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.author).to eq(author)
@@ -253,7 +253,7 @@ XML
end
it "parses new signatures and xml" do
- xml = Nokogiri::XML::Document.parse(new_signature_comment_xml_bob).root
+ xml = Nokogiri::XML(new_signature_comment_xml_bob).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.author).to eq(author)
@@ -261,7 +261,7 @@ XML
end
it "parses new data with legacy xml" do
- xml = Nokogiri::XML::Document.parse(legacy_new_data_comment_xml_bob).root
+ xml = Nokogiri::XML(legacy_new_data_comment_xml_bob).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.author).to eq(author)
@@ -270,7 +270,7 @@ XML
end
it "parses new xml with additional data" do
- xml = Nokogiri::XML::Document.parse(new_data_comment_xml_bob).root
+ xml = Nokogiri::XML(new_data_comment_xml_bob).root
entity = Salmon::XmlPayload.unpack(xml)
expect(entity.author).to eq(author)
diff --git a/spec/lib/diaspora_federation/entities/comment_spec.rb b/spec/lib/diaspora_federation/entities/comment_spec.rb
index 008c369..2ea3099 100644
--- a/spec/lib/diaspora_federation/entities/comment_spec.rb
+++ b/spec/lib/diaspora_federation/entities/comment_spec.rb
@@ -59,7 +59,7 @@ JSON
describe "#created_at" do
it "has a created_at after parse" do
- entity = described_class.from_xml(Nokogiri::XML::Document.parse(xml).root)
+ entity = described_class.from_xml(Nokogiri::XML(xml).root)
expect(entity.created_at).to be_within(1).of(Time.now.utc)
end
diff --git a/spec/lib/diaspora_federation/entities/conversation_spec.rb b/spec/lib/diaspora_federation/entities/conversation_spec.rb
index c13b617..28349ab 100644
--- a/spec/lib/diaspora_federation/entities/conversation_spec.rb
+++ b/spec/lib/diaspora_federation/entities/conversation_spec.rb
@@ -56,7 +56,7 @@ XML
XML
- parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(minimal_xml).root)
+ parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML(minimal_xml).root)
expect(parsed_instance.messages).to eq([])
end
end
diff --git a/spec/lib/diaspora_federation/entities/event_spec.rb b/spec/lib/diaspora_federation/entities/event_spec.rb
index 5a759e6..84afaa3 100644
--- a/spec/lib/diaspora_federation/entities/event_spec.rb
+++ b/spec/lib/diaspora_federation/entities/event_spec.rb
@@ -40,7 +40,7 @@ XML
XML
- parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(minimal_xml).root)
+ parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML(minimal_xml).root)
expect(parsed_instance.end).to be_nil
expect(parsed_instance.all_day).to be_falsey
expect(parsed_instance.timezone).to be_nil
diff --git a/spec/lib/diaspora_federation/entities/like_spec.rb b/spec/lib/diaspora_federation/entities/like_spec.rb
index 2477ae8..ae90f5b 100644
--- a/spec/lib/diaspora_federation/entities/like_spec.rb
+++ b/spec/lib/diaspora_federation/entities/like_spec.rb
@@ -69,7 +69,7 @@ JSON
XML
expect {
- DiasporaFederation::Entities::Like.from_xml(Nokogiri::XML::Document.parse(broken_xml).root)
+ DiasporaFederation::Entities::Like.from_xml(Nokogiri::XML(broken_xml).root)
}.to raise_error Entity::ValidationError, "invalid DiasporaFederation::Entities::Like! missing 'parent_type'."
end
@@ -83,7 +83,7 @@ XML
XML
expect {
- DiasporaFederation::Entities::Like.from_xml(Nokogiri::XML::Document.parse(broken_xml).root)
+ DiasporaFederation::Entities::Like.from_xml(Nokogiri::XML(broken_xml).root)
}.to raise_error Entity::ValidationError, "invalid DiasporaFederation::Entities::Like! missing 'parent_guid'."
end
end
diff --git a/spec/lib/diaspora_federation/entities/photo_spec.rb b/spec/lib/diaspora_federation/entities/photo_spec.rb
index b873218..bb30d9e 100644
--- a/spec/lib/diaspora_federation/entities/photo_spec.rb
+++ b/spec/lib/diaspora_federation/entities/photo_spec.rb
@@ -58,7 +58,7 @@ JSON
XML
- parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(minimal_xml).root)
+ parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML(minimal_xml).root)
expect(parsed_instance.public).to be_falsey
expect(parsed_instance.text).to be_nil
end
diff --git a/spec/lib/diaspora_federation/entities/profile_spec.rb b/spec/lib/diaspora_federation/entities/profile_spec.rb
index 8e2c280..4994ba0 100644
--- a/spec/lib/diaspora_federation/entities/profile_spec.rb
+++ b/spec/lib/diaspora_federation/entities/profile_spec.rb
@@ -59,7 +59,7 @@ JSON
XML
- parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(minimal_xml).root)
+ parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML(minimal_xml).root)
expect(parsed_instance.first_name).to be_nil
expect(parsed_instance.last_name).to be_nil
expect(parsed_instance.image_url).to be_nil
diff --git a/spec/lib/diaspora_federation/entities/relayable_retraction_spec.rb b/spec/lib/diaspora_federation/entities/relayable_retraction_spec.rb
index 67bf69d..09bfce4 100644
--- a/spec/lib/diaspora_federation/entities/relayable_retraction_spec.rb
+++ b/spec/lib/diaspora_federation/entities/relayable_retraction_spec.rb
@@ -32,7 +32,7 @@ XML
context "parse retraction" do
it "parses the xml as a retraction" do
expect(Entities::Retraction).to receive(:fetch_target).and_return(target_entity)
- retraction = Entities::RelayableRetraction.from_xml(Nokogiri::XML::Document.parse(xml).root)
+ retraction = Entities::RelayableRetraction.from_xml(Nokogiri::XML(xml).root)
expect(retraction).to be_a(Entities::Retraction)
expect(retraction.author).to eq(data[:author])
expect(retraction.target_guid).to eq(data[:target_guid])
diff --git a/spec/lib/diaspora_federation/entities/relayable_spec.rb b/spec/lib/diaspora_federation/entities/relayable_spec.rb
index d317871..0ad1d1b 100644
--- a/spec/lib/diaspora_federation/entities/relayable_spec.rb
+++ b/spec/lib/diaspora_federation/entities/relayable_spec.rb
@@ -241,7 +241,7 @@ XML
XML
it "doesn't drop unknown properties" do
- entity = Entities::SomeRelayable.from_xml(Nokogiri::XML::Document.parse(new_xml).root)
+ entity = Entities::SomeRelayable.from_xml(Nokogiri::XML(new_xml).root)
expect(entity).to be_an_instance_of Entities::SomeRelayable
expect(entity.property).to eq(property)
@@ -251,7 +251,7 @@ XML
end
it "hand over the order in the xml to the instance without signatures" do
- entity = Entities::SomeRelayable.from_xml(Nokogiri::XML::Document.parse(new_xml).root)
+ entity = Entities::SomeRelayable.from_xml(Nokogiri::XML(new_xml).root)
expect(entity.xml_order).to eq([:author, :guid, :parent_guid, "new_property", :property])
end
@@ -280,7 +280,7 @@ XML
XML
expect {
- Entities::SomeRelayable.from_xml(Nokogiri::XML::Document.parse(broken_xml).root)
+ Entities::SomeRelayable.from_xml(Nokogiri::XML(broken_xml).root)
}.to raise_error Entity::ValidationError,
"invalid DiasporaFederation::Entities::SomeRelayable! missing 'parent_guid'."
end
diff --git a/spec/lib/diaspora_federation/entities/request_spec.rb b/spec/lib/diaspora_federation/entities/request_spec.rb
index 7d2287e..250c37f 100644
--- a/spec/lib/diaspora_federation/entities/request_spec.rb
+++ b/spec/lib/diaspora_federation/entities/request_spec.rb
@@ -30,7 +30,7 @@ XML
context "parse contact" do
it "parses the xml as a contact" do
- contact = Entities::Request.from_xml(Nokogiri::XML::Document.parse(xml).root)
+ contact = Entities::Request.from_xml(Nokogiri::XML(xml).root)
expect(contact).to be_a(Entities::Contact)
end
end
diff --git a/spec/lib/diaspora_federation/entities/reshare_spec.rb b/spec/lib/diaspora_federation/entities/reshare_spec.rb
index 33d3bc7..e01ecf1 100644
--- a/spec/lib/diaspora_federation/entities/reshare_spec.rb
+++ b/spec/lib/diaspora_federation/entities/reshare_spec.rb
@@ -50,7 +50,7 @@ JSON
XML
- parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(minimal_xml).root)
+ parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML(minimal_xml).root)
expect(parsed_instance.public).to be_truthy
expect(parsed_instance.provider_display_name).to be_nil
end
@@ -61,7 +61,7 @@ XML
expect_callback(:fetch_related_entity, "Post", data[:root_guid]).and_return(nil)
expect(Federation::Fetcher).to receive(:fetch_public).with(data[:root_author], "Post", data[:root_guid])
- Entities::Reshare.from_xml(Nokogiri::XML::Document.parse(xml).root)
+ Entities::Reshare.from_xml(Nokogiri::XML(xml).root)
end
end
end
diff --git a/spec/lib/diaspora_federation/entities/signed_retraction_spec.rb b/spec/lib/diaspora_federation/entities/signed_retraction_spec.rb
index b1582d4..0547299 100644
--- a/spec/lib/diaspora_federation/entities/signed_retraction_spec.rb
+++ b/spec/lib/diaspora_federation/entities/signed_retraction_spec.rb
@@ -25,7 +25,7 @@ XML
context "parse retraction" do
it "parses the xml as a retraction" do
expect(Entities::Retraction).to receive(:fetch_target).and_return(target_entity)
- retraction = Entities::SignedRetraction.from_xml(Nokogiri::XML::Document.parse(xml).root)
+ retraction = Entities::SignedRetraction.from_xml(Nokogiri::XML(xml).root)
expect(retraction).to be_a(Entities::Retraction)
expect(retraction.author).to eq(data[:author])
expect(retraction.target_guid).to eq(data[:target_guid])
diff --git a/spec/lib/diaspora_federation/entities/status_message_spec.rb b/spec/lib/diaspora_federation/entities/status_message_spec.rb
index 39bf89e..1bdf01c 100644
--- a/spec/lib/diaspora_federation/entities/status_message_spec.rb
+++ b/spec/lib/diaspora_federation/entities/status_message_spec.rb
@@ -129,7 +129,7 @@ JSON
XML
- parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(minimal_xml).root)
+ parsed_instance = DiasporaFederation::Salmon::XmlPayload.unpack(Nokogiri::XML(minimal_xml).root)
expect(parsed_instance.photos).to eq([])
expect(parsed_instance.location).to be_nil
expect(parsed_instance.poll).to be_nil
diff --git a/spec/lib/diaspora_federation/entity_spec.rb b/spec/lib/diaspora_federation/entity_spec.rb
index 7e0f6d1..ec0df37 100644
--- a/spec/lib/diaspora_federation/entity_spec.rb
+++ b/spec/lib/diaspora_federation/entity_spec.rb
@@ -99,7 +99,7 @@ module DiasporaFederation
it "replaces invalid XML characters" do
entity = Entities::TestEntity.new(test: "asdf asdf asdf💩asdf\nasdf")
xml = entity.to_xml.to_xml
- parsed = Entities::TestEntity.from_xml(Nokogiri::XML::Document.parse(xml).root).test
+ parsed = Entities::TestEntity.from_xml(Nokogiri::XML(xml).root).test
expect(parsed).to eq("asdf�asdf asdf💩asdf\nasdf")
end
end
@@ -127,7 +127,7 @@ module DiasporaFederation
test2: "qwer",
test3: true
)
- Entities::TestDefaultEntity.from_xml(Nokogiri::XML::Document.parse(<<-XML).root)
+ Entities::TestDefaultEntity.from_xml(Nokogiri::XML(<<-XML).root)
asdf
qwer
@@ -140,11 +140,11 @@ XML
arguments = [{arg1: "value"}]
expect_any_instance_of(DiasporaFederation::Parsers::XmlParser).to receive(:parse).and_return(arguments)
expect(Entities::TestDefaultEntity).to receive(:from_hash).with(*arguments)
- Entities::TestDefaultEntity.from_xml(Nokogiri::XML::Document.parse("").root)
+ Entities::TestDefaultEntity.from_xml(Nokogiri::XML("").root)
end
it "passes input parameter directly to .parse method of the parser" do
- root = Nokogiri::XML::Document.parse("").root
+ root = Nokogiri::XML("").root
expect_any_instance_of(DiasporaFederation::Parsers::XmlParser).to receive(:parse).with(root)
.and_return([{test1: "2", test2: "1"}])
Entities::TestDefaultEntity.from_xml(root)
@@ -444,7 +444,7 @@ JSON
XML
- entity = Entities::TestNestedEntity.from_xml(Nokogiri::XML::Document.parse(xml).root)
+ entity = Entities::TestNestedEntity.from_xml(Nokogiri::XML(xml).root)
expect(entity.asdf).to eq("FDSA")
expect(entity.test).to be_nil
diff --git a/spec/lib/diaspora_federation/federation/receiver_spec.rb b/spec/lib/diaspora_federation/federation/receiver_spec.rb
index 83d2df1..54e4396 100644
--- a/spec/lib/diaspora_federation/federation/receiver_spec.rb
+++ b/spec/lib/diaspora_federation/federation/receiver_spec.rb
@@ -94,7 +94,7 @@ module DiasporaFederation
end
it "redirects exceptions from the receiver" do
- invalid_magic_env = Nokogiri::XML::Document.parse("").root
+ invalid_magic_env = Nokogiri::XML("").root
data = Salmon::EncryptedMagicEnvelope.encrypt(invalid_magic_env, recipient_key.public_key)
expect {
diff --git a/spec/lib/diaspora_federation/parsers/relayable_xml_parser_spec.rb b/spec/lib/diaspora_federation/parsers/relayable_xml_parser_spec.rb
index e40b979..d5ef872 100644
--- a/spec/lib/diaspora_federation/parsers/relayable_xml_parser_spec.rb
+++ b/spec/lib/diaspora_federation/parsers/relayable_xml_parser_spec.rb
@@ -4,7 +4,7 @@ module DiasporaFederation
let(:entity_class) { Entities::SomeRelayable }
let(:xml_parser) { Parsers::RelayableXmlParser.new(entity_class) }
it "passes order of the XML elements as a second argument in the returned list" do
- xml_object = Nokogiri::XML::Document.parse(<<-XML).root
+ xml_object = Nokogiri::XML(<<-XML).root
im a guid
value
diff --git a/spec/lib/diaspora_federation/parsers/xml_parser_spec.rb b/spec/lib/diaspora_federation/parsers/xml_parser_spec.rb
index 010b35c..6f51387 100644
--- a/spec/lib/diaspora_federation/parsers/xml_parser_spec.rb
+++ b/spec/lib/diaspora_federation/parsers/xml_parser_spec.rb
@@ -18,7 +18,7 @@ module DiasporaFederation
XML
expect {
- xml_parser.parse(Nokogiri::XML::Document.parse(xml).root)
+ xml_parser.parse(Nokogiri::XML(xml).root)
}.to raise_error Parsers::BaseParser::InvalidRootNode,
"'unknown_entity' can't be parsed by DiasporaFederation::Entities::TestComplexEntity"
end
@@ -39,7 +39,7 @@ XML
XML
- parsed = Parsers::XmlParser.new(Entities::TestEntityWithXmlName).parse(Nokogiri::XML::Document.parse(xml).root)
+ parsed = Parsers::XmlParser.new(Entities::TestEntityWithXmlName).parse(Nokogiri::XML(xml).root)
expect(parsed[0][:test]).to eq("asdf")
expect(parsed[0][:qwer]).to eq("qwer")
@@ -53,7 +53,7 @@ XML
XML
- parsed = Parsers::XmlParser.new(Entities::TestEntityWithXmlName).parse(Nokogiri::XML::Document.parse(xml).root)
+ parsed = Parsers::XmlParser.new(Entities::TestEntityWithXmlName).parse(Nokogiri::XML(xml).root)
expect(parsed[0][:test]).to eq("asdf")
expect(parsed[0][:qwer]).to eq("qwer")
@@ -68,7 +68,7 @@ XML
XML
- parsed = Parsers::XmlParser.new(Entities::TestDefaultEntity).parse(Nokogiri::XML::Document.parse(xml).root)
+ parsed = Parsers::XmlParser.new(Entities::TestDefaultEntity).parse(Nokogiri::XML(xml).root)
expect(parsed[0][:test1]).to eq("asdf")
expect(parsed[0][:test2]).to eq("qwer")
@@ -82,7 +82,7 @@ XML
XML
- parsed = Parsers::XmlParser.new(Entities::TestEntityWithBoolean).parse(Nokogiri::XML::Document.parse(xml).root)
+ parsed = Parsers::XmlParser.new(Entities::TestEntityWithBoolean).parse(Nokogiri::XML(xml).root)
expect(parsed[0][:test]).to eq(false)
end
@@ -95,7 +95,7 @@ XML
XML
parsed = Parsers::XmlParser.new(Entities::TestEntityWithBoolean).parse(
- Nokogiri::XML::Document.parse(xml).root
+ Nokogiri::XML(xml).root
)
expect(parsed[0][:test]).to be_nil
end
@@ -110,7 +110,7 @@ XML
XML
parsed = Parsers::XmlParser.new(Entities::TestEntityWithInteger).parse(
- Nokogiri::XML::Document.parse(xml).root
+ Nokogiri::XML(xml).root
)
expect(parsed[0][:test]).to be_nil
end
@@ -125,7 +125,7 @@ XML
XML
parsed = Parsers::XmlParser.new(Entities::TestEntityWithTimestamp).parse(
- Nokogiri::XML::Document.parse(xml).root
+ Nokogiri::XML(xml).root
)
expect(parsed[0][:test]).to be_nil
end
@@ -161,7 +161,7 @@ XML
XML
- parsed = Parsers::XmlParser.new(Entities::TestDefaultEntity).parse(Nokogiri::XML::Document.parse(xml).root)
+ parsed = Parsers::XmlParser.new(Entities::TestDefaultEntity).parse(Nokogiri::XML(xml).root)
expect(parsed[0]["test_new"]).to eq("new_value")
end
end
diff --git a/spec/lib/diaspora_federation/salmon/encrypted_magic_envelope_spec.rb b/spec/lib/diaspora_federation/salmon/encrypted_magic_envelope_spec.rb
index f39fa21..7b5f264 100644
--- a/spec/lib/diaspora_federation/salmon/encrypted_magic_envelope_spec.rb
+++ b/spec/lib/diaspora_federation/salmon/encrypted_magic_envelope_spec.rb
@@ -32,7 +32,7 @@ module DiasporaFederation
xml = Salmon::AES.decrypt(json["encrypted_magic_envelope"], key["key"], key["iv"])
- expect(Nokogiri::XML::Document.parse(xml).to_xml).to eq(magic_env.to_xml)
+ expect(Nokogiri::XML(xml).to_xml).to eq(magic_env.to_xml)
end
end
diff --git a/spec/lib/diaspora_federation/salmon/encrypted_slap_spec.rb b/spec/lib/diaspora_federation/salmon/encrypted_slap_spec.rb
index ac6d0d8..b461c67 100644
--- a/spec/lib/diaspora_federation/salmon/encrypted_slap_spec.rb
+++ b/spec/lib/diaspora_federation/salmon/encrypted_slap_spec.rb
@@ -55,7 +55,7 @@ module DiasporaFederation
end
it "generates valid xml" do
- doc = Nokogiri::XML::Document.parse(slap_xml)
+ doc = Nokogiri::XML(slap_xml)
expect(doc.root.name).to eq("diaspora")
expect(doc.at_xpath("d:diaspora/d:encrypted_header", ns).content).to_not be_empty
expect(doc.xpath("d:diaspora/me:env", ns)).to have(1).item
@@ -64,7 +64,7 @@ module DiasporaFederation
it "can generate xml for two people" do
slap = Salmon::EncryptedSlap.prepare(sender, privkey, payload)
- doc1 = Nokogiri::XML::Document.parse(slap.generate_xml(recipient_key.public_key))
+ doc1 = Nokogiri::XML(slap.generate_xml(recipient_key.public_key))
enc_header1 = doc1.at_xpath("d:diaspora/d:encrypted_header", ns).content
cipher_header1 = JSON.parse(Base64.decode64(enc_header1))
header_key1 = JSON.parse(recipient_key.private_decrypt(Base64.decode64(cipher_header1["aes_key"])))
@@ -73,7 +73,7 @@ module DiasporaFederation
Base64.decode64(header_key1["iv"]))
recipient2_key = OpenSSL::PKey::RSA.generate(1024)
- doc2 = Nokogiri::XML::Document.parse(slap.generate_xml(recipient2_key.public_key))
+ doc2 = Nokogiri::XML(slap.generate_xml(recipient2_key.public_key))
enc_header2 = doc2.at_xpath("d:diaspora/d:encrypted_header", ns).content
cipher_header2 = JSON.parse(Base64.decode64(enc_header2))
header_key2 = JSON.parse(recipient2_key.private_decrypt(Base64.decode64(cipher_header2["aes_key"])))
@@ -88,13 +88,13 @@ module DiasporaFederation
end
it "does not add the sender to the magic envelope" do
- doc = Nokogiri::XML::Document.parse(slap_xml)
+ doc = Nokogiri::XML(slap_xml)
expect(doc.at_xpath("d:diaspora/me:env/me:sig", ns)["key_id"]).to be_nil
end
context "header" do
subject {
- doc = Nokogiri::XML::Document.parse(slap_xml)
+ doc = Nokogiri::XML(slap_xml)
doc.at_xpath("d:diaspora/d:encrypted_header", ns).content
}
let(:cipher_header) { JSON.parse(Base64.decode64(subject)) }
@@ -125,7 +125,7 @@ module DiasporaFederation
Base64.decode64(header_key["key"]),
Base64.decode64(header_key["iv"]))
}.not_to raise_error
- header_doc = Nokogiri::XML::Document.parse(header)
+ header_doc = Nokogiri::XML(header)
expect(header_doc.root.name).to eq("decrypted_header")
expect(header_doc.xpath("//iv")).to have(1).item
expect(header_doc.xpath("//aes_key")).to have(1).item
diff --git a/spec/lib/diaspora_federation/salmon/magic_envelope_spec.rb b/spec/lib/diaspora_federation/salmon/magic_envelope_spec.rb
index 90c76a4..79e771f 100644
--- a/spec/lib/diaspora_federation/salmon/magic_envelope_spec.rb
+++ b/spec/lib/diaspora_federation/salmon/magic_envelope_spec.rb
@@ -128,7 +128,7 @@ module DiasporaFederation
it "verifies the envelope structure" do
expect {
- Salmon::MagicEnvelope.unenvelop(Nokogiri::XML::Document.parse("").root, sender)
+ Salmon::MagicEnvelope.unenvelop(Nokogiri::XML("").root, sender)
}.to raise_error Salmon::InvalidEnvelope
end
diff --git a/spec/lib/diaspora_federation/salmon/xml_payload_spec.rb b/spec/lib/diaspora_federation/salmon/xml_payload_spec.rb
index 998640d..c254580 100644
--- a/spec/lib/diaspora_federation/salmon/xml_payload_spec.rb
+++ b/spec/lib/diaspora_federation/salmon/xml_payload_spec.rb
@@ -15,7 +15,7 @@ XML
context "sanity" do
it "expects an Nokogiri::XML::Element as param" do
expect {
- Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(entity_xml).root)
+ Salmon::XmlPayload.unpack(Nokogiri::XML(entity_xml).root)
}.not_to raise_error
end
@@ -29,7 +29,7 @@ XML
end
context "returned object" do
- subject { Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(entity_xml).root) }
+ subject { Salmon::XmlPayload.unpack(Nokogiri::XML(entity_xml).root) }
it "#to_h should match entity.to_h" do
expect(subject.to_h).to eq(entity.to_h)
@@ -47,7 +47,7 @@ XML
XML
- entity = Salmon::XmlPayload.unpack(Nokogiri::XML::Document.parse(xml).root)
+ entity = Salmon::XmlPayload.unpack(Nokogiri::XML(xml).root)
expect(entity).to be_an_instance_of Entities::TestEntity
expect(entity.test).to eq("asdf")