Merge pull request #186 from javierav/support-active-storage

Fix #144. Add support for Tempfile

Author: jamesmartin

CHANGELOG.md

@@ -9,6 +9,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - Freeze string literals. [#172](https://github.com/jamesmartin/inline_svg/pull/172). Thanks, [@tagliala](https://github.com/tagliala)
 - Fix thread-local variable leakage in `with_asset_finder`. [#185](https://github.com/jamesmartin/inline_svg/pull/185). Thanks, [@tagliala](https://github.com/tagliala)
 - Remove unused `InlineSvg::IOResource.default_for` method. [#187](https://github.com/jamesmartin/inline_svg/pull/187). Thanks, [@tagliala](https://github.com/tagliala)
+- Add support for Tempfile. [#186](https://github.com/jamesmartin/inline_svg/pull/186). Thanks, [@javierav](https://github.com/javierav)
 
 ## [1.10.0] - 2024-09-03
 ### Added

README.md

@@ -315,6 +315,12 @@ InlineSvg.configure do |config|
 end
 ```
 
+## ActiveStorage
+
+```erb
+<%= user.avatar.open { |file| inline_svg_tag file } %>
+```
+
 ## Contributing
 
 1. Fork it ( [http://github.com/jamesmartin/inline_svg/fork](http://github.com/jamesmartin/inline_svg/fork) )

lib/inline_svg/io_resource.rb

@@ -3,7 +3,7 @@
 module InlineSvg
   module IOResource
     def self.===(object)
-      object.is_a?(IO) || object.is_a?(StringIO)
+      object.is_a?(IO) || object.is_a?(StringIO) || object.is_a?(Tempfile)
     end
 
     def self.read(object)

spec/io_resource_spec.rb

@@ -24,6 +24,10 @@
       it "for File object" do
         expect(subject === File.new("#{Dir.tmpdir}/testfile", "w")).to be true
       end
+
+      it "for Tempfile object" do
+        expect(subject === Tempfile.new).to be true
+      end
     end
 
     context 'return false' do
@@ -78,5 +82,18 @@
         expect(answer).not_to eq ''
       end
     end
+
+    context 'Tempfile object' do
+      let(:answer) { 'read' }
+      let(:rio) do
+        Tempfile.new.tap do |f|
+          f.write(answer)
+          f.rewind
+        end
+      end
+      let(:wio) { File.new(File::NULL, 'w') } # Tempfile cannot be created for write only mode
+
+      instance_exec(&tests)
+    end
   end
 end