Skip to content

Stub PreferencesController in KeyringController tests#1194

Merged
mikesposito merged 4 commits intomainfrom
test/stub-preferences-controller
Apr 20, 2023
Merged

Stub PreferencesController in KeyringController tests#1194
mikesposito merged 4 commits intomainfrom
test/stub-preferences-controller

Conversation

@mikesposito
Copy link
Member

@mikesposito mikesposito commented Apr 17, 2023

Description

This PR stubs the PreferencesController used to create the KeyringController instance used in tests.

Changes

  • CHANGED: Functions passed to KeyringController constructor during tests

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation for new or updated code as appropriate (note: this will usually be JSDoc)
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate

@mikesposito mikesposito requested a review from a team as a code owner April 17, 2023 12:27
@mikesposito mikesposito force-pushed the test/stub-preferences-controller branch from 00df9d6 to d52da13 Compare April 17, 2023 12:28
Gudahtt
Gudahtt previously approved these changes Apr 17, 2023
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Gudahtt
Gudahtt previously approved these changes Apr 20, 2023
@mikesposito mikesposito merged commit db046ee into main Apr 20, 2023
@mikesposito mikesposito deleted the test/stub-preferences-controller branch April 20, 2023 14:14
@legobeat legobeat mentioned this pull request Apr 25, 2023
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
* test: stub preferences controller in keyring tests

* test: refactor addNewAccountWithoutUpdate

* refactor: keyringController instantiation
MajorLift pushed a commit that referenced this pull request Oct 11, 2023
* test: stub preferences controller in keyring tests

* test: refactor addNewAccountWithoutUpdate

* refactor: keyringController instantiation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Stub PreferencesController in core KeyringController unit tests

2 participants