In the `core` version of this controller, event types are not held in an
enum but are specified directly as strings instead.
Simplifying the event types will make it easier to compare the
NetworkController unit tests between extension and core.
This makes it possible to test that mock functions are called the
correct amount of times (otherwise Jest will think that a mock function
has been called for as many tests as exist in the NetworkController unit
test suite).
On the `core` side, the tests for `NetworkController.initializeProvider`
use the `lookupNetworkTests` to automatically test the `lookupNetwork`
behavior that `initializeProvider` initiates. This commit makes use of
the helper to bring the tests closer to the `core` version.
The NetworkController tests in the `core` repo use a `refreshNetwork`
test helper to exercise the behavior performed by various methods
when switching the network. This commit brings over this test helper and
refactors tests for those methods to use it.
import account without password
---------
Co-authored-by: Mike B <32695229+plasmacorral@users.noreply.github.com>
Co-authored-by: Gustavo Antunes <17601467+gantunesr@users.noreply.github.com>
Co-authored-by: Howard Braham <howrad@gmail.com>
* Added code fences to the following controllers: app-state, metametrics, preferences
* Fixed prettier
* Updated code to be align with new changes in MMI
* Changing code fences
* Remove uneeded files
* Fixed tests
* Fixed code fence
* Changing logic to use async/await
* Removed accountAddress
* Reverted code from develop
* Add DS Popover component
* Update to Popover story
* make role a prop with PopoverRole enum
* update to Hiro design changes
* fix snapshot
* Update ui/components/component-library/popover/README.mdx
Co-authored-by: George Marshall <george.marshall@consensys.net>
* Update ui/components/component-library/popover/README.mdx
Co-authored-by: George Marshall <george.marshall@consensys.net>
* small story changes and removal of unused forwardRef
* add more test coverage
* add more story demos
* add more popover demos
* if escKeyClose is passed then it will add event listener
* isPortal story
* add if statement
* replace Text with Box for now
* add esc test coverage
* add README docs
* fix readme and onEscKeyClose
* onEscKeyClose to onPressEscKey
* Update ui/components/component-library/popover/README.mdx
Co-authored-by: George Marshall <george.marshall@consensys.net>
* change conditional on useEffect
---------
Co-authored-by: legobeat <109787230+legobeat@users.noreply.github.com>
Co-authored-by: George Marshall <george.marshall@consensys.net>
In preparation for introducing `@metamask/network-controller`, we want
to ensure that any packages that depend on this package are up to date
so that we are using the same version of `@metamask/network-controller`
in the dependency tree.
The extension uses two packages that depend on
`@metamask/network-controller`: `@metamask/assets-controllers` and
`@metamask/gas-fee-controller`. This commit upgrades them to the latest
version. There are no breaking changes in the new release of these
packages so there are no accompanying compatibility changes.
* updating the tokenBalance fetch and Token standard type to enable the Total in setSetApproveForAllWarning
* updating the base-10 for the BN.toString() conversion
* updating review comments
* adding string conversion to tokenBalance
* ConfirmPageContainer: mv props to the top
* ConfirmPageContainer: deprecate sinon
* updating coverage-targets
* updating the conversion base
* ConfirmPageContainer: add test for warning modal
---------
Co-authored-by: Niranjana Binoy <43930900+NiranjanaBinoy@users.noreply.github.com>
