Merge pull request #1091 from projecthydra/save_metadata

AF::File#save! should persist metadata

lib/active_fedora/file.rb

@@ -48,12 +48,6 @@ def initialize(identifier = nil, &_block)
       end
     end
 
-    def save
-      super.tap do
-        metadata.save if metadata.changed?
-      end
-    end
-
     def described_by
       raise "#{self} isn't persisted yet" if new_record?
       links['describedby'].first
@@ -176,6 +170,12 @@ def prefix(path)
 
     private
 
+      def create_or_update(*options)
+        super.tap do
+          metadata.save if metadata.changed?
+        end
+      end
+
       # Rack::Test::UploadedFile is often set via content=, however it's not an IO, though it wraps an io object.
       def behaves_like_io?(obj)
         [IO, Tempfile, StringIO].any? { |klass| obj.is_a? klass } || (defined?(Rack) && obj.is_a?(Rack::Test::UploadedFile))

spec/unit/file_spec.rb

@@ -7,6 +7,21 @@
 
   it { is_expected.not_to be_metadata }
 
+  describe "#save!" do
+    context "when updating metadata" do
+      before do
+        file.content = 'foo'
+        file.save!
+        file.mime_type = 'application/pdf'
+      end
+
+      it "Updates metadata" do
+        expect(file.metadata).to receive(:save)
+        file.save!
+      end
+    end
+  end
+
   describe "#behaves_like_io?" do
     subject { file.send(:behaves_like_io?, object) }