2013-02-06 03:16:51 +08:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe ClicksController do
|
|
|
|
|
|
|
|
context 'create' do
|
|
|
|
|
|
|
|
context 'missing params' do
|
|
|
|
it 'raises an error without the url param' do
|
2015-01-10 01:04:02 +08:00
|
|
|
expect { xhr :get, :track, post_id: 123 }.to raise_error(ActionController::ParameterMissing)
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it "redirects to the url even without the topic_id or post_id params" do
|
|
|
|
xhr :get, :track, url: 'http://google.com'
|
2015-01-10 01:04:02 +08:00
|
|
|
expect(response).not_to be_redirect
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'correct params' do
|
2013-07-27 00:14:11 +08:00
|
|
|
let(:url) { "http://discourse.org" }
|
2013-02-06 03:16:51 +08:00
|
|
|
|
|
|
|
before do
|
|
|
|
request.stubs(:remote_ip).returns('192.168.0.1')
|
|
|
|
end
|
|
|
|
|
2013-07-27 00:14:11 +08:00
|
|
|
context "with a made up url" do
|
|
|
|
it "doesn't redirect" do
|
|
|
|
TopicLinkClick.expects(:create_from).returns(nil)
|
|
|
|
xhr :get, :track, url: 'http://discourse.org', post_id: 123
|
2015-01-10 01:04:02 +08:00
|
|
|
expect(response).not_to be_redirect
|
2013-07-27 00:14:11 +08:00
|
|
|
end
|
2013-12-14 01:56:20 +08:00
|
|
|
end
|
2013-07-27 00:14:11 +08:00
|
|
|
|
2013-12-14 01:56:20 +08:00
|
|
|
context "with a query string" do
|
|
|
|
it "tries again without the query if it fails" do
|
|
|
|
TopicLinkClick.expects(:create_from).with(has_entries('url' => 'http://discourse.org/?hello=123')).returns(nil)
|
|
|
|
TopicLinkClick.expects(:create_from).with(has_entries('url' => 'http://discourse.org/')).returns(nil)
|
|
|
|
xhr :get, :track, url: 'http://discourse.org/?hello=123', post_id: 123
|
|
|
|
end
|
2013-07-27 00:14:11 +08:00
|
|
|
end
|
|
|
|
|
2013-02-06 03:16:51 +08:00
|
|
|
context 'with a post_id' do
|
|
|
|
it 'calls create_from' do
|
2013-07-27 00:14:11 +08:00
|
|
|
TopicLinkClick.expects(:create_from).with('url' => url, 'post_id' => '123', 'ip' => '192.168.0.1').returns(url)
|
|
|
|
xhr :get, :track, url: url, post_id: 123
|
2015-01-10 01:04:02 +08:00
|
|
|
expect(response).to redirect_to(url)
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'redirects to the url' do
|
2013-07-27 00:14:11 +08:00
|
|
|
TopicLinkClick.stubs(:create_from).returns(url)
|
|
|
|
xhr :get, :track, url: url, post_id: 123
|
2015-01-10 01:04:02 +08:00
|
|
|
expect(response).to redirect_to(url)
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'will pass the user_id to create_from' do
|
2013-07-27 00:14:11 +08:00
|
|
|
TopicLinkClick.expects(:create_from).with('url' => url, 'post_id' => '123', 'ip' => '192.168.0.1').returns(url)
|
|
|
|
xhr :get, :track, url: url, post_id: 123
|
2015-01-10 01:04:02 +08:00
|
|
|
expect(response).to redirect_to(url)
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
it "doesn't redirect with the redirect=false param" do
|
2013-07-27 00:14:11 +08:00
|
|
|
TopicLinkClick.expects(:create_from).with('url' => url, 'post_id' => '123', 'ip' => '192.168.0.1', 'redirect' => 'false').returns(url)
|
|
|
|
xhr :get, :track, url: url, post_id: 123, redirect: 'false'
|
2015-01-10 01:04:02 +08:00
|
|
|
expect(response).not_to be_redirect
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with a topic_id' do
|
|
|
|
it 'calls create_from' do
|
2013-07-27 00:14:11 +08:00
|
|
|
TopicLinkClick.expects(:create_from).with('url' => url, 'topic_id' => '789', 'ip' => '192.168.0.1').returns(url)
|
|
|
|
xhr :get, :track, url: url, topic_id: 789
|
2015-01-10 01:04:02 +08:00
|
|
|
expect(response).to redirect_to(url)
|
2013-02-26 00:42:20 +08:00
|
|
|
end
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
end
|
|
|
|
|
2013-02-26 00:42:20 +08:00
|
|
|
end
|