From 6549b93caa77fa29fc56eb410c0a8ca8974fc5e5 Mon Sep 17 00:00:00 2001 From: Miguel Serrano Date: Wed, 8 Jun 2022 14:48:58 +0200 Subject: [PATCH] Fixed tests by removing stdout checks that are no longer valid (#8337) * Fixed tests by removing stdout checks that are no longer valid script verbosity was updated in https://github.com/overleaf/internal/commit/c73b46599b51fadb8054cdd7d05e25beb702533d, this checks are no longer valid. After the deleted line there's an extra check that should be good enough for the test case. GitOrigin-RevId: 2756d11cad97fdbeca44f35c24ee192e582a52c1 --- services/web/test/acceptance/src/BackFillDeletedFilesTests.js | 4 +--- .../test/acceptance/src/BackFillDocNameForDeletedDocsTests.js | 4 +--- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/services/web/test/acceptance/src/BackFillDeletedFilesTests.js b/services/web/test/acceptance/src/BackFillDeletedFilesTests.js index 5e9a0927d8..947a900415 100644 --- a/services/web/test/acceptance/src/BackFillDeletedFilesTests.js +++ b/services/web/test/acceptance/src/BackFillDeletedFilesTests.js @@ -85,9 +85,7 @@ describe('BackFillDeletedFiles', function () { logger.error({ error }, 'script failed') throw error } - const { stderr: stdErr, stdout: stdOut } = result - expect(stdOut).to.include(projectId1.toString()) - expect(stdOut).to.include(projectId2.toString()) + const { stderr: stdErr } = result expect(stdErr).to.include(`Completed batch ending ${projectId5}`) } diff --git a/services/web/test/acceptance/src/BackFillDocNameForDeletedDocsTests.js b/services/web/test/acceptance/src/BackFillDocNameForDeletedDocsTests.js index cdc2194aa7..73fa267e6e 100644 --- a/services/web/test/acceptance/src/BackFillDocNameForDeletedDocsTests.js +++ b/services/web/test/acceptance/src/BackFillDocNameForDeletedDocsTests.js @@ -74,9 +74,7 @@ describe('BackFillDocNameForDeletedDocs', function () { logger.error({ error }, 'script failed') throw error } - const { stderr: stdErr, stdout: stdOut } = result - expect(stdOut).to.include(projectId1.toString()) - expect(stdOut).to.include(projectId2.toString()) + const { stderr: stdErr } = result expect(stdErr).to.include(`Completed batch ending ${projectId2}`) }