[refs #2249] Add repositories specs. Fixes perosonal repo factory

spec/controllers/repositories_controller_spec.rb

@@ -0,0 +1,133 @@
+require 'spec_helper'
+require 'shared_examples/repositories_controller'
+
+describe RepositoriesController do
+	before(:each) do
+    @repository = Factory(:repository)
+    @personal_repository = Factory(:personal_repository)
+    @platform = Factory(:platform)
+    @project = Factory(:project)
+    @another_user = Factory(:user)
+    @create_params = {:repository => {:name => 'pro', :description => 'pro2'}, :platform_id => @platform.id}
+	end
+
+	context 'for guest' do
+    [:index, :create].each do |action|
+      it "should not be able to perform #{ action } action" do
+        get action
+        response.should redirect_to(new_user_session_path)
+      end
+    end
+
+    [:show, :new, :add_project, :remove_project, :destroy].each do |action|
+      it "should not be able to perform #{ action } action" do
+        get action, :id => @repository.id
+        response.should redirect_to(new_user_session_path)
+      end
+    end
+  end
+
+  context 'for admin' do
+  	before(:each) do
+  		@admin = Factory(:admin)
+  		set_session_for(@admin)
+		end
+
+    it_should_behave_like 'be_able_to_perform_index_action'
+    it_should_behave_like 'be_able_to_perform_show_action'
+
+    it 'should be able to perform new action' do
+      get :new
+      response.should render_template(:new)
+    end
+
+    it 'should be able to perform create action' do
+      post :create, @create_params
+      response.should redirect_to(platform_repositories_path(@platform.id))
+    end
+
+    it 'should change objects count after create action' do
+      lambda { post :create, @create_params }.should change{ Repository.count }.by(1)
+    end
+
+    it_should_behave_like 'be_able_to_perform_destroy_action'
+    it_should_behave_like 'change_repositories_count_after_destroy'
+    it_should_behave_like 'be_able_to_perform_add_project_action'
+    it_should_behave_like 'be_able_to_perform_add_project_action_with_project_id_param'
+    it_should_behave_like 'add_project_to_repository'
+    it_should_behave_like 'be_able_to_perform_remove_project'
+    it_should_behave_like 'remove_project_from_repository'
+    it_should_behave_like 'not_be_able_to_destroy_personal_repository'
+  end
+
+  context 'for anyone except admin' do
+  	before(:each) do
+  		@user = Factory(:user)
+  		set_session_for(@user)
+		end
+
+    it 'should not be able to perform new action' do
+      get :new
+      response.should redirect_to(forbidden_path)
+    end
+
+    it 'should not be able to perform create action' do
+      post :create, @create_params
+      response.should redirect_to(forbidden_path)
+    end
+
+    it 'should not change objects count after create action' do
+      lambda { post :create, @create_params }.should change{ Repository.count }.by(0)
+    end
+
+    it_should_behave_like 'not_be_able_to_destroy_personal_repository'
+  end
+
+  context 'for owner user' do
+  	before(:each) do
+  		@user = Factory(:user)
+  		set_session_for(@user)
+  		@repository.update_attribute(:owner, @user)
+  		r = @repository.relations.build(:object_type => 'User', :object_id => @user.id, :role => 'admin')
+  		r.save!
+		end
+
+    it_should_behave_like 'be_able_to_perform_index_action'
+    it_should_behave_like 'be_able_to_perform_show_action'
+    it_should_behave_like 'be_able_to_perform_add_project_action'
+    it_should_behave_like 'be_able_to_perform_add_project_action_with_project_id_param'
+    it_should_behave_like 'add_project_to_repository'
+    it_should_behave_like 'be_able_to_perform_remove_project'
+    it_should_behave_like 'remove_project_from_repository'
+    it_should_behave_like 'be_able_to_perform_destroy_action'
+    it_should_behave_like 'change_repositories_count_after_destroy'
+  end
+
+  context 'for reader user' do
+  	before(:each) do
+  		@user = Factory(:user)
+  		set_session_for(@user)
+  		r = @repository.relations.build(:object_type => 'User', :object_id => @user.id, :role => 'reader')
+  		r.save!
+		end
+
+    it_should_behave_like 'be_able_to_perform_index_action'
+    it_should_behave_like 'be_able_to_perform_show_action'
+
+    it 'should not be able to perform add_project action' do
+      get :add_project, :id => @repository.id
+      response.should redirect_to(forbidden_path)
+    end
+
+    it 'should not be able to perform add_project action with project_id param' do
+      get :add_project, :id => @repository.id, :project_id => @project.id
+      response.should redirect_to(forbidden_path)
+    end
+
+    it 'should not be able to perform destroy action' do
+      delete :destroy, :id => @repository.id
+      response.should redirect_to(forbidden_path)
+    end
+  end
+end
+

spec/factories/repository_factory.rb

@@ -13,5 +13,6 @@ Factory.define(:personal_repository, :class => Repository) do |p|
 
   p.after_create { |rep| 
   	rep.platform.platform_type = 'personal'
+    rep.platform.save!
   }
 end

spec/shared_examples/repositories_controller.rb

@@ -0,0 +1,68 @@
+shared_examples_for 'be_able_to_perform_index_action' do
+  it 'should be able to perform index action' do
+    get :index
+    response.should render_template(:index)
+  end
+end
+
+shared_examples_for 'be_able_to_perform_show_action' do
+  it 'should be able to perform show action' do
+    get :show, :id => @repository.id
+    response.should render_template(:show)
+  end
+end
+
+shared_examples_for 'be_able_to_perform_add_project_action' do
+  it 'should be able to perform add_project action' do
+    get :add_project, :id => @repository.id
+    response.should render_template(:projects_list)
+  end
+end
+
+shared_examples_for 'be_able_to_perform_add_project_action_with_project_id_param' do
+  it 'should be able to perform add_project action with project_id param' do
+    get :add_project, :id => @repository.id, :project_id => @project.id
+    response.should redirect_to(repository_path(@repository))
+  end
+end
+
+shared_examples_for 'add_project_to_repository' do
+  it 'should be able to add project to repository' do
+    get :add_project, :id => @repository.id, :project_id => @project.id
+    @repository.projects.exists? :id => @project.id
+  end
+end
+
+shared_examples_for 'be_able_to_perform_remove_project' do
+  it 'should be able to perform remove_project action' do
+    get :remove_project, :id => @repository.id, :project_id => @project.id
+    response.should redirect_to(repository_path(@repository))
+  end
+end
+
+shared_examples_for 'remove_project_from_repository' do
+  it 'should be able to remove project from repository' do
+    get :remove_project, :id => @repository.id, :project_id => @project.id
+    !@repository.projects.exists? :id => @project.id
+  end
+end
+
+shared_examples_for 'be_able_to_perform_destroy_action' do
+  it 'should be able to perform destroy action' do
+    delete :destroy, :id => @repository.id
+    response.should redirect_to(platform_path(@repository.platform.id))
+  end
+end
+
+shared_examples_for 'change_repositories_count_after_destroy' do
+  it 'should change objects count after destroy action' do
+    lambda { delete :destroy, :id => @repository.id }.should change{ Repository.count }.by(-1)
+  end
+end
+
+shared_examples_for 'not_be_able_to_destroy_personal_repository' do
+  it 'should not be able to destroy personal repository' do
+    delete :destroy, :id => @personal_repository.id
+    response.should redirect_to(forbidden_path)
+  end
+end