Add subscribe filter to labels page

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

1 files changed, 45 insertions, 5 deletions

diff --git a/spec/features/groups/labels/subscription_spec.rb b/spec/features/groups/labels/subscription_spec.rb
index d9543bfa97f..22b51b297a6 100644
--- a/spec/features/groups/labels/subscription_spec.rb
+++ b/spec/features/groups/labels/subscription_spec.rb
@@ -3,7 +3,8 @@ require 'spec_helper'
 describe 'Labels subscription' do
   let(:user)     { create(:user) }
   let(:group)    { create(:group) }
-  let!(:feature) { create(:group_label, group: group, title: 'feature') }
+  let!(:label1)  { create(:group_label, group: group, title: 'foo') }
+  let!(:label2)  { create(:group_label, group: group, title: 'bar') }
 
   context 'when signed in' do
     before do
@@ -14,9 +15,9 @@ describe 'Labels subscription' do
     it 'users can subscribe/unsubscribe to group labels', :js do
       visit group_labels_path(group)
 
-      expect(page).to have_content('feature')
+      expect(page).to have_content(label1.title)
 
-      within "#group_label_#{feature.id}" do
+      within "#group_label_#{label1.id}" do
         expect(page).not_to have_button 'Unsubscribe'
 
         click_button 'Subscribe'
@@ -30,15 +31,48 @@ describe 'Labels subscription' do
         expect(page).not_to have_button 'Unsubscribe'
       end
     end
+
+    context 'subscription filter' do
+      before do
+        visit group_labels_path(group)
+      end
+
+      it 'shows only subscribed labels' do
+        label1.subscribe(user)
+
+        click_subscribed_tab
+
+        page.within('.labels-container') do
+          expect(page).to have_content label1.title
+        end
+      end
+
+      it 'shows no subscribed labels message' do
+        click_subscribed_tab
+
+        page.within('.labels-container') do
+          expect(page).not_to have_content label1.title
+          expect(page).to have_content('You do not have any subscriptions yet')
+        end
+      end
+    end
   end
 
   context 'when not signed in' do
-    it 'users can not subscribe/unsubscribe to labels' do
+    before do
       visit group_labels_path(group)
+    end
 
-      expect(page).to have_content 'feature'
+    it 'users can not subscribe/unsubscribe to labels' do
+      expect(page).to have_content label1.title
       expect(page).not_to have_button('Subscribe')
     end
+
+    it 'does not show subscribed tab' do
+      page.within('.nav-tabs') do
+        expect(page).not_to have_link 'Subscribed'
+      end
+    end
   end
 
   def click_link_on_dropdown(text)
@@ -48,4 +82,10 @@ describe 'Labels subscription' do
       find('a.js-subscribe-button', text: text).click
     end
   end
+
+  def click_subscribed_tab
+    page.within('.nav-tabs') do
+      click_link 'Subscribed'
+    end
+  end
 end