nocebo/diaspora
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fixed builder spec

Browse source
This commit is contained in:
danielvincent 2010-09-27 21:28:21 -07:00
parent eb7244e07c
commit 58b688aa68
2 changed files with 8 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/diaspora/ostatus_builder.rb
View file

@ -50,10 +50,10 @@ module Diaspora
end
def self.create_body
@user.raw_visible_posts.all.inject do |xml,curr|
@user.visible_posts(:public=>true).inject("") do |xml,curr|
if curr.respond_to?(:to_activity)
unless xml
xml = curr.to_activity
curr.to_activity
else
xml + curr.to_activity
end

9
spec/lib/diaspora_ostatus_builder_spec.rb
View file

@ -9,13 +9,16 @@ describe Diaspora::OstatusBuilder do
let(:user) { Factory(:user) }
let(:aspect) { user.aspect(:name => "Public People") }
let!(:status_message1) { user.post(:status_message, :message => "One", :to => aspect.id) }
let!(:status_message2) { user.post(:status_message, :message => "Two", :to => aspect.id) }
let!(:status_message1) { user.post(:status_message, :message => "One", :public => true, :to => aspect.id) }
let!(:status_message2) { user.post(:status_message, :message => "Two", :public => true, :to => aspect.id) }
let!(:status_message3) { user.post(:status_message, :message => "Three", :public => false, :to => aspect.id) }
let!(:atom) { Diaspora::OstatusBuilder::build(user) }
it 'should include a users posts' do
atom.should include status_message.message
atom.should include status_message1.message
atom.should include status_message2.message
atom.should_not include status_message3.message
end
end