Commit 6d4b9340596fdcbffa8f1087d638db84ed9e613d
Exists in
spb-stable
and in
3 other branches
Merge pull request #6596 from tsigo/rs-project-service-specs
Move Project's service specs to the correct location
Showing
6 changed files
with
307 additions
and
307 deletions
Show diff stats
spec/services/project_transfer_service_spec.rb
| ... | ... | @@ -1,33 +0,0 @@ |
| 1 | -require 'spec_helper' | |
| 2 | - | |
| 3 | -describe ProjectTransferService do | |
| 4 | - before(:each) { enable_observers } | |
| 5 | - after(:each) {disable_observers} | |
| 6 | - | |
| 7 | - context 'namespace -> namespace' do | |
| 8 | - let(:user) { create(:user) } | |
| 9 | - let(:group) { create(:group) } | |
| 10 | - let(:project) { create(:project, namespace: user.namespace) } | |
| 11 | - | |
| 12 | - before do | |
| 13 | - @result = service.transfer(project, group) | |
| 14 | - end | |
| 15 | - | |
| 16 | - it { @result.should be_true } | |
| 17 | - it { project.namespace.should == group } | |
| 18 | - end | |
| 19 | - | |
| 20 | - context 'namespace -> no namespace' do | |
| 21 | - let(:user) { create(:user) } | |
| 22 | - let(:project) { create(:project, namespace: user.namespace) } | |
| 23 | - | |
| 24 | - it { lambda{service.transfer(project, nil)}.should raise_error(ActiveRecord::RecordInvalid) } | |
| 25 | - end | |
| 26 | - | |
| 27 | - def service | |
| 28 | - service = ProjectTransferService.new | |
| 29 | - service.gitlab_shell.stub(mv_repository: true) | |
| 30 | - service | |
| 31 | - end | |
| 32 | -end | |
| 33 | - |
| ... | ... | @@ -0,0 +1,163 @@ |
| 1 | +require 'spec_helper' | |
| 2 | + | |
| 3 | +describe Projects::CreateService do | |
| 4 | + before(:each) { ActiveRecord::Base.observers.enable(:user_observer) } | |
| 5 | + after(:each) { ActiveRecord::Base.observers.disable(:user_observer) } | |
| 6 | + | |
| 7 | + describe :create_by_user do | |
| 8 | + before do | |
| 9 | + @user = create :user | |
| 10 | + @admin = create :user, admin: true | |
| 11 | + @opts = { | |
| 12 | + name: "GitLab", | |
| 13 | + namespace: @user.namespace | |
| 14 | + } | |
| 15 | + end | |
| 16 | + | |
| 17 | + context 'user namespace' do | |
| 18 | + before do | |
| 19 | + @project = create_project(@user, @opts) | |
| 20 | + end | |
| 21 | + | |
| 22 | + it { @project.should be_valid } | |
| 23 | + it { @project.owner.should == @user } | |
| 24 | + it { @project.namespace.should == @user.namespace } | |
| 25 | + end | |
| 26 | + | |
| 27 | + context 'group namespace' do | |
| 28 | + before do | |
| 29 | + @group = create :group | |
| 30 | + @group.add_owner(@user) | |
| 31 | + | |
| 32 | + @opts.merge!(namespace_id: @group.id) | |
| 33 | + @project = create_project(@user, @opts) | |
| 34 | + end | |
| 35 | + | |
| 36 | + it { @project.should be_valid } | |
| 37 | + it { @project.owner.should == @group } | |
| 38 | + it { @project.namespace.should == @group } | |
| 39 | + end | |
| 40 | + | |
| 41 | + context 'wiki_enabled creates repository directory' do | |
| 42 | + context 'wiki_enabled true creates wiki repository directory' do | |
| 43 | + before do | |
| 44 | + @project = create_project(@user, @opts) | |
| 45 | + @path = GollumWiki.new(@project, @user).send(:path_to_repo) | |
| 46 | + end | |
| 47 | + | |
| 48 | + it { File.exists?(@path).should be_true } | |
| 49 | + end | |
| 50 | + | |
| 51 | + context 'wiki_enabled false does not create wiki repository directory' do | |
| 52 | + before do | |
| 53 | + @opts.merge!(wiki_enabled: false) | |
| 54 | + @project = create_project(@user, @opts) | |
| 55 | + @path = GollumWiki.new(@project, @user).send(:path_to_repo) | |
| 56 | + end | |
| 57 | + | |
| 58 | + it { File.exists?(@path).should be_false } | |
| 59 | + end | |
| 60 | + end | |
| 61 | + | |
| 62 | + context 'respect configured visibility setting' do | |
| 63 | + before(:each) do | |
| 64 | + @settings = double("settings") | |
| 65 | + @settings.stub(:issues) { true } | |
| 66 | + @settings.stub(:merge_requests) { true } | |
| 67 | + @settings.stub(:wiki) { true } | |
| 68 | + @settings.stub(:wall) { true } | |
| 69 | + @settings.stub(:snippets) { true } | |
| 70 | + stub_const("Settings", Class.new) | |
| 71 | + @restrictions = double("restrictions") | |
| 72 | + @restrictions.stub(:restricted_visibility_levels) { [] } | |
| 73 | + Settings.stub_chain(:gitlab).and_return(@restrictions) | |
| 74 | + Settings.stub_chain(:gitlab, :default_projects_features).and_return(@settings) | |
| 75 | + end | |
| 76 | + | |
| 77 | + context 'should be public when setting is public' do | |
| 78 | + before do | |
| 79 | + @settings.stub(:visibility_level) { Gitlab::VisibilityLevel::PUBLIC } | |
| 80 | + @project = create_project(@user, @opts) | |
| 81 | + end | |
| 82 | + | |
| 83 | + it { @project.public?.should be_true } | |
| 84 | + end | |
| 85 | + | |
| 86 | + context 'should be private when setting is private' do | |
| 87 | + before do | |
| 88 | + @settings.stub(:visibility_level) { Gitlab::VisibilityLevel::PRIVATE } | |
| 89 | + @project = create_project(@user, @opts) | |
| 90 | + end | |
| 91 | + | |
| 92 | + it { @project.private?.should be_true } | |
| 93 | + end | |
| 94 | + | |
| 95 | + context 'should be internal when setting is internal' do | |
| 96 | + before do | |
| 97 | + @settings.stub(:visibility_level) { Gitlab::VisibilityLevel::INTERNAL } | |
| 98 | + @project = create_project(@user, @opts) | |
| 99 | + end | |
| 100 | + | |
| 101 | + it { @project.internal?.should be_true } | |
| 102 | + end | |
| 103 | + end | |
| 104 | + | |
| 105 | + context 'respect configured visibility restrictions setting' do | |
| 106 | + before(:each) do | |
| 107 | + @settings = double("settings") | |
| 108 | + @settings.stub(:issues) { true } | |
| 109 | + @settings.stub(:merge_requests) { true } | |
| 110 | + @settings.stub(:wiki) { true } | |
| 111 | + @settings.stub(:wall) { true } | |
| 112 | + @settings.stub(:snippets) { true } | |
| 113 | + @settings.stub(:visibility_level) { Gitlab::VisibilityLevel::PRIVATE } | |
| 114 | + stub_const("Settings", Class.new) | |
| 115 | + @restrictions = double("restrictions") | |
| 116 | + @restrictions.stub(:restricted_visibility_levels) { [ Gitlab::VisibilityLevel::PUBLIC ] } | |
| 117 | + Settings.stub_chain(:gitlab).and_return(@restrictions) | |
| 118 | + Settings.stub_chain(:gitlab, :default_projects_features).and_return(@settings) | |
| 119 | + end | |
| 120 | + | |
| 121 | + context 'should be private when option is public' do | |
| 122 | + before do | |
| 123 | + @opts.merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 124 | + @project = create_project(@user, @opts) | |
| 125 | + end | |
| 126 | + | |
| 127 | + it { @project.private?.should be_true } | |
| 128 | + end | |
| 129 | + | |
| 130 | + context 'should be public when option is public for admin' do | |
| 131 | + before do | |
| 132 | + @opts.merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 133 | + @project = create_project(@admin, @opts) | |
| 134 | + end | |
| 135 | + | |
| 136 | + it { @project.public?.should be_true } | |
| 137 | + end | |
| 138 | + | |
| 139 | + context 'should be private when option is private' do | |
| 140 | + before do | |
| 141 | + @opts.merge!(visibility_level: Gitlab::VisibilityLevel::PRIVATE) | |
| 142 | + @project = create_project(@user, @opts) | |
| 143 | + end | |
| 144 | + | |
| 145 | + it { @project.private?.should be_true } | |
| 146 | + end | |
| 147 | + | |
| 148 | + context 'should be internal when option is internal' do | |
| 149 | + before do | |
| 150 | + @opts.merge!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) | |
| 151 | + @project = create_project(@user, @opts) | |
| 152 | + end | |
| 153 | + | |
| 154 | + it { @project.internal?.should be_true } | |
| 155 | + end | |
| 156 | + end | |
| 157 | + end | |
| 158 | + | |
| 159 | + def create_project(user, opts) | |
| 160 | + Projects::CreateService.new(user, opts).execute | |
| 161 | + end | |
| 162 | +end | |
| 163 | + | ... | ... |
| ... | ... | @@ -0,0 +1,33 @@ |
| 1 | +require 'spec_helper' | |
| 2 | + | |
| 3 | +describe ProjectTransferService do | |
| 4 | + before(:each) { enable_observers } | |
| 5 | + after(:each) {disable_observers} | |
| 6 | + | |
| 7 | + context 'namespace -> namespace' do | |
| 8 | + let(:user) { create(:user) } | |
| 9 | + let(:group) { create(:group) } | |
| 10 | + let(:project) { create(:project, namespace: user.namespace) } | |
| 11 | + | |
| 12 | + before do | |
| 13 | + @result = service.transfer(project, group) | |
| 14 | + end | |
| 15 | + | |
| 16 | + it { @result.should be_true } | |
| 17 | + it { project.namespace.should == group } | |
| 18 | + end | |
| 19 | + | |
| 20 | + context 'namespace -> no namespace' do | |
| 21 | + let(:user) { create(:user) } | |
| 22 | + let(:project) { create(:project, namespace: user.namespace) } | |
| 23 | + | |
| 24 | + it { lambda{service.transfer(project, nil)}.should raise_error(ActiveRecord::RecordInvalid) } | |
| 25 | + end | |
| 26 | + | |
| 27 | + def service | |
| 28 | + service = ProjectTransferService.new | |
| 29 | + service.gitlab_shell.stub(mv_repository: true) | |
| 30 | + service | |
| 31 | + end | |
| 32 | +end | |
| 33 | + | ... | ... |
| ... | ... | @@ -0,0 +1,111 @@ |
| 1 | +require 'spec_helper' | |
| 2 | + | |
| 3 | +describe Projects::UpdateService do | |
| 4 | + before(:each) { ActiveRecord::Base.observers.enable(:user_observer) } | |
| 5 | + after(:each) { ActiveRecord::Base.observers.disable(:user_observer) } | |
| 6 | + | |
| 7 | + describe :update_by_user do | |
| 8 | + before do | |
| 9 | + @user = create :user | |
| 10 | + @admin = create :user, admin: true | |
| 11 | + @project = create :project, creator_id: @user.id, namespace: @user.namespace | |
| 12 | + @opts = { project: {} } | |
| 13 | + end | |
| 14 | + | |
| 15 | + context 'should be private when updated to private' do | |
| 16 | + before do | |
| 17 | + @created_private = @project.private? | |
| 18 | + | |
| 19 | + @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PRIVATE) | |
| 20 | + update_project(@project, @user, @opts) | |
| 21 | + end | |
| 22 | + | |
| 23 | + it { @created_private.should be_true } | |
| 24 | + it { @project.private?.should be_true } | |
| 25 | + end | |
| 26 | + | |
| 27 | + context 'should be internal when updated to internal' do | |
| 28 | + before do | |
| 29 | + @created_private = @project.private? | |
| 30 | + | |
| 31 | + @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) | |
| 32 | + update_project(@project, @user, @opts) | |
| 33 | + end | |
| 34 | + | |
| 35 | + it { @created_private.should be_true } | |
| 36 | + it { @project.internal?.should be_true } | |
| 37 | + end | |
| 38 | + | |
| 39 | + context 'should be public when updated to public' do | |
| 40 | + before do | |
| 41 | + @created_private = @project.private? | |
| 42 | + | |
| 43 | + @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 44 | + update_project(@project, @user, @opts) | |
| 45 | + end | |
| 46 | + | |
| 47 | + it { @created_private.should be_true } | |
| 48 | + it { @project.public?.should be_true } | |
| 49 | + end | |
| 50 | + | |
| 51 | + context 'respect configured visibility restrictions setting' do | |
| 52 | + before(:each) do | |
| 53 | + @restrictions = double("restrictions") | |
| 54 | + @restrictions.stub(:restricted_visibility_levels) { [ Gitlab::VisibilityLevel::PUBLIC ] } | |
| 55 | + Settings.stub_chain(:gitlab).and_return(@restrictions) | |
| 56 | + end | |
| 57 | + | |
| 58 | + context 'should be private when updated to private' do | |
| 59 | + before do | |
| 60 | + @created_private = @project.private? | |
| 61 | + | |
| 62 | + @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PRIVATE) | |
| 63 | + update_project(@project, @user, @opts) | |
| 64 | + end | |
| 65 | + | |
| 66 | + it { @created_private.should be_true } | |
| 67 | + it { @project.private?.should be_true } | |
| 68 | + end | |
| 69 | + | |
| 70 | + context 'should be internal when updated to internal' do | |
| 71 | + before do | |
| 72 | + @created_private = @project.private? | |
| 73 | + | |
| 74 | + @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) | |
| 75 | + update_project(@project, @user, @opts) | |
| 76 | + end | |
| 77 | + | |
| 78 | + it { @created_private.should be_true } | |
| 79 | + it { @project.internal?.should be_true } | |
| 80 | + end | |
| 81 | + | |
| 82 | + context 'should be private when updated to public' do | |
| 83 | + before do | |
| 84 | + @created_private = @project.private? | |
| 85 | + | |
| 86 | + @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 87 | + update_project(@project, @user, @opts) | |
| 88 | + end | |
| 89 | + | |
| 90 | + it { @created_private.should be_true } | |
| 91 | + it { @project.private?.should be_true } | |
| 92 | + end | |
| 93 | + | |
| 94 | + context 'should be public when updated to public by admin' do | |
| 95 | + before do | |
| 96 | + @created_private = @project.private? | |
| 97 | + | |
| 98 | + @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 99 | + update_project(@project, @admin, @opts) | |
| 100 | + end | |
| 101 | + | |
| 102 | + it { @created_private.should be_true } | |
| 103 | + it { @project.public?.should be_true } | |
| 104 | + end | |
| 105 | + end | |
| 106 | + end | |
| 107 | + | |
| 108 | + def update_project(project, user, opts) | |
| 109 | + Projects::UpdateService.new(project, user, opts).execute | |
| 110 | + end | |
| 111 | +end | ... | ... |
spec/services/projects_create_service_spec.rb
| ... | ... | @@ -1,163 +0,0 @@ |
| 1 | -require 'spec_helper' | |
| 2 | - | |
| 3 | -describe Projects::CreateService do | |
| 4 | - before(:each) { ActiveRecord::Base.observers.enable(:user_observer) } | |
| 5 | - after(:each) { ActiveRecord::Base.observers.disable(:user_observer) } | |
| 6 | - | |
| 7 | - describe :create_by_user do | |
| 8 | - before do | |
| 9 | - @user = create :user | |
| 10 | - @admin = create :user, admin: true | |
| 11 | - @opts = { | |
| 12 | - name: "GitLab", | |
| 13 | - namespace: @user.namespace | |
| 14 | - } | |
| 15 | - end | |
| 16 | - | |
| 17 | - context 'user namespace' do | |
| 18 | - before do | |
| 19 | - @project = create_project(@user, @opts) | |
| 20 | - end | |
| 21 | - | |
| 22 | - it { @project.should be_valid } | |
| 23 | - it { @project.owner.should == @user } | |
| 24 | - it { @project.namespace.should == @user.namespace } | |
| 25 | - end | |
| 26 | - | |
| 27 | - context 'group namespace' do | |
| 28 | - before do | |
| 29 | - @group = create :group | |
| 30 | - @group.add_owner(@user) | |
| 31 | - | |
| 32 | - @opts.merge!(namespace_id: @group.id) | |
| 33 | - @project = create_project(@user, @opts) | |
| 34 | - end | |
| 35 | - | |
| 36 | - it { @project.should be_valid } | |
| 37 | - it { @project.owner.should == @group } | |
| 38 | - it { @project.namespace.should == @group } | |
| 39 | - end | |
| 40 | - | |
| 41 | - context 'wiki_enabled creates repository directory' do | |
| 42 | - context 'wiki_enabled true creates wiki repository directory' do | |
| 43 | - before do | |
| 44 | - @project = create_project(@user, @opts) | |
| 45 | - @path = GollumWiki.new(@project, @user).send(:path_to_repo) | |
| 46 | - end | |
| 47 | - | |
| 48 | - it { File.exists?(@path).should be_true } | |
| 49 | - end | |
| 50 | - | |
| 51 | - context 'wiki_enabled false does not create wiki repository directory' do | |
| 52 | - before do | |
| 53 | - @opts.merge!(wiki_enabled: false) | |
| 54 | - @project = create_project(@user, @opts) | |
| 55 | - @path = GollumWiki.new(@project, @user).send(:path_to_repo) | |
| 56 | - end | |
| 57 | - | |
| 58 | - it { File.exists?(@path).should be_false } | |
| 59 | - end | |
| 60 | - end | |
| 61 | - | |
| 62 | - context 'respect configured visibility setting' do | |
| 63 | - before(:each) do | |
| 64 | - @settings = double("settings") | |
| 65 | - @settings.stub(:issues) { true } | |
| 66 | - @settings.stub(:merge_requests) { true } | |
| 67 | - @settings.stub(:wiki) { true } | |
| 68 | - @settings.stub(:wall) { true } | |
| 69 | - @settings.stub(:snippets) { true } | |
| 70 | - stub_const("Settings", Class.new) | |
| 71 | - @restrictions = double("restrictions") | |
| 72 | - @restrictions.stub(:restricted_visibility_levels) { [] } | |
| 73 | - Settings.stub_chain(:gitlab).and_return(@restrictions) | |
| 74 | - Settings.stub_chain(:gitlab, :default_projects_features).and_return(@settings) | |
| 75 | - end | |
| 76 | - | |
| 77 | - context 'should be public when setting is public' do | |
| 78 | - before do | |
| 79 | - @settings.stub(:visibility_level) { Gitlab::VisibilityLevel::PUBLIC } | |
| 80 | - @project = create_project(@user, @opts) | |
| 81 | - end | |
| 82 | - | |
| 83 | - it { @project.public?.should be_true } | |
| 84 | - end | |
| 85 | - | |
| 86 | - context 'should be private when setting is private' do | |
| 87 | - before do | |
| 88 | - @settings.stub(:visibility_level) { Gitlab::VisibilityLevel::PRIVATE } | |
| 89 | - @project = create_project(@user, @opts) | |
| 90 | - end | |
| 91 | - | |
| 92 | - it { @project.private?.should be_true } | |
| 93 | - end | |
| 94 | - | |
| 95 | - context 'should be internal when setting is internal' do | |
| 96 | - before do | |
| 97 | - @settings.stub(:visibility_level) { Gitlab::VisibilityLevel::INTERNAL } | |
| 98 | - @project = create_project(@user, @opts) | |
| 99 | - end | |
| 100 | - | |
| 101 | - it { @project.internal?.should be_true } | |
| 102 | - end | |
| 103 | - end | |
| 104 | - | |
| 105 | - context 'respect configured visibility restrictions setting' do | |
| 106 | - before(:each) do | |
| 107 | - @settings = double("settings") | |
| 108 | - @settings.stub(:issues) { true } | |
| 109 | - @settings.stub(:merge_requests) { true } | |
| 110 | - @settings.stub(:wiki) { true } | |
| 111 | - @settings.stub(:wall) { true } | |
| 112 | - @settings.stub(:snippets) { true } | |
| 113 | - @settings.stub(:visibility_level) { Gitlab::VisibilityLevel::PRIVATE } | |
| 114 | - stub_const("Settings", Class.new) | |
| 115 | - @restrictions = double("restrictions") | |
| 116 | - @restrictions.stub(:restricted_visibility_levels) { [ Gitlab::VisibilityLevel::PUBLIC ] } | |
| 117 | - Settings.stub_chain(:gitlab).and_return(@restrictions) | |
| 118 | - Settings.stub_chain(:gitlab, :default_projects_features).and_return(@settings) | |
| 119 | - end | |
| 120 | - | |
| 121 | - context 'should be private when option is public' do | |
| 122 | - before do | |
| 123 | - @opts.merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 124 | - @project = create_project(@user, @opts) | |
| 125 | - end | |
| 126 | - | |
| 127 | - it { @project.private?.should be_true } | |
| 128 | - end | |
| 129 | - | |
| 130 | - context 'should be public when option is public for admin' do | |
| 131 | - before do | |
| 132 | - @opts.merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 133 | - @project = create_project(@admin, @opts) | |
| 134 | - end | |
| 135 | - | |
| 136 | - it { @project.public?.should be_true } | |
| 137 | - end | |
| 138 | - | |
| 139 | - context 'should be private when option is private' do | |
| 140 | - before do | |
| 141 | - @opts.merge!(visibility_level: Gitlab::VisibilityLevel::PRIVATE) | |
| 142 | - @project = create_project(@user, @opts) | |
| 143 | - end | |
| 144 | - | |
| 145 | - it { @project.private?.should be_true } | |
| 146 | - end | |
| 147 | - | |
| 148 | - context 'should be internal when option is internal' do | |
| 149 | - before do | |
| 150 | - @opts.merge!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) | |
| 151 | - @project = create_project(@user, @opts) | |
| 152 | - end | |
| 153 | - | |
| 154 | - it { @project.internal?.should be_true } | |
| 155 | - end | |
| 156 | - end | |
| 157 | - end | |
| 158 | - | |
| 159 | - def create_project(user, opts) | |
| 160 | - Projects::CreateService.new(user, opts).execute | |
| 161 | - end | |
| 162 | -end | |
| 163 | - |
spec/services/projects_update_service_spec.rb
| ... | ... | @@ -1,111 +0,0 @@ |
| 1 | -require 'spec_helper' | |
| 2 | - | |
| 3 | -describe Projects::UpdateService do | |
| 4 | - before(:each) { ActiveRecord::Base.observers.enable(:user_observer) } | |
| 5 | - after(:each) { ActiveRecord::Base.observers.disable(:user_observer) } | |
| 6 | - | |
| 7 | - describe :update_by_user do | |
| 8 | - before do | |
| 9 | - @user = create :user | |
| 10 | - @admin = create :user, admin: true | |
| 11 | - @project = create :project, creator_id: @user.id, namespace: @user.namespace | |
| 12 | - @opts = { project: {} } | |
| 13 | - end | |
| 14 | - | |
| 15 | - context 'should be private when updated to private' do | |
| 16 | - before do | |
| 17 | - @created_private = @project.private? | |
| 18 | - | |
| 19 | - @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PRIVATE) | |
| 20 | - update_project(@project, @user, @opts) | |
| 21 | - end | |
| 22 | - | |
| 23 | - it { @created_private.should be_true } | |
| 24 | - it { @project.private?.should be_true } | |
| 25 | - end | |
| 26 | - | |
| 27 | - context 'should be internal when updated to internal' do | |
| 28 | - before do | |
| 29 | - @created_private = @project.private? | |
| 30 | - | |
| 31 | - @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) | |
| 32 | - update_project(@project, @user, @opts) | |
| 33 | - end | |
| 34 | - | |
| 35 | - it { @created_private.should be_true } | |
| 36 | - it { @project.internal?.should be_true } | |
| 37 | - end | |
| 38 | - | |
| 39 | - context 'should be public when updated to public' do | |
| 40 | - before do | |
| 41 | - @created_private = @project.private? | |
| 42 | - | |
| 43 | - @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 44 | - update_project(@project, @user, @opts) | |
| 45 | - end | |
| 46 | - | |
| 47 | - it { @created_private.should be_true } | |
| 48 | - it { @project.public?.should be_true } | |
| 49 | - end | |
| 50 | - | |
| 51 | - context 'respect configured visibility restrictions setting' do | |
| 52 | - before(:each) do | |
| 53 | - @restrictions = double("restrictions") | |
| 54 | - @restrictions.stub(:restricted_visibility_levels) { [ Gitlab::VisibilityLevel::PUBLIC ] } | |
| 55 | - Settings.stub_chain(:gitlab).and_return(@restrictions) | |
| 56 | - end | |
| 57 | - | |
| 58 | - context 'should be private when updated to private' do | |
| 59 | - before do | |
| 60 | - @created_private = @project.private? | |
| 61 | - | |
| 62 | - @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PRIVATE) | |
| 63 | - update_project(@project, @user, @opts) | |
| 64 | - end | |
| 65 | - | |
| 66 | - it { @created_private.should be_true } | |
| 67 | - it { @project.private?.should be_true } | |
| 68 | - end | |
| 69 | - | |
| 70 | - context 'should be internal when updated to internal' do | |
| 71 | - before do | |
| 72 | - @created_private = @project.private? | |
| 73 | - | |
| 74 | - @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) | |
| 75 | - update_project(@project, @user, @opts) | |
| 76 | - end | |
| 77 | - | |
| 78 | - it { @created_private.should be_true } | |
| 79 | - it { @project.internal?.should be_true } | |
| 80 | - end | |
| 81 | - | |
| 82 | - context 'should be private when updated to public' do | |
| 83 | - before do | |
| 84 | - @created_private = @project.private? | |
| 85 | - | |
| 86 | - @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 87 | - update_project(@project, @user, @opts) | |
| 88 | - end | |
| 89 | - | |
| 90 | - it { @created_private.should be_true } | |
| 91 | - it { @project.private?.should be_true } | |
| 92 | - end | |
| 93 | - | |
| 94 | - context 'should be public when updated to public by admin' do | |
| 95 | - before do | |
| 96 | - @created_private = @project.private? | |
| 97 | - | |
| 98 | - @opts[:project].merge!(visibility_level: Gitlab::VisibilityLevel::PUBLIC) | |
| 99 | - update_project(@project, @admin, @opts) | |
| 100 | - end | |
| 101 | - | |
| 102 | - it { @created_private.should be_true } | |
| 103 | - it { @project.public?.should be_true } | |
| 104 | - end | |
| 105 | - end | |
| 106 | - end | |
| 107 | - | |
| 108 | - def update_project(project, user, opts) | |
| 109 | - Projects::UpdateService.new(project, user, opts).execute | |
| 110 | - end | |
| 111 | -end |