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 )} let(:moderator) { Fabricate(:moderator) } let(:admin) { Fabricate(:admin) } it "it can find the best responses" do best2 = TopicView.new(topic.id, coding_horror, best: 2) best2.posts.count.should == 2 best2.posts[0].id.should == p2.id best2.posts[1].id.should == p3.id topic.update_status('closed', true, Fabricate(:admin)) topic.posts.count.should == 4 # should not get the status post best = TopicView.new(topic.id, nil, best: 99) best.posts.count.should == 2 best.filtered_post_ids.size.should == 3 best.current_post_ids.should =~ [p2.id, p3.id] # should get no results for trust level too low best = TopicView.new(topic.id, nil, best: 99, min_trust_level: coding_horror.trust_level + 1) best.posts.count.should == 0 # should filter out the posts with a score that is too low best = TopicView.new(topic.id, nil, best: 99, min_score: 99) best.posts.count.should == 0 # should filter out everything if min replies not met best = TopicView.new(topic.id, nil, best: 99, min_replies: 99) best.posts.count.should == 0 # should punch through posts if the score is high enough p2.update_column(:score, 100) best = TopicView.new(topic.id, nil, best: 99, bypass_trust_level_score: 100, min_trust_level: coding_horror.trust_level + 1) best.posts.count.should == 1 # 0 means ignore best = TopicView.new(topic.id, nil, best: 99, bypass_trust_level_score: 0, min_trust_level: coding_horror.trust_level + 1) best.posts.count.should == 0 # If we restrict to posts a moderator liked, return none best = TopicView.new(topic.id, nil, best: 99, only_moderator_liked: true) best.posts.count.should == 0 # It doesn't count likes from admins PostAction.act(admin, p3, PostActionType.types[:like]) best = TopicView.new(topic.id, nil, best: 99, only_moderator_liked: true) best.posts.count.should == 0 # It should find the post liked by the moderator PostAction.act(moderator, p2, PostActionType.types[:like]) best = TopicView.new(topic.id, nil, best: 99, only_moderator_liked: true) best.posts.count.should == 1 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(:p2) { stub(post_number: 2) } let(:topic) do topic = Fabricate(:topic) 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) described_class.any_instance.stubs(:last_post).returns(p2) SiteSetting.stubs(:posts_per_page).returns(2) end it "should return the next page" do described_class.new(1234, user).next_page.should eql(2) 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 '.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_paged' do before { SiteSetting.stubs(:posts_per_page).returns(2) } it 'returns correct posts for all pages' do topic_view.filter_posts_paged(1).should == [p1, p2] topic_view.filter_posts_paged(2).should == [p3, p5] topic_view.filter_posts_paged(100).should == [] 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.desired_post.should == p1 near_view.posts.should == [p1, p2, p3] end it "snaps to the upper boundary" do near_view = topic_view_near(p5) near_view.desired_post.should == p5 near_view.posts.should == [p2, p3, p5] end it "returns the posts in the middle" do near_view = topic_view_near(p2) near_view.desired_post.should == p2 near_view.posts.should == [p1, p2, p3] end it "returns deleted posts to an admin" do coding_horror.admin = true near_view = topic_view_near(p3) near_view.desired_post.should == p3 near_view.posts.should == [p2, p3, p4] end context "when 'posts per page' exceeds the number of posts" do before { SiteSetting.stubs(:posts_per_page).returns(100) } it 'returns all the posts' do near_view = topic_view_near(p5) near_view.posts.should == [p1, p2, p3, p5] end end end end end