Handle multiple file params (#25)

Author: laurynas-convertapi

lib/convert_api/task.rb

@@ -33,10 +33,10 @@ def normalize_params(params)
       result = {}
 
       symbolize_keys(params).each do |key, value|
-        case key
-        when :File
-          result[:File] = FileParam.build(value)
-        when :Files
+        case
+        when key != :StoreFile && key.to_s.end_with?('File')
+          result[key] = FileParam.build(value)
+        when key == :Files
           result[:Files] = files_batch(value)
         else
           result[key] = value

spec/convert_api/task_spec.rb

@@ -27,4 +27,31 @@
       expect(subject).to be_instance_of(ConvertApi::Result)
     end
   end
+
+  context 'when file is instance of ResultFile' do
+    let(:file) { ConvertApi::ResultFile.new('Url' => 'testurl') }
+    let(:expected_params) { hash_including(File: 'testurl') }
+
+    it 'uses file url' do
+      expect(ConvertApi.client).to(
+        receive(:post).with('convert/txt/to/pdf', expected_params, instance_of(Hash)).and_return(result)
+      )
+
+      expect(subject).to be_instance_of(ConvertApi::Result)
+    end
+  end
+
+  context 'when multiple file params' do
+    let(:file) { ConvertApi::ResultFile.new('Url' => 'testurl') }
+    let(:params) { { File: file, CompareFile: file } }
+    let(:expected_params) { hash_including(File: 'testurl', CompareFile: 'testurl') }
+
+    it 'uses multiple file urls' do
+      expect(ConvertApi.client).to(
+        receive(:post).with('convert/txt/to/pdf', expected_params, instance_of(Hash)).and_return(result)
+      )
+
+      expect(subject).to be_instance_of(ConvertApi::Result)
+    end
+  end
 end