Added support for Jenkins unstable status

CHANGELOG-EE

@@ -2,7 +2,8 @@ v 7.14
   - Disable adding, updating and removing members from a group that is synced with LDAP
   - Don't send "Added to group" notifications when group is LDAP synched
   - Fix importing projects from GitHub Enterprise Edition.
-  - Automatic approver suggestions (based on an authority of the code)  
+  - Automatic approver suggestions (based on an authority of the code)
+  - Add support for Jenkins unstable status
 
 v7.13.3
   - Merge community edition changes for version 7.13.3
@@ -21,7 +22,7 @@ v 7.13.1
 v 7.13
   - Fix git hook validation on initial push to master branch.
   - Reset approvals on push
-  - Fix 500 error when the source project of an MR is deleted 
+  - Fix 500 error when the source project of an MR is deleted
   - Ability to define merge request approvers
 
 v 7.12.2
@@ -34,7 +35,7 @@ v 7.12.0
   - Fix error when viewing merge request with a commit that includes "Closes #<issue id>".
   - Enhance LDAP group synchronization to check also for member attributes that only contain "uid=<username>"
   - Enhance LDAP group synchronization to check also for submember attributes
-  - Prevent LDAP group sync from removing a group's last owner 
+  - Prevent LDAP group sync from removing a group's last owner
   - Add Git hook to validate maximum file size.
   - Project setting: approve merge request by N users before accept
 

app/models/project_services/jenkins_service.rb

@@ -71,7 +71,7 @@ class JenkinsService < CiService
     if response.code == 200
       # img.build-caption-status-icon for old jenkins version
       src = Nokogiri.parse(response).css('img.build-caption-status-icon,.build-caption>img').first.attributes['src'].value
-      if src =~ /blue\.png$/
+      if src =~ /(blue\.png|yellow\.png)$/
         'success'
       elsif src =~ /(red\.png|aborted\.png)$/
         'failed'

spec/models/project_services/jenkins_service_spec.rb

@@ -26,6 +26,15 @@ describe JenkinsService do
   end
 
   describe 'commits methods' do
+    def status_body_for_icon(state)
+      body =<<eos
+        <h1 class="build-caption page-headline"><img style="width: 48px; height: 48px; " alt="Success" class="icon-#{state} icon-xlg" src="/static/855d7c3c/images/48x48/#{state}" tooltip="Success" title="Success">
+                Build #188
+              (Oct 15, 2014 9:45:21 PM)
+                    </h1>
+eos
+    end
+
     before do
       @service = JenkinsService.new
       allow(@service).to receive_messages(
@@ -33,19 +42,16 @@ describe JenkinsService do
         project_url: 'http://jenkins.gitlab.org/projects/2',
         token: 'verySecret'
       )
-
-      body =<<eos
-        <h1 class="build-caption page-headline"><img style="width: 48px; height: 48px; " alt="Success" class="icon-blue icon-xlg" src="/static/855d7c3c/images/48x48/blue.png" tooltip="Success" title="Success">
-                Build #188
-              (Oct 15, 2014 9:45:21 PM)
-                    </h1>
-eos
-      stub_request(:get, "http://jenkins.gitlab.org/projects/2/scm/bySHA1/2ab7834c").
-        to_return(status: 200, body: body, headers: {})
     end
 
     describe :commit_status do
-      it { expect(@service.commit_status("2ab7834c", 'master')).to eq("success") }
+      statuses = {'blue.png' => 'success', 'yellow.png' => 'success', 'red.png' => 'failed', 'aborted.png' => 'failed', 'blue-anime.gif' => 'running', 'grey.png' => 'pending'}
+      statuses.each do |icon, state|
+        it "should have a status of #{state} when the icon #{icon} exists." do
+          stub_request(:get, "http://jenkins.gitlab.org/projects/2/scm/bySHA1/2ab7834c").to_return(status: 200, body: status_body_for_icon(icon), headers: {})
+          expect(@service.commit_status("2ab7834c", 'master')).to eq(state)
+        end
+      end
     end
 
     describe :build_page do