Merge pull request #29 from jorgejch/develop

Minor adjustments
jorgejch · Dec 15, 2024 · d477eae · d477eae
2 parents 6529012 + abdc5c4
commit d477eae
Show file tree

Hide file tree

Showing 5 changed files with 334 additions and 135 deletions.
diff --git a/contracts/UniswapV3Arbitrage.sol b/contracts/UniswapV3Arbitrage.sol
@@ -179,14 +179,21 @@ contract UniswapV3Arbitrage is FlashLoanSimpleReceiverBase, Ownable2Step {
         );
         int256 profit = int256(swap3AmountOut) -
             int256(amountOwned + decoded.extraCost);
-
         emit ArbitrageConcluded(
             _executionCounter,
             amount,
             swap3AmountOut,
             profit
         );
-        require(profit > 0, string(abi.encodePacked("not profitable:", Strings.toString(uint256(profit)))));
+        require(
+            profit > 0,
+            string(
+                abi.encodePacked(
+                    "not profitable by: ",
+                    Strings.toString(uint256(-profit))
+                )
+            )
+        );
         IERC20(asset).approve(_poolAddress, amountOwned);
         return true;
     }

diff --git a/contracts/tests/test_UniswapV3Arbitrage.sol b/contracts/tests/test_UniswapV3Arbitrage.sol