[thirdweb] fix: Re-enable flaky smart wallet tests

Author: joaquim-verges

.github/workflows/issue.yml

@@ -26,8 +26,7 @@ describe.runIf(process.env.TW_SECRET_KEY)("verifyTypedData", async () => {
     ).toBe(true);
   });
 
-  // FIXME: flaky test - needs to be fixed
-  test.skip("invalid EOA signature", async () => {
+  test("invalid EOA signature", async () => {
     expect(
       await verifyTypedData({
         ...typedData.basic,
@@ -54,8 +53,7 @@ describe.runIf(process.env.TW_SECRET_KEY)("verifyTypedData", async () => {
     ).toBe(true);
   });
 
-  // FIXME: flaky test - needs to be fixed
-  test.skip("invalid smart account signature", async () => {
+  test("invalid smart account signature", async () => {
     expect(
       await verifyTypedData({
         ...typedData.basic,

packages/thirdweb/src/auth/verify-typed-data.test.ts

@@ -49,11 +49,9 @@ const contract = getContract({
   address: "0xe2cb0eb5147b42095c2FfA6F7ec953bb0bE347D8",
 });
 
-// FIXME: SKIP ALL OF THIS IT IS FLAKY
-describe.skip(
+describe.sequential(
   "SmartWallet 0.7 core tests",
   {
-    retry: 0,
     timeout: 240_000,
   },
   () => {
@@ -103,8 +101,7 @@ describe.skip(
       expect(isValid).toEqual(true);
     });
 
-    // FIXME: flaky test - skipped
-    it.skip("should use ERC-1271 signatures after deployment", async () => {
+    it("should use ERC-1271 signatures after deployment", async () => {
       await deploySmartAccount({
         chain,
         client,
@@ -137,8 +134,7 @@ describe.skip(
       expect(isValid).toEqual(true);
     });
 
-    // FIXME: flaky test - skipped
-    it.skip("should use ERC-1271 typed data signatures after deployment", async () => {
+    it("should use ERC-1271 typed data signatures after deployment", async () => {
       await deploySmartAccount({
         chain,
         client,

packages/thirdweb/src/wallets/smart/smart-wallet-integration-v07.test.ts

@@ -51,7 +51,6 @@ const factoryAddress = "0x564cf6453a1b0FF8DB603E92EA4BbD410dea45F3"; // pre 712
 describe.runIf(process.env.TW_SECRET_KEY).sequential(
   "SmartWallet core tests",
   {
-    retry: 0,
     timeout: 240_000,
   },
   () => {

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

@@ -28,8 +28,7 @@ const client = TEST_CLIENT;
 const DEFAULT_FACTORY_ADDRESS = "0xB1846E893CA01c5Dcdaa40371C1e13f2e0Df5717";
 const DEFAULT_VALIDATOR_ADDRESS = "0x7D3631d823e0De311DC86f580946EeF2eEC81fba";
 
-// FIXME: This test is flaky and needs to be fixed
-describe.skip.sequential(
+describe.sequential(
   "SmartWallet modular tests",
   {
     retry: 0,