RDF::NTriples::Writer#format_uri should escape value

[gkellogg]

Just as literals must be escaped to be represented as valid RDF strings, URIs must also be escaped.

Consider making the following change:

def format_uri(uri, options = {})
  "<%s>" % escaped(uri_for(uri))
end

Here are specs I've used:

describe "utf-8 escaped" do
  {
    %(http://a/D%C3%BCrst)                => %(<http://a/D%C3%BCrst>),
    %(http://a/D\u00FCrst)                => %(<http://a/D\\u00FCrst>),
    %(http://b/Dürst)                     => %(<http://b/D\\u00FCrst>),
    %(http://a/\u{15678}another)          => %(<http://a/\\U00015678another>),
  }.each_pair do |uri, dump|
    it "should dump #{uri} as #{dump}" do
      RDF::URI.new(uri).to_ntriples.should == dump
    end
  end
end

[artob]

Thanks, Gregg. I'll incorporate this into 0.2.3, which I'll probably cut tomorrow.

[artob]

Fixed in 542d20c back in October. RDF.rb 0.3.0 will include this fix.