search

iTwin / itwinjs-core

Monorepo for iTwin.js Library
https://www.itwinjs.org
MIT License
620 stars 210 forks source link

attempt to make logLevel.Trace set LogMask to ALL test less flaky #7185

Closed nick4598 closed 1 month ago

nick4598 commented 1 month ago

I looked at first 10 or so times this test failed using azure's analytics page. 

expected _logInfo to not have been called but was called twice

     _logInfo('CloudSqlite', '16:37:26.631 | container test1 enters DELETE state', undefined) at logInfo (/home/vsts/work/1/s/core/bentley/src/Logger.ts:287:14)     
     _logInfo('CloudSqlite', '16:37:26.631 | container test1 leaves DELETE state', undefined) at logInfo (/home/vsts/work/1/s/core/bentley/src/Logger.ts:287:14)

Now waiting for the only two log messages that were failing in the first ten failures I saw in analytics to show up before resetting history. I took grigas's waitFor function from native which he added when reworking logging.