fix:add standard to test file

src/modules/paymentProcessor/abstracts/CrossChainBase_v1.sol

@@ -23,7 +23,9 @@ import {ICrossChainBase_v1} from "../interfaces/ICrosschainBase_v1.sol";
  *                          In case of any concerns or findings, please refer to our
  *                          Security Policy at security.inverter.network or email us
  *                          directly!
- * @author  Inverter Network
+ * @custom:version 1.0.0
+ * @custom:standard-version 1.0.0
+ * @author  33Audits
  */
 abstract contract CrossChainBase_v1 is ICrossChainBase_v1, Module_v1 {
     // Storage Variables

src/modules/paymentProcessor/abstracts/PP_Crosschain_v1.sol

@@ -39,19 +39,32 @@ import {IPP_Crosschain_v1} from "../interfaces/IPP_Crosschain_v1.sol";
  *                          at security.inverter.network or email us directly!
  *
  * @author  Inverter Network
+ * @custom:version 1.0.0
+ * @custom:standard-version 1.0.0
  */
 abstract contract PP_Crosschain_v1 is CrossChainBase_v1, IPP_Crosschain_v1 {
-    /// @inheritdoc ERC165Upgradeable
-    function supportsInterface(bytes4 interfaceId_)
-        public
-        view
-        virtual
-        override(CrossChainBase_v1)
-        returns (bool)
-    {
-        return interfaceId_ == type(IPP_Crosschain_v1).interfaceId
-            || super.supportsInterface(interfaceId_);
-    }
+    //--------------------------------------------------------------------------
+    // Events
+    event PaymentProcessed(uint indexed paymentId, address indexed client);
+    event PaymentCancelled(uint indexed paymentId, address indexed client);
+    event UnclaimablePaymentClaimed(
+        address indexed client,
+        address indexed token,
+        address indexed receiver,
+        uint amount
+    );
+
+    //--------------------------------------------------------------------------
+    // Constants
+    uint public constant VERSION = 1;
+
+    //--------------------------------------------------------------------------
+    // Storage Variables
+    bytes public executionData;
+    uint public _paymentId;
+
+    /// @dev    Gap for possible future upgrades.
+    uint[50] private __gap;
 
     //--------------------------------------------------------------------------
     // Modifiers
@@ -73,18 +86,19 @@ abstract contract PP_Crosschain_v1 is CrossChainBase_v1, IPP_Crosschain_v1 {
     }
 
     //--------------------------------------------------------------------------
-    // Storage
+    // External/Public Functions
 
-    bytes public executionData;
-
-    /// @dev    Gap for possible future upgrades.
-    uint[50] private __gap;
-
-    //@dev paymentId
-    uint public _paymentId;
-
-    //--------------------------------------------------------------------------
-    // Virtual Functions
+    /// @inheritdoc ERC165Upgradeable
+    function supportsInterface(bytes4 interfaceId_)
+        public
+        view
+        virtual
+        override(CrossChainBase_v1)
+        returns (bool)
+    {
+        return interfaceId_ == type(IPP_Crosschain_v1).interfaceId
+            || super.supportsInterface(interfaceId_);
+    }
 
     /// @notice Process payments for a given payment client
     /// @param client The payment client to process payments for
@@ -134,6 +148,9 @@ abstract contract PP_Crosschain_v1 is CrossChainBase_v1, IPP_Crosschain_v1 {
             && _validPaymentToken(order.paymentToken);
     }
 
+    //--------------------------------------------------------------------------
+    // Internal Functions
+
     /// @dev    Validate address input.
     /// @param  addr Address to validate.
     /// @return True if address is valid.
@@ -162,18 +179,13 @@ abstract contract PP_Crosschain_v1 is CrossChainBase_v1, IPP_Crosschain_v1 {
         pure
         returns (bool)
     {
-        // _start + _cliff should be less or equal to _end
-        // this already implies that _start is not greater than _end
         return _start + _cliff <= _end;
     }
 
     /// @dev    Validate payment token input.
     /// @param  _token Address of the token to validate.
     /// @return True if address is valid.
     function _validPaymentToken(address _token) internal returns (bool) {
-        // Only a basic sanity check that the address supports the balanceOf() function. The corresponding
-        // module should ensure it's sending an ERC20.
-
         (bool success, bytes memory data) = _token.call(
             abi.encodeWithSelector(
                 IERC20(_token).balanceOf.selector, address(this)

test/modules/paymentProcessor/abstracts/PP_Connext_Crosschain_v1_Test.t.sol

@@ -2,68 +2,72 @@
 
 pragma solidity ^0.8.20;
 
+//--------------------------------------------------------------------------
+// Imports
+
 // External Dependencies
 import {Test} from "forge-std/Test.sol";
 import {Clones} from "@oz/proxy/Clones.sol";
+import {IERC20Errors} from "@oz/interfaces/draft-IERC6093.sol";
+import {IWETH} from "src/modules/paymentProcessor/interfaces/IWETH.sol";
+import "forge-std/console2.sol";
 
 // Internal Dependencies
 import {PP_Connext_Crosschain_v1} from
     "src/modules/paymentProcessor/PP_Connext_Crosschain_v1.sol";
-
 import {CrossChainBase_v1} from
     "src/modules/paymentProcessor/abstracts/CrossChainBase_v1.sol";
-import {CrossChainBase_v1_Exposed} from
-    "test/modules/paymentProcessor/abstracts/CrossChainBase_v1_Exposed.sol";
-import {IERC20PaymentClientBase_v1} from
-    "@lm/interfaces/IERC20PaymentClientBase_v1.sol";
 import {ICrossChainBase_v1} from
     "src/modules/paymentProcessor/interfaces/ICrosschainBase_v1.sol";
 import {IPaymentProcessor_v1} from
     "src/modules/paymentProcessor/IPaymentProcessor_v1.sol";
+import {IPP_Crosschain_v1} from
+    "src/modules/paymentProcessor/interfaces/IPP_Crosschain_v1.sol";
+import {IModule_v1, IOrchestrator_v1} from "src/modules/base/IModule_v1.sol";
+import {IERC20PaymentClientBase_v1} from
+    "@lm/interfaces/IERC20PaymentClientBase_v1.sol";
+
+// Tests and Mocks
+import {CrossChainBase_v1_Exposed} from
+    "test/modules/paymentProcessor/abstracts/CrossChainBase_v1_Exposed.sol";
 import {PP_Connext_Crosschain_v1_Exposed} from
     "test/modules/paymentProcessor/abstracts/PP_Connext_Crosschain_v1_Exposed.sol";
-// Tests and Mocks
 import {Mock_EverclearPayment} from
     "test/modules/paymentProcessor/abstracts/mocks/Mock_EverclearPayment.sol";
 import {
     IERC20PaymentClientBase_v1,
     ERC20PaymentClientBaseV1Mock
 } from "test/utils/mocks/modules/paymentClient/ERC20PaymentClientBaseV1Mock.sol";
-import {
-    ModuleTest,
-    IModule_v1,
-    IOrchestrator_v1
-} from "test/modules/ModuleTest.sol";
+import {ModuleTest} from "test/modules/ModuleTest.sol";
 import {OZErrors} from "test/utils/errors/OZErrors.sol";
-import {IERC20Errors} from "@oz/interfaces/draft-IERC6093.sol";
-import {IWETH} from "src/modules/paymentProcessor/interfaces/IWETH.sol";
-import "forge-std/console2.sol";
-import {IPP_Crosschain_v1} from
-    "src/modules/paymentProcessor/interfaces/IPP_Crosschain_v1.sol";
-
-import {IModule_v1, IOrchestrator_v1} from "src/modules/base/IModule_v1.sol";
 
 contract PP_Connext_Crosschain_v1_Test is ModuleTest {
+    //--------------------------------------------------------------------------
+    // Constants
+    uint constant MINTED_SUPPLY = 1000 ether;
+
+    //--------------------------------------------------------------------------
+    // Test Storage
     PP_Connext_Crosschain_v1_Exposed public paymentProcessor;
     Mock_EverclearPayment public everclearPaymentMock;
     ERC20PaymentClientBaseV1Mock paymentClient;
     IPP_Crosschain_v1 public crossChainBase;
-
     IWETH public weth;
-
     uint public chainId;
 
-    // Add these as contract state variables
+    // Bridge-related storage
     address public mockConnextBridge;
     address public mockEverClearSpoke;
     address public mockWeth;
 
+    // Execution data storage
     uint maxFee = 0;
     uint ttl = 1;
     bytes executionData;
     bytes invalidExecutionData;
 
-    uint constant MINTED_SUPPLY = 1000 ether;
+    //--------------------------------------------------------------------------
+    // Setup Function
 
     function setUp() public {
         // Set chain ID for test environment
@@ -104,6 +108,9 @@ contract PP_Connext_Crosschain_v1_Test is ModuleTest {
         _setupInitialBalances();
     }
 
+    //--------------------------------------------------------------------------
+    // Initialization Tests
+
     /* Test initialization
     */
     function testInit() public override(ModuleTest) {
@@ -138,6 +145,9 @@ contract PP_Connext_Crosschain_v1_Test is ModuleTest {
         paymentProcessor.init(_orchestrator, _METADATA, abi.encode(1));
     }
 
+    //--------------------------------------------------------------------------
+    // Payment Processing Tests
+
     /* Test single payment processing
     └── Given single valid payment order
         └── When processing cross-chain payments
@@ -267,6 +277,9 @@ contract PP_Connext_Crosschain_v1_Test is ModuleTest {
         );
     }
 
+    //--------------------------------------------------------------------------
+    // Error Case Tests
+
     /* Test invalid execution data
         └── When processing with invalid Connext parameters
             └── Then it should revert
@@ -628,7 +641,8 @@ contract PP_Connext_Crosschain_v1_Test is ModuleTest {
         );
     }
 
-    // Helper functions
+    //--------------------------------------------------------------------------
+    // Helper Functions
 
     function _setupSinglePayment(address _recipient, uint _amount)
         internal