build_manager.rb in fastlane-2.19.0.beta.20170223010028

- old
+ new

@@ -29,11 +29,12 @@
         UI.important("This means that no changelog will be set and no build will be distributed to testers")
         return
       end
 
       UI.message("If you want to skip waiting for the processing to be finished, use the `skip_waiting_for_build_processing` option")
-      uploaded_build = FastlaneCore::BuildWatcher.wait_for_build(app, platform, config[:wait_processing_interval].to_i)
+      uploaded_build = wait_for_processing_build(options, platform) # this might take a while
+
       distribute(options, uploaded_build)
     end
 
     def distribute(options, build = nil)
       start(options)
@@ -119,9 +120,74 @@
       return row
     end
 
     def should_update_build_information(options)
       options[:changelog].to_s.length > 0 or options[:beta_app_description].to_s.length > 0 or options[:beta_app_feedback_email].to_s.length > 0
+    end
+
+    # This method will takes care of checking for the processing builds every few seconds
+    # @return [Build] The build that we just uploaded
+    def wait_for_processing_build(options, platform)
+      # the upload date of the new buid
+      # we use it to identify the build
+      start = Time.now
+      wait_processing_interval = config[:wait_processing_interval].to_i
+      latest_build = nil
+      UI.message("Waiting for iTunes Connect to process the new build")
+      must_update_build_info = config[:update_build_info_on_upload]
+      loop do
+        sleep(wait_processing_interval)
+
+        # before we look for processing builds, we need to ensure that there
+        #  is a build train for this application; new applications don't
+        #  build trains right away, and if we don't do this check, we will
+        #  get break out of this loop and then generate an error later when we
+        #  have a nil build
+        if app.build_trains(platform: platform).count == 0
+          UI.message("New application; waiting for build train to appear on iTunes Connect")
+        else
+          builds = app.all_processing_builds(platform: platform)
+          break if builds.count == 0
+          latest_build = builds.last
+
+          if latest_build.valid and must_update_build_info
+            # Set the changelog and/or description if necessary
+            if should_update_build_information(options)
+              latest_build.update_build_information!(whats_new: options[:changelog], description: options[:beta_app_description], feedback_email: options[:beta_app_feedback_email])
+              UI.success "Successfully set the changelog and/or description for build"
+            end
+            must_update_build_info = false
+          end
+
+          UI.message("Waiting for iTunes Connect to finish processing the new build (#{latest_build.train_version} - #{latest_build.build_version})")
+        end
+      end
+
+      UI.user_error!("Error receiving the newly uploaded binary, please check iTunes Connect") if latest_build.nil?
+      full_build = nil
+
+      while full_build.nil? || full_build.processing
+        # The build's processing state should go from true to false, and be done. But sometimes it goes true -> false ->
+        # true -> false, where the second true is transient. This causes a spurious failure. Find build by build_version
+        # and ensure it's not processing before proceeding - it had to have already been false before, to get out of the
+        # previous loop.
+        full_build = app.build_trains(platform: platform)[latest_build.train_version].builds.find do |b|
+          b.build_version == latest_build.build_version
+        end
+
+        UI.message("Waiting for iTunes Connect to finish processing the new build (#{latest_build.train_version} - #{latest_build.build_version})")
+        sleep(wait_processing_interval)
+      end
+
+      if full_build && !full_build.processing && full_build.valid
+        minutes = ((Time.now - start) / 60).round
+        UI.success("Successfully finished processing the build")
+        UI.message("You can now tweet: ")
+        UI.important("iTunes Connect #iosprocessingtime #{minutes} minutes")
+        return full_build
+      else
+        UI.user_error!("Error: Seems like iTunes Connect didn't properly pre-process the binary")
+      end
     end
 
     def distribute_build(uploaded_build, options)
       UI.message("Distributing new build to testers")
 