Use double splat to pass opts to File.open (#27)

Author: DanOlson

.gitignore

@@ -5,6 +5,7 @@
 /doc/
 /pkg/
 /spec/reports/
+/spec/protocol/http/body/reader_spec.txt
 /tmp/
 /external/
 

lib/protocol/http/body/reader.rb

@@ -57,9 +57,9 @@ def finish
 				end
 
 				# Write the body of the response to the given file path.
-				def save(path, mode = ::File::WRONLY|::File::CREAT, *args)
+				def save(path, mode = ::File::WRONLY|::File::CREAT, **options)
 					if @body
-						::File.open(path, mode, *args) do |file|
+						::File.open(path, mode, **options) do |file|
 							self.each do |chunk|
 								file.write(chunk)
 							end

spec/protocol/http/body/reader_spec.rb

@@ -0,0 +1,37 @@
+require 'protocol/http/body/reader'
+
+RSpec.describe Protocol::HTTP::Body::Reader do
+	class TestReader
+		include Protocol::HTTP::Body::Reader
+
+		def initialize(body)
+			@body = body
+		end
+	end
+
+	subject do
+		TestReader.new(%w(the quick brown fox))
+	end
+
+	describe '#save' do
+		let(:path) { File.expand_path('reader_spec.txt', __dir__) }
+
+		before do
+			File.open(path, 'w') {}
+		end
+
+		it 'saves to the provided filename' do
+			expect { subject.save(path) }
+				.to change { File.read(path) }
+				.from('')
+				.to('thequickbrownfox')
+		end
+
+		it 'mirrors the interface of File.open' do
+			expect { subject.save(path, nil, mode: 'w') }
+				.to change { File.read(path) }
+				.from('')
+				.to('thequickbrownfox')
+		end
+	end
+end