Use CI commit status for merge request widgetcloser-ci-integration

For temporary compatibility with other services like Jenkins we ask for
CI status via AJAX request if there is no commit status in GitLab
database

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

2 files changed, 37 insertions, 22 deletions

diff --git a/CHANGELOG b/CHANGELOG
index 3144175db89..04e3807451b 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -33,6 +33,7 @@ v 8.1.0 (unreleased)
   - Move CI web hooks page to project settings area
   - Fix User Identities API. It now allows you to properly create or update user's identities.
   - Add user preference to change layout width (Peter Göbel)
+  - Use commit status in merge request widget as preffered source of CI status
 
 v 8.0.3
   - Fix URL shown in Slack notifications
diff --git a/app/views/projects/merge_requests/widget/_heading.html.haml b/app/views/projects/merge_requests/widget/_heading.html.haml
index 10640f746f0..68dda1424cf 100644
--- a/app/views/projects/merge_requests/widget/_heading.html.haml
+++ b/app/views/projects/merge_requests/widget/_heading.html.haml
@@ -1,30 +1,44 @@
 - if @merge_request.has_ci?
-  .mr-widget-heading
-    - [:success, :skipped, :canceled, :failed, :running, :pending].each do |status|
-      .ci_widget{class: "ci-#{status}", style: "display:none"}
-        - if status == :success
-          - status = "passed"
-          = icon("check-circle")
-        - else
-          = icon("circle")
+  - ci_commit = @merge_request.source_project.ci_commit(@merge_request.source_sha)
+  - if ci_commit
+    - status = ci_commit.status
+    .mr-widget-heading
+      .ci_widget{class: "ci-#{status}"}
+        = ci_status_icon(ci_commit)
         %span CI build #{status}
         for #{@merge_request.last_commit_short_sha}.
         %span.ci-coverage
-        - if ci_build_details_path(@merge_request)
-          = link_to "View build details", ci_build_details_path(@merge_request), :"data-no-turbolink" => "data-no-turbolink"
+        = link_to "View build details", ci_status_path(ci_commit)
 
-    .ci_widget
-      = icon("spinner spin")
-      Checking CI status for #{@merge_request.last_commit_short_sha}…
+  - else
+    - # Compatibility with old CI integrations (ex jenkins) when you request status from CI server via AJAX
+    - # Remove in later versions when services like Jenkins will set CI status via Commit status API
+    .mr-widget-heading
+      - [:success, :skipped, :canceled, :failed, :running, :pending].each do |status|
+        .ci_widget{class: "ci-#{status}", style: "display:none"}
+          - if status == :success
+            - status = "passed"
+            = icon("check-circle")
+          - else
+            = icon("circle")
+          %span CI build #{status}
+          for #{@merge_request.last_commit_short_sha}.
+          %span.ci-coverage
+          - if ci_build_details_path(@merge_request)
+            = link_to "View build details", ci_build_details_path(@merge_request), :"data-no-turbolink" => "data-no-turbolink"
 
-    .ci_widget.ci-not_found{style: "display:none"}
-      = icon("times-circle")
-      Could not find CI status for #{@merge_request.last_commit_short_sha}.
+      .ci_widget
+        = icon("spinner spin")
+        Checking CI status for #{@merge_request.last_commit_short_sha}…
 
-    .ci_widget.ci-error{style: "display:none"}
-      = icon("times-circle")
-      Could not connect to the CI server. Please check your settings and try again.
+      .ci_widget.ci-not_found{style: "display:none"}
+        = icon("times-circle")
+        Could not find CI status for #{@merge_request.last_commit_short_sha}.
 
-  :coffeescript
-    $ ->
-      merge_request_widget.getCiStatus()
+      .ci_widget.ci-error{style: "display:none"}
+        = icon("times-circle")
+        Could not connect to the CI server. Please check your settings and try again.
+
+    :coffeescript
+      $ ->
+        merge_request_widget.getCiStatus()