working on authentication

spec/controllers/schools_controller_spec.rb

@@ -20,6 +20,10 @@ require 'rails_helper'
 
 RSpec.describe SchoolsController, type: :controller do
 
+  let!(:school) { School.create(name: 'school') }
+  let!(:user) { User.create(email: 'test@example.com', password: '123456') }
+  let!(:user_school) { user.user_schools.create(school: school) }
+
   # This should return the minimal set of attributes required to create a valid
   # School. As you add validations to School, be sure to
   # adjust the attributes here as well.
@@ -36,14 +40,6 @@ RSpec.describe SchoolsController, type: :controller do
   # SchoolsController. Be sure to keep this updated too.
   let(:valid_session) { {} }
 
-  describe "GET #index" do
-    it "assigns all schools as @schools" do
-      school = School.create! valid_attributes
-      get :index, params: {}, session: valid_session
-      expect(assigns(:schools)).to eq([school])
-    end
-  end
-
   describe "GET #show" do
     it "assigns the requested school as @school" do
       school = School.create! valid_attributes
@@ -54,104 +50,132 @@ RSpec.describe SchoolsController, type: :controller do
 
   describe "GET #new" do
     it "assigns a new school as @school" do
-      get :new, params: {}, session: valid_session
+      sign_in user
+      get :new, params: {}
       expect(assigns(:school)).to be_a_new(School)
     end
   end
 
   describe "GET #edit" do
     it "assigns the requested school as @school" do
+      sign_in user
       school = School.create! valid_attributes
-      get :edit, params: {id: school.to_param}, session: valid_session
+      get :edit, params: {id: school.to_param}
       expect(assigns(:school)).to eq(school)
     end
   end
 
+  describe "GET #admin" do
+    it "assigns the requested school as @school" do
+      sign_in user
+      get :admin, params: {school_id: school.to_param}
+      expect(assigns(:school)).to eq(school)
+    end
+
+    it "redirects if not logged in" do
+      get :admin, params: {school_id: school.to_param}
+      expect(response).to redirect_to(new_user_session_path)
+    end
+
+    it "redirects if user is not associated with school" do
+      another_user = User.create(email: 'test2@test.com', password: '123456')
+      sign_in another_user
+
+      get :admin, params: {school_id: school.to_param}
+      expect(response).to redirect_to(root_path)
+    end
+  end
+
   describe "POST #create" do
+    before :each do
+      sign_in user
+    end
+
     context "with valid params" do
       it "creates a new School" do
         expect {
-          post :create, params: {school: valid_attributes}, session: valid_session
+          post :create, params: {school: valid_attributes}
         }.to change(School, :count).by(1)
       end
 
       it "assigns a newly created school as @school" do
-        post :create, params: {school: valid_attributes}, session: valid_session
+        post :create, params: {school: valid_attributes}
         expect(assigns(:school)).to be_a(School)
         expect(assigns(:school)).to be_persisted
       end
 
       it "redirects to the created school" do
-        post :create, params: {school: valid_attributes}, session: valid_session
+        post :create, params: {school: valid_attributes}
         expect(response).to redirect_to(School.last)
       end
     end
 
     context "with invalid params" do
       it "assigns a newly created but unsaved school as @school" do
-        post :create, params: {school: invalid_attributes}, session: valid_session
+        post :create, params: {school: invalid_attributes}
         expect(assigns(:school)).to be_a_new(School)
       end
 
       it "re-renders the 'new' template" do
-        post :create, params: {school: invalid_attributes}, session: valid_session
+        post :create, params: {school: invalid_attributes}
         expect(response).to render_template("new")
       end
     end
   end
 
   describe "PUT #update" do
+    before :each do
+      sign_in user
+    end
+
     context "with valid params" do
       let(:new_attributes) {
         {name: 'New School'}
       }
 
       it "updates the requested school" do
-        school = School.create! valid_attributes
-        put :update, params: {id: school.to_param, school: new_attributes}, session: valid_session
+        put :update, params: {id: school.to_param, school: new_attributes}
         school.reload
         expect(school.name).to eq('New School')
       end
 
       it "assigns the requested school as @school" do
-        school = School.create! valid_attributes
-        put :update, params: {id: school.to_param, school: valid_attributes}, session: valid_session
+        put :update, params: {id: school.to_param, school: valid_attributes}
         expect(assigns(:school)).to eq(school)
       end
 
       it "redirects to the school" do
-        school = School.create! valid_attributes
-        put :update, params: {id: school.to_param, school: valid_attributes}, session: valid_session
+        put :update, params: {id: school.to_param, school: valid_attributes}
         expect(response).to redirect_to(school)
       end
     end
 
     context "with invalid params" do
       it "assigns the school as @school" do
-        school = School.create! valid_attributes
-        put :update, params: {id: school.to_param, school: invalid_attributes}, session: valid_session
+        put :update, params: {id: school.to_param, school: invalid_attributes}
         expect(assigns(:school)).to eq(school)
       end
 
       it "re-renders the 'edit' template" do
-        school = School.create! valid_attributes
-        put :update, params: {id: school.to_param, school: invalid_attributes}, session: valid_session
+        put :update, params: {id: school.to_param, school: invalid_attributes}
         expect(response).to render_template("edit")
       end
     end
   end
 
   describe "DELETE #destroy" do
+    before :each do
+      sign_in user
+    end
+
     it "destroys the requested school" do
-      school = School.create! valid_attributes
       expect {
-        delete :destroy, params: {id: school.to_param}, session: valid_session
+        delete :destroy, params: {id: school.to_param}
       }.to change(School, :count).by(-1)
     end
 
     it "redirects to the schools list" do
-      school = School.create! valid_attributes
-      delete :destroy, params: {id: school.to_param}, session: valid_session
+      delete :destroy, params: {id: school.to_param}
       expect(response).to redirect_to(schools_url)
     end
   end