From 0444fa560acd07255960284f19b1de6499cd5910 Mon Sep 17 00:00:00 2001
From: Timothy Andrew <mail@timothyandrew.net>
Date: Fri, 12 Feb 2016 20:28:39 +0530
Subject: Original implementation to allow users to subscribe to labels

1. Allow subscribing (the current user) to a label

- Refactor the `Subscription` coffeescript class
  - The main change is that it accepts a container, and conducts all
    DOM queries within its scope. We need this because the labels
    page has multiple instances of `Subscription` on the same page.

2. Creating an issue or MR with labels notifies users subscribed to those labels

- Label `has_many` subscribers through subscriptions.

3. Adding a label to an issue or MR notifies users subscribed to those labels

- This only applies to subscribers of the label that has just been
  added, not all labels for the issue.
---
 features/steps/project/labels.rb | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)
 create mode 100644 features/steps/project/labels.rb

(limited to 'features/steps/project/labels.rb')

diff --git a/features/steps/project/labels.rb b/features/steps/project/labels.rb
new file mode 100644
index 00000000000..3f800a10594
--- /dev/null
+++ b/features/steps/project/labels.rb
@@ -0,0 +1,34 @@
+class Spinach::Features::Labels < Spinach::FeatureSteps
+  include SharedAuthentication
+  include SharedIssuable
+  include SharedProject
+  include SharedNote
+  include SharedPaths
+  include SharedMarkdown
+
+  step 'And I visit project "Shop" labels page' do
+    visit namespace_project_labels_path(project.namespace, project)
+  end
+
+  step 'I should see that I am subscribed' do
+    expect(subscribe_button).to have_content 'Unsubscribe'
+  end
+
+  step 'I should see that I am unsubscribed' do
+    expect(subscribe_button).to have_content 'Subscribe'
+  end
+
+  step 'I click button "Unsubscribe"' do
+    subscribe_button.click
+  end
+
+  step 'I click button "Subscribe"' do
+    subscribe_button.click
+  end
+
+  private
+
+  def subscribe_button
+    first('.subscribe-button span')
+  end
+end
-- 
cgit v1.2.1


From 54ec7e959900493b6e9174bf4dfe09ed0afd1e46 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9my=20Coutable?= <remy@rymai.me>
Date: Tue, 1 Mar 2016 17:33:13 +0100
Subject: Improving the original label-subscribing implementation

1. Make the "subscribed" text in Issuable sidebar reflect the labels
   subscription status

2. Current user mut be logged-in to toggle issue/MR/label subscription
---
 features/steps/project/labels.rb | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'features/steps/project/labels.rb')

diff --git a/features/steps/project/labels.rb b/features/steps/project/labels.rb
index 3f800a10594..17944527e3a 100644
--- a/features/steps/project/labels.rb
+++ b/features/steps/project/labels.rb
@@ -10,19 +10,19 @@ class Spinach::Features::Labels < Spinach::FeatureSteps
     visit namespace_project_labels_path(project.namespace, project)
   end
 
-  step 'I should see that I am subscribed' do
+  step 'I should see that I am subscribed to the "bug" label' do
     expect(subscribe_button).to have_content 'Unsubscribe'
   end
 
-  step 'I should see that I am unsubscribed' do
+  step 'I should see that I am not subscribed to the "bug" label' do
     expect(subscribe_button).to have_content 'Subscribe'
   end
 
-  step 'I click button "Unsubscribe"' do
+  step 'I click button "Unsubscribe" for the "bug" label' do
     subscribe_button.click
   end
 
-  step 'I click button "Subscribe"' do
+  step 'I click button "Subscribe" for the "bug" label' do
     subscribe_button.click
   end
 
-- 
cgit v1.2.1