Still return the key if file already existed and we skipped the upload

Author: AliSoftware

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

@@ -27,7 +27,7 @@ def self.run(params)
           else
             UI.user_error!(message)
           end
-          return
+          return key
         end
 
         UI.message("Uploading #{file_path} to: #{key}")

spec/upload_to_s3_spec.rb

@@ -167,13 +167,14 @@ def stub_s3_response_for_file(key, exists: true)
       allow(FastlaneCore::UI).to receive(:important) { |message| warnings << message }
 
       with_tmp_file(named: 'key') do |file_path|
-        run_described_fastlane_action(
+        key = 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."]
+        expect(warnings).to eq(["File already exists in S3 bucket #{test_bucket} at #{expected_key}. Skipping upload."])
+        expect(key).to eq(expected_key)
       end
     end
   end