Fix DEFAULT_TEST_CONTRACT

[CodeSandwich]

The test contracts have a different address than the one in the constant. I tried printing address(this) in 2 separate tests and they were all showing this address. It makes sense because the msg.sender and tx.origin were always DEFAULT_SENDER in the constructors, setUps and the test functions, and the new DEFAULT_TEST_CONTRACT can be obtained by calculating the CREATE address for DEFAULT_SENDER and nonce 1.

[zerosnacks]

Hi @CodeSandwich thanks for flagging this

We are making the same assertion here in Foundry: https://github.com/foundry-rs/foundry/blob/f6133f9e5fbd37864958cc58457f795ce35cd469/testdata/default/core/ContractEnvironment.t.sol#L16

It appears that when the test contract creates its first contract it will be deployed at 0x5615deb798bb3e4dfa0139dfa1b3d433cc23b72f as seen here

https://github.com/foundry-rs/foundry/blob/f6133f9e5fbd37864958cc58457f795ce35cd469/crates/forge/tests/fixtures/SimpleContractTestVerbose.json#L119-L120

compared to

https://github.com/foundry-rs/foundry/blob/f6133f9e5fbd37864958cc58457f795ce35cd469/crates/forge/tests/fixtures/SimpleContractTestVerbose.json#L79-L80

[CodeSandwich]

You're right, this is all correct and the address doesn't need to be fixed; I simply didn't understand what it means. IMO the current docs are just wrong and need to be fixed, but I'll do it in a separate PR.