UploadedFile#close should leave UploadedFile in a state where #opened…

…? is false (#656)

This means removing the reference to the now-closed (and unusable) IO object. Which meant refactoring an existing test slightly, as it tried to refer to the IO object after UploadedFile#close.

lib/shrine/uploaded_file.rb

@@ -170,6 +170,7 @@ def rewind
       # opened IO object.
       def close
         io.close if opened?
+        @io = nil
       end
 
       # Returns whether the file has already been opened.

test/plugin/rack_response_test.rb

@@ -106,8 +106,11 @@
     uploaded_file = @uploader.upload(fakeio)
     uploaded_file.open
     response = uploaded_file.to_rack_response
+
+    existing_io = uploaded_file.to_io
+
     response[2].close
-    assert uploaded_file.to_io.closed?
+    assert existing_io.closed?
   end
 
   it "returns ranged responses when :range is given" do

test/uploaded_file_test.rb

@@ -241,6 +241,14 @@ def uploaded_file(data = {})
       uploaded_file.storage.expects(:open).never
       uploaded_file.close
     end
+
+    it "leaves the UploadedFile no longer #opened?" do
+      uploaded_file = @uploader.upload(fakeio)
+      uploaded_file.open
+      uploaded_file.close
+
+      refute uploaded_file.opened?
+    end
   end
 
   describe "#url" do