From a69210ae6b95673d0035bdf96ce8f93b176838f0 Mon Sep 17 00:00:00 2001
From: Yoong Kang Lim <yoongkang.lim@gmail.com>
Date: Tue, 9 Dec 2014 22:07:41 +1100
Subject: [PATCH] Explicitly add spec for notification

---
 spec/features/signin_spec.rb | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/spec/features/signin_spec.rb b/spec/features/signin_spec.rb
index 047bd68cb..3ad259213 100644
--- a/spec/features/signin_spec.rb
+++ b/spec/features/signin_spec.rb
@@ -2,10 +2,9 @@ require 'spec_helper'
 
 feature "signin" do
   let(:member){FactoryGirl.create(:member)}
+  let(:recipient){FactoryGirl.create(:member)}
   let(:notification){FactoryGirl.create(:notification)}
 
-  MODELS = [:plantings, :harvests, :posts, :photos, :gardens, :seeds]
-
   scenario "redirect to previous page after signin" do
     visit crops_path # some random page
     click_link 'Sign in'
@@ -30,7 +29,8 @@ feature "signin" do
   end
 
   scenario "after signin, redirect to what you were trying to do" do
-    MODELS.each do |model|
+    models = %w[plantings harvests posts photos gardens seeds]
+    models.each do |model|
       visit "/#{model}/new"
       current_path.should eq new_member_session_path
       fill_in 'Login', with: member.login_name
@@ -41,4 +41,13 @@ feature "signin" do
     end
   end
 
+  scenario "after signin, redirect to new notifications page" do
+    visit new_notification_path(recipient: recipient)
+    current_path.should eq new_member_session_path
+    fill_in 'Login', with: member.login_name
+    fill_in 'Password', with: member.password
+    click_button 'Sign in'
+    current_path.should eq new_notification_path
+  end
+
 end