🐺 More extensive check for provider object in renderWeb3Hook (#830)

TrueFiEng · Jun 20, 2022 · 55a28a2 · 55a28a2 · github-actions · Jun 20, 2022
1 parent 0600742
commit 55a28a2
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/.changeset/silent-fans-tap.md b/.changeset/silent-fans-tap.md
@@ -0,0 +1,5 @@
+---
+"@usedapp/core": patch
+---
+
+🐺 More extensive check for provider object in `renderWeb3Hook`
diff --git a/packages/core/src/testing/renderWeb3Hook.tsx b/packages/core/src/testing/renderWeb3Hook.tsx
@@ -5,6 +5,7 @@ import React from 'react'
 import { deployMulticall, getWaitUtils, IdentityWrapper, mineBlock } from './utils'
 import { BlockNumbersProvider } from '../providers/blockNumber/blockNumbers'
 import { ReadonlyNetworksProvider } from '../providers/network'
+import { providers as ethersProviders } from 'ethers'
 
 export interface renderWeb3HookOptions<Tprops> {
   mockProvider?: MockProvider | Record<number /* ChainId */, MockProvider>
@@ -55,9 +56,17 @@ export const renderWeb3Hook = async <Tprops, TResult>(
   }
 
   const providerObject = options?.mockProvider || new MockProvider()
-  if (providerObject instanceof MockProvider) {
-    defaultProvider = providerObject
-    await addSingleProvider(providerObject)
+  if (
+    providerObject instanceof MockProvider ||
+    /**
+     * We still expect this to be a MockProvider,
+     * but this is an override in case a different instance of the provider package is used
+     * and the TS types are not matching.
+     */
+    ethersProviders.Web3Provider.isProvider(providerObject)
+  ) {
+    defaultProvider = providerObject as MockProvider
+    await addSingleProvider(providerObject as MockProvider)
   } else {
     for (const chainIdString in providerObject) {
       const chainId = Number(chainIdString)