Fix test snapshot exclusion

[RensR]

This PR cleans up the base contracts for all tests, removing the MockRMN in favor of RMNRemote and moving logic further up the inheritance chain. This should result in (minor) compilation gains. This PR also renames the reverts tests to _RevertWhen_ tests so the snapshot excludes them. This was done using regex so not all names are perfect. The ongoing initiative of fixing all test names in any file you touch should continue, but at least we've addresses the snapshot part of it.

This PR

➜  contracts git:(fix-test-naming) forge build --no-cache     
[⠊] Compiling...
[⠊] Compiling 386 files with Solc 0.8.24
[⠆] Solc 0.8.24 finished in 101.08s
Compiler run successful!
➜  contracts git:(fix-test-naming) forge build --no-cache
[⠊] Compiling...
[⠒] Compiling 386 files with Solc 0.8.24
[⠘] Solc 0.8.24 finished in 98.60s
Compiler run successful!

Develop

➜  contracts git:(develop) forge build --no-cache
[⠊] Compiling...
[⠔] Compiling 388 files with Solc 0.8.24
[⠑] Solc 0.8.24 finished in 102.75s
Compiler run successful!
➜  contracts git:(develop) forge build --no-cache
[⠊] Compiling...
[⠢] Compiling 388 files with Solc 0.8.24
[⠰] Solc 0.8.24 finished in 99.21s
Compiler run successful!

It appears to be ~1% faster while improving our test suite.

[github-actions]

AER Report: CI Core ran successfully ✅

aer_workflow , commit

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

[github-actions]

Static analysis results are available

Hey @RensR, you can view Slither reports in the job summary here or download them as artifact here.
Please check them before merging and make sure you have addressed all issues.