Don't reload on initialize. Fixes #87

lib/active_triples/resource.rb

@@ -50,7 +50,9 @@ def type_registry
       #
       # @return [ActiveTriples::Resource] a Resource with
       def from_uri(uri,vals=nil)
-        new(uri, vals)
+        new(uri, vals).tap do |o|
+          o.reload
+        end
       end
 
       def property(*)
@@ -83,8 +85,8 @@ def initialize(*args, &block)
       self.parent = args.shift unless args.first.is_a?(Hash)
       set_subject!(resource_uri) if resource_uri
       super(*args, &block)
-      reload
       # Append type to graph if necessary.
+      @loaded = false
       self.get_values(:type) << self.class.type if self.class.type.kind_of?(RDF::URI) && type.empty?
     end
 
@@ -265,6 +267,7 @@ def persisted?
     #
     # @return [true, false]
     def reload
+      @loaded = true
       @term_cache ||= {}
       return false unless repository
       self << repository.query(subject: rdf_subject)
@@ -286,6 +289,7 @@ def reload
     #
     # @note This method will delete existing statements with the correct subject and predicate from the graph
     def set_value(*args)
+      reload unless loaded?
       # Add support for legacy 3-parameter syntax
       if args.length > 3 || args.length < 2
         raise ArgumentError, "wrong number of arguments (#{args.length} for 2-3)"
@@ -294,6 +298,10 @@ def set_value(*args)
       get_term(args).set(values)
     end
 
+    def loaded?
+      @loaded
+    end
+
     ##
     # Adds or updates a property with supplied values.
     #
@@ -327,6 +335,7 @@ def [](uri_or_term_property)
 
 
     def get_term(args)
+      reload unless loaded?
       @term_cache ||= {}
       term = Term.new(self, args)
       @term_cache["#{term.send(:rdf_subject)}/#{term.property}/#{term.term_args}"] ||= term

spec/active_triples/resource_spec.rb

@@ -292,7 +292,7 @@ class DummyResourceWithBaseURI < ActiveTriples::Resource
 
     it "should warn when the repo doesn't exist" do
       allow(DummyLicense).to receive(:repository).and_return('repo2')
-      expect { subject }.to raise_error ActiveTriples::RepositoryNotFoundError, 'The class DummyLicense expects a repository called repo2, but none was declared'
+      expect { subject.title }.to raise_error ActiveTriples::RepositoryNotFoundError, 'The class DummyLicense expects a repository called repo2, but none was declared'
     end
   end