From 5d531230950c27c0b519e43f1e2a601bc45f3dca Mon Sep 17 00:00:00 2001 From: Jakob Ackermann Date: Tue, 23 Jan 2024 15:48:55 +0000 Subject: [PATCH] Merge pull request #16659 from overleaf/jpa-loose-progress-assertion [document-updater] loose assertion on progress with low LIMIT GitOrigin-RevId: f5046e0ed46b889e5a49a69b9752961f847d361a --- .../test/acceptance/js/CheckRedisMongoSyncStateTests.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/services/document-updater/test/acceptance/js/CheckRedisMongoSyncStateTests.js b/services/document-updater/test/acceptance/js/CheckRedisMongoSyncStateTests.js index c230229d8d..8738dcf500 100644 --- a/services/document-updater/test/acceptance/js/CheckRedisMongoSyncStateTests.js +++ b/services/document-updater/test/acceptance/js/CheckRedisMongoSyncStateTests.js @@ -245,17 +245,18 @@ describe('CheckRedisMongoSyncState', 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.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( - '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 () { const result = await runScript({ - LIMIT: '2', + LIMIT: '4', FLUSH_IN_SYNC_PROJECTS: 'true', }) expect(result.code).to.equal(0)