test: enable SmartWallet policy tests

Author: joaquim-verges

.changeset/smooth-walls-glow.md

@@ -0,0 +1,5 @@
+---
+"thirdweb": patch
+---
+
+Ensure smart accounts are deployed before validating signatures

packages/thirdweb/src/wallets/smart/index.ts

@@ -249,13 +249,19 @@ async function createSmartAccount(
         import("../../utils/hashing/hashMessage.js"),
         import("../../extensions/erc1271/checkContractWalletSignature.js"),
       ]);
-      const isDeployed = await isContractDeployed(accountContract);
+      let isDeployed = await isContractDeployed(accountContract);
       if (!isDeployed) {
         await _deployAccount({
           options,
           account,
           accountContract,
         });
+        // the bundler and rpc might not be in sync, so while the bundler has a transaction hash for the deployment,
+        // the rpc might not have it yet, so we wait until the rpc confirms the contract is deployed
+        while (!isDeployed) {
+          await new Promise((resolve) => setTimeout(resolve, 500));
+          isDeployed = await isContractDeployed(accountContract);
+        }
       }
 
       const originalMsgHash = hashMessage(message);
@@ -337,13 +343,19 @@ async function createSmartAccount(
         return options.personalAccount.signTypedData(typedData);
       }
 
-      const isDeployed = await isContractDeployed(accountContract);
+      let isDeployed = await isContractDeployed(accountContract);
       if (!isDeployed) {
         await _deployAccount({
           options,
           account,
           accountContract,
         });
+        // the bundler and rpc might not be in sync, so while the bundler has a transaction hash for the deployment,
+        // the rpc might not have it yet, so we wait until the rpc confirms the contract is deployed
+        while (!isDeployed) {
+          await new Promise((resolve) => setTimeout(resolve, 500));
+          isDeployed = await isContractDeployed(accountContract);
+        }
       }
 
       const originalMsgHash = hashTypedData(typedData);

packages/thirdweb/src/wallets/smart/smart-wallet-dev.test.ts

@@ -2,7 +2,6 @@ import { beforeAll, describe, expect, it } from "vitest";
 import { TEST_CLIENT } from "../../../test/src/test-clients.js";
 import { arbitrumSepolia } from "../../chains/chain-definitions/arbitrum-sepolia.js";
 import { type ThirdwebContract, getContract } from "../../contract/contract.js";
-
 import { balanceOf } from "../../extensions/erc1155/__generated__/IERC1155/read/balanceOf.js";
 import { claimTo } from "../../extensions/erc1155/drops/write/claimTo.js";
 import { sendAndConfirmTransaction } from "../../transaction/actions/send-and-confirm-transaction.js";
@@ -62,6 +61,12 @@ describe.runIf(process.env.TW_SECRET_KEY).skip.sequential(
       expect(smartWalletAddress).toHaveLength(42);
     });
 
+    it("can sign a msg", async () => {
+      await smartAccount.signMessage({ message: "hello world" });
+      const isDeployed = await isContractDeployed(accountContract);
+      expect(isDeployed).toEqual(true);
+    });
+
     it("can execute a tx", async () => {
       const tx = await sendAndConfirmTransaction({
         transaction: claimTo({