Apply suggestions from code review

Co-authored-by: Zied Guesmi <[email protected]>

contracts/beacon/Voucher.sol

@@ -436,16 +436,16 @@ contract Voucher is Initializable, IVoucher {
             // Transfer non-sponsored amount from the iExec account of the
             // requester to the iExec account of the voucher
             //slither-disable-start arbitrary-send-erc20
-            // Note: We can disable this check since request order is signed by requester so he agrees to pay for it
+            // Note: We can disable this check since the requester signed the request order and agreed to pay for the deal.
             // & caller is only authorized.
+            // SRLC
             if (
                 !IERC20(iexecPoco).transferFrom(
                     requestOrder.requester,
                     address(this),
                     dealPrice - sponsoredAmount
                 )
             ) {
-                // SRLC
                 revert("Voucher: Transfer of non-sponsored amount failed");
             }
             //slither-disable-end arbitrary-send-erc20

test/VoucherHub.test.ts

@@ -668,6 +668,7 @@ describe('VoucherHub', function () {
                 ),
             ).to.be.revertedWith('VoucherHub: type index out of bounds');
         });
+
         it('Should not create voucher when SLRC transfer fails', async function () {
             const { voucherOwner1 } = await loadFixture(deployFixture);
             await voucherHubWithAssetEligibilityManagerSigner.createVoucherType(

test/beacon/Voucher.test.ts

@@ -1071,6 +1071,7 @@ describe('Voucher', function () {
                 );
             });
         });
+
         describe('Should not claim task when SLRC transfer fails', async () => {
             it('Classic', async () => await runTest(voucherMatchOrders, claim));
             it('Boost', async () => await runTest(voucherMatchOrdersBoost, claimBoost));