Add test for the case when user can't prioritize labels

1 files changed, 77 insertions, 47 deletions

diff --git a/spec/features/projects/labels/update_prioritization_spec.rb b/spec/features/projects/labels/update_prioritization_spec.rb
index 2fcaed84169..2a0bf6548db 100644
--- a/spec/features/projects/labels/update_prioritization_spec.rb
+++ b/spec/features/projects/labels/update_prioritization_spec.rb
@@ -3,76 +3,106 @@ require 'spec_helper'
 feature 'Prioritize labels', feature: true do
   include WaitForAjax
 
-  let(:user)    { create(:user) }
-  let(:project) { create(:project, name: 'test', namespace: user.namespace) }
+  context 'when project belongs to user' do
+    let(:user)    { create(:user) }
+    let(:project) { create(:project, name: 'test', namespace: user.namespace) }
 
-  scenario 'user can prioritize a label', js: true do
-    bug     = create(:label, title: 'bug')
-    wontfix = create(:label, title: 'wontfix')
+    scenario 'user can prioritize a label', js: true do
+      bug     = create(:label, title: 'bug')
+      wontfix = create(:label, title: 'wontfix')
 
-    project.labels << bug
-    project.labels << wontfix
+      project.labels << bug
+      project.labels << wontfix
 
-    login_as user
-    visit namespace_project_labels_path(project.namespace, project)
+      login_as user
+      visit namespace_project_labels_path(project.namespace, project)
 
-    expect(page).to have_content('No prioritized labels yet')
+      expect(page).to have_content('No prioritized labels yet')
 
-    page.within('.other-labels') do
-      first('.js-toggle-priority').click
-      wait_for_ajax
-      expect(page).not_to have_content('bug')
+      page.within('.other-labels') do
+        first('.js-toggle-priority').click
+        wait_for_ajax
+        expect(page).not_to have_content('bug')
+      end
+
+      page.within('.prioritized-labels') do
+        expect(page).not_to have_content('No prioritized labels yet')
+        expect(page).to have_content('bug')
+      end
     end
 
-    page.within('.prioritized-labels') do
-      expect(page).not_to have_content('No prioritized labels yet')
+    scenario 'user can unprioritize a label', js: true do
+      bug     = create(:label, title: 'bug', priority: 1)
+      wontfix = create(:label, title: 'wontfix')
+
+      project.labels << bug
+      project.labels << wontfix
+
+      login_as user
+      visit namespace_project_labels_path(project.namespace, project)
+
       expect(page).to have_content('bug')
+
+      page.within('.prioritized-labels') do
+        first('.js-toggle-priority').click
+        wait_for_ajax
+        expect(page).not_to have_content('bug')
+      end
+
+      page.within('.other-labels') do
+        expect(page).to have_content('bug')
+        expect(page).to have_content('wontfix')
+      end
     end
-  end
 
-  scenario 'user can unprioritize a label', js: true do
-    bug     = create(:label, title: 'bug', priority: 1)
-    wontfix = create(:label, title: 'wontfix')
+    scenario 'user can sort prioritized labels', js: true do
+      bug     = create(:label, title: 'bug', priority: 1)
+      wontfix = create(:label, title: 'wontfix', priority: 2)
 
-    project.labels << bug
-    project.labels << wontfix
+      project.labels << bug
+      project.labels << wontfix
 
-    login_as user
-    visit namespace_project_labels_path(project.namespace, project)
+      login_as user
+      visit namespace_project_labels_path(project.namespace, project)
 
-    expect(page).to have_content('bug')
+      expect(page).to have_content 'bug'
+      expect(page).to have_content 'wontfix'
 
-    page.within('.prioritized-labels') do
-      first('.js-toggle-priority').click
-      wait_for_ajax
-      expect(page).not_to have_content('bug')
-    end
+      # Sort labels
+      find("#label_#{bug.id}").drag_to find("#label_#{wontfix.id}")
 
-    page.within('.other-labels') do
-      expect(page).to have_content('bug')
-      expect(page).to have_content('wontfix')
+      page.within('.prioritized-labels') do
+        expect(first('li')).to have_content('wontfix')
+        expect(page.all('li').last).to have_content('bug')
+      end
     end
   end
 
-  scenario 'user can sort prioritized labels', js: true do
-    bug     = create(:label, title: 'bug', priority: 1)
-    wontfix = create(:label, title: 'wontfix', priority: 2)
+  context 'as a guest' do
+    it 'can not prioritize labels' do
+      user = create(:user)
+      guest = create(:user)
+      project = create(:project, name: 'test', namespace: user.namespace)
+
+      create(:label, title: 'bug')
 
-    project.labels << bug
-    project.labels << wontfix
+      login_as guest
+      visit namespace_project_labels_path(project.namespace, project)
+
+      expect(page).not_to have_css('.prioritized-labels')
+    end
+  end
 
-    login_as user
-    visit namespace_project_labels_path(project.namespace, project)
+  context 'as a non signed in user' do
+    it 'can not prioritize labels' do
+      user = create(:user)
+      project = create(:project, name: 'test', namespace: user.namespace)
 
-    expect(page).to have_content 'bug'
-    expect(page).to have_content 'wontfix'
+      create(:label, title: 'bug')
 
-    # Sort labels
-    find("#label_#{bug.id}").drag_to find("#label_#{wontfix.id}")
+      visit namespace_project_labels_path(project.namespace, project)
 
-    page.within('.prioritized-labels') do
-      expect(first('li')).to have_content('wontfix')
-      expect(page.all('li').last).to have_content('bug')
+      expect(page).not_to have_css('.prioritized-labels')
     end
   end
 end