gaku 0.1.1 → 0.2.0

data/frontend/spec/controllers/gaku/class_groups/class_groups_controller_spec.rb

@@ -12,18 +12,24 @@ describe Gaku::ClassGroupsController do
       describe 'GET #index' do
         let(:class_group_for_semester) { create(:class_group) }
         let(:semester) { create(:semester) }
-        let(:semester_class_group) { create(:semester_class_group, semester: semester, class_group: class_group) }
+        let(:semester_connector_class_group) do
+          create(:semester_connector_class_group, semester: semester, semesterable: class_group)
+        end
 
         before do
           class_group
-          semester_class_group
+          semester_connector_class_group
           gaku_get :index
         end
 
         it { should respond_with 200 }
-        xit('assigns @class_groups') { expect(assigns(:class_groups)).to eq [semester_class_group] }
-        xit('assigns @class_groups_without_semester') { expect(assigns(:class_groups_without_semester)).to eq [class_group] }
-        it('assigns @count') { expect(assigns(:count)).to eq 1 }
+        xit('assigns @class_groups') { expect(assigns(:class_groups)).to eq [semester_connector_class_group] }
+
+        xit('assigns @class_groups_without_semester') do
+          expect(assigns(:class_groups_without_semester)).to eq [class_group]
+        end
+
+        xit('assigns @count') { expect(assigns(:count)).to eq 1 }
         it('renders :index template') { template? :index }
       end
 
@@ -32,7 +38,6 @@ describe Gaku::ClassGroupsController do
 
         it { should respond_with 200 }
         it('assigns @class_group') { expect(assigns(:class_group)).to eq class_group }
-        it('assigns @class_group_course_enrollment') { expect(assigns(:class_group_course_enrollment)).to be_a_new(Gaku::ClassGroupCourseEnrollment) }
         it('renders the :edit template') { template? :edit }
       end
 

data/frontend/spec/controllers/gaku/contacts_controller_spec.rb

@@ -18,7 +18,11 @@ describe Gaku::ContactsController do
         it { should respond_with 200 }
         it('assigns @contact') { expect(assigns(:contact)).to be_a_new(Gaku::Contact) }
         it('assigns @contact_types') { expect(assigns(:contact_types)).to eq [contact_type] }
-        it('assigns @polymorphic_resource_name') { expect(assigns(:polymorphic_resource_name)).to eq 'student-contact' }
+
+        it('assigns @polymorphic_resource_name') do
+          expect(assigns(:polymorphic_resource_name)).to eq 'student-contact'
+        end
+
         it('assigns @polymorphic_resource') { expect(assigns(:polymorphic_resource)).to eq student }
         it('assigns @nested_resources') { expect(assigns(:nested_resources)).to eq [] }
         it('renders the :new template') { template? :new }
@@ -27,7 +31,8 @@ describe Gaku::ContactsController do
       describe 'XHR POST #create' do
         context 'with valid attributes' do
           let(:valid_js_create) do
-            gaku_js_post :create, contact: attributes_for(:contact, contact_type_id: contact_type.id), student_id: student.id
+            gaku_js_post :create, contact: attributes_for(:contact, contact_type_id: contact_type.id),
+                                  student_id: student.id
           end
 
           it 'creates new contact' do
@@ -64,7 +69,8 @@ describe Gaku::ContactsController do
 
         context 'with invalid attributes' do
           let(:invalid_js_create) do
-            gaku_js_post :create, contact: attributes_for(:invalid_contact, contact_type_id: contact_type.id), student_id: student.id
+            gaku_js_post :create, contact: attributes_for(:invalid_contact, contact_type_id: contact_type.id),
+                                  student_id: student.id
           end
 
           it 'does not save the new contact' do
@@ -91,7 +97,11 @@ describe Gaku::ContactsController do
         it { should respond_with 200 }
         it('assigns @contact') { expect(assigns(:contact)).to eq contact }
         it('assigns @contact_types') { expect(assigns(:contact_types)).to eq [contact_type] }
-        it('assigns @polymorphic_resource_name') { expect(assigns(:polymorphic_resource_name)).to eq 'student-contact' }
+
+        it('assigns @polymorphic_resource_name') do
+          expect(assigns(:polymorphic_resource_name)).to eq 'student-contact'
+        end
+
         it('assigns @polymorphic_resource') { expect(assigns(:polymorphic_resource)).to eq student }
         it('assigns @nested_resources') { expect(assigns(:nested_resources)).to eq [] }
         it('renders the :edit template') { template? :edit }
@@ -100,12 +110,18 @@ describe Gaku::ContactsController do
       describe 'PATCH #update' do
         context 'with valid attributes' do
           before do
-            gaku_js_patch :update, id: contact.id, student_id: student.id, contact: attributes_for(:contact, data: 'mobifon')
+            gaku_js_patch :update, id: contact.id,
+                                   student_id: student.id,
+                                   contact: attributes_for(:contact, data: 'mobifon')
           end
 
           it { should respond_with 200 }
           it('assigns @contact') { expect(assigns(:contact)).to eq contact }
-          it('assigns @polymorphic_resource_name') { expect(assigns(:polymorphic_resource_name)).to eq 'student-contact' }
+
+          it('assigns @polymorphic_resource_name') do
+            expect(assigns(:polymorphic_resource_name)).to eq 'student-contact'
+          end
+
           it('assigns @polymorphic_resource') { expect(assigns(:polymorphic_resource)).to eq student }
           it('assigns @nested_resources') { expect(assigns(:nested_resources)).to eq [] }
           it('sets flash') { flash_updated? }
@@ -116,7 +132,9 @@ describe Gaku::ContactsController do
 
         context 'with invalid attributes' do
           before do
-            gaku_js_patch :update, id: contact.id, student_id: student.id, contact: attributes_for(:invalid_contact, data: '')
+            gaku_js_patch :update, id: contact.id,
+                                   student_id: student.id,
+                                   contact: attributes_for(:invalid_contact, data: '')
           end
 
           it { should respond_with 200 }
@@ -128,7 +146,6 @@ describe Gaku::ContactsController do
         end
       end
 
-
     end
   end
 

data/frontend/spec/controllers/gaku/course_groups/course_group_enrollments_controller.rb

@@ -4,7 +4,7 @@ describe Gaku::CourseGroups::CourseGroupEnrollmentsController do
 
   let(:course_group) { create(:course_group) }
   let(:course) { create(:course) }
-  let(:course_group_enrollment) { create(:course_group_enrollment, course_group: course_group, course: course)}
+  let(:course_group_enrollment) { create(:course_group_enrollment, course_group: course_group, course: course) }
 
   context 'as admin' do
     before { as :admin }
@@ -15,14 +15,21 @@ describe Gaku::CourseGroups::CourseGroupEnrollmentsController do
         before { gaku_js_get :new, course_group_id: course_group }
 
         it { should respond_with 200 }
-        it('assigns @course_group_enrollment') { expect(assigns(:course_group_enrollment)).to be_a_new(Gaku::CourseGroupEnrollment) }
+
+        it('assigns @course_group_enrollment') do
+          expect(assigns(:course_group_enrollment)).to be_a_new(Gaku::CourseGroupEnrollment)
+        end
+
         it('renders the :new template') { template? :new }
       end
 
       describe 'JS POST #create' do
         context 'with valid attributes' do
           let(:valid_js_create) do
-            gaku_js_post :create, course_group_id: course_group, course_group_enrollment: attributes_for(:course_group_enrollment, course_id: course.id, course_group_id: course_group.id)
+            gaku_js_post :create, course_group_id: course_group,
+                                  course_group_enrollment: attributes_for(:course_group_enrollment,
+                                                                          course_id: course.id,
+                                                                          course_group_id: course_group.id)
           end
 
           it 'creates new course_group' do
@@ -44,7 +51,8 @@ describe Gaku::CourseGroups::CourseGroupEnrollmentsController do
 
         context 'with invalid attributes' do
           let(:invalid_js_create) do
-            gaku_js_post :create, course_group_id: course_group, course_group_enrollment: attributes_for(:course_group, course_id: nil)
+            gaku_js_post :create, course_group_id: course_group,
+                                  course_group_enrollment: attributes_for(:course_group, course_id: nil)
           end
 
           xit 'does not save the new course_group' do

data/frontend/spec/controllers/gaku/course_groups/course_groups_controller_spec.rb

@@ -117,7 +117,6 @@ describe Gaku::CourseGroupsController do
         end
       end
 
-
       describe 'JS DELETE #destroy' do
         it 'deletes the course_group' do
           course_group

data/frontend/spec/controllers/gaku/exams/exam_portions_controller_spec.rb

@@ -14,10 +14,12 @@ describe Gaku::Exams::ExamPortionsController do
       describe 'PATCH #update' do
         context 'with valid attributes' do
           before do
-            gaku_patch :update, exam_id: exam, id: exam_portion, exam_portion: attributes_for(:exam_portion, name: 'mobifon')
+            gaku_js_patch :update, exam_id: exam,
+                                   id: exam_portion,
+                                   exam_portion: attributes_for(:exam_portion, name: 'mobifon')
           end
 
-          it { should respond_with 302 }
+          it { should respond_with 200 }
           it('assigns @exam_portion') { expect(assigns(:exam_portion)).to eq exam_portion }
           it('sets flash') { flash_updated? }
           it "changes exam_portion's attributes" do
@@ -28,7 +30,9 @@ describe Gaku::Exams::ExamPortionsController do
 
         context 'with invalid attributes' do
           before do
-            gaku_patch :update, id: exam_portion, exam_id: exam, exam_portion: attributes_for(:invalid_exam_portion, name: '')
+            gaku_patch :update, id: exam_portion,
+                                exam_id: exam,
+                                exam_portion: attributes_for(:invalid_exam_portion, name: '')
           end
 
           it { should respond_with 200 }
@@ -103,7 +107,7 @@ describe Gaku::Exams::ExamPortionsController do
 
           it "doesn't increment @count" do
             invalid_js_create
-            #because there are autocreated master portion
+            # because there are autocreated master portion
             expect(assigns(:count)).to eq 0
           end
         end

data/frontend/spec/controllers/gaku/exams/exam_sessions_controller_spec.rb

@@ -0,0 +1,147 @@
+require 'spec_helper_controllers'
+
+describe Gaku::Exams::ExamSessionsController do
+
+  let(:exam) { create(:exam) }
+  let(:exam_session) { create(:exam_session) }
+  let(:invalid_exam_session) { create(:invalid_exam_session) }
+
+  context 'as student' do
+    before do
+      exam_session
+      as :student
+    end
+
+    pending 'GET #edit' do
+      before { gaku_get :edit, id: exam_session }
+
+      it { should respond_with 302 }
+      it('redirects') { redirect_to? gaku.root_path }
+      it('sets unauthorized flash') { flash_unauthorized? }
+    end
+  end
+
+  context 'as admin' do
+    before { as :admin }
+
+    context 'html' do
+      describe 'XHR #edit' do
+        before { gaku_get :edit, id: exam_session }
+
+        it { should respond_with 200 }
+        it('assigns @exam_session') { expect(assigns(:exam_session)).to eq exam_session }
+        it('renders the :edit template') { template? :edit }
+      end
+
+      describe 'PATCH #update' do
+        context 'with valid attributes' do
+          before do
+            gaku_patch :update, id: exam_session, exam_session: attributes_for(:exam_session, name: 'new method')
+          end
+
+          it('assigns @exam_session') { expect(assigns(:exam_session)).to eq exam_session }
+          it('sets flash') { flash_updated? }
+          it { should respond_with 302 }
+          it('redirects') { redirect_to? gaku.edit_exam_session_path(exam_session) }
+          it "changes exam_session's attributes" do
+            exam_session.reload
+            expect(exam_session.name).to eq 'new method'
+          end
+        end
+
+        context 'with invalid attributes' do
+          before do
+            gaku_js_patch :update, id: exam_session, exam_session: attributes_for(:invalid_exam_session, name: '')
+          end
+
+          it { should respond_with 200 }
+          it('assigns @exam_session') { expect(assigns(:exam_session)).to eq exam_session }
+
+          it "does not change exam_session's attributes" do
+            exam_session.reload
+            expect(exam_session.name).not_to eq ''
+          end
+        end
+      end
+
+      describe 'XHR DELETE #destroy' do
+        it 'deletes the exam_session' do
+          exam_session
+          expect do
+            gaku_delete :destroy, id: exam_session
+          end.to change(Gaku::ExamSession, :count).by(-1)
+        end
+
+        it 'sets flash' do
+          gaku_delete :destroy, id: exam_session
+          flash_destroyed?
+        end
+
+        it 'redirect' do
+          gaku_delete :destroy, id: exam_session
+          redirect_to? gaku.exams_path
+        end
+      end
+
+    end
+
+    context 'js' do
+
+      describe 'XHR #new' do
+        before { gaku_js_get :new }
+
+        it { should respond_with 200 }
+        it('assigns @exam_session') { expect(assigns(:exam_session)).to be_a_new(Gaku::ExamSession) }
+        it('renders the :new template') { template? :new }
+      end
+
+      describe 'POST #create' do
+        context 'with valid attributes' do
+          let(:valid_js_create) do
+            exam
+            gaku_js_post :create, exam_session: build(:exam_session, exam_id: exam.id).attributes.symbolize_keys
+          end
+
+          it 'creates new exam_session' do
+            expect do
+              valid_js_create
+            end.to change(Gaku::ExamSession, :count).by(1)
+          end
+
+          it 'renders flash' do
+            valid_js_create
+            flash_created?
+          end
+
+          it 'increments @count' do
+            valid_js_create
+            expect(assigns(:count)).to eq 1
+          end
+        end
+
+        context 'with invalid attributes' do
+          let(:invalid_js_create) do
+            gaku_js_post :create, exam_session: attributes_for(:invalid_exam_session)
+          end
+
+          it 'does not save the new exam_session' do
+            expect do
+              invalid_js_create
+            end.to_not change(Gaku::ExamSession, :count)
+          end
+
+          it 're-renders the new method' do
+            invalid_js_create
+            template? :create
+          end
+
+          it "doesn't increment @count" do
+            invalid_js_create
+            expect(assigns(:count)).to eq 0
+          end
+        end
+      end
+    end
+
+  end
+end

data/frontend/spec/controllers/gaku/exams/exams_controller_spec.rb

@@ -22,7 +22,6 @@ describe Gaku::ExamsController do
         it('renders :index template') { template? :index }
       end
 
-
       describe 'PATCH #update' do
         context 'with valid attributes' do
           before do
@@ -147,7 +146,6 @@ describe Gaku::ExamsController do
         end
       end
 
-
     end
   end
 

data/frontend/spec/controllers/gaku/extracurricular_activities_controller_spec.rb

@@ -16,63 +16,43 @@ describe Gaku::ExtracurricularActivitiesController do
         end
 
         it { should respond_with 200 }
-        it('assigns @extracurricular_activitys') { expect(assigns(:extracurricular_activities)).to eq [extracurricular_activity] }
+
+        it('assigns @extracurricular_activitys') do
+          expect(assigns(:extracurricular_activities)).to eq [extracurricular_activity]
+        end
+
         it('assigns @count') { expect(assigns(:count)).to eq 1 }
         it('renders :index template') { template? :index }
       end
 
-
       describe 'GET #edit' do
         before { gaku_get :edit, id: extracurricular_activity }
 
         it { should respond_with 200 }
-        it('assigns @extracurricular_activity') { expect(assigns(:extracurricular_activity)).to eq extracurricular_activity }
-        it('renders the :edit template') { template? :edit }
-      end
-
-      describe 'PATCH #update' do
-        context 'with valid attributes' do
-          before do
-            gaku_patch :update, id: extracurricular_activity, extracurricular_activity: attributes_for(:extracurricular_activity, name: 'mobifon')
-          end
 
-          it { should respond_with 302 }
-          it('assigns @extracurricular_activity') { expect(assigns(:extracurricular_activity)).to eq extracurricular_activity }
-          it('sets flash') { flash_updated? }
-          it "changes extracurricular_activity's attributes" do
-            extracurricular_activity.reload
-            expect(extracurricular_activity.name).to eq 'mobifon'
-          end
+        it('assigns @extracurricular_activity') do
+          expect(assigns(:extracurricular_activity)).to eq extracurricular_activity
         end
 
-        context 'with invalid attributes' do
-          before do
-            gaku_patch :update, id: extracurricular_activity, extracurricular_activity: attributes_for(:invalid_extracurricular_activity, name: '')
-          end
-
-          it { should respond_with 200 }
-          it('assigns @extracurricular_activity') { expect(assigns(:extracurricular_activity)).to eq extracurricular_activity }
-
-          it "does not change extracurricular_activity's attributes" do
-            extracurricular_activity.reload
-            expect(extracurricular_activity.name).not_to eq ''
-          end
-        end
+        it('renders the :edit template') { template? :edit }
       end
-
     end
 
     context 'js' do
 
-      describe 'XHR #new' do
+      describe 'XHR GET #new' do
         before { gaku_js_get :new }
 
         it { should respond_with 200 }
-        it('assigns @extracurricular_activity') { expect(assigns(:extracurricular_activity)).to be_a_new(Gaku::ExtracurricularActivity) }
+
+        it('assigns @extracurricular_activity') do
+          expect(assigns(:extracurricular_activity)).to be_a_new(Gaku::ExtracurricularActivity)
+        end
+
         it('renders the :new template') { template? :new }
       end
 
-      describe 'POST #create' do
+      describe 'XHR POST #create' do
         context 'with valid attributes' do
           let(:valid_js_create) do
             gaku_js_post :create, extracurricular_activity: attributes_for(:extracurricular_activity)
@@ -118,6 +98,57 @@ describe Gaku::ExtracurricularActivitiesController do
         end
       end
 
+      describe 'XHR GET #edit' do
+        before { gaku_js_get :edit, id: extracurricular_activity }
+
+        it { should respond_with 200 }
+
+        it('assigns @extracurricular_activity') do
+          expect(assigns(:extracurricular_activity)).to eq extracurricular_activity
+        end
+
+        it('renders the :edit template') { template? :edit }
+      end
+
+      describe 'XHR PATCH #update' do
+        context 'with valid attributes' do
+          before do
+            gaku_js_patch :update, id: extracurricular_activity,
+                                extracurricular_activity: attributes_for(:extracurricular_activity, name: 'mobifon')
+          end
+
+          it { should respond_with 200 }
+
+          it('assigns @extracurricular_activity') do
+            expect(assigns(:extracurricular_activity)).to eq extracurricular_activity
+          end
+
+          it('sets flash') { flash_updated? }
+          it "changes extracurricular_activity's attributes" do
+            extracurricular_activity.reload
+            expect(extracurricular_activity.name).to eq 'mobifon'
+          end
+        end
+
+        context 'with invalid attributes' do
+          before do
+            gaku_js_patch :update, id: extracurricular_activity,
+                                extracurricular_activity: attributes_for(:invalid_extracurricular_activity, name: '')
+          end
+
+          it { should respond_with 200 }
+
+          it('assigns @extracurricular_activity') do
+            expect(assigns(:extracurricular_activity)).to eq extracurricular_activity
+          end
+
+          it "does not change extracurricular_activity's attributes" do
+            extracurricular_activity.reload
+            expect(extracurricular_activity.name).not_to eq ''
+          end
+        end
+      end
+
       describe 'XHR DELETE #destroy' do
         it 'deletes the extracurricular_activity' do
           extracurricular_activity