Skip to content

fix: memoize deadline + smart contract check calls #5253

fix: memoize deadline + smart contract check calls

fix: memoize deadline + smart contract check calls #5253

Triggered via pull request August 16, 2023 11:21
@iamacookiamacook
opened #2411
Status Success
Total duration 15s
Artifacts

cla.yml

on: pull_request_target
CLAAssistant
6s
CLAAssistant
Fit to window
Zoom out
Zoom in

Annotations

1 error and 1 warning
useSafeTokenAllocation > add not redeemed allocations if deadline has not passed: src/hooks/__tests__/useSafeTokenAllocation.test.ts#L272
Error: expect(received).toEqual(expected) // deep equality Expected: 2000 Received: 0 Ignored nodes: comments, script, style <html> <head /> <body> <div /> </body> </html> at toEqual (/home/runner/work/safe-wallet-web/safe-wallet-web/src/hooks/__tests__/useSafeTokenAllocation.test.ts:323:45) at runWithExpensiveErrorDiagnosticsDisabled (/home/runner/work/safe-wallet-web/safe-wallet-web/node_modules/@testing-library/dom/dist/config.js:52:12) at checkCallback (/home/runner/work/safe-wallet-web/safe-wallet-web/node_modules/@testing-library/dom/dist/wait-for.js:141:77) at checkRealTimersCallback (/home/runner/work/safe-wallet-web/safe-wallet-web/node_modules/@testing-library/dom/dist/wait-for.js:133:16) at Timeout.task [as _onTimeout] (/home/runner/work/safe-wallet-web/safe-wallet-web/node_modules/jsdom/lib/jsdom/browser/Window.js:514:19) at listOnTimeout (node:internal/timers:569:17) at processTimers (node:internal/timers:512:7)
CLAAssistant
The following actions uses node12 which is deprecated and will be forced to run on node16: contributor-assistant/[email protected]. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/