8000 Clean up bls provider and signer by JohnGuilding · Pull Request #599 · getwax/bls-wallet · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Nov 5, 2023. It is now read-only.

Clean up bls provider and signer #599

Merged
merged 6 commits into from
May 4, 2023
Merged

Conversation

JohnGuilding
Copy link
Collaborator
@JohnGuilding JohnGuilding commented May 4, 2023

What is this PR doing?

  • Updates signMessage method
  • Resolves some TODOs
  • Cleans up a few tests
  • Removes unnecessary integration tests

Viewing the changes by each commit may make it easier to understand which tests were moved rather than deleted

How can these changes be manually tested?

Observing all tests pass

Does this PR resolve or contribute to any issues?

None

Checklist

  • I have manually tested these changes
  • Post a link to the PR in the group chat

Guidelines

  • If your PR is not ready, mark it as a draft
  • The resolve conversation button is for reviewers, not authors
    • (But add a 'done' comment or similar)

- estimateGas test has an actual test in contract interaction tests
- list accounts is basically being tested in test below it (ethers logic just calls `.send("eth_accounts", [])`
- getBalance tested everywhere implicitly
@github-actions github-actions bot added clients contracts Smart contract related labels May 4, 2023
@JohnGuilding JohnGuilding force-pushed the clean-up-bls-provider-and-signer branch from f6ea284 to c5d83f4 Compare May 4, 2023 18:52
@JohnGuilding JohnGuilding marked this pull request as ready for review May 4, 2023 19:08
@jacque006 jacque006 merged commit 8edaab3 into main May 4, 2023
@jacque006 jacque006 deleted the clean-up-bls-provider-and-signer branch May 4, 2023 20:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
clients contracts Smart contract related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0