fastlane 2.74.0.beta.20180106010004 → 2.74.0.beta.20180107010004

data/spaceship/lib/spaceship/test_flight/build_trains.rb

@@ -1,3 +1,6 @@
+require_relative 'base'
+require_relative 'build'
+
 module Spaceship::TestFlight
   class BuildTrains < Base
     ##
@@ -15,7 +18,7 @@ module Spaceship::TestFlight
 
       data.each do |train_version|
         builds_data = client.get_builds_for_train(app_id: app_id, platform: platform, train_version: train_version, retry_count: retry_count)
-        trains[train_version] = builds_data.map { |attrs| Build.new(attrs) }
+        trains[train_version] = builds_data.map { |attrs| Spaceship::TestFlight::Build.new(attrs) }
       end
 
       self.new(trains)

data/spaceship/lib/spaceship/test_flight/client.rb

@@ -1,323 +1,327 @@
-module Spaceship::TestFlight
-  class Client < Spaceship::Client
-    ##
-    # Spaceship HTTP client for the testflight API.
-    #
-    # This client is solely responsible for the making HTTP requests and
-    # parsing their responses. Parameters should be either named parameters, or
-    # for large request data bodies, pass in anything that can resond to
-    # `to_json`.
-    #
-    # Each request method should validate the required parameters. A required parameter is one that would result in 400-range response if it is not supplied.
-    # Each request method should make only one request. For more high-level logic, put code in the data models.
-
-    def self.hostname
-      'https://itunesconnect.apple.com/testflight/v2/'
-    end
+require_relative '../client'
+
+module Spaceship
+  module TestFlight
+    class Client < Spaceship::Client
+      ##
+      # Spaceship HTTP client for the testflight API.
+      #
+      # This client is solely responsible for the making HTTP requests and
+      # parsing their responses. Parameters should be either named parameters, or
+      # for large request data bodies, pass in anything that can resond to
+      # `to_json`.
+      #
+      # Each request method should validate the required parameters. A required parameter is one that would result in 400-range response if it is not supplied.
+      # Each request method should make only one request. For more high-level logic, put code in the data models.
+
+      def self.hostname
+        'https://itunesconnect.apple.com/testflight/v2/'
+      end
 
-    ##
-    # @!group Build trains API
-    ##
+      ##
+      # @!group Build trains API
+      ##
 
-    # Returns an array of all available build trains (not the builds they include)
-    def get_build_trains(app_id: nil, platform: "ios")
-      assert_required_params(__method__, binding)
+      # Returns an array of all available build trains (not the builds they include)
+      def get_build_trains(app_id: nil, platform: "ios")
+        assert_required_params(__method__, binding)
 
-      response = request(:get, "providers/#{team_id}/apps/#{app_id}/platforms/#{platform}/trains")
+        response = request(:get, "providers/#{team_id}/apps/#{app_id}/platforms/#{platform}/trains")
 
-      handle_response(response)
-    end
-
-    def get_builds_for_train(app_id: nil, platform: "ios", train_version: nil, retry_count: 3)
-      assert_required_params(__method__, binding)
-      with_retry(retry_count) do
-        response = request(:get, "providers/#{team_id}/apps/#{app_id}/platforms/#{platform}/trains/#{train_version}/builds", nil, {}, true)
         handle_response(response)
       end
-    end
 
-    ##
-    # @!group Builds API
-    ##
+      def get_builds_for_train(app_id: nil, platform: "ios", train_version: nil, retry_count: 3)
+        assert_required_params(__method__, binding)
+        with_retry(retry_count) do
+          response = request(:get, "providers/#{team_id}/apps/#{app_id}/platforms/#{platform}/trains/#{train_version}/builds", nil, {}, true)
+          handle_response(response)
+        end
+      end
 
-    def get_build(app_id: nil, build_id: nil)
-      assert_required_params(__method__, binding)
+      ##
+      # @!group Builds API
+      ##
 
-      response = request(:get, "providers/#{team_id}/apps/#{app_id}/builds/#{build_id}")
-      handle_response(response)
-    end
+      def get_build(app_id: nil, build_id: nil)
+        assert_required_params(__method__, binding)
 
-    def put_build(app_id: nil, build_id: nil, build: nil)
-      assert_required_params(__method__, binding)
-
-      response = request(:put) do |req|
-        req.url "providers/#{team_id}/apps/#{app_id}/builds/#{build_id}"
-        req.body = build.to_json
-        req.headers['Content-Type'] = 'application/json'
+        response = request(:get, "providers/#{team_id}/apps/#{app_id}/builds/#{build_id}")
+        handle_response(response)
       end
-      handle_response(response)
-    end
 
-    def post_for_testflight_review(app_id: nil, build_id: nil, build: nil)
-      assert_required_params(__method__, binding)
+      def put_build(app_id: nil, build_id: nil, build: nil)
+        assert_required_params(__method__, binding)
 
-      response = request(:post) do |req|
-        req.url "providers/#{team_id}/apps/#{app_id}/builds/#{build_id}/review"
-        req.body = build.to_json
-        req.headers['Content-Type'] = 'application/json'
+        response = request(:put) do |req|
+          req.url "providers/#{team_id}/apps/#{app_id}/builds/#{build_id}"
+          req.body = build.to_json
+          req.headers['Content-Type'] = 'application/json'
+        end
+        handle_response(response)
       end
-      handle_response(response)
-    end
 
-    def expire_build(app_id: nil, build_id: nil, build: nil)
-      assert_required_params(__method__, binding)
+      def post_for_testflight_review(app_id: nil, build_id: nil, build: nil)
+        assert_required_params(__method__, binding)
 
-      response = request(:post) do |req|
-        req.url "providers/#{team_id}/apps/#{app_id}/builds/#{build_id}/expire"
-        req.body = build.to_json
-        req.headers['Content-Type'] = 'application/json'
+        response = request(:post) do |req|
+          req.url "providers/#{team_id}/apps/#{app_id}/builds/#{build_id}/review"
+          req.body = build.to_json
+          req.headers['Content-Type'] = 'application/json'
+        end
+        handle_response(response)
       end
-      handle_response(response)
-    end
 
-    ##
-    # @!group Groups API
-    ##
+      def expire_build(app_id: nil, build_id: nil, build: nil)
+        assert_required_params(__method__, binding)
+
+        response = request(:post) do |req|
+          req.url "providers/#{team_id}/apps/#{app_id}/builds/#{build_id}/expire"
+          req.body = build.to_json
+          req.headers['Content-Type'] = 'application/json'
+        end
+        handle_response(response)
+      end
 
-    # Returns a list of available testing groups
-    # e.g.
-    #   {"b6f65dbd-c845-4d91-bc39-0b661d608970" => "Boarding",
-    #    "70402368-9deb-409f-9a26-bb3f215dfee3" => "Automatic"}
-    def get_groups(app_id: nil)
-      return @cached_groups if @cached_groups
-      assert_required_params(__method__, binding)
+      ##
+      # @!group Groups API
+      ##
 
-      response = request(:get, "/testflight/v2/providers/#{provider_id}/apps/#{app_id}/groups")
-      @cached_groups = handle_response(response)
-    end
+      # Returns a list of available testing groups
+      # e.g.
+      #   {"b6f65dbd-c845-4d91-bc39-0b661d608970" => "Boarding",
+      #    "70402368-9deb-409f-9a26-bb3f215dfee3" => "Automatic"}
+      def get_groups(app_id: nil)
+        return @cached_groups if @cached_groups
+        assert_required_params(__method__, binding)
 
-    def add_group_to_build(app_id: nil, group_id: nil, build_id: nil)
-      assert_required_params(__method__, binding)
-
-      body = {
-        'groupId' => group_id,
-        'buildId' => build_id
-      }
-      response = request(:put) do |req|
-        req.url "providers/#{team_id}/apps/#{app_id}/groups/#{group_id}/builds/#{build_id}"
-        req.body = body.to_json
-        req.headers['Content-Type'] = 'application/json'
+        response = request(:get, "/testflight/v2/providers/#{provider_id}/apps/#{app_id}/groups")
+        @cached_groups = handle_response(response)
       end
-      handle_response(response)
-    end
 
-    def create_group_for_app(app_id: nil, group_name: nil)
-      assert_required_params(__method__, binding)
-      body = {
-          'name' => group_name
-      }
+      def create_group_for_app(app_id: nil, group_name: nil)
+        assert_required_params(__method__, binding)
+        body = {
+            'name' => group_name
+        }
 
-      response = request(:post) do |req|
-        req.url "providers/#{team_id}/apps/#{app_id}/groups"
-        req.body = body.to_json
-        req.headers['Content-Type'] = 'application/json'
+        response = request(:post) do |req|
+          req.url "providers/#{team_id}/apps/#{app_id}/groups"
+          req.body = body.to_json
+          req.headers['Content-Type'] = 'application/json'
+        end
+        handle_response(response)
       end
-      handle_response(response)
-    end
 
-    #####################################################
-    # @!group Testers
-    #####################################################
-    def testers(tester)
-      url = tester.url[:index]
-      r = request(:get, url)
-      parse_response(r, 'data')['users']
-    end
-
-    def testers_by_app(tester, app_id, group_id: nil)
-      if group_id.nil?
-        group_ids = get_groups(app_id: app_id).map do |group|
-          group['id']
+      def add_group_to_build(app_id: nil, group_id: nil, build_id: nil)
+        assert_required_params(__method__, binding)
+
+        body = {
+          'groupId' => group_id,
+          'buildId' => build_id
+        }
+        response = request(:put) do |req|
+          req.url "providers/#{team_id}/apps/#{app_id}/groups/#{group_id}/builds/#{build_id}"
+          req.body = body.to_json
+          req.headers['Content-Type'] = 'application/json'
         end
+        handle_response(response)
       end
-      group_ids ||= [group_id]
-      testers = []
 
-      group_ids.each do |json_group_id|
-        url = tester.url(app_id, provider_id, json_group_id)[:index_by_app]
+      #####################################################
+      # @!group Testers
+      #####################################################
+      def testers(tester)
+        url = tester.url[:index]
         r = request(:get, url)
-        testers += parse_response(r, 'data')['users']
+        parse_response(r, 'data')['users']
       end
 
-      testers
-    end
+      def testers_by_app(tester, app_id, group_id: nil)
+        if group_id.nil?
+          group_ids = get_groups(app_id: app_id).map do |group|
+            group['id']
+          end
+        end
+        group_ids ||= [group_id]
+        testers = []
 
-    ##
-    # @!group Testers API
-    ##
-
-    def testers_for_app(app_id: nil)
-      assert_required_params(__method__, binding)
-      page_size = 40 # that's enforced by the iTC servers
-      resulting_array = []
-      initial_url = "providers/#{team_id}/apps/#{app_id}/testers?limit=#{page_size}&sort=email&order=asc"
-      response = request(:get, initial_url)
-      link_from_response = proc do |r|
-        # I weep for Swift nil chaining
-        (l = r.headers['link']) && (m = l.match(/<(.*)>/)) && m.captures.first
+        group_ids.each do |json_group_id|
+          url = tester.url(app_id, provider_id, json_group_id)[:index_by_app]
+          r = request(:get, url)
+          testers += parse_response(r, 'data')['users']
+        end
+
+        testers
       end
-      next_link = link_from_response.call(response)
-      result = Array(handle_response(response))
-      resulting_array += result
-      return resulting_array if result.count == 0
 
-      until next_link.nil?
-        response = request(:get, next_link)
-        result = Array(handle_response(response))
+      ##
+      # @!group Testers API
+      ##
+
+      def testers_for_app(app_id: nil)
+        assert_required_params(__method__, binding)
+        page_size = 40 # that's enforced by the iTC servers
+        resulting_array = []
+        initial_url = "providers/#{team_id}/apps/#{app_id}/testers?limit=#{page_size}&sort=email&order=asc"
+        response = request(:get, initial_url)
+        link_from_response = proc do |r|
+          # I weep for Swift nil chaining
+          (l = r.headers['link']) && (m = l.match(/<(.*)>/)) && m.captures.first
+        end
         next_link = link_from_response.call(response)
-
-        break if result.count == 0
-
+        result = Array(handle_response(response))
         resulting_array += result
-      end
-      return resulting_array.uniq
-    end
+        return resulting_array if result.count == 0
 
-    def delete_tester_from_app(app_id: nil, tester_id: nil)
-      assert_required_params(__method__, binding)
-      url = "providers/#{team_id}/apps/#{app_id}/testers/#{tester_id}"
-      response = request(:delete, url)
-      handle_response(response)
-    end
+        until next_link.nil?
+          response = request(:get, next_link)
+          result = Array(handle_response(response))
+          next_link = link_from_response.call(response)
 
-    def search_for_tester_in_app(app_id: nil, text: nil)
-      assert_required_params(__method__, binding)
-      text = CGI.escape(text)
-      url = "providers/#{team_id}/apps/#{app_id}/testers?order=asc&search=#{text}&sort=status"
-      response = request(:get, url)
-      handle_response(response)
-    end
+          break if result.count == 0
 
-    def resend_invite_to_external_tester(app_id: nil, tester_id: nil)
-      assert_required_params(__method__, binding)
-      url = "/testflight/v1/invites/#{app_id}/resend?testerId=#{tester_id}"
-      response = request(:post, url)
-      handle_response(response)
-    end
+          resulting_array += result
+        end
+        return resulting_array.uniq
+      end
 
-    def create_app_level_tester(app_id: nil, first_name: nil, last_name: nil, email: nil)
-      assert_required_params(__method__, binding)
-      url = "providers/#{team_id}/apps/#{app_id}/testers"
-      response = request(:post) do |req|
-        req.url url
-        req.body = {
-          "email" => email,
-          "firstName" => first_name,
-          "lastName" => last_name
-        }.to_json
-        req.headers['Content-Type'] = 'application/json'
+      def delete_tester_from_app(app_id: nil, tester_id: nil)
+        assert_required_params(__method__, binding)
+        url = "providers/#{team_id}/apps/#{app_id}/testers/#{tester_id}"
+        response = request(:delete, url)
+        handle_response(response)
       end
-      handle_response(response)
-    end
 
-    def post_tester_to_group(app_id: nil, email: nil, first_name: nil, last_name: nil, group_id: nil)
-      assert_required_params(__method__, binding)
-
-      # Then we can add the tester to the group that allows the app to test
-      # This is easy enough, we already have all this data. We don't need any response from the previous request
-      url = "providers/#{team_id}/apps/#{app_id}/groups/#{group_id}/testers"
-      response = request(:post) do |req|
-        req.url url
-        req.body = [{
-          "email" => email,
-          "firstName" => first_name,
-          "lastName" => last_name
-        }].to_json
-        req.headers['Content-Type'] = 'application/json'
+      def search_for_tester_in_app(app_id: nil, text: nil)
+        assert_required_params(__method__, binding)
+        text = CGI.escape(text)
+        url = "providers/#{team_id}/apps/#{app_id}/testers?order=asc&search=#{text}&sort=status"
+        response = request(:get, url)
+        handle_response(response)
       end
-      handle_response(response)
-    end
 
-    def delete_tester_from_group(group_id: nil, tester_id: nil, app_id: nil)
-      assert_required_params(__method__, binding)
+      def resend_invite_to_external_tester(app_id: nil, tester_id: nil)
+        assert_required_params(__method__, binding)
+        url = "/testflight/v1/invites/#{app_id}/resend?testerId=#{tester_id}"
+        response = request(:post, url)
+        handle_response(response)
+      end
 
-      url = "providers/#{team_id}/apps/#{app_id}/groups/#{group_id}/testers/#{tester_id}"
-      response = request(:delete) do |req|
-        req.url url
-        req.headers['Content-Type'] = 'application/json'
+      def create_app_level_tester(app_id: nil, first_name: nil, last_name: nil, email: nil)
+        assert_required_params(__method__, binding)
+        url = "providers/#{team_id}/apps/#{app_id}/testers"
+        response = request(:post) do |req|
+          req.url url
+          req.body = {
+            "email" => email,
+            "firstName" => first_name,
+            "lastName" => last_name
+          }.to_json
+          req.headers['Content-Type'] = 'application/json'
+        end
+        handle_response(response)
+      end
+
+      def post_tester_to_group(app_id: nil, email: nil, first_name: nil, last_name: nil, group_id: nil)
+        assert_required_params(__method__, binding)
+
+        # Then we can add the tester to the group that allows the app to test
+        # This is easy enough, we already have all this data. We don't need any response from the previous request
+        url = "providers/#{team_id}/apps/#{app_id}/groups/#{group_id}/testers"
+        response = request(:post) do |req|
+          req.url url
+          req.body = [{
+            "email" => email,
+            "firstName" => first_name,
+            "lastName" => last_name
+          }].to_json
+          req.headers['Content-Type'] = 'application/json'
+        end
+        handle_response(response)
       end
-      handle_response(response)
-    end
 
-    ##
-    # @!group AppTestInfo
-    ##
+      def delete_tester_from_group(group_id: nil, tester_id: nil, app_id: nil)
+        assert_required_params(__method__, binding)
 
-    def get_app_test_info(app_id: nil)
-      assert_required_params(__method__, binding)
+        url = "providers/#{team_id}/apps/#{app_id}/groups/#{group_id}/testers/#{tester_id}"
+        response = request(:delete) do |req|
+          req.url url
+          req.headers['Content-Type'] = 'application/json'
+        end
+        handle_response(response)
+      end
 
-      response = request(:get, "providers/#{team_id}/apps/#{app_id}/testInfo")
-      handle_response(response)
-    end
+      ##
+      # @!group AppTestInfo
+      ##
 
-    def put_app_test_info(app_id: nil, app_test_info: nil)
-      assert_required_params(__method__, binding)
+      def get_app_test_info(app_id: nil)
+        assert_required_params(__method__, binding)
 
-      response = request(:put) do |req|
-        req.url "providers/#{team_id}/apps/#{app_id}/testInfo"
-        req.body = app_test_info.to_json
-        req.headers['Content-Type'] = 'application/json'
+        response = request(:get, "providers/#{team_id}/apps/#{app_id}/testInfo")
+        handle_response(response)
       end
-      handle_response(response)
-    end
 
-    protected
+      def put_app_test_info(app_id: nil, app_test_info: nil)
+        assert_required_params(__method__, binding)
 
-    def handle_response(response)
-      if (200...300).cover?(response.status) && (response.body.nil? || response.body.empty?)
-        return
+        response = request(:put) do |req|
+          req.url "providers/#{team_id}/apps/#{app_id}/testInfo"
+          req.body = app_test_info.to_json
+          req.headers['Content-Type'] = 'application/json'
+        end
+        handle_response(response)
       end
 
-      raise InternalServerError, "Server error got #{response.status}" if (500...600).cover?(response.status)
+      protected
 
-      unless response.body.kind_of?(Hash)
-        raise UnexpectedResponse, response.body
-      end
+      def handle_response(response)
+        if (200...300).cover?(response.status) && (response.body.nil? || response.body.empty?)
+          return
+        end
 
-      raise UnexpectedResponse, response.body['error'] if response.body['error']
+        raise InternalServerError, "Server error got #{response.status}" if (500...600).cover?(response.status)
 
-      raise UnexpectedResponse, "Temporary iTunes Connect error: #{response.body}" if response.body['statusCode'] == 'ERROR'
+        unless response.body.kind_of?(Hash)
+          raise UnexpectedResponse, response.body
+        end
 
-      return response.body['data'] if response.body['data']
+        raise UnexpectedResponse, response.body['error'] if response.body['error']
 
-      return response.body
-    end
+        raise UnexpectedResponse, "Temporary iTunes Connect error: #{response.body}" if response.body['statusCode'] == 'ERROR'
 
-    private
+        return response.body['data'] if response.body['data']
+
+        return response.body
+      end
 
-    # used to assert all of the named parameters are supplied values
-    #
-    # @raises NameError if the values are nil
-    def assert_required_params(method_name, binding)
-      parameter_names = method(method_name).parameters.map { |k, v| v }
-      parameter_names.each do |name|
-        if local_variable_get(binding, name).nil?
-          raise NameError, "`#{name}' is a required parameter"
+      private
+
+      # used to assert all of the named parameters are supplied values
+      #
+      # @raises NameError if the values are nil
+      def assert_required_params(method_name, binding)
+        parameter_names = method(method_name).parameters.map { |k, v| v }
+        parameter_names.each do |name|
+          if local_variable_get(binding, name).nil?
+            raise NameError, "`#{name}' is a required parameter"
+          end
         end
       end
-    end
 
-    def local_variable_get(binding, name)
-      if binding.respond_to?(:local_variable_get)
-        binding.local_variable_get(name)
-      else
-        binding.eval(name.to_s)
+      def local_variable_get(binding, name)
+        if binding.respond_to?(:local_variable_get)
+          binding.local_variable_get(name)
+        else
+          binding.eval(name.to_s)
+        end
       end
-    end
 
-    def provider_id
-      return team_id if self.provider.nil?
-      self.provider.provider_id
+      def provider_id
+        return team_id if self.provider.nil?
+        self.provider.provider_id
+      end
     end
   end
 end