From d0d0e869d0d734238d7c90023d8a876f9df1e390 Mon Sep 17 00:00:00 2001
From: Gerardo Nardelli <gerardo.nardelli@altoros.com>
Date: Fri, 13 Nov 2020 12:30:14 -0300
Subject: [PATCH] Fix flash loan fee usage in _getAmountsIn

---
 contracts/adapters/BaseUniswapAdapter.sol | 4 ++--
 test/uniswapAdapters.spec.ts              | 8 ++++----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/contracts/adapters/BaseUniswapAdapter.sol b/contracts/adapters/BaseUniswapAdapter.sol
index e5e779f7..7a727b7d 100644
--- a/contracts/adapters/BaseUniswapAdapter.sol
+++ b/contracts/adapters/BaseUniswapAdapter.sol
@@ -335,8 +335,8 @@ contract BaseUniswapAdapter {
 
     uint256[] memory amounts = UNISWAP_ROUTER.getAmountsIn(amountOut, path);
 
-    // Subtract flash loan fee
-    uint256 finalAmountIn = amounts[0].sub(amounts[0].mul(FLASHLOAN_PREMIUM_TOTAL).div(10000));
+    // Add flash loan fee
+    uint256 finalAmountIn = amounts[0].add(amounts[0].mul(FLASHLOAN_PREMIUM_TOTAL).div(10000));
 
     uint256 reserveInDecimals = _getDecimals(reserveIn);
     uint256 reserveOutDecimals = _getDecimals(reserveOut);
diff --git a/test/uniswapAdapters.spec.ts b/test/uniswapAdapters.spec.ts
index 2a086fb8..6f40530f 100644
--- a/test/uniswapAdapters.spec.ts
+++ b/test/uniswapAdapters.spec.ts
@@ -152,7 +152,7 @@ makeSuite('Uniswap adapters', (testEnv: TestEnv) => {
 
         const amountIn = parseEther('1');
         const flashloanPremium = amountIn.mul(9).div(10000);
-        const amountToSwap = amountIn.sub(flashloanPremium);
+        const amountToSwap = amountIn.add(flashloanPremium);
 
         const wethPrice = await oracle.getAssetPrice(weth.address);
         const daiPrice = await oracle.getAssetPrice(dai.address);
@@ -197,7 +197,7 @@ makeSuite('Uniswap adapters', (testEnv: TestEnv) => {
 
         const amountIn = parseEther('10');
         const flashloanPremium = amountIn.mul(9).div(10000);
-        const amountToSwap = amountIn.sub(flashloanPremium);
+        const amountToSwap = amountIn.add(flashloanPremium);
 
         const aavePrice = await oracle.getAssetPrice(aave.address);
         const usdcPrice = await oracle.getAssetPrice(usdc.address);
@@ -241,7 +241,7 @@ makeSuite('Uniswap adapters', (testEnv: TestEnv) => {
     });
   });
 
-  describe('UniswapLiquiditySwapAdapter', () => {
+  describe.only('UniswapLiquiditySwapAdapter', () => {
     describe('constructor', () => {
       it('should deploy with correct parameters', async () => {
         const {addressesProvider} = testEnv;
@@ -1959,7 +1959,7 @@ makeSuite('Uniswap adapters', (testEnv: TestEnv) => {
     });
   });
 
-  describe('UniswapRepayAdapter', () => {
+  describe.only('UniswapRepayAdapter', () => {
     describe('constructor', () => {
       it('should deploy with correct parameters', async () => {
         const {addressesProvider} = testEnv;