From 2b3eeb32302f250b98c61b722a832aed0ef47528 Mon Sep 17 00:00:00 2001
From: 0xchin <77933451+0xChin@users.noreply.github.com>
Date: Wed, 22 Oct 2025 17:42:35 -0300
Subject: [PATCH] chore: improve test naming

---
 packages/contracts-bedrock/test/L2/FeeVault.t.sol | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/packages/contracts-bedrock/test/L2/FeeVault.t.sol b/packages/contracts-bedrock/test/L2/FeeVault.t.sol
index 03ae05ec6b5..1cfee1b57ea 100644
--- a/packages/contracts-bedrock/test/L2/FeeVault.t.sol
+++ b/packages/contracts-bedrock/test/L2/FeeVault.t.sol
@@ -33,13 +33,6 @@ abstract contract FeeVault_Uncategorized_Test is CommonTest {
         feeVault.setWithdrawalNetwork(Types.WithdrawalNetwork.L2);
     }
 
-    /// @notice Tests that the l1 fee wallet is correct.
-    function test_constructor_succeeds() external view {
-        assertEq(feeVault.RECIPIENT(), recipient);
-        assertEq(feeVault.MIN_WITHDRAWAL_AMOUNT(), minWithdrawalAmount);
-        assertEq(uint8(feeVault.WITHDRAWAL_NETWORK()), uint8(withdrawalNetwork));
-    }
-
     /// @notice Tests that the initialize function succeeds.
     function test_initialize_succeeds() external view {
         assertEq(feeVault.recipient(), recipient);
@@ -55,6 +48,13 @@ abstract contract FeeVault_Uncategorized_Test is CommonTest {
         feeVault.initialize(recipient, minWithdrawalAmount, Types.WithdrawalNetwork.L1);
     }
 
+    /// @notice Tests that the immutable values match the storage getters.
+    function test_immutableMatchesStorageVariables_succeeds() external view {
+        assertEq(feeVault.RECIPIENT(), feeVault.recipient());
+        assertEq(feeVault.MIN_WITHDRAWAL_AMOUNT(), feeVault.minWithdrawalAmount());
+        assertEq(uint8(feeVault.WITHDRAWAL_NETWORK()), uint8(feeVault.withdrawalNetwork()));
+    }
+
     /// @notice Tests that the fee feeVault is able to receive ETH.
     function test_receive_succeeds() external {
         uint256 balance = address(feeVault).balance;