From 7d07012994be8014da1ce9e4d1857b220322ba41 Mon Sep 17 00:00:00 2001
From: Clement Ho <ClemMakesApps@gmail.com>
Date: Tue, 16 May 2017 14:51:15 -0500
Subject: Fix form spec

---
 spec/features/issues/form_spec.rb          | 20 ++++++++------------
 spec/features/issues/issue_sidebar_spec.rb | 10 ----------
 2 files changed, 8 insertions(+), 22 deletions(-)

diff --git a/spec/features/issues/form_spec.rb b/spec/features/issues/form_spec.rb
index 7bac9d6db74..5c0907e26df 100644
--- a/spec/features/issues/form_spec.rb
+++ b/spec/features/issues/form_spec.rb
@@ -25,7 +25,7 @@ describe 'New/edit issue', :feature, :js do
     end
 
     describe 'single assignee' do
-      it 'hides assignee after selection' do
+      before do
         click_button 'Unassigned'
 
         wait_for_ajax
@@ -36,14 +36,12 @@ describe 'New/edit issue', :feature, :js do
           click_link user2.name
         end
 
+        click_button user2.name
+
         page.within '.dropdown-menu-user' do
           click_link 'Unassigned'
         end
 
-        page.within '.js-assignee-search' do
-          expect(page).to have_content 'Unassigned'
-        end
-
         expect(find('input[name="issue[assignee_ids][]"]', visible: false).value).to match('0')
       end
 
@@ -54,6 +52,8 @@ describe 'New/edit issue', :feature, :js do
 
         expect(find('a', text: 'Assign to me', visible: false)).not_to be_visible
 
+        click_button user.name
+
         page.within('.dropdown-menu-user') do
           click_link user.name
         end
@@ -161,18 +161,14 @@ describe 'New/edit issue', :feature, :js do
         click_link user.name
       end
 
-      expect(find('input[name="issue[assignee_ids][]"]', visible: false).value).to match(user.id.to_s)
-      expect(find('.dropdown-menu-user a.is-active').first(:xpath, '..')['data-user-id']).to eq(user.id.to_s)
-      # check the ::before pseudo element to ensure checkmark icon is present
-      expect(before_for_selector('.dropdown-menu-selectable a.is-active')).not_to eq('')
-      expect(before_for_selector('.dropdown-menu-selectable a:not(.is-active)')).to eq('')
+      expect(find('.js-assignee-search')).to have_content(user.name)
+      click_button user.name
 
       page.within '.dropdown-menu-user' do
         click_link user2.name
       end
 
-      expect(find('input[name="issue[assignee_ids][]"]', visible: false).value).to match(user2.id.to_s)
-      expect(find('.dropdown-menu-user a.is-active').first(:xpath, '..')['data-user-id']).to eq(user2.id.to_s)
+      expect(find('.js-assignee-search')).to have_content(user2.name)
     end
   end
 
diff --git a/spec/features/issues/issue_sidebar_spec.rb b/spec/features/issues/issue_sidebar_spec.rb
index dd8c46f1044..0de0f93089a 100644
--- a/spec/features/issues/issue_sidebar_spec.rb
+++ b/spec/features/issues/issue_sidebar_spec.rb
@@ -57,16 +57,6 @@ feature 'Issue Sidebar', feature: true do
         expect(page.find('.dropdown-menu-user-link.is-active')).to have_content(user.name)
       end
     end
-
-    context 'single assignee' do
-      it 'hides assignee after selection' do
-        page.within('.dropdown-menu-user') do
-          click_link user.name
-        end
-
-        expect(page.find('.block.assignee .selectbox', visible: false)).not_to be_visible
-      end
-    end
   end
 
   context 'as a allowed user' do
-- 
cgit v1.2.1