test(contracts): improve OptimismSuperchainERC20Beacon test coverage

[devin-ai-integration]

Summary

Two small improvements to OptimismSuperchainERC20Beacon.t.sol:

1. Naming fix: Rename test_implementation_isCorrect_works → test_implementation_succeeds to follow the [method]_[functionName]_[outcome] naming convention.
2. Add version test: New OptimismSuperchainERC20Beacon_Version_Test contract with test_version_succeeds() using SemverComp.parse() to validate the version() getter returns a valid semver string. This follows the same pattern used in L1FeeVault.t.sol, WETH.t.sol, and other test files.

Note: All tests in this file are currently skipped via vm.skip(true) in OptimismSuperchainERC20Beacon_TestInit pending reintegration of the beacon contract.

Review & Testing Checklist for Human

• Confirm the version test pattern (SemverComp.parse(ISemver(...).version())) is appropriate for this contract — the contract is currently behind vm.skip(true) so the test only compiles but does not execute at runtime

Notes

• just pr passes all 16/16 checks locally
• Import whitespace changes (e.g. { CommonTest } → {CommonTest}) are from forge fmt

Link to Devin session: https://app.devin.ai/sessions/0127a225aa8348c0b518d08f3df0ef3d
Requested by: @aliersh

[devin-ai-integration]

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

• Address comments on this PR that start with 'DevinAI' or '@devin'.
• Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

• Disable automatic comment and CI monitoring