From 1183e5517bee3a8c5b8925846b7dbcb354006606 Mon Sep 17 00:00:00 2001
From: Siddarth Kumar <siddarthkay@gmail.com>
Date: Tue, 1 Oct 2024 09:22:50 +0530
Subject: [PATCH] chore!: try to fetch both teams from github

---
 .github/workflows/breaking-change-review.yml | 80 ++++++++++++++++----
 1 file changed, 66 insertions(+), 14 deletions(-)

diff --git a/.github/workflows/breaking-change-review.yml b/.github/workflows/breaking-change-review.yml
index 328388d457a..928159451ab 100644
--- a/.github/workflows/breaking-change-review.yml
+++ b/.github/workflows/breaking-change-review.yml
@@ -18,7 +18,7 @@ jobs:
   check_breaking_change:
     runs-on: ubuntu-latest
     steps:
-      - name: Check for breaking_change label
+      - name: Check for breaking change label
         id: check_label
         uses: actions/github-script@v6
         with:
@@ -30,20 +30,48 @@ jobs:
               issue_number: context.issue.number,
             });
             const hasBreakingChange = labels.some(label => label.name === 'breaking change');
-            console.log(`Has breaking_change label: ${hasBreakingChange}`);
+            console.log(`Has breaking change label: ${hasBreakingChange}`);
             return hasBreakingChange;
 
+      - name: Fetch QA team members
+        if: steps.check_label.outputs.result == 'true'
+        id: fetch_qa_teams
+        uses: actions/github-script@v6
+        with:
+          github-token: ${{secrets.GITHUB_TOKEN}}
+          script: |
+            async function getTeamMembers(teamSlug) {
+              const { data: members } = await github.rest.teams.listMembersInOrg({
+                org: 'status-im',
+                team_slug: teamSlug,
+              });
+              return members.map(member => member.login);
+            }
+            
+            const mobileQA = await getTeamMembers('mobile-qa');
+            const desktopQA = await getTeamMembers('desktop-qa');
+            
+            console.log('Mobile QA team:', mobileQA);
+            console.log('Desktop QA team:', desktopQA);
+            
+            core.setOutput('mobile_qa', mobileQA.join(','));
+            core.setOutput('desktop_qa', desktopQA.join(','));
+
       - name: Request QA reviews
         if: steps.check_label.outputs.result == 'true'
         uses: actions/github-script@v6
         with:
           github-token: ${{secrets.GITHUB_TOKEN}}
           script: |
+            const mobileQA = '${{ steps.fetch_qa_teams.outputs.mobile_qa }}'.split(',');
+            const desktopQA = '${{ steps.fetch_qa_teams.outputs.desktop_qa }}'.split(',');
+            const reviewers = [...mobileQA, ...desktopQA];
+            
             await github.rest.pulls.requestReviewers({
               owner: context.repo.owner,
               repo: context.repo.repo,
               pull_number: context.issue.number,
-              reviewers: ['churik']
+              reviewers: reviewers
             });
 
       - name: Check QA approvals
@@ -59,17 +87,23 @@ jobs:
               pull_number: context.issue.number,
             });
             
-            const desktopQAApproved = reviews.some(review => 
-              review.state === 'APPROVED' && review.user.login.includes('desktop-qa')
-            );
+            const mobileQA = '${{ steps.fetch_qa_teams.outputs.mobile_qa }}'.split(',');
+            const desktopQA = '${{ steps.fetch_qa_teams.outputs.desktop_qa }}'.split(',');
+            
             const mobileQAApproved = reviews.some(review => 
-              review.state === 'APPROVED' && review.user.login.includes('mobile-qa')
+              review.state === 'APPROVED' && mobileQA.includes(review.user.login)
+            );
+            const desktopQAApproved = reviews.some(review => 
+              review.state === 'APPROVED' && desktopQA.includes(review.user.login)
             );
             
-            console.log(`Desktop QA approved: ${desktopQAApproved}`);
             console.log(`Mobile QA approved: ${mobileQAApproved}`);
+            console.log(`Desktop QA approved: ${desktopQAApproved}`);
+            
+            core.setOutput('mobile-qa-approved', mobileQAApproved);
+            core.setOutput('desktop-qa-approved', desktopQAApproved);
             
-            return desktopQAApproved && mobileQAApproved;
+            return mobileQAApproved && desktopQAApproved;
 
       - name: Block PR if conditions not met
         if: steps.check_label.outputs.result == 'true' && steps.check_approvals.outputs.result != 'true'
@@ -77,12 +111,30 @@ jobs:
         with:
           github-token: ${{secrets.GITHUB_TOKEN}}
           script: |
-            await github.rest.pulls.createReview({
+            const mobileQAApproved = ${{ steps.check_approvals.outputs.mobile-qa-approved }};
+            const desktopQAApproved = ${{ steps.check_approvals.outputs.desktop-qa-approved }};
+            
+            let message = 'This PR has the breaking change label and requires approval from both mobile-qa and desktop-qa teams before it can be merged.\n\n';
+            
+            if (!mobileQAApproved && !desktopQAApproved) {
+              message += 'Both mobile-qa and desktop-qa teams have not approved this PR yet.';
+            } else if (!mobileQAApproved) {
+              message += 'The mobile-qa team has not approved this PR yet.';
+            } else if (!desktopQAApproved) {
+              message += 'The desktop-qa team has not approved this PR yet.';
+            }
+            
+            await github.rest.checks.create({
               owner: context.repo.owner,
               repo: context.repo.repo,
-              pull_number: context.issue.number,
-              event: 'REQUEST_CHANGES',
-              body: 'This PR has the breaking_change label and requires approval from both @status-im/desktop-qa and @status-im/mobile-qa teams before it can be merged.'
+              name: 'QA Approval Check',
+              head_sha: context.payload.pull_request.head.sha,
+              status: 'completed',
+              conclusion: 'failure',
+              output: {
+                title: 'QA Approval Required',
+                summary: message
+              }
             });
 
       - name: Allow PR merge if conditions are met
@@ -96,5 +148,5 @@ jobs:
               repo: context.repo.repo,
               pull_number: context.issue.number,
               event: 'APPROVE',
-              body: 'All conditions have been met. This PR can now be merged.'
+              body: 'Breaking changes have been tested by Mobile and Desktop QA teams. This PR can now be merged.'
             });
\ No newline at end of file