From 58b688aa6872df557f6d645bd1286a4ac455ed15 Mon Sep 17 00:00:00 2001 From: danielvincent Date: Mon, 27 Sep 2010 21:28:21 -0700 Subject: [PATCH] fixed builder spec --- lib/diaspora/ostatus_builder.rb | 4 ++-- spec/lib/diaspora_ostatus_builder_spec.rb | 9 ++++++--- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/lib/diaspora/ostatus_builder.rb b/lib/diaspora/ostatus_builder.rb index 0770f4c5c..ce4ebfeb4 100644 --- a/lib/diaspora/ostatus_builder.rb +++ b/lib/diaspora/ostatus_builder.rb @@ -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 diff --git a/spec/lib/diaspora_ostatus_builder_spec.rb b/spec/lib/diaspora_ostatus_builder_spec.rb index cc46bd0c5..0670755f7 100644 --- a/spec/lib/diaspora_ostatus_builder_spec.rb +++ b/spec/lib/diaspora_ostatus_builder_spec.rb @@ -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