435 lines
16 KiB
Ruby
435 lines
16 KiB
Ruby
# -*- encoding : utf-8 -*-
|
|
require 'spec_helper'
|
|
|
|
describe Projects::BuildListsController do
|
|
|
|
shared_examples_for 'show build list' do
|
|
it 'should be able to perform show action' do
|
|
get :show, @show_params
|
|
response.should be_success
|
|
end
|
|
|
|
it 'should be able to perform index action in project scope' do
|
|
get :index, :owner_name => @project.owner.uname, :project_name => @project.name
|
|
response.should be_success
|
|
end
|
|
end
|
|
|
|
shared_examples_for 'not show build list' do
|
|
it 'should not be able to perform show action' do
|
|
get :show, @show_params
|
|
response.should redirect_to(forbidden_url)
|
|
end
|
|
|
|
it 'should not be able to perform index action in project scope' do
|
|
get :index, :owner_name => @project.owner.uname, :project_name => @project.name
|
|
response.should redirect_to(forbidden_url)
|
|
end
|
|
end
|
|
|
|
shared_examples_for 'create build list' do
|
|
before {
|
|
@project.update_attribute(:repositories, @platform.repositories)
|
|
}
|
|
|
|
it 'should be able to perform new action' do
|
|
get :new, :owner_name => @project.owner.uname, :project_name => @project.name
|
|
response.should render_template(:new)
|
|
end
|
|
|
|
it 'should be able to perform create action' do
|
|
post :create, {:owner_name => @project.owner.uname, :project_name => @project.name}.merge(@create_params)
|
|
response.should redirect_to project_build_lists_path(@project)
|
|
end
|
|
|
|
it 'should save correct commit_hash for branch based build' do
|
|
post :create, {:owner_name => @project.owner.uname, :project_name => @project.name}.merge(@create_params).deep_merge(:build_list => {:project_version => "master"})
|
|
@project.build_lists.last.commit_hash.should == @project.repo.commits('master').first.id
|
|
end
|
|
|
|
it 'should save correct commit_hash for tag based build' do
|
|
system("cd #{@project.repo.path} && git tag 4.7.5.3") # TODO REDO through grit
|
|
post :create, {:owner_name => @project.owner.uname, :project_name => @project.name}.merge(@create_params).deep_merge(:build_list => {:project_version => "4.7.5.3"})
|
|
@project.build_lists.last.commit_hash.should == @project.repo.commits('4.7.5.3').first.id
|
|
end
|
|
|
|
it 'should not be able to create with wrong project version' do
|
|
lambda{ post :create, {:owner_name => @project.owner.uname, :project_name => @project.name}.merge(@create_params).deep_merge(:build_list => {:project_version => "wrong", :commit_hash => nil})}.should change{@project.build_lists.count}.by(0)
|
|
end
|
|
|
|
it 'should not be able to create with wrong git hash' do
|
|
lambda{ post :create, {:owner_name => @project.owner.uname, :project_name => @project.name}.merge(@create_params).deep_merge(:build_list => {:commit_hash => 'wrong'})}.should change{@project.build_lists.count}.by(0)
|
|
end
|
|
end
|
|
|
|
shared_examples_for 'not create build list' do |skip_new = false|
|
|
before {
|
|
@project.update_attribute(:repositories, @platform.repositories)
|
|
}
|
|
|
|
it 'should not be able to perform new action' do
|
|
get :new, :owner_name => @project.owner.uname, :project_name => @project.name
|
|
response.should redirect_to(forbidden_url)
|
|
end unless skip_new
|
|
|
|
it 'should not be able to perform create action' do
|
|
post :create, {:owner_name => @project.owner.uname, :project_name => @project.name}.merge(@create_params)
|
|
response.should redirect_to(forbidden_url)
|
|
end
|
|
end
|
|
|
|
before { stub_symlink_methods }
|
|
|
|
context 'crud' do
|
|
before(:each) do
|
|
@platform = FactoryGirl.create(:platform_with_repos)
|
|
@create_params = {
|
|
:build_list => {
|
|
:project_version => 'master',
|
|
:save_to_repository_id => @platform.repositories.first.id,
|
|
:update_type => 'security',
|
|
:include_repos => [@platform.repositories.first.id]
|
|
},
|
|
:arches => [FactoryGirl.create(:arch).id],
|
|
:build_for_platforms => [@platform.id]
|
|
}
|
|
any_instance_of(Project, :versions => ['v1.0', 'v2.0'])
|
|
stub_redis
|
|
end
|
|
|
|
context 'for guest' do
|
|
it 'should be able to perform index action', :anonymous_access => true do
|
|
get :index
|
|
response.should be_success
|
|
end
|
|
|
|
it 'should not be able to perform index action', :anonymous_access => false do
|
|
get :index
|
|
response.should redirect_to(new_user_session_path)
|
|
end
|
|
|
|
end
|
|
|
|
context 'for user' do
|
|
before(:each) do
|
|
any_instance_of(BuildList, :current_duration => 100)
|
|
@build_list = FactoryGirl.create(:build_list)
|
|
@project = @build_list.project
|
|
@owner_user = @project.owner
|
|
@member_user = FactoryGirl.create(:user)
|
|
rel = @project.relations.build(:role => 'reader')
|
|
rel.actor = @member_user
|
|
rel.save
|
|
@user = FactoryGirl.create(:user)
|
|
set_session_for(@user)
|
|
@show_params = {:owner_name => @project.owner.uname, :project_name => @project.name, :id => @build_list.id}
|
|
@build_list.save_to_repository.update_column(:publish_without_qa, false)
|
|
@request.env['HTTP_REFERER'] = build_list_path(@build_list)
|
|
end
|
|
|
|
context "do reject_publish" do
|
|
before(:each) {@build_list.save_to_repository.update_column(:publish_without_qa, true)}
|
|
|
|
def do_reject_publish
|
|
put :reject_publish, :id => @build_list
|
|
end
|
|
|
|
context 'if user is project owner' do
|
|
before(:each) do
|
|
set_session_for(@owner_user)
|
|
@build_list.update_column(:status, BuildList::SUCCESS)
|
|
@build_list.save_to_platform.update_column(:released, true)
|
|
do_reject_publish
|
|
end
|
|
|
|
context "if it has :success status" do
|
|
it 'should return 302 response code' do
|
|
response.status.should == 302
|
|
end
|
|
|
|
it "should reject publish build list" do
|
|
@build_list.reload.status.should == BuildList::REJECTED_PUBLISH
|
|
end
|
|
end
|
|
|
|
context "if it has another status" do
|
|
before(:each) do
|
|
@build_list.update_column(:status, BuildList::BUILD_ERROR)
|
|
do_reject_publish
|
|
end
|
|
|
|
it "should not change status of build list" do
|
|
@build_list.reload.status.should == BuildList::BUILD_ERROR
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'if user is not project owner' do
|
|
before(:each) do
|
|
@build_list.update_column(:status, BuildList::SUCCESS)
|
|
@build_list.save_to_platform.update_column(:released, true)
|
|
do_reject_publish
|
|
end
|
|
|
|
it "should redirect to forbidden page" do
|
|
response.should redirect_to(forbidden_url)
|
|
end
|
|
|
|
it "should not change status of build list" do
|
|
do_reject_publish
|
|
@build_list.reload.status.should == BuildList::SUCCESS
|
|
end
|
|
end
|
|
|
|
context 'if user is project reader' do
|
|
before(:each) do
|
|
@another_user = FactoryGirl.create(:user)
|
|
@build_list.update_column(:status, BuildList::SUCCESS)
|
|
@build_list.save_to_repository.update_column(:publish_without_qa, true)
|
|
@build_list.project.collaborators.create(:actor_type => 'User', :actor_id => @another_user.id, :role => 'reader')
|
|
set_session_for(@another_user)
|
|
do_reject_publish
|
|
end
|
|
|
|
it "should redirect to forbidden page" do
|
|
response.should redirect_to(forbidden_url)
|
|
end
|
|
|
|
it "should not change status of build list" do
|
|
do_reject_publish
|
|
@build_list.reload.status.should == BuildList::SUCCESS
|
|
end
|
|
end
|
|
|
|
context 'if user is project writer' do
|
|
before(:each) do
|
|
@writer_user = FactoryGirl.create(:user)
|
|
@build_list.update_column(:status, BuildList::SUCCESS)
|
|
@build_list.save_to_repository.update_column(:publish_without_qa, true)
|
|
@build_list.project.relations.create!(:actor_type => 'User', :actor_id => @writer_user.id, :role => 'writer')
|
|
set_session_for(@writer_user)
|
|
do_reject_publish
|
|
end
|
|
|
|
it 'should return 302 response code' do
|
|
response.status.should == 302
|
|
end
|
|
|
|
it "should reject publish build list" do
|
|
@build_list.reload.status.should == BuildList::REJECTED_PUBLISH
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'for all build lists' do
|
|
before(:each) do
|
|
@build_list1 = FactoryGirl.create(:build_list)
|
|
|
|
@build_list2 = FactoryGirl.create(:build_list)
|
|
@build_list2.project.update_column(:visibility, 'hidden')
|
|
|
|
project = FactoryGirl.create(:project_with_commit, :visibility => 'hidden', :owner => @user)
|
|
@build_list3 = FactoryGirl.create(:build_list_with_attaching_project, :project => project)
|
|
|
|
@build_list4 = FactoryGirl.create(:build_list)
|
|
@build_list4.project.update_column(:visibility, 'hidden')
|
|
@build_list4.project.relations.create! :role => 'reader', :actor_id => @user.id, :actor_type => 'User'
|
|
end
|
|
|
|
it 'should be able to perform index action' do
|
|
get :index
|
|
response.should be_success
|
|
end
|
|
|
|
it 'should show only accessible build_lists' do
|
|
get :index, :filter => {:ownership => 'everything'}
|
|
assigns(:build_lists).should include(@build_list1)
|
|
assigns(:build_lists).should_not include(@build_list2)
|
|
assigns(:build_lists).should include(@build_list3)
|
|
assigns(:build_lists).should include(@build_list4)
|
|
end
|
|
end
|
|
|
|
context 'for open project' do
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'not create build list'
|
|
|
|
context 'if user is project owner' do
|
|
before(:each) {set_session_for(@owner_user)}
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'create build list'
|
|
|
|
context 'no ability to read build_for_platform' do
|
|
before do
|
|
repository = FactoryGirl.create(:repository)
|
|
repository.platform.change_visibility
|
|
Platform.where(:id => @platform.id).update_all(:platform_type => 'personal')
|
|
@create_params[:build_list].merge!({:include_repos => [repository.id]})
|
|
@create_params[:build_for_platforms] = [repository.platform_id]
|
|
end
|
|
it_should_behave_like 'not create build list', true
|
|
end
|
|
|
|
end
|
|
|
|
context 'if user is project read member' do
|
|
before(:each) {set_session_for(@member_user)}
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'not create build list'
|
|
end
|
|
end
|
|
|
|
context 'for hidden project' do
|
|
before(:each) do
|
|
@project.visibility = 'hidden'
|
|
@project.save
|
|
end
|
|
|
|
it_should_behave_like 'not show build list'
|
|
it_should_behave_like 'not create build list'
|
|
|
|
context 'if user is project owner' do
|
|
before(:each) {set_session_for(@owner_user)}
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'create build list'
|
|
end
|
|
|
|
context 'if user is project read member' do
|
|
before(:each) {set_session_for(@member_user)}
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'not create build list'
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'for group' do
|
|
before(:each) do
|
|
|
|
@user = FactoryGirl.create(:user)
|
|
set_session_for(@user)
|
|
|
|
@build_list = FactoryGirl.create(:build_list_by_group_project)
|
|
@project = @build_list.project
|
|
@owner_group = @build_list.project.owner
|
|
@owner_user = @owner_group.owner
|
|
|
|
@member_group = FactoryGirl.create(:group)
|
|
@member_user = FactoryGirl.create(:user)
|
|
@member_group.actors.create :role => 'reader', :actor_id => @member_user.id, :actor_type => 'User'
|
|
@project.relations.create :role => 'reader', :actor_id => @member_group.id, :actor_type => 'Group'
|
|
|
|
@show_params = {:owner_name => @project.owner.uname, :project_name => @project.name, :id => @build_list.id}
|
|
end
|
|
|
|
context 'for all build lists' do
|
|
before(:each) do
|
|
@build_list1 = FactoryGirl.create(:build_list)
|
|
|
|
@build_list2 = FactoryGirl.create(:build_list)
|
|
@build_list2.project.update_column(:visibility, 'hidden')
|
|
|
|
project = FactoryGirl.create(:project_with_commit, :visibility => 'hidden', :owner => @user)
|
|
@build_list3 = FactoryGirl.create(:build_list_with_attaching_project, :project => project)
|
|
|
|
@build_list4 = FactoryGirl.create(:build_list)
|
|
@build_list4.project.update_column(:visibility, 'hidden')
|
|
@build_list4.project.relations.create! :role => 'reader', :actor_id => @user.id, :actor_type => 'User'
|
|
end
|
|
|
|
it 'should be able to perform index action' do
|
|
get :index
|
|
response.should be_success
|
|
end
|
|
|
|
it 'should show only accessible build_lists' do
|
|
get :index, :filter => {:ownership => 'everything'}
|
|
assigns(:build_lists).should include(@build_list1)
|
|
assigns(:build_lists).should_not include(@build_list2)
|
|
assigns(:build_lists).should include(@build_list3)
|
|
assigns(:build_lists).should include(@build_list4)
|
|
end
|
|
end
|
|
|
|
context 'for open project' do
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'not create build list'
|
|
|
|
context 'if user is group owner' do
|
|
before(:each) {set_session_for(@owner_user)}
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'create build list'
|
|
end
|
|
|
|
context 'if user is group read member' do
|
|
before(:each) {set_session_for(@member_user)}
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'not create build list'
|
|
end
|
|
end
|
|
|
|
context 'for hidden project' do
|
|
before(:each) do
|
|
@project.visibility = 'hidden'
|
|
@project.save
|
|
end
|
|
|
|
it_should_behave_like 'not show build list'
|
|
it_should_behave_like 'not create build list'
|
|
|
|
context 'if user is group owner' do
|
|
before(:each) {set_session_for(@owner_user)}
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'create build list'
|
|
end
|
|
|
|
context 'if user is group read member' do
|
|
before(:each) {set_session_for(@member_user)}
|
|
it_should_behave_like 'show build list'
|
|
it_should_behave_like 'not create build list'
|
|
end
|
|
end
|
|
|
|
end
|
|
end
|
|
|
|
context 'filter' do
|
|
|
|
before(:each) do
|
|
set_session_for FactoryGirl.create(:admin)
|
|
|
|
@build_list1 = FactoryGirl.create(:build_list)
|
|
@build_list2 = FactoryGirl.create(:build_list)
|
|
@build_list3 = FactoryGirl.create(:build_list)
|
|
@build_list4 = FactoryGirl.create(:build_list, :updated_at => (Time.now - 1.day),
|
|
:project => @build_list3.project, :save_to_platform => @build_list3.save_to_platform,
|
|
:arch => @build_list3.arch)
|
|
end
|
|
|
|
it 'should filter by id' do
|
|
get :index, :filter => {:id => @build_list1.id, :project_name => 'fdsfdf', :any_other_field => 'do not matter'}
|
|
assigns[:build_lists].should include(@build_list1)
|
|
assigns[:build_lists].should_not include(@build_list2)
|
|
assigns[:build_lists].should_not include(@build_list3)
|
|
end
|
|
|
|
it 'should filter by project_name' do
|
|
# Project.where(:id => build_list2.project.id).update_all(:name => 'project_name')
|
|
get :index, :filter => {:project_name => @build_list2.project.name, :ownership => 'everything'}
|
|
assigns[:build_lists].should_not include(@build_list1)
|
|
assigns[:build_lists].should include(@build_list2)
|
|
assigns[:build_lists].should_not include(@build_list3)
|
|
end
|
|
|
|
it 'should filter by project_name and update_date' do
|
|
get :index, :filter => {:project_name => @build_list3.project.name, :ownership => 'everything',
|
|
"updated_at_start" => @build_list3.updated_at.strftime('%d/%m/%Y')}
|
|
assigns[:build_lists].should_not include(@build_list1)
|
|
assigns[:build_lists].should_not include(@build_list2)
|
|
assigns[:build_lists].should include(@build_list3)
|
|
assigns[:build_lists].should_not include(@build_list4)
|
|
end
|
|
end
|
|
|
|
after(:all) {clean_projects_dir}
|
|
end
|