Renaming tests to follow best practices from Foundry Book

[milosdjurica]

This PR puts the forge-std test naming conventions in line with the Best Practices from Foundry Book.

I am looking for a response if I should additionally rename tests as stated here in #599 . After I get the feedback, I will make changes to other test files too.

This PR closes #273 and #599 . Additionally, I could adapt it to fit suggestion on the book best practice test naming changes if this suggestion gets accepted.

[milosdjurica]

As I mentioned earlier, I have suggested small changes to best practices.

Here is a table overview of methods from this file.
It shows how tests were named before I changed them, how they look now, and how they would look if new best practices are accepted.

Old names	Current names	New names
test_ChainRpcInitialization()	test_ChainRpcInitialization()	test_ChainRpcInitialization()
test_ChainNoDefault()	test_RevertIf_ChainNotFound()	test_GetChain_RevertIf_ChainNotFound()
test_SetChainFirstFails()	test_RevertIf_SetChain_ChainIdExist_FirstTest()	test_SetChain_RevertIf_ChainIdExist_FirstTest()
test_ChainBubbleUp()	test_RevertIf_ChainBubbleUp()	test_GetChain_RevertIf_ChainBubbleUp()
test_CannotSetChain_ChainIdExists()	test_RevertIf_SetChain_ChainIdExists_SecondTest()	test_SetChain_RevertIf_ChainIdExists_SecondTest()
test_SetChain()	test_SetChain()	test_SetChain_MultipleAssertions()
test_SetNoEmptyAlias()	test_RevertIf_SetEmptyAlias()	test_SetChain_RevertIf_SetEmptyAlias()
test_SetNoChainId0()	test_RevertIf_SetNoChainId0()	test_SetChain_RevertIf_ChainId0()
test_GetNoChainId0()	test_RevertIf_GetNoChainId0()	test_GetChain_RevertIf_ChainId0()
test_GetNoEmptyAlias()	test_RevertIf_GetNoEmptyAlias()	test_GetChain_RevertIf_EmptyAlias()
test_ChainIdNotFound()	test_RevertIf_ChainIdNotFound()	test_GetChain_RevertIf_ChainIdNotFound()
test_ChainAliasNotFound()	test_RevertIf_ChainAliasNotFound()	test_GetChain_RevertIf_ChainAliasNotFound()
test_SetChain_ExistingOne()	test_SetChain_ExistingOne()	test_SetChain_ChangesChainDataSuccessfully()
test_DontUseDefaultRpcUrl()	test_RevertIf_DontUseDefaultRpcUrl()	test_GetChain_RevertIf_DontUseDefaultRpcUrl()

I feel that the third approach is most clear and also makes it very easy to search for specific tests, e.g. forge test --mt test_GetChain would run all tests for GetChain method.

[mds1]

Thank you! Per foundry-rs/book#1279 (comment) let's keep the current convention that you have now, this looks great