298 lines
9.6 KiB
Ruby
298 lines
9.6 KiB
Ruby
require 'spec_helper'
|
|
require 'topic_view'
|
|
|
|
describe TopicView do
|
|
|
|
let(:topic) { Fabricate(:topic) }
|
|
let(:coding_horror) { Fabricate(:coding_horror) }
|
|
let(:first_poster) { topic.user }
|
|
|
|
let(:topic_view) { TopicView.new(topic.id, coding_horror) }
|
|
|
|
it "raises a not found error if the topic doesn't exist" do
|
|
lambda { TopicView.new(1231232, coding_horror) }.should raise_error(Discourse::NotFound)
|
|
end
|
|
|
|
it "raises an error if the user can't see the topic" do
|
|
Guardian.any_instance.expects(:can_see?).with(topic).returns(false)
|
|
lambda { topic_view }.should raise_error(Discourse::InvalidAccess)
|
|
end
|
|
|
|
context "with a few sample posts" do
|
|
let!(:p1) { Fabricate(:post, topic: topic, user: first_poster, percent_rank: 1 )}
|
|
let!(:p2) { Fabricate(:post, topic: topic, user: coding_horror, percent_rank: 0.5 )}
|
|
let!(:p3) { Fabricate(:post, topic: topic, user: first_poster, percent_rank: 0 )}
|
|
|
|
it "it can the best 2 responses" do
|
|
best2 = TopicView.new(topic.id, nil, best: 2)
|
|
best2.posts.count.should == 2
|
|
best2.posts[0].id.should == p2.id
|
|
best2.posts[1].id.should == p3.id
|
|
end
|
|
|
|
|
|
it "raises NotLoggedIn if the user isn't logged in and is trying to view a private message" do
|
|
Topic.any_instance.expects(:private_message?).returns(true)
|
|
lambda { TopicView.new(topic.id, nil) }.should raise_error(Discourse::NotLoggedIn)
|
|
end
|
|
|
|
it "provides an absolute url" do
|
|
topic_view.absolute_url.should be_present
|
|
end
|
|
|
|
it "provides a summary of the first post" do
|
|
topic_view.summary.should be_present
|
|
end
|
|
|
|
describe "#get_canonical_path" do
|
|
let(:user) { Fabricate(:user) }
|
|
let(:topic) { Fabricate(:topic) }
|
|
let(:path) { "/1234" }
|
|
|
|
before do
|
|
topic.expects(:relative_url).returns(path)
|
|
described_class.any_instance.expects(:find_topic).with(1234).returns(topic)
|
|
end
|
|
|
|
context "without a post number" do
|
|
context "without a page" do
|
|
it "generates a canonical path for a topic" do
|
|
described_class.new(1234, user).canonical_path.should eql(path)
|
|
end
|
|
end
|
|
|
|
context "with a page" do
|
|
let(:path_with_page) { "/1234?page=5" }
|
|
|
|
it "generates a canonical path for a topic" do
|
|
described_class.new(1234, user, page: 5).canonical_path.should eql(path_with_page)
|
|
end
|
|
end
|
|
end
|
|
context "with a post number" do
|
|
let(:path_with_page) { "/1234?page=10" }
|
|
before { SiteSetting.stubs(:posts_per_page).returns(5) }
|
|
|
|
it "generates a canonical path for a topic" do
|
|
described_class.new(1234, user, post_number: 50).canonical_path.should eql(path_with_page)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "#next_page" do
|
|
let(:posts) { [stub(post_number: 1), stub(post_number: 2)] }
|
|
let(:topic) do
|
|
topic = Fabricate(:topic)
|
|
topic.stubs(:posts).returns(posts)
|
|
topic.stubs(:highest_post_number).returns(5)
|
|
topic
|
|
end
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
before do
|
|
described_class.any_instance.expects(:find_topic).with(1234).returns(topic)
|
|
described_class.any_instance.stubs(:filter_posts)
|
|
SiteSetting.stubs(:posts_per_page).returns(2)
|
|
end
|
|
|
|
it "should return the next page" do
|
|
described_class.new(1234, user).next_page.should eql(1)
|
|
end
|
|
end
|
|
|
|
context '.post_counts_by_user' do
|
|
it 'returns the two posters with their counts' do
|
|
topic_view.post_counts_by_user.to_a.should =~ [[first_poster.id, 2], [coding_horror.id, 1]]
|
|
end
|
|
end
|
|
|
|
context '.participants' do
|
|
it 'returns the two participants hashed by id' do
|
|
topic_view.participants.to_a.should =~ [[first_poster.id, first_poster], [coding_horror.id, coding_horror]]
|
|
end
|
|
end
|
|
|
|
context '.all_post_actions' do
|
|
it 'is blank at first' do
|
|
topic_view.all_post_actions.should be_blank
|
|
end
|
|
|
|
it 'returns the like' do
|
|
PostAction.act(coding_horror, p1, PostActionType.types[:like])
|
|
topic_view.all_post_actions[p1.id][PostActionType.types[:like]].should be_present
|
|
end
|
|
end
|
|
|
|
context '.post_action_visibility' do
|
|
it "is allows users to see likes" do
|
|
topic_view.post_action_visibility.include?(PostActionType.types[:like]).should be_true
|
|
end
|
|
end
|
|
|
|
context '.read?' do
|
|
it 'is unread with no logged in user' do
|
|
TopicView.new(topic.id).read?(1).should be_false
|
|
end
|
|
|
|
it 'makes posts as unread by default' do
|
|
topic_view.read?(1).should be_false
|
|
end
|
|
|
|
it 'knows a post is read when it has a PostTiming' do
|
|
PostTiming.create(topic: topic, user: coding_horror, post_number: 1, msecs: 1000)
|
|
topic_view.read?(1).should be_true
|
|
end
|
|
end
|
|
|
|
context '.topic_user' do
|
|
it 'returns nil when there is no user' do
|
|
TopicView.new(topic.id, nil).topic_user.should be_blank
|
|
end
|
|
|
|
it 'returns a record once the user has some data' do
|
|
TopicView.new(topic.id, coding_horror).topic_user.should be_present
|
|
end
|
|
end
|
|
|
|
context '#recent_posts' do
|
|
before do
|
|
24.times do # our let()s have already created 3
|
|
Fabricate(:post, topic: topic, user: first_poster)
|
|
end
|
|
end
|
|
it 'returns at most 25 recent posts ordered newest first' do
|
|
recent_posts = topic_view.recent_posts
|
|
|
|
# count
|
|
recent_posts.count.should == 25
|
|
|
|
# ordering
|
|
recent_posts.include?(p1).should be_false
|
|
recent_posts.include?(p3).should be_true
|
|
recent_posts.first.created_at.should > recent_posts.last.created_at
|
|
end
|
|
end
|
|
|
|
end
|
|
|
|
context '.posts' do
|
|
|
|
# Create the posts in a different order than the sort_order
|
|
let!(:p5) { Fabricate(:post, topic: topic, user: coding_horror)}
|
|
let!(:p2) { Fabricate(:post, topic: topic, user: coding_horror)}
|
|
let!(:p4) { Fabricate(:post, topic: topic, user: coding_horror, deleted_at: Time.now)}
|
|
let!(:p1) { Fabricate(:post, topic: topic, user: first_poster)}
|
|
let!(:p3) { Fabricate(:post, topic: topic, user: first_poster)}
|
|
|
|
before do
|
|
SiteSetting.stubs(:posts_per_page).returns(3)
|
|
|
|
# Update them to the sort order we're checking for
|
|
[p1, p2, p3, p4, p5].each_with_index do |p, idx|
|
|
p.sort_order = idx + 1
|
|
p.save
|
|
end
|
|
end
|
|
|
|
describe "filter_posts_after" do
|
|
it "returns undeleted posts after a post" do
|
|
topic_view.filter_posts_after(p1.post_number).map(&:id).should == [p2.id, p3.id, p5.id]
|
|
topic_view.should_not be_initial_load
|
|
topic_view.index_offset.should == 1
|
|
topic_view.index_reverse.should be_false
|
|
end
|
|
|
|
it "clips to the end boundary" do
|
|
topic_view.filter_posts_after(p2.post_number).should == [p3, p5]
|
|
topic_view.index_offset.should == 2
|
|
topic_view.index_reverse.should be_false
|
|
end
|
|
|
|
it "returns nothing after the last post" do
|
|
topic_view.filter_posts_after(p5.post_number).should be_blank
|
|
end
|
|
|
|
it "returns nothing after an invalid post number" do
|
|
topic_view.filter_posts_after(1000).should be_blank
|
|
end
|
|
|
|
it "returns deleted posts to an admin" do
|
|
coding_horror.admin = true
|
|
topic_view.filter_posts_after(p1.post_number).should == [p2, p3, p4]
|
|
topic_view.index_offset.should == 1
|
|
topic_view.index_reverse.should be_false
|
|
end
|
|
end
|
|
|
|
describe "fitler_posts_before" do
|
|
it "returns undeleted posts before a post" do
|
|
topic_view.filter_posts_before(p5.post_number).should == [p3, p2, p1]
|
|
topic_view.should_not be_initial_load
|
|
topic_view.index_offset.should == 3
|
|
topic_view.index_reverse.should be_true
|
|
end
|
|
|
|
it "clips to the beginning boundary" do
|
|
topic_view.filter_posts_before(p3.post_number).should == [p2, p1]
|
|
topic_view.index_offset.should == 2
|
|
topic_view.index_reverse.should be_true
|
|
end
|
|
|
|
it "returns nothing before the first post" do
|
|
topic_view.filter_posts_before(p1.post_number).should be_blank
|
|
end
|
|
|
|
it "returns nothing before an invalid post number" do
|
|
topic_view.filter_posts_before(-10).should be_blank
|
|
topic_view.filter_posts_before(1000).should be_blank
|
|
end
|
|
|
|
it "returns deleted posts to an admin" do
|
|
coding_horror.admin = true
|
|
topic_view.filter_posts_before(p5.post_number).should == [p4, p3, p2]
|
|
topic_view.index_offset.should == 4
|
|
topic_view.index_reverse.should be_true
|
|
end
|
|
end
|
|
|
|
describe "filter_posts_near" do
|
|
|
|
def topic_view_near(post)
|
|
TopicView.new(topic.id, coding_horror, post_number: post.post_number)
|
|
end
|
|
|
|
it "snaps to the lower boundary" do
|
|
near_view = topic_view_near(p1)
|
|
near_view.posts.should == [p1, p2, p3]
|
|
near_view.index_offset.should == 0
|
|
near_view.index_reverse.should be_false
|
|
end
|
|
|
|
it "snaps to the upper boundary" do
|
|
near_view = topic_view_near(p5)
|
|
near_view.posts.should == [p2, p3, p5]
|
|
near_view.index_offset.should == 1
|
|
near_view.index_reverse.should be_false
|
|
end
|
|
|
|
it "returns the posts in the middle" do
|
|
near_view = topic_view_near(p2)
|
|
near_view.posts.should == [p1, p2, p3]
|
|
near_view.index_offset.should == 0
|
|
near_view.index_reverse.should be_false
|
|
end
|
|
|
|
it "returns deleted posts to an admin" do
|
|
coding_horror.admin = true
|
|
near_view = topic_view_near(p3)
|
|
near_view.posts.should == [p2, p3, p4]
|
|
near_view.index_offset.should == 1
|
|
near_view.index_reverse.should be_false
|
|
end
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|