SparqlTransmission 1 → 1.1

data/SparqlTransmission.gemspec

@@ -3,8 +3,8 @@ Gem::Specification.new do |spec|
   spec.name = 'SparqlTransmission'
   spec.author = 'Michael Sokol'
   spec.email = 'mikaa123@gmail.com'
-  spec.homepage = 'http://s139459221.onlinehome.fr/MichaelSokol/'
-  spec.version = '1'
+  spec.homepage = 'http://github.com/mikaa123/SparqlTransmission'
+  spec.version = '1.1'
   spec.summary = 'SparqlTransmission handles SPARQL queries over distant
     end-point and format the results in a ruby hash'
   spec.files = %w[
@@ -14,8 +14,14 @@ Gem::Specification.new do |spec|
     lib/SparqlTransmission.rb
     tests/SparqlTransmissionTest.rb
     tests/sparql_response.xml
+    tests/sparql_foaf_response.xml
     examples/example.rb
   ]
+  spec.test_files = %w[
+    tests/SparqlTransmissionTest.rb
+    tests/sparql_response.xml
+    tests/sparql_foaf_response.xml    
+  ]
                   
   
 end

data/lib/SparqlTransmission.rb

@@ -80,7 +80,15 @@ class SparqlTransmission
     unless req.is_a? String
       raise ArgumentError, "The response must be a String"
     end
-    doc = REXML::Document.new(req)
+
+    begin
+      doc = REXML::Document.new(req)
+    rescue
+      # Handle illegal characters here, we are being more permissive over badly
+      # formatted XMLs
+      req.sub!("&", '')
+      retry
+    end
     
     # Getting the list of sparql variables
     res = {:variables => doc.elements.collect('sparql/head/variable') do |element|

data/tests/SparqlTransmissionTest.rb

@@ -27,6 +27,9 @@ class SparqlTransmission_Test < Test::Unit::TestCase
     assert_equal("subject", result[:variables][0], "Wrong requested variable")
     assert_equal("http://www.openlinksw.com/schemas/virtrdf#DefaultQuadMap-G", result[:results][0][:subject], "Wrong result")
     assert_raise(ArgumentError, "ArgumentError not raised") { sq.send(:parse_request, 1) }
+    res = String.new
+    File.open('tests/sparql_foaf_response.xml').each { |line| res += line }
+    result = sq.send(:parse_request, res)
   end
   
   def test_fail