mirror of
https://github.com/kremalicious/metamask-extension.git
synced 2024-11-25 20:02:58 +01:00
c2163434db
* Fix and test log.info calls run for each migration In migrator/index.js, log.info is called before an after each migration. These calls are intended to produce breadcrumbs to be captured by sentry in cases where errors happen during or shortly after migrations are run. These calls were not causing any output to the console because the log.setLevel calls in ui/index.js were setting a 'warn value in local storage that was being used by logLevel in the background. This commit fixes the problem by setting the `persist` param of setLevel to false, so that the background no longer reads the ui's log level. Tests are added to verify that these logs are captured in sentry breadcrumbs when there is a migration error due to an invariant state. * Improve breadcrumb message matching The test modified in this commit asserts eqaulity of messages from breadcrumbs and hard coded expected results. This could cause failures, as sometimes the messages contain whitespace characters. This commit ensures the assertions only check that the expected string is within the message string, ignoring extra characters. |
||
|---|---|---|
| .. | ||
| account-import-strategies | ||
| constants | ||
| controllers | ||
| lib | ||
| migrations | ||
| platforms | ||
| app-init.js | ||
| background.js | ||
| contentscript.js | ||
| detect-multiple-instances.js | ||
| detect-multiple-instances.test.js | ||
| disable-console.js | ||
| first-time-state.js | ||
| init-globals.js | ||
| inpage.js | ||
| lockdown-more.js | ||
| lockdown-run.js | ||
| metamask-controller.actions.test.js | ||
| metamask-controller.js | ||
| metamask-controller.test.js | ||
| mmi-keyring-builder-factory.js | ||
| sentry-install.js | ||
| translate.test.ts | ||
| translate.ts | ||
| ui.js | ||
| use-snow.js | ||