2013-02-05 14:16:51 -05:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe TopicLink do
|
|
|
|
|
|
|
|
it { should belong_to :topic }
|
|
|
|
it { should belong_to :post }
|
|
|
|
it { should belong_to :user }
|
|
|
|
it { should have_many :topic_link_clicks }
|
|
|
|
it { should validate_presence_of :url }
|
|
|
|
|
|
|
|
def test_uri
|
|
|
|
URI.parse(Discourse.base_url)
|
|
|
|
end
|
|
|
|
|
|
|
|
before do
|
|
|
|
@topic = Fabricate(:topic, title: 'unique topic name')
|
|
|
|
@user = @topic.user
|
|
|
|
end
|
|
|
|
|
|
|
|
it "can't link to the same topic" do
|
|
|
|
ftl = TopicLink.new(url: "/t/#{@topic.id}",
|
|
|
|
topic_id: @topic.id,
|
|
|
|
link_topic_id: @topic.id)
|
|
|
|
ftl.valid?.should be_false
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'external links' do
|
|
|
|
before do
|
|
|
|
@post = Fabricate(:post_with_external_links, user: @user, topic: @topic)
|
|
|
|
TopicLink.extract_from(@post)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'has the forum topic links' do
|
|
|
|
@topic.topic_links.count.should == 4
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'works with markdown links' do
|
|
|
|
@topic.topic_links.exists?(url: "http://forumwarz.com").should be_true
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'works with markdown links followed by a period' do
|
|
|
|
@topic.topic_links.exists?(url: "http://www.codinghorror.com/blog").should be_true
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
describe 'internal links' do
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
context 'topic link' do
|
|
|
|
before do
|
|
|
|
@other_topic = Fabricate(:topic, user: @user)
|
|
|
|
@other_post = @other_topic.posts.create(user: @user, raw: "some content")
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
@url = "http://#{test_uri.host}/t/topic-slug/#{@other_topic.id}"
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
@topic.posts.create(user: @user, raw: 'initial post')
|
|
|
|
@post = @topic.posts.create(user: @user, raw: "Link to another topic: #{@url}")
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
TopicLink.extract_from(@post)
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
@link = @topic.topic_links.first
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'extracted the link' do
|
|
|
|
@link.should be_present
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'is set to internal' do
|
|
|
|
@link.should be_internal
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'has the correct url' do
|
|
|
|
@link.url.should == @url
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'has the extracted domain' do
|
|
|
|
@link.domain.should == test_uri.host
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'should have the id of the linked forum' do
|
|
|
|
@link.link_topic_id == @other_topic.id
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'should not be the reflection' do
|
|
|
|
@link.should_not be_reflection
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
describe 'reflection in the other topic' do
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
before do
|
|
|
|
@reflection = @other_topic.topic_links.first
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'exists' do
|
|
|
|
@reflection.should be_present
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'is a reflection' do
|
|
|
|
@reflection.should be_reflection
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'has a post_id' do
|
|
|
|
@reflection.post_id.should be_present
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'has the correct host' do
|
|
|
|
@reflection.domain.should == test_uri.host
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'has the correct url' do
|
|
|
|
@reflection.url.should == "http://#{test_uri.host}/t/unique-topic-name/#{@topic.id}/#{@post.post_number}"
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'links to the original forum topic' do
|
|
|
|
@reflection.link_topic_id.should == @topic.id
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'links to the original post' do
|
|
|
|
@reflection.link_post_id.should == @post.id
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'has the user id of the original link' do
|
|
|
|
@reflection.user_id.should == @link.user_id
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
end
|
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
context 'removing a link' do
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
before do
|
|
|
|
@post.revise(@post.user, "no more linkies")
|
|
|
|
TopicLink.extract_from(@post)
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'should remove the link' do
|
|
|
|
@topic.topic_links.where(post_id: @post.id).should be_blank
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'should remove the reflected link' do
|
|
|
|
@reflection = @other_topic.topic_links.should be_blank
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 12:27:32 -05:00
|
|
|
context "link to a user on discourse" do
|
2013-02-11 11:11:48 -05:00
|
|
|
let(:post) { @topic.posts.create(user: @user, raw: "<a href='/users/#{@user.username_lower}'>user</a>") }
|
2013-02-05 14:16:51 -05:00
|
|
|
before do
|
2013-02-11 11:11:48 -05:00
|
|
|
TopicLink.extract_from(post)
|
2013-02-05 14:16:51 -05:00
|
|
|
end
|
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'does not extract a link' do
|
|
|
|
@topic.topic_links.should be_blank
|
2013-02-05 14:16:51 -05:00
|
|
|
end
|
2013-02-11 11:11:48 -05:00
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 12:27:32 -05:00
|
|
|
context "link to a discourse resource like a FAQ" do
|
|
|
|
let(:post) { @topic.posts.create(user: @user, raw: "<a href='/faq'>faq link here</a>") }
|
|
|
|
before do
|
|
|
|
TopicLink.extract_from(post)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not extract a link' do
|
|
|
|
@topic.topic_links.should be_present
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
context "@mention links" do
|
|
|
|
let(:post) { @topic.posts.create(user: @user, raw: "Hey @#{@user.username_lower}") }
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
before do
|
|
|
|
TopicLink.extract_from(post)
|
|
|
|
end
|
2013-02-05 14:16:51 -05:00
|
|
|
|
2013-02-11 11:11:48 -05:00
|
|
|
it 'does not extract a link' do
|
|
|
|
@topic.topic_links.should be_blank
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2013-02-05 14:16:51 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
describe 'internal link from pm' do
|
|
|
|
before do
|
|
|
|
@pm = Fabricate(:topic, user: @user, archetype: 'private_message')
|
|
|
|
@other_post = @pm.posts.create(user: @user, raw: "some content")
|
|
|
|
|
|
|
|
@url = "http://#{test_uri.host}/t/topic-slug/#{@topic.id}"
|
|
|
|
|
|
|
|
@pm.posts.create(user: @user, raw: 'initial post')
|
|
|
|
@linked_post = @pm.posts.create(user: @user, raw: "Link to another topic: #{@url}")
|
|
|
|
|
|
|
|
TopicLink.extract_from(@linked_post)
|
|
|
|
|
|
|
|
@link = @topic.topic_links.first
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'should not create a reflection' do
|
|
|
|
@topic.topic_links.first.should be_nil
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'should not create a normal link' do
|
|
|
|
@pm.topic_links.first.should_not be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'internal link with non-standard port' do
|
|
|
|
it 'includes the non standard port if present' do
|
|
|
|
@other_topic = Fabricate(:topic, user: @user)
|
|
|
|
SiteSetting.stubs(:port).returns(5678)
|
|
|
|
alternate_uri = URI.parse(Discourse.base_url)
|
|
|
|
@url = "http://#{alternate_uri.host}:5678/t/topic-slug/#{@other_topic.id}"
|
|
|
|
@post = @topic.posts.create(user: @user,
|
|
|
|
raw: "Link to another topic: #{@url}")
|
|
|
|
TopicLink.extract_from(@post)
|
|
|
|
@reflection = @other_topic.topic_links.first
|
|
|
|
@reflection.url.should == "http://#{alternate_uri.host}:5678/t/unique-topic-name/#{@topic.id}"
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|