fix tests, add docs

Author: 0xFirekeeper

Thirdweb.Tests/Thirdweb.Extensions/Thirdweb.Extensions.Tests.cs

@@ -442,7 +442,7 @@ public async Task ERC20_Approve()
     [Fact(Timeout = 120000)]
     public async Task ERC721A_TokensOfOwner()
     {
-        var contract = await this.GetDrop721Contract();
+        var contract = await this.GetERC721AContract();
         var ownerAddress = Constants.ADDRESS_ZERO;
         var tokens = await contract.ERC721A_TokensOfOwner(ownerAddress);
         Assert.NotNull(tokens);
@@ -452,7 +452,7 @@ public async Task ERC721A_TokensOfOwner()
     [Fact(Timeout = 120000)]
     public async Task ERC721A_TokensOfOwnerIn()
     {
-        var contract = await this.GetDrop721Contract();
+        var contract = await this.GetERC721AContract();
         var ownerAddress = Constants.ADDRESS_ZERO;
         var tokens = await contract.ERC721A_TokensOfOwnerIn(ownerAddress, 0, 1);
         Assert.NotNull(tokens);

Thirdweb/Thirdweb.Utils/Utils.cs

@@ -26,6 +26,7 @@ public static partial class Utils
     private static readonly Dictionary<BigInteger, bool> _eip155EnforcedCache = new();
     private static readonly Dictionary<BigInteger, ThirdwebChainData> _chainDataCache = new();
     private static readonly Dictionary<string, string> _ensCache = new();
+    private static readonly List<string[]> _errorSubstringsComposite = new() { new string[] { "account", "not found!" }, new[] { "wrong", "chainid" } };
 
     /// <summary>
     /// Computes the client ID from the given secret key.
@@ -305,7 +306,7 @@ public static string GenerateSIWE(LoginPayloadData loginPayloadData)
     /// <returns>True if it is a zkSync chain ID, otherwise false.</returns>
     public static async Task<bool> IsZkSync(ThirdwebClient client, BigInteger chainId)
     {
-        if (chainId.Equals(324) || chainId.Equals(300) || chainId.Equals(302) || chainId.Equals(11124) || chainId.Equals(4654) || chainId.Equals(333271))
+        if (chainId.Equals(324) || chainId.Equals(300) || chainId.Equals(302) || chainId.Equals(11124) || chainId.Equals(4654) || chainId.Equals(333271) || chainId.Equals(37111))
         {
             return true;
         }
@@ -659,9 +660,7 @@ public static async Task<bool> IsEip155Enforced(ThirdwebClient client, BigIntege
             else
             {
                 // Check if all substrings in any of the composite substrings are present
-                var errorSubstringsComposite = new List<string[]> { new string[] { "account", "not found!" }, new[] { "wrong", "chainid" } };
-
-                result = errorSubstringsComposite.Any(arr => arr.All(substring => errorMsg.Contains(substring)));
+                result = _errorSubstringsComposite.Any(arr => arr.All(substring => errorMsg.Contains(substring)));
             }
         }
 

Thirdweb/Thirdweb.Wallets/IThirdwebWallet.cs

@@ -130,6 +130,19 @@ Task<string> RecoverAddressFromTypedDataV4<T, TDomain>(T data, TypedData<TDomain
     /// </summary>
     Task Disconnect();
 
+    /// <summary>
+    /// Links a new account (auth method) to the current wallet. The current wallet must be connected and the wallet being linked must not be fully connected ie created.
+    /// </summary>
+    /// <param name="walletToLink">The wallet to link.</param>
+    /// <param name="otp">The OTP code if the wallet to link is an email or phone wallet.</param>
+    /// <param name="isMobile">Set to true if linking OAuth on mobile.</param>
+    /// <param name="browserOpenAction">The action to open the browser if linking OAuth.</param>
+    /// <param name="mobileRedirectScheme">The redirect scheme if linking OAuth on mobile.</param>
+    /// <param name="browser">The browser to use if linking OAuth.</param>
+    /// <param name="chainId">The chain ID if linking an external wallet (SIWE).</param>
+    /// <param name="jwt">The JWT token if linking custom JWT auth.</param>
+    /// <param name="payload">The login payload if linking custom AuthEndpoint auth.</param>
+    /// <returns>A list of <see cref="LinkedAccount"/> objects.</returns>
     Task<List<LinkedAccount>> LinkAccount(
         IThirdwebWallet walletToLink,
         string otp = null,
@@ -142,6 +155,10 @@ Task<List<LinkedAccount>> LinkAccount(
         string payload = null
     );
 
+    /// <summary>
+    /// Returns a list of linked accounts to the current wallet.
+    /// </summary>
+    /// <returns>A list of <see cref="LinkedAccount"/> objects.</returns>
     Task<List<LinkedAccount>> GetLinkedAccounts();
 }