Commit 3a92292df02c113c92eeb0fd9d64e5abe6abfcac
1 parent
02130572
Exists in
master
and in
1 other branch
Fixed error helper and user show specs
Showing
2 changed files
with
5 additions
and
15 deletions
Show diff stats
spec/helpers/errs_helper_spec.rb
| @@ -22,23 +22,13 @@ describe ErrsHelper do | @@ -22,23 +22,13 @@ describe ErrsHelper do | ||
| 22 | end | 22 | end |
| 23 | 23 | ||
| 24 | it "should render image_tag with correct alt and src" do | 24 | it "should render image_tag with correct alt and src" do |
| 25 | - expected = "<img alt=\"#{email}\" src=\"#{base_url}?d=identicon&s=48\" />" | ||
| 26 | - helper.gravatar_tag(email).should eq(expected) | 25 | + expected = "<img alt=\"#{email}\" class=\"gravatar\" src=\"#{base_url}?d=identicon&s=48\" />" |
| 26 | + helper.gravatar_tag(email, :s => 48).should eq(expected) | ||
| 27 | end | 27 | end |
| 28 | 28 | ||
| 29 | it "should override :d" do | 29 | it "should override :d" do |
| 30 | - expected = "<img alt=\"#{email}\" src=\"#{base_url}?d=retro&s=48\" />" | ||
| 31 | - helper.gravatar_tag(email, :d => 'retro').should eq(expected) | ||
| 32 | - end | ||
| 33 | - | ||
| 34 | - it "should override alt" do | ||
| 35 | - expected = "<img alt=\"gravatar\" src=\"#{base_url}?d=identicon&s=48\" />" | ||
| 36 | - helper.gravatar_tag(email, :alt => 'gravatar').should eq(expected) | ||
| 37 | - end | ||
| 38 | - | ||
| 39 | - it "should pass other options to image_tag" do | ||
| 40 | - expected = "<img align=\"left\" alt=\"#{email}\" src=\"#{base_url}?d=identicon&s=48\" />" | ||
| 41 | - helper.gravatar_tag(email, :align => :left).should eq(expected) | 30 | + expected = "<img alt=\"#{email}\" class=\"gravatar\" src=\"#{base_url}?d=retro&s=48\" />" |
| 31 | + helper.gravatar_tag(email, :d => 'retro', :s => 48).should eq(expected) | ||
| 42 | end | 32 | end |
| 43 | end | 33 | end |
| 44 | end | 34 | end |
spec/views/users/show.html.haml_spec.rb
| @@ -2,7 +2,7 @@ require 'spec_helper' | @@ -2,7 +2,7 @@ require 'spec_helper' | ||
| 2 | 2 | ||
| 3 | describe 'users/show.html.haml' do | 3 | describe 'users/show.html.haml' do |
| 4 | let(:user) do | 4 | let(:user) do |
| 5 | - stub_model(User, :created_at => Time.now) | 5 | + stub_model(User, :created_at => Time.now, :email => "test@example.com") |
| 6 | end | 6 | end |
| 7 | 7 | ||
| 8 | before do | 8 | before do |