Add endpoint for recent boards

Returns last 5 visited boards

app/models/board_group_recent_visit.rb

@@ -19,7 +19,10 @@ class BoardGroupRecentVisit < ActiveRecord::Base
     retry
   end
 
-  def self.latest(user, group, count = nil)
-    by_user_group(user, group).order(updated_at: :desc).first(count)
+  def self.latest(user, group, count: nil)
+    visits = by_user_group(user, group).order(updated_at: :desc)
+    visits = visits.preload(:board) if count && count > 1
+
+    visits.first(count)
   end
 end

app/models/board_project_recent_visit.rb

@@ -19,7 +19,10 @@ class BoardProjectRecentVisit < ActiveRecord::Base
     retry
   end
 
-  def self.latest(user, project, count = nil)
-    by_user_project(user, project).order(updated_at: :desc).first(count)
+  def self.latest(user, project, count: nil)
+    visits = by_user_project(user, project).order(updated_at: :desc)
+    visits = visits.preload(:board) if count && count > 1
+
+    visits.first(count)
   end
 end

app/services/boards/visits/latest_service.rb

@@ -6,13 +6,7 @@ module Boards
       def execute
         return nil unless current_user
 
-        relation = recent_visit_model
-
-        if params[:count] && params[:count] > 1
-          relation = relation.preload(:board)
-        end
-
-        relation.latest(current_user, parent, params[:count])
+        recent_visit_model.latest(current_user, parent, count: params[:count])
       end
 
       private

config/routes/group.rb

@@ -67,7 +67,7 @@ constraints(::Constraints::GroupUrlConstrainer.new) do
       end
     end
 
-    resources :boards, only: [:index, :show]
+    resources :boards, only: [:index, :show], constraints: { id: /\d+/ }
 
     resources :runners, only: [:index, :edit, :update, :destroy, :show] do
       member do

config/routes/project.rb

@@ -475,8 +475,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
 
       get 'noteable/:target_type/:target_id/notes' => 'notes#index', as: 'noteable_notes'
 
-      # On CE only index and show are needed
-      resources :boards, only: [:index, :show, :create, :update, :destroy]
+      resources :boards, only: [:index, :show], constraints: { id: /\d+/ }
 
       resources :todos, only: [:create]
 

ee/app/controllers/ee/boards/boards_controller.rb

@@ -12,6 +12,13 @@ module EE
         before_action :authorize_admin_board!, only: [:create, :update, :destroy]
       end
 
+      def recent
+        recent_visits = ::Boards::Visits::LatestService.new(parent, current_user, count: 5).execute
+        recent_boards = recent_visits.map(&:board)
+
+        render json: serialize_as_json(recent_boards)
+      end
+
       def create
         board = ::Boards::CreateService.new(parent, current_user, board_params).execute
 

ee/config/routes/group.rb

@@ -67,7 +67,11 @@ constraints(::Constraints::GroupUrlConstrainer.new) do
     end
 
     resources :todos, only: [:create]
-    resources :boards, only: [:create, :update, :destroy]
+    resources :boards, only: [:create, :update, :destroy] do
+      collection do
+        get :recent
+      end
+    end
 
     namespace :security do
       resource :dashboard, only: [:show], controller: :dashboard

ee/config/routes/project.rb

@@ -17,6 +17,12 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
         end
       end
 
+      resources :boards, only: [:create, :update, :destroy] do
+        collection do
+          get :recent
+        end
+      end
+
       resources :web_ide_terminals, path: :ide_terminals, only: [:create, :show], constraints: { id: /\d+/, format: :json } do
         member do
           post :cancel

ee/spec/controllers/groups/boards_controller_spec.rb

@@ -52,6 +52,12 @@ describe Groups::BoardsController do
     end
   end
 
+  describe 'GET recent' do
+    let(:parent) { group }
+
+    it_behaves_like 'returns recently visited boards'
+  end
+
   describe 'GET show' do
     context 'for multiple issue boards' do
       let(:parent) { group }

ee/spec/controllers/projects/boards_controller_spec.rb

@@ -38,6 +38,12 @@ describe Projects::BoardsController do
     end
   end
 
+  describe 'GET recent' do
+    let(:parent) { project }
+
+    it_behaves_like 'returns recently visited boards'
+  end
+
   describe 'GET show' do
     let(:parent) { project }
 

ee/spec/support/shared_examples/controllers/recent_boards.rb

+# frozen_string_literal: true
+
+require 'spec_helper'
+
+shared_examples 'returns recently visited boards' do
+  let(:boards) { create_list(:board, 8, parent: parent) }
+
+  it 'returns last 5 visited boards' do
+    [0, 2, 5, 3, 7, 1].each_with_index do |board_index, i|
+      visit_board(boards[board_index], Time.now + i.minutes)
+    end
+
+    get_recent_boards
+
+    expect(json_response.length).to eq(5)
+    expect(json_response.map { |b| b['id'] }).to eq([1, 7, 3, 5, 2].map { |i| boards[i].id })
+  end
+
+  def visit_board(board, time)
+    if parent.is_a?(Group)
+      create(:board_group_recent_visit, group: parent, board: board, user: user, updated_at: time)
+    else
+      create(:board_project_recent_visit, project: parent, board: board, user: user, updated_at: time)
+    end
+  end
+
+  def get_recent_boards
+    params = if parent.is_a?(Group)
+               { group_id: parent }
+             else
+               { namespace_id: parent.namespace, project_id: parent }
+             end
+
+    get :recent, params: params
+  end
+end

spec/models/board_group_recent_visit_spec.rb

@@ -50,21 +50,25 @@ describe BoardGroupRecentVisit do
   end
 
   describe '#latest' do
+    def create_visit(time)
+      create :board_group_recent_visit, group: group, user: user, updated_at: time
+    end
+
     it 'returns the most recent visited' do
-      create :board_group_recent_visit, group: group, user: user, updated_at: 7.days.ago
-      create :board_group_recent_visit, group: group, user: user, updated_at: 5.days.ago
-      recent = create :board_group_recent_visit, group: group, user: user, updated_at: 1.day.ago
+      create_visit(7.days.ago)
+      create_visit(5.days.ago)
+      recent = create_visit(1.day.ago)
 
       expect(described_class.latest(user, group)).to eq recent
     end
 
     it 'returns last 3 visited boards' do
-      create :board_group_recent_visit, group: group, user: user, updated_at: 7.days.ago
-      visit1 = create :board_group_recent_visit, group: group, user: user, updated_at: 3.days.ago
-      visit2 = create :board_group_recent_visit, group: group, user: user, updated_at: 2.days.ago
-      visit3 = create :board_group_recent_visit, group: group, user: user, updated_at: 5.days.ago
+      create_visit(7.days.ago)
+      visit1 = create_visit(3.days.ago)
+      visit2 = create_visit(2.days.ago)
+      visit3 = create_visit(5.days.ago)
 
-      expect(described_class.latest(user, group, 3)).to eq([visit2, visit1, visit3])
+      expect(described_class.latest(user, group, count: 3)).to eq([visit2, visit1, visit3])
     end
   end
 end

spec/models/board_project_recent_visit_spec.rb

@@ -50,21 +50,25 @@ describe BoardProjectRecentVisit do
   end
 
   describe '#latest' do
+    def create_visit(time)
+      create :board_project_recent_visit, project: project, user: user, updated_at: time
+    end
+
     it 'returns the most recent visited' do
-      create :board_project_recent_visit, project: project, user: user, updated_at: 7.days.ago
-      create :board_project_recent_visit, project: project, user: user, updated_at: 5.days.ago
-      recent = create :board_project_recent_visit, project: project, user: user, updated_at: 1.day.ago
+      create_visit(7.days.ago)
+      create_visit(5.days.ago)
+      recent = create_visit(1.day.ago)
 
       expect(described_class.latest(user, project)).to eq recent
     end
 
     it 'returns last 3 visited boards' do
-      create :board_project_recent_visit, project: project, user: user, updated_at: 7.days.ago
-      visit1 = create :board_project_recent_visit, project: project, user: user, updated_at: 3.days.ago
-      visit2 = create :board_project_recent_visit, project: project, user: user, updated_at: 2.days.ago
-      visit3 = create :board_project_recent_visit, project: project, user: user, updated_at: 5.days.ago
+      create_visit(7.days.ago)
+      visit1 = create_visit(3.days.ago)
+      visit2 = create_visit(2.days.ago)
+      visit3 = create_visit(5.days.ago)
 
-      expect(described_class.latest(user, project, 3)).to eq([visit2, visit1, visit3])
+      expect(described_class.latest(user, project, count: 3)).to eq([visit2, visit1, visit3])
     end
   end
 end

spec/services/boards/visits/latest_service_spec.rb

@@ -25,7 +25,7 @@ describe Boards::Visits::LatestService do
       end
 
       it 'queries for last N visits' do
-        expect(BoardProjectRecentVisit).to receive(:latest).with(user, project, 5).once
+        expect(BoardProjectRecentVisit).to receive(:latest).with(user, project, count: 5).once
 
         described_class.new(project_board.parent, user, count: 5).execute
       end
@@ -50,7 +50,7 @@ describe Boards::Visits::LatestService do
       end
 
       it 'queries for last N visits' do
-        expect(BoardGroupRecentVisit).to receive(:latest).with(user, group, 5).once
+        expect(BoardGroupRecentVisit).to receive(:latest).with(user, group, count: 5).once
 
         described_class.new(group_board.parent, user, count: 5).execute
       end