diff --git a/app/controllers/home_controller.rb b/app/controllers/home_controller.rb index 0be13de..0f7115e 100644 --- a/app/controllers/home_controller.rb +++ b/app/controllers/home_controller.rb @@ -86,7 +86,7 @@ class HomeController < ApplicationController by_user = User.where(:username => params[:user]).first! @stories, @show_more = get_from_cache(by_user: by_user) { - paginate stories.by_user(by_user) + paginate stories.newest_by_user(by_user) } @heading = @title = "Newest Stories by #{by_user.username}" diff --git a/app/models/story_repository.rb b/app/models/story_repository.rb index 3f1b8c2..7597633 100644 --- a/app/models/story_repository.rb +++ b/app/models/story_repository.rb @@ -31,8 +31,8 @@ class StoryRepository newest.order("stories.created_at DESC") end - def by_user(user) - base_scope.where(user_id: user.id) + def newest_by_user(user) + base_scope.where(user_id: user.id).order("stories.id DESC") end def recent diff --git a/spec/controllers/home_controller_spec.rb b/spec/controllers/home_controller_spec.rb index a50d9cf..2d1d145 100644 --- a/spec/controllers/home_controller_spec.rb +++ b/spec/controllers/home_controller_spec.rb @@ -70,7 +70,7 @@ describe HomeController do let(:scope) { double 'Newest By User Scope' } let(:user) { User.make! } - before { StoryRepository.any_instance.should_receive(:by_user).with(user) } + before { StoryRepository.any_instance.should_receive(:newest_by_user).with(user) } before { get 'newest_by_user', user: user.username } context 'assigns' do diff --git a/spec/models/story_repository_spec.rb b/spec/models/story_repository_spec.rb index 8967234..e1a4747 100644 --- a/spec/models/story_repository_spec.rb +++ b/spec/models/story_repository_spec.rb @@ -87,17 +87,17 @@ describe StoryRepository do end end - describe '#by_user' do + describe '#newest_by_user' do let(:another_user) { User.make! } - subject { stories.by_user(another_user) } + subject { stories.newest_by_user(another_user) } - it 'orders by created_at' do + it 'orders by id descending' do story1 = Story.make! user_id: another_user.id story2 = Story.make! story3 = Story.make! user_id: another_user.id - expect(subject).to eq [story1, story3] + expect(subject).to eq [story3, story1] end end