From f71642017ebfd409e20735b621dd3a9fe09add12 Mon Sep 17 00:00:00 2001
From: Jacob Schatz <jacobschatz@Jacobs-MBP.fios-router.home>
Date: Wed, 6 Jan 2016 07:33:50 -0500
Subject: adds tests (and passes them) for ajax open and close merge requests.

---
 .../fixtures/merge_requests_show.html.haml         | 12 ++-
 spec/javascripts/issue_spec.js.coffee              |  2 +-
 spec/javascripts/merge_request_spec.js.coffee      | 88 ++++++++++++++++++++++
 3 files changed, 100 insertions(+), 2 deletions(-)

diff --git a/spec/javascripts/fixtures/merge_requests_show.html.haml b/spec/javascripts/fixtures/merge_requests_show.html.haml
index 8447dfdda32..fdfa8a273e2 100644
--- a/spec/javascripts/fixtures/merge_requests_show.html.haml
+++ b/spec/javascripts/fixtures/merge_requests_show.html.haml
@@ -1,4 +1,14 @@
-%a.btn-close
+:css
+  .hidden { display: none !important }
+
+.flash-container
+  .flash-alert
+  .flash-notice
+
+.status-box.status-box-open Open
+.status-box.status-box-closed.hidden Closed
+%a.btn-close{"href" => "http://gitlab.com/merge_requests/6/close"} Close
+%a.btn-reopen.hidden{"href" => "http://gitlab.com/merge_requests/6/reopen"} Reopen
 
 .detail-page-description
   .description.js-task-list-container
diff --git a/spec/javascripts/issue_spec.js.coffee b/spec/javascripts/issue_spec.js.coffee
index 7e67c778861..e1c22860da7 100644
--- a/spec/javascripts/issue_spec.js.coffee
+++ b/spec/javascripts/issue_spec.js.coffee
@@ -44,7 +44,7 @@ describe 'reopen/close issue', ->
     expect($('div.status-box-closed')).toBeVisible()
     expect($('div.status-box-open')).toBeHidden()
 
-  it 'fails to closes an issue with success:false', ->
+  it 'fails to close an issue with success:false', ->
 
     $.ajax = (obj) ->
       expect(obj.type).toBe('PUT')
diff --git a/spec/javascripts/merge_request_spec.js.coffee b/spec/javascripts/merge_request_spec.js.coffee
index 22ebc7039d1..e21bfde38ad 100644
--- a/spec/javascripts/merge_request_spec.js.coffee
+++ b/spec/javascripts/merge_request_spec.js.coffee
@@ -21,3 +21,91 @@ describe 'MergeRequest', ->
         expect(req.data.merge_request.description).not.toBe(null)
 
       $('.js-task-list-field').trigger('tasklist:changed')
+
+  describe 'reopen/close merge request', ->
+    fixture.preload('merge_requests_show.html')
+    beforeEach ->
+      fixture.load('merge_requests_show.html')
+      @merge_request = new MergeRequest({})
+    it 'closes a merge request', ->
+      $.ajax = (obj) ->
+        expect(obj.type).toBe('PUT')
+        expect(obj.url).toBe('http://gitlab.com/merge_requests/6/close')
+        obj.success saved:true
+
+      $btnClose = $('a.btn-close')
+      $btnReopen = $('a.btn-reopen')
+      expect($btnReopen).toBeHidden()
+      expect($btnClose.text()).toBe('Close')
+      expect(typeof $btnClose.prop('disabled')).toBe('undefined')
+
+      $btnClose.trigger('click')
+
+      expect($btnReopen).toBeVisible()
+
+      expect($btnClose).toBeHidden()
+      expect($('div.status-box-closed')).toBeVisible()
+      expect($('div.status-box-open')).toBeHidden()
+
+    it 'fails to close a merge request with success:false', ->
+
+      $.ajax = (obj) ->
+        expect(obj.type).toBe('PUT')
+        expect(obj.url).toBe('http://goesnowhere.nothing/whereami')
+        obj.success saved:false
+
+      $btnClose = $('a.btn-close')
+      $btnReopen = $('a.btn-reopen')
+      $btnClose.attr('href','http://goesnowhere.nothing/whereami')
+      expect($btnReopen).toBeHidden()
+      expect($btnClose.text()).toBe('Close')
+      expect(typeof $btnClose.prop('disabled')).toBe('undefined')
+
+      $btnClose.trigger('click')
+
+      expect($btnReopen).toBeHidden()
+      expect($btnClose).toBeVisible()
+      expect($('div.status-box-closed')).toBeHidden()
+      expect($('div.status-box-open')).toBeVisible()
+      expect($('div.flash-alert')).toBeVisible()
+      expect($('div.flash-alert').text()).toBe('Unable to update this merge request at this time.')
+
+    it 'fails to closes an issue with HTTP error', ->
+
+      $.ajax = (obj) ->
+        expect(obj.type).toBe('PUT')
+        expect(obj.url).toBe('http://goesnowhere.nothing/whereami')
+        obj.error()
+      
+      $btnClose = $('a.btn-close')
+      $btnReopen = $('a.btn-reopen')
+      $btnClose.attr('href','http://goesnowhere.nothing/whereami')
+      expect($btnReopen).toBeHidden()
+      expect($btnClose.text()).toBe('Close')
+      expect(typeof $btnClose.prop('disabled')).toBe('undefined')
+
+      $btnClose.trigger('click')
+      
+      expect($btnReopen).toBeHidden()
+      expect($btnClose).toBeVisible()
+      expect($('div.status-box-closed')).toBeHidden()
+      expect($('div.status-box-open')).toBeVisible()
+      expect($('div.flash-alert')).toBeVisible()
+      expect($('div.flash-alert').text()).toBe('Unable to update this merge request at this time.')    
+    
+    it 'reopens a merge request', ->
+      $.ajax = (obj) ->
+        expect(obj.type).toBe('PUT')
+        expect(obj.url).toBe('http://gitlab.com/merge_requests/6/reopen')
+        obj.success saved: true
+
+      $btnClose = $('a.btn-close')
+      $btnReopen = $('a.btn-reopen')
+      expect($btnReopen.text()).toBe('Reopen')
+
+      $btnReopen.trigger('click')
+
+      expect($btnReopen).toBeHidden()
+      expect($btnClose).toBeVisible()
+      expect($('div.status-box-open')).toBeVisible()
+      expect($('div.status-box-closed')).toBeHidden()
\ No newline at end of file
-- 
cgit v1.2.1