Эх сурвалжийг харах

Merge pull request #16659 from overleaf/jpa-loose-progress-assertion

[document-updater] loose assertion on progress with low LIMIT

GitOrigin-RevId: f5046e0ed46b889e5a49a69b9752961f847d361a
Jakob Ackermann 2 жил өмнө
parent
commit
216e3b2091

+ 5 - 4
services/document-updater/test/acceptance/js/CheckRedisMongoSyncStateTests.js

@@ -245,17 +245,18 @@ describe('CheckRedisMongoSyncState', function () {
     }
     }
 
 
     it('should flag limit', async function () {
     it('should flag limit', async function () {
-      const result = await runScript({ LIMIT: '2' })
+      const result = await runScript({ LIMIT: '4' })
       expect(result.code).to.equal(2)
       expect(result.code).to.equal(2)
-      expect(result.stdout).to.include('Processed 2 projects')
+      // A redis SCAN may return more than COUNT (aka LIMIT) entries. Match loosely.
+      expect(result.stdout).to.match(/Processed \d+ projects/)
       expect(result.stderr).to.include(
       expect(result.stderr).to.include(
-        'Found too many un-flushed projects (LIMIT=2). Please fix the reported projects first, then try again.'
+        'Found too many un-flushed projects (LIMIT=4). Please fix the reported projects first, then try again.'
       )
       )
     })
     })
 
 
     it('should continue with auto-flush', async function () {
     it('should continue with auto-flush', async function () {
       const result = await runScript({
       const result = await runScript({
-        LIMIT: '2',
+        LIMIT: '4',
         FLUSH_IN_SYNC_PROJECTS: 'true',
         FLUSH_IN_SYNC_PROJECTS: 'true',
       })
       })
       expect(result.code).to.equal(0)
       expect(result.code).to.equal(0)