Provide the URI as part of the error message when object recreation i…

…s attempted
samvera · Jan 6, 2015 · 7e8dd02 · 7e8dd02
1 parent 63e9aed
commit 7e8dd02
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/active_fedora/core.rb b/lib/active_fedora/core.rb
@@ -36,7 +36,7 @@ def ldp_source
     def initialize(attributes_or_resource_or_url = nil, &block)
       init_internals
       attributes = initialize_resource_and_attributes(attributes_or_resource_or_url)
-      raise IllegalOperation, "Attempting to recreate existing ldp_source" unless ldp_source.new?
+      raise IllegalOperation, "Attempting to recreate existing ldp_source: `#{ldp_source.subject}'" unless ldp_source.new?
       assert_content_model
       load_attached_files
       self.attributes = attributes if attributes

diff --git a/spec/integration/base_spec.rb b/spec/integration/base_spec.rb
@@ -46,7 +46,7 @@ class MockAFBaseRelationship < ActiveFedora::Base
     end
     describe "when trying to create it again" do
       it "should raise an error" do
-        expect { MockAFBaseRelationship.create(id: @release.id) }.to raise_error(ActiveFedora::IllegalOperation)
+        expect { MockAFBaseRelationship.create(id: @release.id) }.to raise_error(ActiveFedora::IllegalOperation, "Attempting to recreate existing ldp_source: `#{@release.uri}'")
         @release.reload
         expect(@release.foo.person).to include('test foo content')
       end