Fix Doc2Text::Odt::Document.parse_and_save when using tmp files as input

lib/doc2text/xml_based_document_file.rb

@@ -10,11 +10,10 @@ def initialize(document_path)
       def unpack
         Zip::File.open(@document_path) {
             |zip_file|
-          Dir.mkdir(extract_path)
           zip_file.each do |entry|
             zipped_file_extract_path = File.join extract_path, entry.name
             FileUtils.mkdir_p File.dirname(zipped_file_extract_path)
-            zip_file.extract entry, zipped_file_extract_path
+            zip_file.extract entry, zipped_file_extract_path, destination_directory: "/"
           end
         }
       end
@@ -41,7 +40,7 @@ def extract_extension
       end
 
       def extract_path
-        File.join File.dirname(@document_path), ".#{File.basename(@document_path)}_#{extract_extension}"
+        @extract_path ||= Dir.mktmpdir(".#{File.basename(@document_path)}_#{extract_extension}")
       end
     end
   end

spec/unpack_odt_spec.rb

@@ -17,4 +17,23 @@ def rspec_extract_odt
     rspec_extract_odt
     rspec_extract_odt
   end
+
+  context "when the odt is a temporary file" do
+    it "runs from a temp file" do
+      tempfile = Tempfile.new('text_styles.odt')
+      tempfile.write File.read(File.join 'spec', 'fixtures', 'text_styles.odt')
+      tempfile.rewind
+      tempfile.close
+
+      @odt = Doc2Text::Odt::Document.new tempfile
+      @odt.unpack
+
+      entries = Dir.glob "#{@odt.extract_path}/**/*"
+      mandatory_files = %w(manifest.rdf content.xml settings.xml styles.xml META-INF META-INF/manifest.xml meta.xml mimetype).map { |entry|
+        File.join @odt.extract_path, entry }
+      expect(entries.to_set.subset? mandatory_files.to_set)
+
+      @odt.clean
+    end
+  end
 end