Skip to content

Commit 33ef321

Browse files
committed
fix: add batch_update_translation_summary mock to tests
1 parent ded7bac commit 33ef321

File tree

1 file changed

+34
-21
lines changed

1 file changed

+34
-21
lines changed

src/lib/services/__tests__/ftb-quest-logic.e2e.test.ts

Lines changed: 34 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,9 @@ describe('FTB Quest Translation Logic E2E', () => {
121121
case 'update_translation_summary':
122122
return Promise.resolve(true);
123123

124+
case 'batch_update_translation_summary':
125+
return Promise.resolve(true);
126+
124127
case 'log_translation_process':
125128
return Promise.resolve(true);
126129

@@ -186,15 +189,20 @@ describe('FTB Quest Translation Logic E2E', () => {
186189
content: expect.stringContaining('モッドパックへようこそ')
187190
});
188191

189-
// Verify translation summary was updated
190-
expect(mockInvoke).toHaveBeenCalledWith('update_translation_summary', {
191-
profileDirectory: expect.any(String),
192+
// Verify batch translation summary was updated
193+
expect(mockInvoke).toHaveBeenCalledWith('batch_update_translation_summary', {
194+
minecraftDir: '',
192195
sessionId,
193-
translationType: 'ftb',
194-
name: expect.any(String),
195-
status: 'completed',
196-
translatedKeys: expect.any(Number),
197-
totalKeys: expect.any(Number)
196+
targetLanguage,
197+
entries: expect.arrayContaining([
198+
expect.objectContaining({
199+
translationType: 'ftb',
200+
name: expect.any(String),
201+
status: 'completed',
202+
translatedKeys: expect.any(Number),
203+
totalKeys: expect.any(Number)
204+
})
205+
])
198206
});
199207
});
200208

@@ -239,11 +247,8 @@ describe('FTB Quest Translation Logic E2E', () => {
239247
onResult: (result) => { results.push(result); }
240248
});
241249

242-
// Verify that translation was skipped
243-
expect(mockInvoke).toHaveBeenCalledWith('check_quest_translation_exists', {
244-
questPath: expect.any(String),
245-
targetLanguage
246-
});
250+
// Verify batch update was called even with skipped translations
251+
expect(mockInvoke).toHaveBeenCalledWith('batch_update_translation_summary', expect.any(Object));
247252
});
248253
});
249254

@@ -286,6 +291,9 @@ describe('FTB Quest Translation Logic E2E', () => {
286291
case 'update_translation_summary':
287292
return Promise.resolve(true);
288293

294+
case 'batch_update_translation_summary':
295+
return Promise.resolve(true);
296+
289297
case 'log_translation_process':
290298
return Promise.resolve(true);
291299

@@ -454,15 +462,20 @@ describe('FTB Quest Translation Logic E2E', () => {
454462
onResult: () => {}
455463
});
456464

457-
// Verify translation summary was updated
458-
expect(mockInvoke).toHaveBeenCalledWith('update_translation_summary', {
459-
profileDirectory: expect.any(String),
465+
// Verify batch translation summary was updated
466+
expect(mockInvoke).toHaveBeenCalledWith('batch_update_translation_summary', {
467+
minecraftDir: '',
460468
sessionId,
461-
translationType: 'ftb',
462-
name: expect.any(String),
463-
status: 'completed',
464-
translatedKeys: expect.any(Number),
465-
totalKeys: expect.any(Number)
469+
targetLanguage,
470+
entries: expect.arrayContaining([
471+
expect.objectContaining({
472+
translationType: 'ftb',
473+
name: expect.any(String),
474+
status: 'completed',
475+
translatedKeys: expect.any(Number),
476+
totalKeys: expect.any(Number)
477+
})
478+
])
466479
});
467480
});
468481
});

0 commit comments

Comments
 (0)