2013-02-06 03:16:51 +08:00
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
|
|
require 'cooked_post_processor'
|
|
|
|
|
|
|
|
|
|
describe CookedPostProcessor do
|
2013-02-19 14:57:14 +08:00
|
|
|
let :cpp do
|
|
|
|
|
post = Fabricate.build(:post_with_youtube)
|
|
|
|
|
post.id = 123
|
|
|
|
|
CookedPostProcessor.new(post)
|
|
|
|
|
end
|
2013-02-06 03:16:51 +08:00
|
|
|
|
|
|
|
|
context 'process_onebox' do
|
|
|
|
|
|
|
|
|
|
before do
|
|
|
|
|
post = Fabricate.build(:post_with_youtube)
|
|
|
|
|
post.id = 123
|
|
|
|
|
@cpp = CookedPostProcessor.new(post, invalidate_oneboxes: true)
|
|
|
|
|
Oneboxer.expects(:onebox).with("http://www.youtube.com/watch?v=9bZkp7q19f0", post_id: 123, invalidate_oneboxes: true).returns('GANGNAM STYLE')
|
|
|
|
|
@cpp.post_process_oneboxes
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
it 'should be dirty' do
|
|
|
|
|
@cpp.should be_dirty
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
it 'inserts the onebox' do
|
2013-02-12 22:46:45 +08:00
|
|
|
@cpp.html.should == <<EXPECTED
|
|
|
|
|
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">
|
|
|
|
|
<html><body>GANGNAM STYLE</body></html>
|
|
|
|
|
EXPECTED
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
context 'process_images' do
|
|
|
|
|
|
|
|
|
|
it "has no topic image if there isn't one in the post" do
|
|
|
|
|
@post = Fabricate(:post)
|
|
|
|
|
@post.topic.image_url.should be_blank
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
context 'with sized images in the post' do
|
|
|
|
|
before do
|
|
|
|
|
@topic = Fabricate(:topic)
|
|
|
|
|
@post = Fabricate.build(:post_with_image_url, topic: @topic, user: @topic.user)
|
|
|
|
|
@cpp = CookedPostProcessor.new(@post, :image_sizes => {'http://www.forumwarz.com/images/header/logo.png' => {'width' => 111, 'height' => 222}})
|
2013-02-19 14:57:14 +08:00
|
|
|
@cpp.expects(:get_size).returns([111,222])
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
|
|
it "doesn't call image_dimensions because it knows the size" do
|
2013-02-19 14:57:14 +08:00
|
|
|
@cpp.expects(:image_dimensions).never
|
2013-02-06 03:16:51 +08:00
|
|
|
@cpp.post_process_images
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
it "adds the width from the image sizes provided" do
|
|
|
|
|
@cpp.post_process_images
|
|
|
|
|
@cpp.html.should =~ /width=\"111\"/
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
context 'with unsized images in the post' do
|
2013-03-19 01:55:34 +08:00
|
|
|
let(:user) { Fabricate(:user) }
|
|
|
|
|
let(:topic) { Fabricate(:topic, user: user) }
|
|
|
|
|
|
2013-02-06 03:16:51 +08:00
|
|
|
before do
|
2013-02-19 14:57:14 +08:00
|
|
|
FastImage.stubs(:size).returns([123, 456])
|
|
|
|
|
CookedPostProcessor.any_instance.expects(:image_dimensions).returns([123, 456])
|
2013-03-19 01:55:34 +08:00
|
|
|
creator = PostCreator.new(user, raw: Fabricate.build(:post_with_images).raw, topic_id: topic.id)
|
|
|
|
|
@post = creator.create
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
2013-02-26 00:42:20 +08:00
|
|
|
it "adds a topic image if there's one in the post" do
|
2013-02-06 03:16:51 +08:00
|
|
|
@post.topic.reload
|
|
|
|
|
@post.topic.image_url.should == "/path/to/img.jpg"
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
it "adds the height and width to images that don't have them" do
|
|
|
|
|
@post.reload
|
|
|
|
|
@post.cooked.should =~ /width=\"123\" height=\"456\"/
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
end
|
|
|
|
|
end
|
|
|
|
|
|
2013-02-26 00:42:20 +08:00
|
|
|
context 'link convertor' do
|
|
|
|
|
before do
|
2013-02-19 14:57:14 +08:00
|
|
|
SiteSetting.stubs(:crawl_images?).returns(true)
|
|
|
|
|
end
|
2013-02-26 00:42:20 +08:00
|
|
|
|
|
|
|
|
let :post_with_img do
|
2013-02-19 14:57:14 +08:00
|
|
|
Fabricate.build(:post, cooked: '<p><img src="http://hello.com/image.png"></p>')
|
|
|
|
|
end
|
|
|
|
|
|
2013-02-26 00:42:20 +08:00
|
|
|
let :cpp_for_post do
|
2013-02-19 14:57:14 +08:00
|
|
|
CookedPostProcessor.new(post_with_img)
|
|
|
|
|
end
|
|
|
|
|
|
2013-02-26 00:42:20 +08:00
|
|
|
it 'convert img tags to links if they are sized down' do
|
2013-02-19 14:57:14 +08:00
|
|
|
cpp_for_post.expects(:get_size).returns([2000,2000]).twice
|
2013-02-26 00:42:20 +08:00
|
|
|
cpp_for_post.post_process
|
2013-02-19 14:57:14 +08:00
|
|
|
cpp_for_post.html.should =~ /a href/
|
|
|
|
|
end
|
|
|
|
|
|
2013-02-26 00:42:20 +08:00
|
|
|
it 'does not convert img tags to links if they are small' do
|
2013-02-19 14:57:14 +08:00
|
|
|
cpp_for_post.expects(:get_size).returns([200,200]).twice
|
2013-02-26 00:42:20 +08:00
|
|
|
cpp_for_post.post_process
|
2013-02-19 14:57:14 +08:00
|
|
|
(cpp_for_post.html !~ /a href/).should be_true
|
|
|
|
|
end
|
2013-02-26 00:42:20 +08:00
|
|
|
|
2013-02-19 14:57:14 +08:00
|
|
|
end
|
2013-02-06 03:16:51 +08:00
|
|
|
|
|
|
|
|
context 'image_dimensions' do
|
|
|
|
|
it "returns unless called with a http or https url" do
|
2013-02-19 14:57:14 +08:00
|
|
|
cpp.image_dimensions('/tmp/image.jpg').should be_blank
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
|
|
context 'with valid url' do
|
|
|
|
|
before do
|
|
|
|
|
@url = 'http://www.forumwarz.com/images/header/logo.png'
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
it "doesn't call fastimage if image crawling is disabled" do
|
|
|
|
|
SiteSetting.expects(:crawl_images?).returns(false)
|
|
|
|
|
FastImage.expects(:size).never
|
2013-02-19 14:57:14 +08:00
|
|
|
cpp.image_dimensions(@url)
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
|
|
|
|
|
it "calls fastimage if image crawling is enabled" do
|
|
|
|
|
SiteSetting.expects(:crawl_images?).returns(true)
|
|
|
|
|
FastImage.expects(:size).with(@url)
|
2013-02-19 14:57:14 +08:00
|
|
|
cpp.image_dimensions(@url)
|
2013-02-26 00:42:20 +08:00
|
|
|
end
|
2013-02-06 03:16:51 +08:00
|
|
|
end
|
|
|
|
|
end
|
|
|
|
|
|
|
|
|
|
end
|
