FIxes errors on Achievement Controller Spec with Achievements creation.

Author: valdeci-dev

spec/controllers/achievements_controller_spec.rb

@@ -3,21 +3,23 @@
 describe AchievementsController do
 
   shared_examples 'public access to achievements' do
+    let(:user) {FactoryGirl.create(:user)}
     describe 'GET index' do
       it 'renders :index template' do
         get :index
         expect(response).to render_template(:index)
       end
 
       it 'assigns only public achievements to template' do
-        public_achievement = FactoryGirl.create(:public_achievement)
+        public_achievement = FactoryGirl.create(:public_achievement, user: user)
+        private_achievement = FactoryGirl.create(:private_achievement, user: user)
         get :index
         expect(assigns(:achievements)).to match_array([public_achievement])
       end
     end
 
     describe 'GET show' do
-      let(:achievement) {FactoryGirl.create(:public_achievement)}
+      let(:achievement) {FactoryGirl.create(:public_achievement, user: user)}
 
       it 'renders :show template' do
         get :show, params: {id: achievement}
@@ -51,32 +53,6 @@
       end
     end
 
-    describe 'GET edit' do
-      it 'redirects to login page' do
-        get :edit, params: {id: FactoryGirl.create(:public_achievement)}
-        expect(response).to redirect_to(new_user_session_url)
-      end
-    end
-
-    describe 'PUT update' do
-      let(:achievement) {FactoryGirl.create(:public_achievement)}
-      let(:valid_data) {FactoryGirl.attributes_for(:public_achievement, title: 'New Title')}
-
-      it 'redirects to login page' do
-        put :update, params: {id: achievement, achievement: valid_data}
-        expect(response).to redirect_to(new_user_session_url)
-      end
-    end
-
-    describe 'DELETE destroy' do
-      let(:achievement) {FactoryGirl.create(:public_achievement)}
-
-      it 'redirects to login page' do
-        delete :destroy, params: {id: achievement}
-        expect(response).to redirect_to(new_user_session_url)
-      end
-    end
-
   end
 
   describe 'authenticated_user' do
@@ -103,7 +79,7 @@
 
     describe 'POST create' do
       context 'valid data' do
-        let(:valid_data) {FactoryGirl.attributes_for(:public_achievement)}
+        let(:valid_data) {FactoryGirl.attributes_for(:public_achievement, user: user)}
         it 'redirect to achievement#show' do
           post :create, params: {achievement: valid_data}
           expect(response).to redirect_to(achievement_path(assigns[:achievement]))
@@ -133,15 +109,21 @@
     end
 
     context 'is not the owner of the achievement' do
+      let(:user) {FactoryGirl.create(:user)}
+      let(:author) {FactoryGirl.create(:user)}
+			let(:achievement) {FactoryGirl.create(:public_achievement, user: author)}
+
+      before {sign_in(user)}
+
       describe 'GET edit' do
         it 'redirects to achievement page' do
-          get :edit, params: {id: FactoryGirl.create(:public_achievement)}
+          get :edit, params: {id: achievement.id}
           expect(response).to redirect_to(achievements_path)
         end
       end
 
       describe 'PUT update' do
-        let(:achievement) {FactoryGirl.create(:public_achievement)}
+        let(:achievement) {FactoryGirl.create(:public_achievement, user: author)}
         let(:valid_data) {FactoryGirl.attributes_for(:public_achievement, title: 'New Title')}
 
         it 'redirects to achievement page' do
@@ -151,7 +133,7 @@
       end
 
       describe 'DELETE destroy' do
-        let(:achievement) {FactoryGirl.create(:public_achievement,)}
+        let(:achievement) {FactoryGirl.create(:public_achievement, user: user)}
 
         it 'redirects to achievement page' do
           delete :destroy, params: {id: achievement}

spec/mailers/user_mailer_spec.rb

@@ -1,7 +1,7 @@
 require 'rails_helper'
 
 RSpec.describe UserMailer, type: :mailer do
-	
+
 	include Rails.application.routes.url_helpers
 
 	let(:achievement_id) {1}