From 594c320851afbf4c8dd1c78600a0195f12d6ce41 Mon Sep 17 00:00:00 2001
From: Filipa Lacerda <filipa@gitlab.com>
Date: Mon, 10 Oct 2016 21:44:29 +0100
Subject: [PATCH 1/2] Adds tests to verify if tabs are rendered

---
 spec/features/environments_spec.rb | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/spec/features/environments_spec.rb b/spec/features/environments_spec.rb
index 3b38a7f5007..99246589eae 100644
--- a/spec/features/environments_spec.rb
+++ b/spec/features/environments_spec.rb
@@ -18,11 +18,26 @@ feature 'Environments', feature: true do
     before do
       visit namespace_project_environments_path(project.namespace, project)
     end
+    
+    context 'shows two tabs' do
+      scenario 'does show Available tab with link' do
+        expect(page).to have_link('Available')
+      end
+
+      scenario 'does show Stopped tab with link' do
+        expect(page).to have_link('Stopped')
+      end
+    end
 
     context 'without environments' do
       scenario 'does show no environments' do
         expect(page).to have_content('You don\'t have any environments right now.')
       end
+      
+      scenario 'does show 0 as counter for environments in both tabs' do
+        expect(page.find('.js-avaibale-environments-count').text).to eq('0')
+        expect(page.find('.js-stopped-environments-count').text).to eq('0')
+      end
     end
 
     context 'with environments' do

From 7d12683de51721e75b314b29272f4f024f7f0655 Mon Sep 17 00:00:00 2001
From: Filipa Lacerda <filipa@gitlab.com>
Date: Tue, 11 Oct 2016 11:03:11 +0100
Subject: [PATCH 2/2] Fixes broken tests

---
 spec/features/environments_spec.rb            | 47 +++++++++++++++++--
 .../merge_when_build_succeeds_spec.rb         |  9 ++++
 2 files changed, 53 insertions(+), 3 deletions(-)

diff --git a/spec/features/environments_spec.rb b/spec/features/environments_spec.rb
index 99246589eae..a8244ca89c6 100644
--- a/spec/features/environments_spec.rb
+++ b/spec/features/environments_spec.rb
@@ -46,6 +46,14 @@ feature 'Environments', feature: true do
       scenario 'does show environment name' do
         expect(page).to have_link(environment.name)
       end
+      
+      scenario 'does show number of opened environments in Availabe tab' do
+        expect(page.find('.js-avaibale-environments-count').text).to eq('1')
+      end
+
+      scenario 'does show number of closed environments in Stopped tab' do
+        expect(page.find('.js-stopped-environments-count').text).to eq('0')
+      end
 
       context 'without deployments' do
         scenario 'does show no deployments' do
@@ -76,6 +84,16 @@ feature 'Environments', feature: true do
             expect(page).to have_content(manual.name)
             expect(manual.reload).to be_pending
           end
+          
+          scenario 'does show close button' do 
+            # TODO: Add test to verify if close button is visible
+            # This needs to be true: if local_assigns.fetch(:allow_close, false) && deployment.closeable?
+          end
+          
+          scenario 'does allow to close environment' do 
+            # TODO: Add test to verify if close environment works
+            # This needs to be true: if local_assigns.fetch(:allow_close, false) && deployment.closeable?
+          end
         end
       end
     end
@@ -137,6 +155,16 @@ feature 'Environments', feature: true do
             expect(page).to have_content(manual.name)
             expect(manual.reload).to be_pending
           end
+          
+          scenario 'does show close button' do 
+            # TODO: Add test to verify if close button is visible
+            # This needs to be true: if local_assigns.fetch(:allow_close, false) && deployment.closeable?
+          end
+          
+          scenario 'does allow to close environment' do 
+            # TODO: Add test to verify if close environment works
+            # This needs to be true: if local_assigns.fetch(:allow_close, false) && deployment.closeable?
+          end
         end
       end
     end
@@ -194,9 +222,22 @@ feature 'Environments', feature: true do
     context 'when logged as master' do
       given(:role) { :master }
 
-      scenario 'does close environment' do
-        click_link 'Close'
-        expect(page).not_to have_link(environment.name)
+      scenario 'does not have a Close link' do
+        expect(page).not_to have_link('Close')
+      end
+      
+      context 'when environment is opened and can be closed' do
+        let(:project)     { create(:project) }
+        let(:environment) { create(:environment, project: project) }
+
+        let!(:deployment) do
+          create(:deployment, environment: environment, sha: project.commit('master').id)
+        end
+
+        scenario 'does have a Close link' do
+          # TODO: Add missing validation. In order to have Close link 
+          # this must be true: last_deployment.try(:close_action)
+        end
       end
     end
 
diff --git a/spec/features/merge_requests/merge_when_build_succeeds_spec.rb b/spec/features/merge_requests/merge_when_build_succeeds_spec.rb
index 60bc07bd1a0..2c1a45af596 100644
--- a/spec/features/merge_requests/merge_when_build_succeeds_spec.rb
+++ b/spec/features/merge_requests/merge_when_build_succeeds_spec.rb
@@ -79,6 +79,15 @@ feature 'Merge When Build Succeeds', feature: true, js: true do
     end
   end
 
+  context 'Has Environment' do
+    let(:environment) { create(:environment, project: project) }
+    
+    it 'does show link to close the environment' do
+        # TODO add test to verify if the button is visible when this condition
+        # is met: if environment.closeable?
+    end
+  end
+  
   def visit_merge_request(merge_request)
     visit namespace_project_merge_request_path(merge_request.project.namespace, merge_request.project, merge_request)
   end