starknet-foundry icon indicating copy to clipboard operation
starknet-foundry copied to clipboard

docs: update snforge version

Open TAdev0 opened this issue 9 months ago • 3 comments

Introduced changes

Updates snforge version in Getting Started of the documentation

TAdev0 avatar May 02 '24 14:05 TAdev0

hi @Arcticae i just noticed versions where outdated in the book, pushed this pr to update it. I have no idea why tests keep failing as i'm just modifying a .md file. any idea?

it seems to be related to the SEPOLIA_RPC_URL during CI

TAdev0 avatar May 15 '24 12:05 TAdev0

@TAdev0 seems you need to set up the URL for sepolia for the tests in the env variables (in your fork). That's why they are failing https://foundry-rs.github.io/starknet-foundry/development/environment-setup.html#environmental-variables

You can try using a public sepolia endpoint

Arcticae avatar May 15 '24 17:05 Arcticae

oh sorry my bad... will do that, thanks!

TAdev0 avatar May 15 '24 18:05 TAdev0

@Arcticae i'm closing the issue, its not a big deal anyway ^^

I populated a .env file with :

SEPOLIA_RPC_URL="https://starknet-sepolia.public.blastapi.io/rpc/v0_7"

also tried:

SEPOLIA_RPC_URL=https://starknet-sepolia.public.blastapi.io/rpc/v0_7

and with a nethermind public endpoint as well.

I still have failing tests locally (4), dont know what the issue is

My errors are like

| error: Plugin diagnostic: Expected fork config must be of the form `url: <double quote string>, block_id: <snforge_std::BlockId>`.
|  --> [..]/src/lib.cairo[..]
|     #[fork(url: "https://lib.com")]
|  --> [..]/tests/test_fork.cairo[..]
| #[fork(url: "https://test.com")]
|       ^***********************^
| 
| error: Plugin diagnostic: Expected fork config must be of the form `url: <double quote string>, block_id: <snforge_std::BlockId>`.
|  --> [..]/tests/test_fork.cairo[..]
| #[fork(url: "https://starknet-sepolia.public.blastapi.io/rpc/v0_7", block_id: BlockId::Number(Latest))]
-       ^*****************************************************************************^
| error: Plugin diagnostic: Expected fuzzer config must be of the form `runs: <u32>, seed: <u64>`
|  --> [..]/tests/test_fuzzer.cairo[..]
| #[fuzzer(test: 10)]
|         ^********^

TAdev0 avatar May 17 '24 00:05 TAdev0

@TAdev0 we will investigate and probably update dox to make it easier for people - thanks for the interest in contribution nonetheless :)

Arcticae avatar May 17 '24 09:05 Arcticae