Merge branch 'ph/widgetPipelinePollingFix' into 'master'

Remove pipeline from widget GraphQL request

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/167161



Merged-by: Jose Ivan Vargas <jvargas@gitlab.com>
Approved-by: Jose Ivan Vargas <jvargas@gitlab.com>
Co-authored-by: Phil Hughes <me@iamphill.com>

app/assets/javascripts/vue_merge_request_widget/queries/get_state.query.graphql

@@ -20,12 +20,6 @@ query getState($projectPath: ID!, $iid: String!) {
       mergeStatus
       mergeable
       mergeableDiscussionsState
-      headPipeline {
-        id
-        iid
-        status
-        warnings
-      }
       shouldBeRebased
       sourceBranchExists
       state

app/assets/javascripts/vue_merge_request_widget/stores/mr_widget_store.js

@@ -141,6 +141,7 @@ export default class MergeRequestStore {
     this.isPipelineSkipped = this.ciStatus === 'skipped';
     this.pipelineDetailedStatus = pipelineStatus;
     this.isPipelineActive = data.pipeline ? data.pipeline.active : false;
+    this.pipelineIid = data.pipeline?.iid;
     this.isPipelineBlocked =
       data.only_allow_merge_if_pipeline_succeeds && pipelineStatus?.group === 'manual';
     this.faviconOverlayPath = data.favicon_overlay_path;
@@ -181,7 +182,6 @@ export default class MergeRequestStore {
 
   setGraphqlData(project) {
     const { mergeRequest } = project;
-    const pipeline = mergeRequest.headPipeline;
 
     this.updateStatusState(mergeRequest.state);
 
@@ -193,11 +193,6 @@ export default class MergeRequestStore {
     this.autoMergeEnabled = mergeRequest.autoMergeEnabled;
     this.canBeMerged = mergeRequest.mergeStatus === 'can_be_merged';
     this.canMerge = mergeRequest.userPermissions.canMerge;
-    this.ciStatus = pipeline?.status.toLowerCase();
-
-    if (pipeline?.warnings && this.ciStatus === 'success') {
-      this.ciStatus = `${this.ciStatus}-with-warnings`;
-    }
 
     this.commitsCount = mergeRequest.commitCount;
     this.branchMissing =
@@ -213,7 +208,6 @@ export default class MergeRequestStore {
     this.draft = mergeRequest.draft;
     this.mergeRequestState = mergeRequest.state;
     this.detailedMergeStatus = mergeRequest.detailedMergeStatus;
-    this.pipelineIid = pipeline?.iid;
 
     this.setState();
   }

app/serializers/merge_requests/pipeline_entity.rb

@@ -4,6 +4,7 @@ class MergeRequests::PipelineEntity < Grape::Entity
   include RequestAwareEntity
 
   expose :id
+  expose :iid
   expose :active?, as: :active
   expose :name
 

spec/frontend/vue_merge_request_widget/mr_widget_options_spec.js

@@ -347,7 +347,7 @@ describe('MrWidgetOptions', () => {
           jest.spyOn(notify, 'notifyMe').mockImplementation(() => {});
           const logoFilename = 'logo.png';
           await createComponent({
-            updatedMrData: { gitlabLogo: logoFilename },
+            updatedMrData: { gitlabLogo: logoFilename, ci_status: 'failed' },
           });
           eventHub.$emit('MRWidgetUpdateRequested');
           await waitForPromises();
@@ -522,10 +522,10 @@ describe('MrWidgetOptions', () => {
           beforeEach(() => {
             mock
               .onGet(mockData.merge_request_widget_path)
-              .reply(HTTP_STATUS_OK, { ...mockData, ...updatedMrData });
+              .reply(HTTP_STATUS_OK, { ...mockData, ...updatedMrData, ci_status: 'failed' });
             mock
               .onGet(mockData.merge_request_cached_widget_path)
-              .reply(HTTP_STATUS_OK, { ...mockData, ...updatedMrData });
+              .reply(HTTP_STATUS_OK, { ...mockData, ...updatedMrData, ci_status: 'failed' });
           });
 
           it('should call notifyMe', async () => {