ldp 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5891836d3638c426292b938c2e815bc78963a486
-  data.tar.gz: e846d7a47515ee66db76b539979b01024b39f56d
+  metadata.gz: a0192dec60a69549a92992ee9a8a17dc7a2e7bea
+  data.tar.gz: 9f1d63533db1609c91b7ca86d807e645a1148343
 SHA512:
-  metadata.gz: f82a7c37efd44e165cfba252c64073255d805074ba78e92a3a5ea2d988407b13987b9947614102b120b006578a15ddb43f68e30904cdcc006765ab79c48dec36
-  data.tar.gz: 44771b0779cbd68c5c0f8b2917176ca9de4a36290b3d433318bf859b77688e1e6164e28150af0ee3a66612ea304944ce235c886ffd5f93f8ac7cb626645e1bb7
+  metadata.gz: a2576375233b5f9c8fb7879e26c187c75a0cb30e740db682d084ebd57f88db616d81efaeb6c04d4289fcb10095fe34dca5199f7936ebc1c65028e08f56bae31d
+  data.tar.gz: 0bcd587f427cefcc0d4d702d57da18389a2b9c602b898a06708d829c28cbfd3a8cb8abff4ed195c7f51d25cd67584a77140d03fbb3f493f6609849bcef1427e6

data/.travis.yml

@@ -1,4 +1,9 @@
 language: ruby
+cache: bundler
+sudo: false
+env:
+  global:
+    - NOKOGIRI_USE_SYSTEM_LIBRARIES=true
 rvm:
   - 1.9.3
   - 2.0.0

data/Gemfile

@@ -3,5 +3,6 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in ldp-client.gemspec
 gemspec
 
-gem 'byebug', :platforms => [:mri_20]
+gem 'slop', '~> 3.6' if RUBY_PLATFORM == "java"
+gem 'byebug', platforms: :mri_20
 gem 'activesupport'

data/lib/ldp/resource/binary_source.rb

@@ -4,20 +4,27 @@ module Ldp
 
     def initialize client, subject, content_or_response = nil, base_path = ''
       super
-      
+
       case content_or_response
       when Faraday::Response
       else
         @content = content_or_response
       end
     end
-    
+
     def content
       @content ||= get.body
     end
-    
+
     def described_by
       client.find_or_initialize Array(Ldp::Response.links(self)["describedby"]).first
     end
+
+    # Override inspect so that `content` is never shown. It is typically too big to be helpful
+    def inspect
+      string = "#<#{self.class.name}:#{self.object_id} "
+      fields = [:subject].map{|field| "#{field}=\"#{self.send(field)}\""}
+      string << fields.join(", ") << ">"
+    end
   end
 end

data/lib/ldp/response.rb

@@ -60,6 +60,17 @@ module Ldp
       container?(response) || Array(links(response)["type"]).include?(Ldp.rdf_source)
     end
 
+    def dup
+      super.tap do |new_resp|
+        new_resp.send(:extend, Ldp::Response)
+        if ::RUBY_VERSION < '2.0'
+          new_resp.send(:remove_instance_variable, :@graph)
+        else
+          new_resp.remove_instance_variable(:@graph)
+        end
+      end
+    end
+
     ##
     # Link: headers from the HTTP response
     def links
@@ -109,7 +120,7 @@ module Ldp
     # Get the graph for the resource (or a blank graph if there is no metadata for the resource)
     def graph
       @graph ||= begin
-        raise UnexpectedContentType, "The resource #{page_subject} is not an RDFSource" unless rdf_source?
+        raise UnexpectedContentType, "The resource at #{page_subject} is not an RDFSource" unless rdf_source?
         graph = RDF::Graph.new
 
         if resource?

data/lib/ldp/version.rb

@@ -1,3 +1,3 @@
 module Ldp
-  VERSION = "0.2.0"
+  VERSION = "0.2.1"
 end

data/spec/lib/ldp/resource/binary_source_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe Ldp::Resource::BinarySource do
+  let(:client) { double }
+  let(:uri) { 'http://example.com/foo/bar' }
+  let(:content) { 'somecontent' }
+  subject { described_class.new(client, uri, content) }
+
+  it "should not display content to inspect" do
+    expect(subject.inspect).to match /subject=\"http:\/\/example\.com\/foo\/bar\"/
+    expect(subject.inspect).not_to match /somecontent/
+  end
+
+end

data/spec/lib/ldp/response_spec.rb

@@ -4,7 +4,7 @@ describe Ldp::Response do
   LDP_RDF_RESOURCE_HEADERS = { "Link" => "<#{Ldp.resource.to_s}>;rel=\"type\", <#{Ldp.direct_container.to_s}>;rel=\"type\""}
   LDP_NON_RDF_SOURCE_HEADERS = { "Link" => "<#{Ldp.resource.to_s}>;rel=\"type\", <#{Ldp.non_rdf_source.to_s}>;rel=\"type\""}
 
-  let(:mock_response) { double(headers: {}, env: { url: "info:a" }) }
+  let(:mock_response) { double("mock response", headers: {}, env: { url: "info:a" }) }
   let(:mock_client) { double(Ldp::Client) }
 
   subject do
@@ -18,6 +18,34 @@ describe Ldp::Response do
     end
   end
 
+  describe "#dup" do
+    let(:simple_container_graph) { "<> a <http://www.w3.org/ns/ldp#Container> ." }
+    let(:link) { ["<http://www.w3.org/ns/ldp#Resource>;rel=\"type\"","<http://www.w3.org/ns/ldp#BasicContainer>;rel=\"type\""] }
+
+    let(:conn_stubs) do
+      Faraday::Adapter::Test::Stubs.new do |stub|
+        stub.get('/a_container') { [200, {"Link" => link}, simple_container_graph] }
+      end
+    end
+
+    let(:mock_conn) { Faraday.new { |builder| builder.adapter :test, conn_stubs } }
+    let(:client) { Ldp::Client.new mock_conn }
+    let(:raw_response) { client.get "a_container" }
+
+    let(:response) { Ldp::Response.wrap mock_client, raw_response }
+    subject { response.dup }
+    it { is_expected.to respond_to :links }
+
+    it "should not have duplicated the graph" do
+      expect(response.graph.object_id).not_to eq subject.graph.object_id
+    end
+
+    it "should have duplicated the body" do
+      expect(response.body.object_id).to eq subject.body.object_id
+    end
+  end
+
+
   describe ".links" do
     it "should extract link headers with relations as a hash" do
       allow(mock_response).to receive(:headers).and_return(
@@ -26,7 +54,7 @@ describe Ldp::Response do
             "<abc>;rel=\"some-multi-rel\"",
             "<123>;rel=\"some-multi-rel\"",
             "<vanilla-link>"
-          ] 
+          ]
         )
       h = Ldp::Response.links mock_response
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ldp
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Chris Beer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-11 00:00:00.000000000 Z
+date: 2015-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -142,6 +142,7 @@ files:
 - spec/lib/integration/integration_spec.rb
 - spec/lib/ldp/client_spec.rb
 - spec/lib/ldp/orm/orm_spec.rb
+- spec/lib/ldp/resource/binary_source_spec.rb
 - spec/lib/ldp/resource/rdf_source_spec.rb
 - spec/lib/ldp/resource_spec.rb
 - spec/lib/ldp/response_spec.rb
@@ -174,6 +175,7 @@ test_files:
 - spec/lib/integration/integration_spec.rb
 - spec/lib/ldp/client_spec.rb
 - spec/lib/ldp/orm/orm_spec.rb
+- spec/lib/ldp/resource/binary_source_spec.rb
 - spec/lib/ldp/resource/rdf_source_spec.rb
 - spec/lib/ldp/resource_spec.rb
 - spec/lib/ldp/response_spec.rb