From 10b784aa88a5ec350e61b40781c86b41ab70d555 Mon Sep 17 00:00:00 2001 From: Sheetal Nandi Date: Fri, 19 Apr 2024 09:48:19 -0700 Subject: [PATCH] Add try catch and defer.reject for checking sys tests (#58252) --- .../unittests/sys/symlinkWatching.ts | 24 ++++++++++++++----- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/src/testRunner/unittests/sys/symlinkWatching.ts b/src/testRunner/unittests/sys/symlinkWatching.ts index 4d6b1014858..f0deb2c0c94 100644 --- a/src/testRunner/unittests/sys/symlinkWatching.ts +++ b/src/testRunner/unittests/sys/symlinkWatching.ts @@ -45,10 +45,16 @@ describe("unittests:: sys:: symlinkWatching::", () => { watcher: sys.watchFile!( toWatch, (fileName, eventKind, modifiedTime) => { - assert.equal(fileName, toWatch); - assert.equal(eventKind, ts.FileWatcherEventKind.Changed); - const actual = modifiedTimeToString(modifiedTime); - assert(actual === undefined || actual === modifiedTimeToString(sys.getModifiedTime!(file))); + try { + assert.equal(fileName, toWatch); + assert.equal(eventKind, ts.FileWatcherEventKind.Changed); + const actual = modifiedTimeToString(modifiedTime); + assert(actual === undefined || actual === modifiedTimeToString(sys.getModifiedTime!(file))); + } + catch (e) { + result.deferred.reject(e); + return; + } result.deferred.resolve(); }, 10, @@ -118,8 +124,14 @@ describe("unittests:: sys:: symlinkWatching::", () => { const deferred = defer(); delayedOp(() => { if (opType !== "init") { - verifyEventAndFileNames(`${opType}:: dir`, dirResult.actual, expectedResult); - verifyEventAndFileNames(`${opType}:: link`, linkResult.actual, expectedResult); + try { + verifyEventAndFileNames(`${opType}:: dir`, dirResult.actual, expectedResult); + verifyEventAndFileNames(`${opType}:: link`, linkResult.actual, expectedResult); + } + catch (e) { + deferred.reject(e); + return; + } } deferred.resolve(); }, !!process.env.CI ? 1000 : 500);