diff --git a/spec/controllers/repositories_controller_spec.rb b/spec/controllers/repositories_controller_spec.rb new file mode 100644 index 000000000..4c526adfc --- /dev/null +++ b/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 + diff --git a/spec/factories/repository_factory.rb b/spec/factories/repository_factory.rb index a730f7f4b..4590b321b 100644 --- a/spec/factories/repository_factory.rb +++ b/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 diff --git a/spec/shared_examples/repositories_controller.rb b/spec/shared_examples/repositories_controller.rb new file mode 100644 index 000000000..78d6cf3fa --- /dev/null +++ b/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