chore(protocol-contracts): save lz gas in FeesSenderToBurner (#1269)

* chore(protocol-contracts): save lz gas in FeesSenderToBurner

* chore(protocol-contracts): optimize enforced executor option

* test(protocol-contracts): fix fees burner tests

* chore(protocol-contracts): fix comment

Author: jatZama

protocol-contracts/feesBurner/contracts/FeesSenderToBurner.sol

@@ -48,7 +48,7 @@ contract FeesSenderToBurner {
 
         if (amountNormalized == 0) revert NotEnoughZAMAToSend();
 
-        bytes memory options = OptionsBuilder.newOptions().addExecutorLzReceiveOption(80000, 0);
+        bytes memory options = OptionsBuilder.newOptions();
 
         SendParam memory sendParam = SendParam({
             dstEid: DESTINATION_EID,
@@ -72,7 +72,7 @@ contract FeesSenderToBurner {
         uint256 amountNormalized = (amount/(_DECIMAL_CONVERSION_RATE))*_DECIMAL_CONVERSION_RATE;
         if (amountNormalized == 0) revert NotEnoughZAMAToSend();
 
-        bytes memory options = OptionsBuilder.newOptions().addExecutorLzReceiveOption(80000, 0);
+        bytes memory options = OptionsBuilder.newOptions();
 
         SendParam memory sendParam = SendParam({
             dstEid: DESTINATION_EID,

protocol-contracts/feesBurner/test/hardhat/feesBurner.test.ts

@@ -68,12 +68,30 @@ describe("FeesBurner", () => {
     await zamaERC20.grantRole(MINTER_ROLE, deployer.address);
 
     // Setting destination endpoints in the LZEndpoint mock for each MyOFT instance
-
     await mockEndpointV2A.setDestLzEndpoint(zamaOFTAddress, mockEndpointV2BAddress);
     await mockEndpointV2B.setDestLzEndpoint(zamaOFTAdapterAddress, mockEndpointV2AAddress);
+    
     // Setting each OFT instance as a peer of the other in the mock LZEndpoint
     await zamaOFTAdapter.connect(deployer).setPeer(eidB, ethers.zeroPadValue(zamaOFTAddress, 32));
     await zamaOFT.connect(deployer).setPeer(eidA, ethers.zeroPadValue(zamaOFTAdapterAddress, 32));
+    
+    //  Set enforced executor options
+    const enforcedOptionsOFT = [
+      {
+        eid: eidA,
+        msgType: await zamaOFT.SEND(),
+        options: Options.newOptions().addExecutorLzReceiveOption(50000, 0).toHex().toString(),
+      },
+    ];
+    await zamaOFT.setEnforcedOptions(enforcedOptionsOFT);
+    const enforcedOptionsOFTAdapter = [
+      {
+        eid: eidB,
+        msgType: await zamaOFTAdapter.SEND(),
+        options: Options.newOptions().addExecutorLzReceiveOption(50000, 0).toHex().toString(),
+      },
+    ];
+    await zamaOFTAdapter.setEnforcedOptions(enforcedOptionsOFTAdapter);
   });
 
   describe("ProtocolFeesBurner", () => {
@@ -224,10 +242,7 @@ describe("FeesBurner", () => {
       it("should properly quote native fees for bridging", async () => {
         const quotedFees = await feesSenderToBurner.quote();
 
-        await expect(feesSenderToBurner.sendFeesToBurner({ value: quotedFees - 1n })).to.be.revertedWithCustomError(
-          feesSenderToBurnerInterface,
-          "InsufficientFee",
-        );
+        await expect(feesSenderToBurner.sendFeesToBurner({ value: quotedFees - 1n })).to.be.revertedWith('LayerZeroMock: not enough native for fees');
         expect(await feesSenderToBurner.sendFeesToBurner({ value: quotedFees })).not.to.be.reverted;
       });
 

protocol-contracts/token/layerzero.config.mainnet.ts

@@ -26,7 +26,7 @@ const EVM_ENFORCED_OPTIONS: OAppEnforcedOption[] = [
     {
         msgType: 1,
         optionType: ExecutorOptionType.LZ_RECEIVE,
-        gas: 80000,
+        gas: 50000,
         value: 0,
     },
 ]

protocol-contracts/token/layerzero.config.testnet.ts

@@ -25,7 +25,7 @@ const EVM_ENFORCED_OPTIONS: OAppEnforcedOption[] = [
     {
         msgType: 1,
         optionType: ExecutorOptionType.LZ_RECEIVE,
-        gas: 80000,
+        gas: 50000,
         value: 0,
     },
 ]