fixed breaking spec

pullflow · Feb 15, 2024 · f514c47 · f514c47
1 parent 3f3a385
commit f514c47
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/src/pullRequestQuickActions/pullRequestQuickActions.test.ts b/src/pullRequestQuickActions/pullRequestQuickActions.test.ts
@@ -39,6 +39,7 @@ describe('Pull Request Quick Actions', () => {
   })
 
   it('adds assignee to pull request', async () => {
+    mockSpaceUsers.SpaceUsers.get.mockReturnValue([])
     await subject().addAssignee({
       codeReview: mockModel,
       context: mockModel,
@@ -49,13 +50,15 @@ describe('Pull Request Quick Actions', () => {
         context: mockModel,
         placeholder: 'Select a user',
         title: 'Add assignee to pull request',
+        spaceUsers: [],
       })
     )
     expect(mockPresence.Presence.set).toBeCalled()
     expect(mockPullRequestState.PullRequestState.updateWithDelay).toBeCalled()
   })
 
   it('adds reviewer on pull request', async () => {
+    mockSpaceUsers.SpaceUsers.get.mockReturnValue([])
     await subject().requestReview({
       codeReview: mockModel,
       context: mockModel,
@@ -66,6 +69,7 @@ describe('Pull Request Quick Actions', () => {
         context: mockModel,
         placeholder: 'Select a user',
         title: 'Add reviewer to pull request',
+        spaceUsers: [],
       })
     )
     expect(mockPresence.Presence.set).toBeCalled()
@@ -84,6 +88,7 @@ const subject = () => {
   jest.mock('../utils/pullRequestsState', () => mockPullRequestState)
   jest.mock('../views/quickpicks/spaceUserPicker', () => mockSpaceUserPicker)
   jest.mock('../views/quickpicks/timePicker', () => mockTimerPicker)
+  jest.mock('../models/spaceUsers', () => mockSpaceUsers)
   return require('./pullRequestQuickActions').PullRequestQuickActions
 }
 
@@ -109,6 +114,11 @@ const mockPullRequestState = {
     updateWithDelay: jest.fn(),
   },
 }
+const mockSpaceUsers = {
+  SpaceUsers: {
+    get: jest.fn(),
+  },
+}
 const mockSpaceUserPicker = {
   spaceUserPicker: jest.fn(),
 }