Introduce recent to Follow (#3247)

Introduce recent to Follow, as Account and other models have.
This change also adds specs for the scope and the dependents.

diff --git a/app/controllers/follower_accounts_controller.rb b/app/controllers/follower_accounts_controller.rb
index d56bd2a4dd43cd36e0fb94d89dd73917f32876c3..1e7c7c406632504cce1e8af247db595d6e104621 100644 (file)
--- a/app/controllers/follower_accounts_controller.rb
+++ b/app/controllers/follower_accounts_controller.rb
@@ -4,6 +4,6 @@ class FollowerAccountsController < ApplicationController
   include AccountControllerConcern
 
   def index
-    @follows = Follow.where(target_account: @account).order(id: :desc).page(params[:page]).per(FOLLOW_PER_PAGE).preload(:account)
+    @follows = Follow.where(target_account: @account).recent.page(params[:page]).per(FOLLOW_PER_PAGE).preload(:account)
   end
 end

diff --git a/app/controllers/following_accounts_controller.rb b/app/controllers/following_accounts_controller.rb
index 925647864467df9328dbe9bcd05e791c2272ea8b..f4488eef5ea6a30322f333eed05061d0dadd40b3 100644 (file)
--- a/app/controllers/following_accounts_controller.rb
+++ b/app/controllers/following_accounts_controller.rb
@@ -4,6 +4,6 @@ class FollowingAccountsController < ApplicationController
   include AccountControllerConcern
 
   def index
-    @follows = Follow.where(account: @account).order(id: :desc).page(params[:page]).per(FOLLOW_PER_PAGE).preload(:target_account)
+    @follows = Follow.where(account: @account).recent.page(params[:page]).per(FOLLOW_PER_PAGE).preload(:target_account)
   end
 end

diff --git a/app/models/follow.rb b/app/models/follow.rb
index 863710f5d41ca85775e7c9a14e04da5b8bb49702..62f6fb670b7df555af533cc51ea105ccd26019e7 100644 (file)
--- a/app/models/follow.rb
+++ b/app/models/follow.rb
@@ -23,4 +23,6 @@ class Follow < ApplicationRecord
   has_one :notification, as: :activity, dependent: :destroy
 
   validates :account_id, uniqueness: { scope: :target_account_id }
+
+  scope :recent, -> { reorder(id: :desc) }
 end

diff --git a/spec/controllers/follower_accounts_controller_spec.rb b/spec/controllers/follower_accounts_controller_spec.rb
index f2082f601f09589b87e9679401d7b6527ed64b92..b9b7fef73eed8f2ade8f2243f54fbc4a621e4252 100644 (file)
--- a/spec/controllers/follower_accounts_controller_spec.rb
+++ b/spec/controllers/follower_accounts_controller_spec.rb
@@ -4,11 +4,21 @@ describe FollowerAccountsController do
   render_views
 
   let(:alice) { Fabricate(:account, username: 'alice') }
+  let(:follower0) { Fabricate(:account) }
+  let(:follower1) { Fabricate(:account) }
 
   describe 'GET #index' do
-    it 'returns http success' do
+    it 'assigns follows' do
+      follow0 = follower0.follow!(alice)
+      follow1 = follower1.follow!(alice)
+
       get :index, params: { account_username: alice.username }
 
+      assigned = assigns(:follows).to_a
+      expect(assigned.size).to eq 2
+      expect(assigned[0]).to eq follow1
+      expect(assigned[1]).to eq follow0
+
       expect(response).to have_http_status(:success)
     end
   end

diff --git a/spec/controllers/following_accounts_controller_spec.rb b/spec/controllers/following_accounts_controller_spec.rb
index f4a7e88f9237c8a1617be6412aab0e5a74e8ee89..55e7265c76d5b7b1f7da6971778577033d38a064 100644 (file)
--- a/spec/controllers/following_accounts_controller_spec.rb
+++ b/spec/controllers/following_accounts_controller_spec.rb
@@ -4,11 +4,21 @@ describe FollowingAccountsController do
   render_views
 
   let(:alice) { Fabricate(:account, username: 'alice') }
+  let(:followee0) { Fabricate(:account) }
+  let(:followee1) { Fabricate(:account) }
 
   describe 'GET #index' do
-    it 'returns http success' do
+    it 'assigns followees' do
+      follow0 = alice.follow!(followee0)
+      follow1 = alice.follow!(followee1)
+
       get :index, params: { account_username: alice.username }
 
+      assigned = assigns(:follows).to_a
+      expect(assigned.size).to eq 2
+      expect(assigned[0]).to eq follow1
+      expect(assigned[1]).to eq follow0
+
       expect(response).to have_http_status(:success)
     end
   end

diff --git a/spec/models/follow_spec.rb b/spec/models/follow_spec.rb
index 0fae2535293416dc20c53f0ce2b2e82d5245a854..43175d8526aded12266038f046e9efe75bc82acf 100644 (file)
--- a/spec/models/follow_spec.rb
+++ b/spec/models/follow_spec.rb
@@ -4,9 +4,9 @@ RSpec.describe Follow, type: :model do
   let(:alice) { Fabricate(:account, username: 'alice') }
   let(:bob)   { Fabricate(:account, username: 'bob') }
 
-  subject { Follow.new(account: alice, target_account: bob) }
-
   describe 'validations' do
+    subject { Follow.new(account: alice, target_account: bob) }
+
     it 'has a valid fabricator' do
       follow = Fabricate.build(:follow)
       expect(follow).to be_valid
@@ -24,4 +24,17 @@ RSpec.describe Follow, type: :model do
       expect(follow).to model_have_error_on_field(:target_account)
     end
   end
+
+  describe 'recent' do
+    it 'sorts so that more recent follows comes earlier' do
+      follow0 = Follow.create!(account: alice, target_account: bob)
+      follow1 = Follow.create!(account: bob, target_account: alice)
+
+      a = Follow.recent.to_a
+
+      expect(a.size).to eq 2
+      expect(a[0]).to eq follow1
+      expect(a[1]).to eq follow0
+    end
+  end
 end