Fix: pass config parameter to useConnectorClient in useSimulateContract and useEstimateGas

[Aerilym]

Added the config parameter to useConnectorClient parameters when used in useSimulateContract and useEstimateGas. useConnectorClient would always call useConfig as config wasn't provided in the parameters, this broke providing a config to either hook and would mean useConfig was triggered in both hooks. This caused an error when WagmiProvider wasn't available in the context, as useConnectorClient would call useContext(WagmiContext) even if a config parameter was provided to the parent hooks.

[changeset-bot]

🦋 Changeset detected

Latest commit: 7ec2998

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
wagmi	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
wagmi	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 9, 2024 6:10am

[Aerilym]

Looks like some of the tests are throwing errors, not 100% if its an issue with my changes or with the tests, will take a look.

[Aerilym]

Ran the tests locally, looks like 2 tests are timing out, and as far as i can tell the issue is unrelated to the code changes:

 FAIL  |@wagmi/core| packages/core/src/actions/getBlock.test.ts > default
 FAIL  |@wagmi/core| packages/core/src/actions/getBlock.test.ts > args: blockNumber
 FAIL  |@wagmi/core| packages/core/src/actions/getBlock.test.ts > args: includeTransactions
 
 FAIL  |wagmi| packages/react/src/hooks/useInfiniteReadContracts.test.ts > default

[tmm]

Nice catch!

[tmm]

Suggested change

	Fixed config parameter passing in useSimulateContract and useEstimateGas
	Fixed `useSimulateContract` and `useEstimateGas` behavior by passing `config` parameter to internal `useConnectorClient`.