refactor: remove args from modifier

Author: PavelNaydanov

src/tokenDistribution/MetaTokenDistributor.sol

@@ -7,7 +7,7 @@ import {Clones} from "@openzeppelin/contracts/proxy/Clones.sol";
 import {MetaToken} from "src/MetaToken.sol";
 
 import {IMetaTokenDistributor} from "./interfaces/IMetaTokenDistributor.sol";
-import {Vesting, IVesting} from "./vesting/Vesting.sol";
+import {IVesting} from "./vesting/Vesting.sol";
 import {IVestingParams} from "./vesting/interfaces/IVestingParams.sol";
 import {Schedule, Beneficiary, VestingType} from "./utils/Common.sol";
 
@@ -34,7 +34,7 @@ contract MetaTokenDistributor is IMetaTokenDistributor {
     mapping(VestingType vestingType => address vesting) private _vestingAddresses;
 
     /// @dev Validate start vesting time
-    modifier validateTime(VestingType vestingType) {
+    modifier validateTime() {
         uint64 startTime = _vestingParams.getStartTime();
 
         if (block.timestamp < startTime) {
@@ -65,7 +65,7 @@ contract MetaTokenDistributor is IMetaTokenDistributor {
      * @return vesting Address of created vesting contract
      * @dev Any can start vesting after start time
      */
-    function startVesting(VestingType vestingType) external validateTime(vestingType) returns (address vesting) {
+    function startVesting(VestingType vestingType) external validateTime returns (address vesting) {
         if (_vestingAddresses[vestingType] != address(0)) {
             revert VestingHasAlreadyStarted();
         }

test/tokenDistribution/vesting/Vesting.t.sol

@@ -11,7 +11,7 @@ import {MetaToken} from "src/MetaToken.sol";
 import {VestingType, Beneficiary, Schedule, Period} from "src/tokenDistribution/utils/Common.sol";
 
 contract VestingTest is Test {
-    uint256 public constant MONTH = 30;
+    uint256 public constant MONTH = 30 days;
     uint256 public constant BASIS_POINTS = 10_000; // the same vesting contract
 
     uint256 public constant SCHEDULE_TGE_PORTION = 2000;