events comments

src/issuer/DirectBuyIssuer.sol

@@ -17,7 +17,9 @@ contract DirectBuyIssuer is BuyOrderIssuer {
 
     error UnreturnedEscrow();
 
+    /// @dev Emitted when `amount` of escrowed payment is taken for `orderId`
     event EscrowTaken(bytes32 indexed orderId, address indexed recipient, uint256 amount);
+    /// @dev Emitted when `amount` of escrowed payment is returned for `orderId`
     event EscrowReturned(bytes32 indexed orderId, address indexed recipient, uint256 amount);
 
     /// ------------------ State ------------------ ///

src/issuer/OrderFees.sol

@@ -16,6 +16,7 @@ contract OrderFees is Ownable2Step, IOrderFees {
     error FeeTooLarge();
     error DecimalsTooLarge();
 
+    /// @dev Emitted when `perOrderFee` and `percentageFeeRate` are set
     event FeeSet(uint64 perOrderFee, uint64 percentageFeeRate);
 
     /// ------------------ Constants ------------------ ///

src/issuer/OrderProcessor.sol

@@ -60,9 +60,11 @@ abstract contract OrderProcessor is
     error DuplicateOrder();
     error AmountTooLarge();
 
-    // Events
+    /// @dev Emitted when `treasury` is set
     event TreasurySet(address indexed treasury);
-    event OrderFeesSet(IOrderFees orderFees);
+    /// @dev Emitted when `orderFees` is set
+    event OrderFeesSet(IOrderFees indexed orderFees);
+    /// @dev Emitted when orders are paused/unpaused
     event OrdersPaused(bool paused);
 
     /// ------------------ Constants ------------------ ///

test/SellOrderProcessor.t.sol

@@ -7,7 +7,8 @@ import "openzeppelin-contracts/contracts/proxy/ERC1967/ERC1967Proxy.sol";
 import "./utils/mocks/MockBridgedERC20.sol";
 import "./utils/SigUtils.sol";
 import "../src/issuer/SellOrderProcessor.sol";
-import {OrderFees} from "../src/issuer/OrderFees.sol";
+import "../src/issuer/IOrderBridge.sol";
+import {OrderFees, IOrderFees} from "../src/issuer/OrderFees.sol";
 import "openzeppelin-contracts/contracts/utils/Strings.sol";
 
 contract SellOrderProcessorTest is Test {