Allow upload_to_s3 not to crash if file exists

Author: AliSoftware

lib/fastlane/plugin/wpmreleasetoolkit/actions/common/upload_to_s3.rb

@@ -20,7 +20,15 @@ def self.run(params)
           key = [file_name_hash, key].join('/')
         end
 
-        UI.user_error!("File already exists in S3 bucket #{bucket} at #{key}") if file_is_already_uploaded?(bucket, key)
+        if file_is_already_uploaded?(bucket, key)
+          message = "File already exists in S3 bucket #{bucket} at #{key}"
+          if params[:skip_if_exists]
+            UI.important("#{message}. Skipping upload.")
+          else
+            UI.user_error!(message)
+          end
+          return
+        end
 
         UI.message("Uploading #{file_path} to: #{key}")
 
@@ -101,6 +109,13 @@ def self.available_options
             default_value: true,
             type: Boolean
           ),
+          FastlaneCore::ConfigItem.new(
+            key: :skip_if_exists,
+            description: 'If the file already exists in the S3 bucket, just log the error instead of failing with `user_error!`',
+            optional: true,
+            default_value: false,
+            type: Boolean
+          ),
         ]
       end
 

spec/upload_to_s3_spec.rb

@@ -144,7 +144,7 @@ def stub_s3_response_for_file(key, exists: true)
       end.to raise_error(FastlaneCore::Interface::FastlaneError, 'Path `this-file-does-not-exist.txt` does not exist.')
     end
 
-    it 'fails if the file already exists on S3' do
+    it 'fails if the file already exists on S3 and skip_if_exists:false' do
       expected_key = 'a62f2225bf70bfaccbc7f1ef2a397836717377de/key'
       stub_s3_response_for_file(expected_key)
 
@@ -158,5 +158,23 @@ def stub_s3_response_for_file(key, exists: true)
         end.to raise_error(FastlaneCore::Interface::FastlaneError, "File already exists in S3 bucket #{test_bucket} at #{expected_key}")
       end
     end
+
+    it 'just logs if the file already exists on S3 and skip_if_exists:true' do
+      expected_key = 'a62f2225bf70bfaccbc7f1ef2a397836717377de/key'
+      stub_s3_response_for_file(expected_key)
+
+      warnings = []
+      allow(FastlaneCore::UI).to receive(:important) { |message| warnings << message }
+
+      with_tmp_file(named: 'key') do |file_path|
+        run_described_fastlane_action(
+          bucket: test_bucket,
+          key: 'key',
+          file: file_path,
+          skip_if_exists: true
+        )
+        expect(warnings).to eq ["File already exists in S3 bucket #{test_bucket} at #{expected_key}. Skipping upload."]
+      end
+    end
   end
 end