From 5c6352397257d82e75b90cc5e7de1a3922bde7bd Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Wed, 31 May 2017 15:36:24 -0400 Subject: Spec coverage and refactor for the api/v1/accounts controllers (#3451) --- .../v1/accounts/follower_accounts_controller.rb | 68 ++++++++++++++++++++++ 1 file changed, 68 insertions(+) create mode 100644 app/controllers/api/v1/accounts/follower_accounts_controller.rb (limited to 'app/controllers/api/v1/accounts/follower_accounts_controller.rb') diff --git a/app/controllers/api/v1/accounts/follower_accounts_controller.rb b/app/controllers/api/v1/accounts/follower_accounts_controller.rb new file mode 100644 index 000000000..3e9da29e3 --- /dev/null +++ b/app/controllers/api/v1/accounts/follower_accounts_controller.rb @@ -0,0 +1,68 @@ +# frozen_string_literal: true + +class Api::V1::Accounts::FollowerAccountsController < ApiController + before_action -> { doorkeeper_authorize! :read } + before_action :set_account + after_action :insert_pagination_headers + + respond_to :json + + def index + @accounts = load_accounts + render 'api/v1/accounts/index' + end + + private + + def set_account + @account = Account.find(params[:account_id]) + end + + def load_accounts + default_accounts.merge(paginated_follows).to_a + end + + def default_accounts + Account.includes(:active_relationships).references(:active_relationships) + end + + def paginated_follows + Follow.where(target_account: @account).paginate_by_max_id( + limit_param(DEFAULT_ACCOUNTS_LIMIT), + params[:max_id], + params[:since_id] + ) + end + + def insert_pagination_headers + set_pagination_headers(next_path, prev_path) + end + + def next_path + if records_continue? + api_v1_account_followers_url pagination_params(max_id: pagination_max_id) + end + end + + def prev_path + unless @accounts.empty? + api_v1_account_followers_url pagination_params(since_id: pagination_since_id) + end + end + + def pagination_max_id + @accounts.last.active_relationships.first.id + end + + def pagination_since_id + @accounts.first.active_relationships.first.id + end + + def records_continue? + @accounts.size == limit_param(DEFAULT_ACCOUNTS_LIMIT) + end + + def pagination_params(core_params) + params.permit(:limit).merge(core_params) + end +end -- cgit