diff options
author | Ashley Dumaine <adumaine@datto.com> | 2017-08-30 13:58:19 -0400 |
---|---|---|
committer | Sean McGivern <sean@gitlab.com> | 2017-09-06 10:07:47 +0100 |
commit | 959e6d221d6b11c64d17fc9d4734200806c293ab (patch) | |
tree | aecb32f12f3115f3573735ce9342bc58841461e7 /spec/requests/api/projects_spec.rb | |
parent | ef4b3a39bc705dfa47762868ea064d2dbadc16e6 (diff) | |
download | gitlab-ce-959e6d221d6b11c64d17fc9d4734200806c293ab.tar.gz |
add unit tests for new collapse_outdated_diff_comments toggle
Diffstat (limited to 'spec/requests/api/projects_spec.rb')
-rw-r--r-- | spec/requests/api/projects_spec.rb | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/spec/requests/api/projects_spec.rb b/spec/requests/api/projects_spec.rb index 4490e50702b..5e3fdbd468e 100644 --- a/spec/requests/api/projects_spec.rb +++ b/spec/requests/api/projects_spec.rb @@ -414,6 +414,7 @@ describe API::Projects do jobs_enabled: false, merge_requests_enabled: false, wiki_enabled: false, + collapse_outdated_diff_comments: false, only_allow_merge_if_pipeline_succeeds: false, request_access_enabled: true, only_allow_merge_if_all_discussions_are_resolved: false, @@ -477,6 +478,18 @@ describe API::Projects do expect(json_response['avatar_url']).to eq("http://localhost/uploads/-/system/project/avatar/#{project_id}/banana_sample.gif") end + it 'sets a project as allowing outdated diff comments to collapse regardless of discussion resolution' do + project = attributes_for(:project, { collapse_outdated_diff_comments: false }) + post api('/projects', user), project + expect(json_response['collapse_outdated_diff_comments']).to be_falsey + end + + it 'sets a project as allowing outdated diff comments to collapse if collapse_outdated_diff_comments' do + project = attributes_for(:project, { collapse_outdated_diff_comments: true }) + post api('/projects', user), project + expect(json_response['collapse_outdated_diff_comments']).to be_truthy + end + it 'sets a project as allowing merge even if build fails' do project = attributes_for(:project, { only_allow_merge_if_pipeline_succeeds: false }) post api('/projects', user), project @@ -642,6 +655,22 @@ describe API::Projects do expect(json_response['visibility']).to eq('private') end + it 'sets a project as allowing outdated diff comments to collapse regardless of discussion resolution' do + project = attributes_for(:project, { collapse_outdated_diff_comments: false }) + + post api("/projects/user/#{user.id}", admin), project + + expect(json_response['collapse_outdated_diff_comments']).to be_falsey + end + + it 'sets a project as allowing outdated diff comments to collapse only if collapse_outdated_diff_comments' do + project = attributes_for(:project, { collapse_outdated_diff_comments: true }) + + post api("/projects/user/#{user.id}", admin), project + + expect(json_response['collapse_outdated_diff_comments']).to be_truthy + end + it 'sets a project as allowing merge even if build fails' do project = attributes_for(:project, { only_allow_merge_if_pipeline_succeeds: false }) post api("/projects/user/#{user.id}", admin), project @@ -732,6 +761,7 @@ describe API::Projects do expect(json_response['wiki_enabled']).to be_present expect(json_response['jobs_enabled']).to be_present expect(json_response['snippets_enabled']).to be_present + expect(json_response['collapse_outdated_diff_comments']).to eq(project.collapse_outdated_diff_comments) expect(json_response['container_registry_enabled']).to be_present expect(json_response['created_at']).to be_present expect(json_response['last_activity_at']).to be_present |