Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Fix test failure on redirect #6

Merged
merged 1 commit into from over 1 year ago

2 participants

pozorvlak Courtney Webster
pozorvlak

The "create" method was never being called. Removing the sender_id
parameter fixed that. Routing problem, maybe?

pozorvlak pozorvlak Fix test failure on redirect
The "create" method was never being called. Removing the sender_id
parameter fixed that. Routing problem, maybe?
98eabe0
Courtney Webster phazel merged commit 5bb7a28 into from
Courtney Webster phazel closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Mar 25, 2013
pozorvlak pozorvlak Fix test failure on redirect
The "create" method was never being called. Removing the sender_id
parameter fixed that. Routing problem, maybe?
98eabe0
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 2 deletions. Show diff stats Hide diff stats

  1. +1 2  spec/controllers/notifications_controller_spec.rb
3  spec/controllers/notifications_controller_spec.rb
@@ -88,8 +88,7 @@ def valid_session
88 88 it "redirects to the recipient's profile" do
89 89 @sender = FactoryGirl.create(:member)
90 90 @recipient = FactoryGirl.create(:member)
91   - post :create, { :notification => { :recipient_id => @recipient.id,
92   - :sender_id => @sender.id}}
  91 + post :create, { :notification => { :recipient_id => @recipient.id } }
93 92 response.should redirect_to(@recipient)
94 93 end
95 94 end

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.