refactor

ExpediaGroup · Jan 3, 2024 · e8fcefd · e8fcefd
1 parent 1d12eb9
commit e8fcefd
Showing 1 changed file with 7 additions and 10 deletions.
diff --git a/test/helpers/manage-merge-queue.test.ts b/test/helpers/manage-merge-queue.test.ts
@@ -149,16 +149,17 @@ describe('manageMergeQueue', () => {
 
   describe('pr ready for merge case queued #1', () => {
     const queuedPrs = [{ labels: [{ name: READY_FOR_MERGE_PR_LABEL }] }];
+    const pullRequest = {
+      merged: false,
+      head: { sha: 'sha' },
+      labels: [{ name: READY_FOR_MERGE_PR_LABEL }]
+    };
     beforeEach(async () => {
       (octokit.pulls.list as unknown as Mocktokit).mockImplementation(async ({ page }) => ({
         data: page === 1 ? queuedPrs : []
       }));
       (octokit.pulls.get as unknown as Mocktokit).mockImplementation(async () => ({
-        data: {
-          merged: false,
-          head: { sha: 'sha' },
-          labels: [{ name: READY_FOR_MERGE_PR_LABEL }]
-        }
+        data: pullRequest
       }));
       await manageMergeQueue();
     });
@@ -181,11 +182,7 @@ describe('manageMergeQueue', () => {
     });
 
     it('should update PR with default branch', () => {
-      expect(updatePrWithDefaultBranch).toHaveBeenCalledWith({
-        merged: false,
-        head: { sha: 'sha' },
-        labels: [{ name: READY_FOR_MERGE_PR_LABEL }]
-      });
+      expect(updatePrWithDefaultBranch).toHaveBeenCalledWith(pullRequest);
     });
 
     it('should enable auto-merge', () => {