Skip to content

Commit 1cb3199

Browse files
committed
Update tests for current EIP-2930 support across backends (ethers-io#1364).
1 parent 25c3024 commit 1cb3199

File tree

1 file changed

+21
-5
lines changed

1 file changed

+21
-5
lines changed

packages/tests/src.ts/test-providers.ts

Lines changed: 21 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -247,6 +247,7 @@ const blockchainData: { [ network: string ]: TestCases } = {
247247
},
248248
],
249249
transactions: [
250+
// Berlin tests
250251
{
251252
hash: "0x48bff7b0e603200118a672f7c622ab7d555a28f98938edb8318803eed7ea7395",
252253
type: 1,
@@ -692,9 +693,8 @@ Object.keys(blockchainData).forEach((network) => {
692693

693694
return tx;
694695
}, test, (provider: string, network: string, test: TestDescription) => {
695-
// Temporary
696-
if (network === "ropsten" && provider === "PocketProvider") {
697-
console.log(`Skipping ${ provider }; incomplete Berlin support`);
696+
// Temporary; Pocket is having issues with old transactions on some testnets
697+
if ((network === "ropsten" || network === "goerli") && provider === "PocketProvider") {
698698
return true;
699699
}
700700

@@ -716,7 +716,14 @@ Object.keys(blockchainData).forEach((network) => {
716716
delete receipt.confirmations;
717717

718718
return receipt;
719-
}, test);
719+
}, test, (provider: string, network: string, test: TestDescription) => {
720+
// Temporary; Pocket is having issues with old transactions on some testnets
721+
if ((network === "ropsten" || network === "goerli") && provider === "PocketProvider") {
722+
return true;
723+
}
724+
725+
return false;
726+
});
720727
});
721728
});
722729

@@ -737,16 +744,25 @@ Object.keys(blockchainData).forEach((network) => {
737744
});
738745
}
739746

747+
/*
748+
@TODO: Use this for testing pre-EIP-155 transactions on specific networks
740749
addErrorTest(ethers.utils.Logger.errors.NONCE_EXPIRED, async (provider: ethers.providers.Provider) => {
741750
return provider.sendTransaction("0xf86480850218711a0082520894000000000000000000000000000000000000000002801ba038aaddcaaae7d3fa066dfd6f196c8348e1bb210f2c121d36cb2c24ef20cea1fba008ae378075d3cd75aae99ab75a70da82161dffb2c8263dabc5d8adecfa9447fa");
742751
});
752+
*/
753+
754+
// Wallet(id("foobar1234"))
755+
addErrorTest(ethers.utils.Logger.errors.NONCE_EXPIRED, async (provider: ethers.providers.Provider) => {
756+
return provider.sendTransaction("0xf86480850218711a00825208940000000000000000000000000000000000000000038029a04320fd28c8e6c95da9229d960d14ffa3de81f83abe3ad9c189642c83d7d951f3a009aac89e04a8bafdcf618e21fed5e7b1144ca1083a301fd5fde28b0419eb63ce");
757+
});
743758

744759
addErrorTest(ethers.utils.Logger.errors.INSUFFICIENT_FUNDS, async (provider: ethers.providers.Provider) => {
745760

746761
const txProps = {
747762
to: "0x8ba1f109551bD432803012645Ac136ddd64DBA72",
748763
gasPrice: 9000000000,
749764
gasLimit: 21000,
765+
chainId: 3,
750766
value: 1
751767
};
752768

@@ -809,7 +825,7 @@ testFunctions.push({
809825
networks: [ "ropsten" ], // Only test on Ropsten
810826
checkSkip: (provider: string, network: string, test: TestDescription) => {
811827
// Temporary
812-
return (provider === "PocketProvider" || provider === "EtherscanProvider");
828+
return false;
813829
},
814830
execute: async (provider: ethers.providers.Provider) => {
815831
const wallet = fundWallet.connect(provider);

0 commit comments

Comments
 (0)