On 7/6/07, David Chelimsky <[EMAIL PROTECTED]> wrote:
On 7/6/07, Daniel N <[EMAIL PROTECTED]> wrote:
> On 7/6/07, Daniel N <[EMAIL PROTECTED]> wrote:
> > On 7/6/07, David Chelimsky < [EMAIL PROTECTED]> wrote:
> > > On 7/5/07, Daniel N <[EMAIL PROTECTED]> wrote:
> > > > Hi,
> > > >
> > > > I'm very new to rspec so please be patient with me.
> > > >
> > > > I've tried to take some of my tests out of the controller specs to
> check for
> > > > things that are rendered.
> > > >
> > > > This has not worked so well, since my views have the controller
method
> > > >
> > > > current_user
> > > >
> > > > in quite a few places.
> > > >
> > > > Is there any way that I can define this so that my views will be
> executed?
> > > > Will this same thing occur for all helper methods that are
definied in
> the
> > > > controller?
> > >
> > > If I understand you correctly, you are trying to take tests for
views
> > > that were previously rails functional tests and turn them into rspec
> > > view examples. If that is the case, you should be able to do this:
> > >
> > >
> template.stub!(:current_user).and_return(mock_model(User))
> > >
> > > If not, please provide an explicit example so we can better
understand
> > > what you're talking about.
> > >
> > > Cheers,
> > > David
> >
> >
> >
> > Thanx David,
> >
> > That looks like what I was looking for.
> >
> > I will try it when I get home.
> >
> > Cheers
> > Daniel
>
> Ok I've started to have a crack at this but it's getting way out of
hand.
> Everytime I stub a method there's another one to do.
>
> I've also found that when there's a partial _detail and I've passed the
> :collection => @things to it it blows up complaining that the local
variable
> is nil in
> dom_id( detail )
If you're using the trunk, you can do this:
template.expects_render(:partial => 'detail', :collection => @things)
>
> Am I doing someting wrong? The start of my specs is
>
> before do
> @u = mock_model( User )
> @book = mock_model( Book )
>
> public_book = mock_model( Book )
> private_book = mock_model( Book )
> public_book.stub!(:title=).and_return( "Public Title" )
> private_book.stub!(:title=).and_return( "Private Title"
> )
> public_book.stub!( :title ).and_return( "Public Title" )
> private_book.stub!( :title ).and_return( "Private Title" )
Why are you stubbing the same calls twice each?
>
> @u.stub!( :public_books ).and_return( [public_book] )
> @u.stub!( :private_books ).and_return( [private_book] )
> @clip = mock_model( Clip )
> @clip.stub!( :id ).and_return( 1 )
> @clips = [ @clip ]
>
> template.stub!( :current_user ).and_return( @u )
> end
>
> and I've only started. Is it normal to have to stub this much for a
view?
Depends on how much stuff is in your view :)
You've got a couple of options. You could create instances of the
model instead. As long as you're not saving and retrieving them
there's very little db interaction - just enough for AR to discover
the attributes.
If you prefer to keep it all mocked/stubbed, you could clean up a bit like
this:
before do
@u = mock_model( User )
@book = mock_model( Book )
public_book = mock_model(Book, :title => 'Public Title')
private_book = mock_model( Book, :title => 'Private Title')
@u.stub!( :public_books ).and_return( [public_book] )
@u.stub!( :private_books ).and_return( [private_book] )
@clips = [ @clip = mock_model( Clip, :id => 1 ) ]
template.stub!( :current_user ).and_return( @u )
end
That stubs the same amount of stuff, but its a little cleaner. You
could also write a shared behaviour that stubs current user:
describe "authenticated page view", :shared => true do
before(:each) do
template.stub!( :current_user ).and_return( @u )
end
end
describe "/some/page"
it_should_behave_like "authenticated page view"
before(:each) do
@u = mock_model( User )
@book = mock_model( Book )
public_book = mock_model(Book, :title => 'Public Title')
private_book = mock_model( Book, :title => 'Private Title')
@u.stub!( :public_books ).and_return( [public_book] )
@u.stub!( :private_books ).and_return( [private_book] )
@clips = [ @clip = mock_model( Clip, :id => 1 ) ]
end
end
There's probably a bit more to clean up but I'd have to see the view
code. Would you mind letting us see it?
>
> Cheers
> Daniel
index.html.erb
<% content_for :action_bar do %>
<% @public_books = current_user.public_books %>
<% @private_books = current_user.private_books %>
<%= render :file => 'books/_book_list.html.erb' -%>
<% end %>
<% if @book %>
<div class='book_clip_list' id='<%= dom_id( @book )-%>'>
<h2 class='book_header'><%= @book.title -%></h2>
<ul id="book_clip_menu">
<li>
<button class='collapse_all'>Collapse All</button>
<button class='expand_all'>Expand All</button>
</li>
</ul>
<%= render :partial => 'detail', :collection => @clips %>
</div>
<% else %>
<%= render :partial => 'detail', :collection => @clips -%>
<% end %>
detail.html.erb
<div id='<%= dom_id( detail ) -%>' class='clip_container'>
<div class='clip_header'>
<button class='minimize'>-</button>
<h3 id='<%= "#{dom_id( detail )}_header" -%>'><%= detail.title-%></h3>
<div>
<a href='<%= detail.url -%>' target='_blank'><%= URI.parse(
detail.url).host -%></a><br/>
<%= detail.created_at.to_s( :long ) -%>
</div>
</div>
<div class='content_container' id='<%= "#{dom_id( detail
)}_content_container" -%>'>
<div class='clip_menu'>
<ul class='action_menu'>
<%= render :partial => 'action_menu_links', :locals => { :clip
=> detail, :book => @book } -%>
</ul>
<ul class='view_menu'>
<%= render :partial => 'alternative_content_links', :locals => {
:clip => detail } -%>
</ul>
</div>
<blockquote id='<%= "#{dom_id( detail )}_blockquote"-%>'>
<% render_as = @view_type || 'html' %>
<% render_as = 'html' unless Clip.available_types.include?( render_as )
%>
<% case render_as %>
<% when 'quote' %>
<%= render :partial => 'text', :locals => { :clip => detail
} %>
<% when 'images' %>
<%= render :partial => 'images', :locals => { :clip => detail
} %>
<% when 'html' %>
<%= render :partial => 'html', :locals => { :clip =>
detail } %>
<% end %>
</blockquote>
</div>
</div>
The _text, _images, _html partials are very simple so I won't show them
here. This will be enough I think to evaluate it.
Thankyou for helping with this.
Daniel
_______________________________________________
rspec-users mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/rspec-users