chore(tests): skip tests that will fail without secret key set (#2604)

Author: jnsdls

legacy_packages/storage/test/ipfs.test.ts

@@ -12,7 +12,9 @@ import { prepareAutoFactoryDeployTransaction } from "./deploy-via-autofactory.js
 import { initialize } from "../../extensions/prebuilts/__generated__/DropERC721/write/initialize.js";
 import { getDeployedInfraContract } from "./utils/infra.js";
 
-describe("deployFromMetadata", () => {
+// skip this test suite if there is no secret key available to test with
+// TODO: remove reliance on secret key during unit tests entirely
+describe.runIf(process.env.TW_SECRET_KEY)("deployFromMetadata", () => {
   it("should deploy published contract with existing infra", async () => {
     const cloneFactoryContract = await getDeployedCloneFactoryContract({
       chain: FORKED_ETHEREUM_CHAIN,

packages/thirdweb/src/contract/deployment/deploy-via-autofactory.test.ts

@@ -7,7 +7,9 @@ import { getDeployedCreate2Factory } from "./create-2-factory.js";
 import { getDeployedInfraContract } from "./infra.js";
 import { fail } from "assert";
 
-describe("bootstrap", () => {
+// skip this test suite if there is no secret key available to test with
+// TODO: remove reliance on secret key during unit tests entirely
+describe.runIf(process.env.TW_SECRET_KEY)("bootstrap", () => {
   it("should bootstrap onchain infra", async () => {
     await deployCloneFactory({
       chain: ANVIL_CHAIN,

packages/thirdweb/src/contract/deployment/utils/bootstrap.test.ts

@@ -7,7 +7,9 @@ import {
 import { transferEvent } from "../../extensions/erc721/__generated__/IERC721A/events/Transfer.js";
 import { prepareEvent } from "../prepare-event.js";
 
-describe("getEvents", () => {
+// skip this test suite if there is no secret key available to test with
+// TODO: remove reliance on secret key during unit tests entirely
+describe.runIf(process.env.TW_SECRET_KEY)("getEvents", () => {
   it("should get all events", async () => {
     const events = await getContractEvents({
       contract: USDC_CONTRACT,

packages/thirdweb/src/event/actions/get-events.test.ts

@@ -2,7 +2,9 @@ import { describe, it, expect } from "vitest";
 import { resolveAddress } from "./resolve-address.js";
 import { TEST_CLIENT } from "../../../test/src/test-clients.js";
 
-describe("ENS", () => {
+// skip this test suite if there is no secret key available to test with
+// TODO: remove reliance on secret key during unit tests entirely
+describe.runIf(process.env.TW_SECRET_KEY)("ENS", () => {
   it("should resolve ENS", async () => {
     const name = "vitalik.eth";
     const address = await resolveAddress({

packages/thirdweb/src/extensions/ens/resolve-address.test.ts

@@ -4,7 +4,7 @@ import { getNFT } from "./getNFT.js";
 import { DROP1155_CONTRACT } from "~test/test-contracts.js";
 
 describe("erc1155.getNFT", () => {
-  it.runIf(process.env.TW_SECRET_KEY)("without owner", async () => {
+  it("without owner", async () => {
     const nft = await getNFT({
       contract: DROP1155_CONTRACT,
       tokenId: 2n,

packages/thirdweb/src/extensions/erc1155/read/getNFT.test.ts

@@ -3,7 +3,7 @@ import { describe, it, expect } from "vitest";
 import { getAllOwners } from "./getAllOwners.js";
 import { AZUKI_CONTRACT, DOODLES_CONTRACT } from "~test/test-contracts.js";
 
-describe.runIf(process.env.TW_SECRET_KEY)("erc721.getAllOwners", () => {
+describe("erc721.getAllOwners", () => {
   it("works for azuki", async () => {
     const nfts = await getAllOwners({
       contract: AZUKI_CONTRACT,

packages/thirdweb/src/extensions/erc721/read/getAllOwners.test.ts

@@ -4,7 +4,7 @@ import { getNFT } from "./getNFT.js";
 import { DOODLES_CONTRACT } from "~test/test-contracts.js";
 
 describe("erc721.getNFT", () => {
-  it.runIf(process.env.TW_SECRET_KEY)("without owner", async () => {
+  it("without owner", async () => {
     const nft = await getNFT({
       contract: { ...DOODLES_CONTRACT },
       tokenId: 1n,
@@ -47,7 +47,7 @@ describe("erc721.getNFT", () => {
     `);
   });
 
-  it.runIf(process.env.TW_SECRET_KEY)("with owner", async () => {
+  it("with owner", async () => {
     const nft = await getNFT({
       contract: { ...DOODLES_CONTRACT },
       tokenId: 1n,

packages/thirdweb/src/extensions/erc721/read/getNFT.test.ts

@@ -3,7 +3,7 @@ import { describe, it, expect } from "vitest";
 import { getNFTs } from "./getNFTs.js";
 import { AZUKI_CONTRACT, DOODLES_CONTRACT } from "~test/test-contracts.js";
 
-describe.runIf(process.env.TW_SECRET_KEY)("erc721.getNFTs", () => {
+describe("erc721.getNFTs", () => {
   it("works for azuki", async () => {
     const nfts = await getNFTs({
       contract: AZUKI_CONTRACT,

packages/thirdweb/src/extensions/erc721/read/getNFTs.test.ts

@@ -16,7 +16,9 @@ import { USDC_CONTRACT_ADDRESS } from "../../../../test/src/test-contracts.js";
 import { toHex } from "../../../utils/encoding/hex.js";
 import { sendTransaction } from "../../../transaction/actions/send-transaction.js";
 
-describe(
+// skip this test suite if there is no secret key available to test with
+// TODO: remove reliance on secret key during unit tests entirely
+describe.runIf(process.env.TW_SECRET_KEY)(
   "generateMintSignature",
   {
     timeout: 60000,