Skip to content

Commit 589b5db

Browse files
harshitha-cstkharshitha-cstk
committed
refactor: update test assertions to use 'items' instead of 'entries' for sync operations
1 parent 163c327 commit 589b5db

1 file changed

Lines changed: 57 additions & 56 deletions

File tree

‎test/api/sync-operations-comprehensive.spec.ts‎

Lines changed: 57 additions & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,13 @@ describe('Sync Operations Comprehensive Tests', () => {
5252
}
5353

5454
expect(result).toBeDefined();
55-
expect(result.entries).toBeDefined();
56-
expect(Array.isArray(result.entries)).toBe(true);
55+
expect(result.items).toBeDefined();
56+
expect(Array.isArray(result.items)).toBe(true);
5757
expect(result.sync_token).toBeDefined();
5858

5959
console.log('Initial sync completed:', {
6060
duration: `${duration}ms`,
61-
entriesCount: result.entries.length,
61+
entriesCount: result.items.length,
6262
syncToken: result.sync_token,
6363
contentType: COMPLEX_CT
6464
});
@@ -81,18 +81,18 @@ describe('Sync Operations Comprehensive Tests', () => {
8181
}
8282

8383
expect(result).toBeDefined();
84-
expect(result.entries).toBeDefined();
85-
expect(Array.isArray(result.entries)).toBe(true);
84+
expect(result.items).toBeDefined();
85+
expect(Array.isArray(result.items)).toBe(true);
8686
expect(result.sync_token).toBeDefined();
8787

8888
console.log('Initial sync (all content types):', {
8989
duration: `${duration}ms`,
90-
entriesCount: result.entries.length,
90+
entriesCount: result.items.length,
9191
syncToken: result.sync_token
9292
});
9393

9494
// Should get more entries without content type filter
95-
expect(result.entries.length).toBeGreaterThanOrEqual(0);
95+
expect(result.items.length).toBeGreaterThanOrEqual(0);
9696
});
9797

9898
it('should perform initial sync with locale filter', async () => {
@@ -112,20 +112,20 @@ describe('Sync Operations Comprehensive Tests', () => {
112112
}
113113

114114
expect(result).toBeDefined();
115-
expect(result.entries).toBeDefined();
116-
expect(Array.isArray(result.entries)).toBe(true);
115+
expect(result.items).toBeDefined();
116+
expect(Array.isArray(result.items)).toBe(true);
117117
expect(result.sync_token).toBeDefined();
118118

119119
console.log('Initial sync with locale filter:', {
120120
duration: `${duration}ms`,
121-
entriesCount: result.entries.length,
121+
entriesCount: result.items.length,
122122
syncToken: result.sync_token,
123123
locale: 'en-us'
124124
});
125125

126126
// Verify entries are in the specified locale
127-
if (result.entries.length > 0) {
128-
result.entries.forEach((entry: any) => {
127+
if (result.items.length > 0) {
128+
result.items.forEach((entry: any) => {
129129
if (entry.locale) {
130130
expect(entry.locale).toBe('en-us');
131131
}
@@ -169,14 +169,14 @@ describe('Sync Operations Comprehensive Tests', () => {
169169
}
170170

171171
expect(result).toBeDefined();
172-
expect(result.entries).toBeDefined();
173-
expect(Array.isArray(result.entries)).toBe(true);
172+
expect(result.items).toBeDefined();
173+
expect(Array.isArray(result.items)).toBe(true);
174174
expect(result.sync_token).toBeDefined();
175175
expect(result.sync_token).not.toBe(initialSyncToken);
176176

177177
console.log('Delta sync completed:', {
178178
duration: `${duration}ms`,
179-
entriesCount: result.entries.length,
179+
entriesCount: result.items.length,
180180
newSyncToken: result.sync_token,
181181
previousSyncToken: initialSyncToken
182182
});
@@ -198,16 +198,16 @@ describe('Sync Operations Comprehensive Tests', () => {
198198
}));
199199

200200
expect(result).toBeDefined();
201-
expect(result.entries).toBeDefined();
202-
expect(Array.isArray(result.entries)).toBe(true);
201+
expect(result.items).toBeDefined();
202+
expect(Array.isArray(result.items)).toBe(true);
203203

204204
console.log('Delta sync (no changes):', {
205-
entriesCount: result.entries.length,
205+
entriesCount: result.items.length,
206206
syncToken: result.sync_token
207207
});
208208

209209
// Should handle no changes gracefully
210-
expect(result.entries.length).toBeGreaterThanOrEqual(0);
210+
expect(result.items.length).toBeGreaterThanOrEqual(0);
211211
});
212212

213213
it('should perform multiple delta syncs', async () => {
@@ -228,7 +228,7 @@ describe('Sync Operations Comprehensive Tests', () => {
228228

229229
syncResults.push({
230230
iteration: i + 1,
231-
entriesCount: result.entries.length,
231+
entriesCount: result.items.length,
232232
syncToken: result.sync_token
233233
});
234234

@@ -261,19 +261,19 @@ describe('Sync Operations Comprehensive Tests', () => {
261261
}
262262

263263
expect(result).toBeDefined();
264-
expect(result.entries).toBeDefined();
265-
expect(Array.isArray(result.entries)).toBe(true);
264+
expect(result.items).toBeDefined();
265+
expect(Array.isArray(result.items)).toBe(true);
266266
expect(result.sync_token).toBeDefined();
267267

268268
console.log('Sync with pagination:', {
269269
duration: `${duration}ms`,
270-
entriesCount: result.entries.length,
270+
entriesCount: result.items.length,
271271
limit: 5,
272272
syncToken: result.sync_token
273273
});
274274

275275
// Should respect the limit
276-
expect(result.entries.length).toBeLessThanOrEqual(5);
276+
expect(result.items.length).toBeLessThanOrEqual(5);
277277
});
278278

279279
it('should handle sync pagination with skip', async () => {
@@ -292,20 +292,20 @@ describe('Sync Operations Comprehensive Tests', () => {
292292
}
293293

294294
expect(result).toBeDefined();
295-
expect(result.entries).toBeDefined();
296-
expect(Array.isArray(result.entries)).toBe(true);
295+
expect(result.items).toBeDefined();
296+
expect(Array.isArray(result.items)).toBe(true);
297297
expect(result.sync_token).toBeDefined();
298298

299299
console.log('Sync with pagination and skip:', {
300300
duration: `${duration}ms`,
301-
entriesCount: result.entries.length,
301+
entriesCount: result.items.length,
302302
limit: 3,
303303
skip: 2,
304304
syncToken: result.sync_token
305305
});
306306

307307
// Should respect both limit and skip
308-
expect(result.entries.length).toBeLessThanOrEqual(3);
308+
expect(result.items.length).toBeLessThanOrEqual(3);
309309
});
310310
});
311311

@@ -326,20 +326,20 @@ describe('Sync Operations Comprehensive Tests', () => {
326326
}
327327

328328
expect(result).toBeDefined();
329-
expect(result.entries).toBeDefined();
330-
expect(Array.isArray(result.entries)).toBe(true);
329+
expect(result.items).toBeDefined();
330+
expect(Array.isArray(result.items)).toBe(true);
331331
expect(result.sync_token).toBeDefined();
332332

333333
console.log('Sync with multiple content types:', {
334334
duration: `${duration}ms`,
335-
entriesCount: result.entries.length,
335+
entriesCount: result.items.length,
336336
contentTypes: [COMPLEX_CT, MEDIUM_CT],
337337
syncToken: result.sync_token
338338
});
339339

340340
// Verify entries belong to specified content types
341-
if (result.entries.length > 0) {
342-
result.entries.forEach((entry: any) => {
341+
if (result.items.length > 0) {
342+
result.items.forEach((entry: any) => {
343343
expect([COMPLEX_CT, MEDIUM_CT]).toContain(entry._content_type_uid);
344344
});
345345
}
@@ -362,13 +362,13 @@ describe('Sync Operations Comprehensive Tests', () => {
362362
}
363363

364364
expect(result).toBeDefined();
365-
expect(result.entries).toBeDefined();
366-
expect(Array.isArray(result.entries)).toBe(true);
365+
expect(result.items).toBeDefined();
366+
expect(Array.isArray(result.items)).toBe(true);
367367
expect(result.sync_token).toBeDefined();
368368

369369
console.log('Sync with environment filter:', {
370370
duration: `${duration}ms`,
371-
entriesCount: result.entries.length,
371+
entriesCount: result.items.length,
372372
environment: process.env.ENVIRONMENT || 'development',
373373
syncToken: result.sync_token
374374
});
@@ -391,13 +391,13 @@ describe('Sync Operations Comprehensive Tests', () => {
391391
}
392392

393393
expect(result).toBeDefined();
394-
expect(result.entries).toBeDefined();
395-
expect(Array.isArray(result.entries)).toBe(true);
394+
expect(result.items).toBeDefined();
395+
expect(Array.isArray(result.items)).toBe(true);
396396
expect(result.sync_token).toBeDefined();
397397

398398
console.log('Sync with publish type filter:', {
399399
duration: `${duration}ms`,
400-
entriesCount: result.entries.length,
400+
entriesCount: result.items.length,
401401
publishType: 'entry_published',
402402
syncToken: result.sync_token
403403
});
@@ -419,14 +419,14 @@ describe('Sync Operations Comprehensive Tests', () => {
419419
}
420420

421421
expect(result).toBeDefined();
422-
expect(result.entries).toBeDefined();
423-
expect(Array.isArray(result.entries)).toBe(true);
422+
expect(result.items).toBeDefined();
423+
expect(Array.isArray(result.items)).toBe(true);
424424

425425
console.log('Large sync performance:', {
426426
duration: `${duration}ms`,
427-
entriesCount: result.entries.length,
427+
entriesCount: result.items.length,
428428
limit: 50,
429-
avgTimePerEntry: result.entries.length > 0 ? (duration / result.entries.length).toFixed(2) + 'ms' : 'N/A'
429+
avgTimePerEntry: result.items.length > 0 ? (duration / result.items.length).toFixed(2) + 'ms' : 'N/A'
430430
});
431431

432432
// Performance should be reasonable
@@ -462,8 +462,8 @@ describe('Sync Operations Comprehensive Tests', () => {
462462
console.log('Sync performance comparison:', {
463463
initialSync: `${initialTime}ms`,
464464
deltaSync: `${deltaTime}ms`,
465-
initialEntries: initialResult.entries.length,
466-
deltaEntries: deltaResult.entries.length,
465+
initialEntries: initialResult.items.length,
466+
deltaEntries: deltaResult.items.length,
467467
ratio: initialTime / deltaTime
468468
});
469469

@@ -498,16 +498,16 @@ describe('Sync Operations Comprehensive Tests', () => {
498498
expect(validResults.length).toBeGreaterThan(0);
499499

500500
results.forEach((result, index) => {
501-
expect(result.entries).toBeDefined();
502-
expect(Array.isArray(result.entries)).toBe(true);
501+
expect(result.items).toBeDefined();
502+
expect(Array.isArray(result.items)).toBe(true);
503503
expect(result.sync_token).toBeDefined();
504504
});
505505

506506
console.log('Concurrent sync operations:', {
507507
duration: `${duration}ms`,
508508
results: results.map((r, i) => ({
509509
contentType: [COMPLEX_CT, MEDIUM_CT, SIMPLE_CT][i],
510-
entriesCount: r.entries.length
510+
entriesCount: r.items.length
511511
}))
512512
});
513513

@@ -525,7 +525,7 @@ describe('Sync Operations Comprehensive Tests', () => {
525525
}));
526526

527527
console.log('Invalid sync token handled:', {
528-
entriesCount: result.entries.length,
528+
entriesCount: result.items.length,
529529
syncToken: result.sync_token
530530
});
531531
} catch (error) {
@@ -541,12 +541,12 @@ describe('Sync Operations Comprehensive Tests', () => {
541541
}))
542542

543543
expect(result).toBeDefined();
544-
expect(result.entries).toBeDefined();
545-
expect(Array.isArray(result.entries)).toBe(true);
546-
expect(result.entries.length).toBe(0);
544+
expect(result.items).toBeDefined();
545+
expect(Array.isArray(result.items)).toBe(true);
546+
expect(result.items.length).toBe(0);
547547

548548
console.log('Non-existent content type handled:', {
549-
entriesCount: result.entries.length,
549+
entriesCount: result.items.length,
550550
syncToken: result.sync_token
551551
});
552552
} catch (error) {
@@ -564,7 +564,7 @@ describe('Sync Operations Comprehensive Tests', () => {
564564
for (const params of invalidParams) {
565565
try {
566566
const result = await safeSyncOperation(() => stack.sync(params as any));
567-
console.log('Invalid params handled:', { params, entriesCount: result.entries.length });
567+
console.log('Invalid params handled:', { params, entriesCount: result.items.length });
568568
} catch (error) {
569569
console.log('Invalid params properly rejected:', { params, error: (error as Error).message });
570570
}
@@ -582,7 +582,7 @@ describe('Sync Operations Comprehensive Tests', () => {
582582

583583
console.log('Large sync completed:', {
584584
duration: `${duration}ms`,
585-
entriesCount: result.entries.length
585+
entriesCount: result.items.length
586586
});
587587

588588
// Should complete within reasonable time
@@ -628,6 +628,7 @@ describe('Sync Operations Comprehensive Tests', () => {
628628
return;
629629
}
630630

631+
console.log("🚀 ~ deltaResult:", deltaResult)
631632
expect(deltaResult.sync_token).toBeDefined();
632633
expect(typeof deltaResult.sync_token).toBe('string');
633634
expect(deltaResult.sync_token).not.toBe(initialResult.sync_token);
@@ -654,7 +655,7 @@ describe('Sync Operations Comprehensive Tests', () => {
654655
}));
655656

656657
console.log('Sync token still valid:', {
657-
entriesCount: result.entries.length,
658+
entriesCount: result.items.length,
658659
newToken: result.sync_token
659660
});
660661
} catch (error) {

0 commit comments

Comments
 (0)