fix browser undefined (#430)

* fix browser undefined * fix test * fix test * fix
AcalaNetwork · Oct 10, 2023 · 94a7a87 · 94a7a87
1 parent 856c16a
commit 94a7a87
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 17 deletions.
diff --git a/packages/core/src/blockchain/block-builder.ts b/packages/core/src/blockchain/block-builder.ts
@@ -298,7 +298,7 @@ export const buildBlock = async (
 
   const storageDiff = await newBlock.storageDiff()
 
-  if (logger.isLevelEnabled('trace')) {
+  if (logger.level.toLowerCase() === 'trace') {
     logger.trace(
       Object.entries(storageDiff).map(([key, value]) => [key, truncate(value)]),
       'Final block',

diff --git a/packages/core/src/blockchain/txpool.ts b/packages/core/src/blockchain/txpool.ts
@@ -238,20 +238,21 @@ export class TxPool {
         onApplyExtrinsicError: (extrinsic, error) => {
           this.event.emit(APPLY_EXTRINSIC_ERROR, [extrinsic, error])
         },
-        onPhaseApplied: logger.isLevelEnabled('trace')
-          ? (phase, resp) => {
-              switch (phase) {
-                case 'initialize':
-                  logger.trace(truncate(resp.storageDiff), 'Initialize block')
-                  break
-                case 'finalize':
-                  logger.trace(truncate(resp.storageDiff), 'Finalize block')
-                  break
-                default:
-                  logger.trace(truncate(resp.storageDiff), `Apply extrinsic ${phase}`)
+        onPhaseApplied:
+          logger.level.toLowerCase() === 'trace'
+            ? (phase, resp) => {
+                switch (phase) {
+                  case 'initialize':
+                    logger.trace(truncate(resp.storageDiff), 'Initialize block')
+                    break
+                  case 'finalize':
+                    logger.trace(truncate(resp.storageDiff), 'Finalize block')
+                    break
+                  default:
+                    logger.trace(truncate(resp.storageDiff), `Apply extrinsic ${phase}`)
+                }
               }
-            }
-          : undefined,
+            : undefined,
       },
       params.unsafeBlockHeight,
     )

diff --git a/packages/web-test/tests/index.spec.ts b/packages/web-test/tests/index.spec.ts
@@ -19,10 +19,7 @@ test.describe('index', () => {
     await expect(page.locator('#blocks-section')).toHaveText(/4000000/, { timeout: 60_000 })
     // wait for new block
     await expect(page.locator('#blocks-section')).toHaveText(/4000001/, { timeout: 200_000 })
-  })
 
-  test('dry run extrinsics', async ({ page }) => {
-    test.setTimeout(5 * 60 * 1000)
     await page.getByText(/dry run call/i).click()
     await expect(page.getByText('Loading dry run result...')).toBeVisible()
     await expect(page.locator('#extrinsic-section')).toHaveText(/outcome/, { timeout: 200_000 })