Adapt to eip 1193 provider changes

[cryptodev-2s]

Explanation

After updating the SafeEventEmitterProvider to support EIP-1193, I have made the following changes:

• Deprecated the use of sendAsync and replaced it with the request method.
  • Updated instances in the core repository:
    • NetworkController tests
  • Modified areas where we previously expected the provider returned by NetworkController to have a sendAsync method. These now use the request method with the correct method signature:
    • SelectedNetworkController tests
    • createAutoManagedNetworkClient tests

References

• Fixes Adapt to EIP-1193 provider changes #4100

Changelog

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've highlighted breaking changes using the "BREAKING" category above as appropriate

[socket-security]

No dependency changes detected. Learn more about Socket for GitHub ↗︎

👍 No dependency changes detected in pull request

[mikesposito]

These changes look good!

I noticed we left unchanged FakeProvider: are we supposed to remove sendAsync from the class or any other changes related to this work?

[cryptodev-2s]

These changes look good!

I noticed we left unchanged FakeProvider: are we supposed to remove sendAsync from the class or any other changes related to this work?

Hi Michele,

FakeProvider has already been updated in a previous PR to include the request method. I had to make this change along with the SafeEventEmitterProvider updates; otherwise, the tests would not pass. You can see the updated FakeProvider here.