Merge pull request #288 from fastlane/add-release-links

Print out release links

Author: rebehe

lib/fastlane/plugin/firebase_app_distribution/actions/firebase_app_distribution_action.rb

@@ -46,7 +46,8 @@ def self.run(params)
 
         upload_timeout = get_upload_timeout(params)
 
-        release_name = fad_api_client.upload(app_name, binary_path, platform.to_s, upload_timeout)
+        upload_status_response = fad_api_client.upload(app_name, binary_path, platform.to_s, upload_timeout)
+        release_name = upload_status_response.release_name
 
         if binary_type == :AAB && aab_info && !aab_info.certs_provided?
           updated_aab_info = fad_api_client.get_aab_info(app_name)
@@ -69,6 +70,18 @@ def self.run(params)
         group_aliases = string_to_array(groups)
         fad_api_client.distribute(release_name, emails, group_aliases)
         UI.success("🎉 App Distribution upload finished successfully.")
+
+        if upload_status_response.firebase_console_uri
+          UI.message("🔗 View this release in the Firebase console: #{upload_status_response.firebase_console_uri}")
+        end
+
+        if upload_status_response.testing_uri
+          UI.message("🔗 Share this release with testers: #{upload_status_response.testing_uri}")
+        end
+
+        if upload_status_response.binary_download_uri
+          UI.message("🔗 Download the release binary (link expires in 1 hour): #{upload_status_response.binary_download_uri}")
+        end
       end
 
       def self.description

lib/fastlane/plugin/firebase_app_distribution/client/firebase_app_distribution_api_client.rb

@@ -171,7 +171,7 @@ def upload(app_name, binary_path, platform, timeout)
           UI.crash!("It took longer than expected to process your #{binary_type}, please try again.")
         end
 
-        upload_status_response.release_name
+        upload_status_response
       end
 
       # Fetches the status of an uploaded binary

lib/fastlane/plugin/firebase_app_distribution/helper/upload_status_response.rb

@@ -31,6 +31,18 @@ def release_version
     end
   end
 
+  def firebase_console_uri
+    release ? release[:firebaseConsoleUri] : nil
+  end
+
+  def testing_uri
+    release ? release[:testingUri] : nil
+  end
+
+  def binary_download_uri
+    release ? release[:binaryDownloadUri] : nil
+  end
+
   def status
     response ? response[:result] : nil
   end

spec/firebase_app_distribution_api_client_spec.rb

@@ -184,7 +184,7 @@
       stub_const("Fastlane::Client::FirebaseAppDistributionApiClient::POLLING_INTERVAL_SECONDS", 0)
     end
 
-    it 'uploads the app binary then returns the release name' do
+    it 'uploads the app binary then returns the upload response' do
       expect(api_client).to receive(:upload_binary)
         .with(app_name, fake_binary_path, "android", upload_timeout)
         .and_return(operation_name)
@@ -199,7 +199,7 @@
         .at_most(:once)
 
       result = api_client.upload(app_name, fake_binary_path, "android", upload_timeout)
-      expect(result).to eq(release_name)
+      expect(result).to eq(upload_status_response_success)
     end
 
     it 'uploads the app binary for an existing unmodified binary' do
@@ -217,7 +217,7 @@
         .at_most(:once)
 
       result = api_client.upload(app_name, fake_binary_path, "android", upload_timeout)
-      expect(result).to eq(release_name)
+      expect(result).to eq(upload_status_response_release_unmodified)
     end
 
     it 'uploads the app binary for an existing updated binary' do
@@ -235,7 +235,7 @@
         .at_most(:once)
 
       result = api_client.upload(app_name, fake_binary_path, "android", upload_timeout)
-      expect(result).to eq(release_name)
+      expect(result).to eq(upload_status_response_release_updated)
     end
 
     it 'raises an error after polling MAX_POLLING_RETRIES times' do
@@ -273,7 +273,7 @@
         .and_return(upload_status_response_success)
 
       result = api_client.upload(app_name, fake_binary_path, "android", upload_timeout)
-      expect(result).to eq(release_name)
+      expect(result).to eq(upload_status_response_success)
     end
 
     it 'crashes after failing to upload with status error' do