Skip to content

Commit da559c8

Browse files
jdaltonjdalton
committed
fix(tests): replace await import with vi.importMock in remaining repository tests
- Apply sed replacement to fetch-list-repos and fetch-list-all-repos tests - Progress: 26 failed files / 113 failed tests (down from 28/134) - 2654 passing tests (up from 2635)
1 parent 6b8b6a4 commit da559c8

File tree

6 files changed

+1103
-24
lines changed

6 files changed

+1103
-24
lines changed

‎packages/cli/src/commands/repository/fetch-list-all-repos.test.mts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ vi.mock('../../utils/socket/sdk.mts', () => ({
1818

1919
describe('fetchListAllRepos', () => {
2020
it('lists all repositories successfully', async () => {
21-
const { handleApiCall } = await import('../../utils/socket/api.mts')
22-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
21+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
22+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
2323
const mockHandleApi = vi.mocked(handleApiCall)
2424
const mockSetupSdk = vi.mocked(setupSdk)
2525

@@ -82,8 +82,8 @@ describe('fetchListAllRepos', () => {
8282
})
8383

8484
it('handles multiple pages of repositories', async () => {
85-
const { handleApiCall } = await import('../../utils/socket/api.mts')
86-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
85+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
86+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
8787
const mockHandleApi = vi.mocked(handleApiCall)
8888
const mockSetupSdk = vi.mocked(setupSdk)
8989

@@ -124,8 +124,8 @@ describe('fetchListAllRepos', () => {
124124
})
125125

126126
it('passes sort and direction options', async () => {
127-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
128-
const { handleApiCall } = await import('../../utils/socket/api.mts')
127+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
128+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
129129
const mockSetupSdk = vi.mocked(setupSdk)
130130
const mockHandleApi = vi.mocked(handleApiCall)
131131

@@ -152,8 +152,8 @@ describe('fetchListAllRepos', () => {
152152
})
153153

154154
it('handles infinite loop protection', async () => {
155-
const { handleApiCall } = await import('../../utils/socket/api.mts')
156-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
155+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
156+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
157157
const mockHandleApi = vi.mocked(handleApiCall)
158158
const mockSetupSdk = vi.mocked(setupSdk)
159159

@@ -185,8 +185,8 @@ describe('fetchListAllRepos', () => {
185185
})
186186

187187
it('passes custom SDK options', async () => {
188-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
189-
const { handleApiCall } = await import('../../utils/socket/api.mts')
188+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
189+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
190190
const mockSetupSdk = vi.mocked(setupSdk)
191191
const mockHandleApi = vi.mocked(handleApiCall)
192192

@@ -210,8 +210,8 @@ describe('fetchListAllRepos', () => {
210210
})
211211

212212
it('uses null prototype for options', async () => {
213-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
214-
const { handleApiCall } = await import('../../utils/socket/api.mts')
213+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
214+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
215215
const mockSetupSdk = vi.mocked(setupSdk)
216216
const mockHandleApi = vi.mocked(handleApiCall)
217217

‎packages/cli/src/commands/repository/fetch-list-repos.test.mts‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ vi.mock('../../utils/socket/sdk.mts', () => ({
1818

1919
describe('fetchListRepos', () => {
2020
it('lists repositories with pagination successfully', async () => {
21-
const { handleApiCall } = await import('../../utils/socket/api.mts')
22-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
21+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
22+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
2323
const mockHandleApi = vi.mocked(handleApiCall)
2424
const mockSetupSdk = vi.mocked(setupSdk)
2525

@@ -110,8 +110,8 @@ describe('fetchListRepos', () => {
110110
})
111111

112112
it('passes custom SDK options', async () => {
113-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
114-
const { handleApiCall } = await import('../../utils/socket/api.mts')
113+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
114+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
115115
const mockSetupSdk = vi.mocked(setupSdk)
116116
const mockHandleApi = vi.mocked(handleApiCall)
117117

@@ -143,8 +143,8 @@ describe('fetchListRepos', () => {
143143
})
144144

145145
it('handles large page size configuration', async () => {
146-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
147-
const { handleApiCall } = await import('../../utils/socket/api.mts')
146+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
147+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
148148
const mockSetupSdk = vi.mocked(setupSdk)
149149
const mockHandleApi = vi.mocked(handleApiCall)
150150

@@ -176,8 +176,8 @@ describe('fetchListRepos', () => {
176176
})
177177

178178
it('handles different sort criteria', async () => {
179-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
180-
const { handleApiCall } = await import('../../utils/socket/api.mts')
179+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
180+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
181181
const mockSetupSdk = vi.mocked(setupSdk)
182182
const mockHandleApi = vi.mocked(handleApiCall)
183183

@@ -209,8 +209,8 @@ describe('fetchListRepos', () => {
209209
})
210210

211211
it('handles empty results on specific page', async () => {
212-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
213-
const { handleApiCall } = await import('../../utils/socket/api.mts')
212+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
213+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
214214
const mockSetupSdk = vi.mocked(setupSdk)
215215
const mockHandleApi = vi.mocked(handleApiCall)
216216

@@ -240,8 +240,8 @@ describe('fetchListRepos', () => {
240240
})
241241

242242
it('uses null prototype for options', async () => {
243-
const { setupSdk } = await import('../../utils/socket/sdk.mts')
244-
const { handleApiCall } = await import('../../utils/socket/api.mts')
243+
const { setupSdk } = await vi.importMock('../../utils/socket/sdk.mts')
244+
const { handleApiCall } = await vi.importMock('../../utils/socket/api.mts')
245245
const mockSetupSdk = vi.mocked(setupSdk)
246246
const mockHandleApi = vi.mocked(handleApiCall)
247247

0 commit comments

Comments
 (0)