[SMR-1975] TODO cleanup

[tsnewnami]

This PR cleans up TODOs in tests. Other TODOs such as

zkevm-bridge-contracts/src/root/flowrate/FlowRateWithdrawalQueue.sol

Line 85 in 6b383e3

// @TODO look at using a mapping instead of an array to make the withdraw function simpler

and

zkevm-bridge-contracts/script/DeployRootContracts.s.sol

Line 18 in 421221e

were left unchanged as these are more complex tasks that will be subsumed by subsequent tasks

[platform-sa]

📃CI Report

Compiling 148 files with 0.8.19
Solc 0.8.19 finished in 16.35s
Compiler run �[32msuccessful!�[0m
Analysing contracts...
Running tests...

File	% Lines	% Statements	% Branches	% Funcs
script/DeployChildContracts.s.sol	0.00% (0/22)	0.00% (0/32)	100.00% (0/0)	0.00% (0/1)
script/DeployRootContracts.s.sol	0.00% (0/23)	0.00% (0/33)	0.00% (0/2)	0.00% (0/1)
script/InitializeChildContracts.s.sol	0.00% (0/11)	0.00% (0/17)	100.00% (0/0)	0.00% (0/1)
script/InitializeRootContracts.s.sol	0.00% (0/11)	0.00% (0/17)	100.00% (0/0)	0.00% (0/1)
script/Utils.sol	0.00% (0/12)	0.00% (0/16)	0.00% (0/2)	0.00% (0/2)
src/child/ChildAxelarBridgeAdaptor.sol	100.00% (37/37)	100.00% (55/55)	100.00% (14/14)	100.00% (6/6)
src/child/ChildERC20.sol	100.00% (14/14)	100.00% (15/15)	50.00% (1/2)	100.00% (7/7)
src/child/ChildERC20Bridge.sol	97.16% (137/141)	97.89% (186/190)	92.68% (76/82)	100.00% (16/16)
src/child/WIMX.sol	0.00% (0/19)	0.00% (0/22)	0.00% (0/8)	0.00% (0/6)
src/common/AdaptorRoles.sol	100.00% (6/6)	100.00% (6/6)	100.00% (0/0)	100.00% (6/6)
src/common/BridgeRoles.sol	100.00% (8/8)	100.00% (8/8)	100.00% (0/0)	100.00% (8/8)
src/lib/EIP712MetaTransaction.sol	8.00% (2/25)	9.68% (3/31)	8.33% (1/12)	14.29% (1/7)
src/lib/EIP712Upgradeable.sol	73.33% (11/15)	60.87% (14/23)	0.00% (0/2)	50.00% (2/4)
src/root/RootAxelarBridgeAdaptor.sol	100.00% (37/37)	100.00% (55/55)	100.00% (14/14)	100.00% (6/6)
src/root/RootERC20Bridge.sol	94.52% (138/146)	96.24% (205/213)	90.79% (69/76)	90.91% (20/22)
src/root/flowrate/FlowRateDetection.sol	90.00% (27/30)	90.91% (30/33)	78.57% (11/14)	100.00% (4/4)
src/root/flowrate/FlowRateWithdrawalQueue.sol	67.39% (31/46)	61.67% (37/60)	50.00% (7/14)	85.71% (6/7)
src/root/flowrate/RootERC20BridgeFlowRate.sol	100.00% (43/43)	98.00% (49/50)	100.00% (10/10)	100.00% (9/9)

For a full HTML report run: forge coverage --report lcov && genhtml --ignore-errors category --branch-coverage --output-dir coverage lcov.info