Optionally invalidate onebox in PostAnalyzer#cook
This commit is contained in:
parent
6a9484155d
commit
ef76e18215
|
@ -20,7 +20,8 @@ class PostAnalyzer
|
||||||
# wait for the post processor.
|
# wait for the post processor.
|
||||||
dirty = false
|
dirty = false
|
||||||
result = Oneboxer.apply(cooked) do |url, elem|
|
result = Oneboxer.apply(cooked) do |url, elem|
|
||||||
Oneboxer.render_from_cache(url)
|
Oneboxer.invalidate(url) if args.last[:invalidate_oneboxes]
|
||||||
|
Oneboxer.onebox url
|
||||||
end
|
end
|
||||||
|
|
||||||
cooked = result.to_html if result.changed?
|
cooked = result.to_html if result.changed?
|
||||||
|
|
|
@ -2,10 +2,39 @@ require 'spec_helper'
|
||||||
|
|
||||||
describe PostAnalyzer do
|
describe PostAnalyzer do
|
||||||
|
|
||||||
let(:topic) { Fabricate(:topic) }
|
let(:default_topic_id) { 12 }
|
||||||
let(:default_topic_id) { topic.id }
|
|
||||||
let(:post_args) do
|
describe '#cook' do
|
||||||
{user: topic.user, topic: topic}
|
let(:post_analyzer) { described_class.new nil, nil }
|
||||||
|
|
||||||
|
let(:args) { [raw, options] }
|
||||||
|
let(:raw) { "Here's a tweet:\n#{url}" }
|
||||||
|
let(:options) { {} }
|
||||||
|
|
||||||
|
let(:url) {
|
||||||
|
'https://twitter.com/evil_trout/status/345954894420787200'
|
||||||
|
}
|
||||||
|
|
||||||
|
before { Oneboxer.stubs(:onebox) }
|
||||||
|
|
||||||
|
it 'fetches the onebox for any urls in the post' do
|
||||||
|
Oneboxer.expects(:onebox).with url
|
||||||
|
post_analyzer.cook(*args)
|
||||||
|
end
|
||||||
|
|
||||||
|
it 'does not invalidate the onebox cache' do
|
||||||
|
Oneboxer.expects(:invalidate).with(url).never
|
||||||
|
post_analyzer.cook(*args)
|
||||||
|
end
|
||||||
|
|
||||||
|
context 'when invalidating oneboxes' do
|
||||||
|
let(:options) {{ invalidate_oneboxes: true }}
|
||||||
|
|
||||||
|
it 'invalidates the oneboxes for urls in the post' do
|
||||||
|
Oneboxer.expects(:invalidate).with url
|
||||||
|
post_analyzer.cook(*args)
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
context "links" do
|
context "links" do
|
||||||
|
@ -115,6 +144,7 @@ describe PostAnalyzer do
|
||||||
end
|
end
|
||||||
|
|
||||||
it "finds links from markdown" do
|
it "finds links from markdown" do
|
||||||
|
Oneboxer.stubs :onebox
|
||||||
post_analyzer = PostAnalyzer.new(raw_post_one_link_md, default_topic_id)
|
post_analyzer = PostAnalyzer.new(raw_post_one_link_md, default_topic_id)
|
||||||
post_analyzer.link_count.should == 1
|
post_analyzer.link_count.should == 1
|
||||||
end
|
end
|
||||||
|
|
|
@ -2,6 +2,8 @@ require 'spec_helper'
|
||||||
require_dependency 'post_destroyer'
|
require_dependency 'post_destroyer'
|
||||||
|
|
||||||
describe Post do
|
describe Post do
|
||||||
|
before { Oneboxer.stubs :onebox }
|
||||||
|
|
||||||
# Help us build a post with a raw body
|
# Help us build a post with a raw body
|
||||||
def post_with_body(body, user=nil)
|
def post_with_body(body, user=nil)
|
||||||
args = post_args.merge(raw: body)
|
args = post_args.merge(raw: body)
|
||||||
|
|
Loading…
Reference in New Issue