Add onchain governor addresses to timelockRolesUpgrader test (#1)

* Add onchain governor addrs to timelockRolesUpgrader tests

* Remove addr arguments

Author: garyghayrat

script/DeployTimelockRolesUpgrader.s.sol

@@ -9,18 +9,15 @@ import {TimelockRolesUpgrader} from
 import {SharedGovernorConstants} from "script/SharedGovernorConstants.sol";
 
 contract DeployTimelockRolesUpgrader is BaseDeployer, SharedGovernorConstants {
-    function run(address _newCoreGovernor, address _newTreasuryGovernor)
-        public
-        returns (TimelockRolesUpgrader timelockRolesUpgrader)
-    {
+    function run() public returns (TimelockRolesUpgrader timelockRolesUpgrader) {
         vm.startBroadcast();
         timelockRolesUpgrader = new TimelockRolesUpgrader(
             L2_CORE_GOVERNOR_TIMELOCK,
             L2_CORE_GOVERNOR,
-            _newCoreGovernor,
+            L2_CORE_GOVERNOR_ONCHAIN,
             L2_TREASURY_GOVERNOR_TIMELOCK,
             L2_TREASURY_GOVERNOR,
-            _newTreasuryGovernor
+            L2_TREASURY_GOVERNOR_ONCHAIN
         );
         vm.stopBroadcast();
     }

test/util/SetupNewGovernors.sol

@@ -75,8 +75,7 @@ abstract contract SetupNewGovernors is SharedGovernorConstants, Test {
         // Prepare the script to submit upgrade proposal
         submitUpgradeProposalScript = new SubmitUpgradeProposalScript();
         DeployTimelockRolesUpgrader deployTimelockRolesUpgrader = new DeployTimelockRolesUpgrader();
-        timelockRolesUpgrader =
-            deployTimelockRolesUpgrader.run(address(newCoreGovernor), address(newTreasuryGovernor));
+        timelockRolesUpgrader = deployTimelockRolesUpgrader.run();
     }
 }