Skip to content

Commit ea3f268

Browse files
manhlx3006ilanDoronAnyhowclickayobuenavista
authored
Fix merging conflict (#604)
* update git ignore (#544) * Kyber Bancor bridge reserve (#546) * Shifted arbitrage check to FeeBurner * Shifted relevant tests from expectedRate to feeBurner * Modified failing test, as it requires less gas to fail * Removed redundan >0 rate check, test for feeBurner * Added test for src==dest token * Istanbul fix, zero rate handling * Arbitrage check shifted to FeeBurner (#433) * Shifted arbitrage check to FeeBurner * Shifted relevant tests from expectedRate to feeBurner * Modified failing test, as it requires less gas to fail * Removed redundan >0 rate check, test for feeBurner * Added test for src==dest token * Revert "Merge branch 'clicky/zero_rate' into development" This reverts commit 795d2f1, reversing changes made to 80af391. * Clicky/buidl integration (#459) * Pulling stuff from KN/clicky/buidl-integration branch * Pull changes from expected rate and fee burner * Update kyberNetwork test file * Update kyberNetwork.js * Merge Mike's PR on enhanced step functions (#458) * [WIP] Add new conversion rate contract with step function enhancement (#416) * [WIP] Add new conversion rate contract with step function enhancement * Add more tests to kyberReserve for testing new conversion rate contract * Add more tests for kybetNetwork with new conversion rate contract * Add verify qty and imbalance step function amounts must be increasing * Add case set qty and balance step func with only one or zero element * Check for qty step function all amounts are non negative * Update conversion rate contract and kyber network tests * Fallback to old logics when qty is 0 * Update new conversion rate with inherite from old one, update logics for some edge cases and tests * Fixed typo * Add check for empty step function before executing step function * Fix issue when qty is negative and greater than all step qty * Fix issue with executing step func with 0 as qty (#417) * Remove unnecessary tests in conversionRate2, add more cases to test for new execute step function * Fix issue with execute step func for 0 qty * expose getImbalance function in conversionRate2.sol file (#419) * Update conversion rate contract 2 with new logics for executing step function * Resolve some comments, change name new conversion rates contract to EnhancedStepFunctions * Remove qty step function, compress data for imbalance steps * Resolve solhint max line length * Avoid modifying function params in execute step func * Update comments for each test case of getting correct bps for step func * Update with some comments, add more tests * Add await keyword * Update some comments, fix some old tests in kyber reserve * Fix wrong calculate imbalance of token in reserve test * Change default max imbalance to return when overflow, fix some tests with missing await keyword * Blocking setting qty step function in new conversion rate * Update execute step func with blocking rate, remove updatedRateBlock in getImbalance * Remove redundant stepX value in last step * Add check data when encoding step func, using MAX_VAL as default for last step X * Add checking for max value of step X and tests * Add check for max bps adjustment, check max_imbalance value test * Fixed Failing Tests (#462) * Istanbul fix * Fixed failing tests as per Ilan's feedback * Fixed failing tests * Minor fixes * Minor fixes * Sync enhanced step functions test file * Reduce getDecimals call in findBestRateTokenToToken in network contract (#467) * Reduce getDecimals call in findBestRateTokenToToken in network contract * Fix new line end file * Remove some tests in utils3 * Avoid decimals call when calling handleFees on FeeBurner (#468) * Ilan/merge master (#460) * reserve deployment scripts (#233) * reserve deployment scripts * read valid block duration from config file. (#423) * Weth Reserve Implementation (#304) * Weth Reserve Implementation Includes testing. Also includes an addition in helper to make throw catching more neat. * Add weth reserve test * Add deploy script for oasis reserve * Organize folder structure and update WrapConversionRate.sol #441 && #452 (#453) * Organize folder structure according to issue #441 this includes organizing audit folder and organizing reserves. Remove multisig support from WrapConversionRate.sol according to issue #452 * enable admin to call transfer admin of base contract * enable wrapConversionRate to control all adminOnly functions in conversionRate (#461) * Ilan/modify wrap conversion rate (#466) * enable wrapConversionRate to control all adminOnly functions in conversionRate * enable wrap conversion rate to control all admin functions in conversion rate contract * set admin as msg.sender. to follow base contract logic * remove redundant test. adopt to chainge in admin logic * add CR LF * rename folders. (#469) * rename folders. (remove contract from folder name.) * reset package - lock * move new mock contracts to mock folder. * create web3 object * Upgraded to ganache-cli@istanbul * Removed header * Istanbul fix (#457) * Return zero rate for same src and dest token (#475) * Return zero rate for same src & dest token * Removed unnecessary tests * Echo fork choice, fix broken logic * Try committing new package-lock.json * Added more info * Bumped to node v12.13.0 * Update package fix npm (#474) * Update package fix npm * Update package-lock file * Remove unused packages * Removed eslint * Add colour for better readability of fork choice * Add spacing between example commands * Resolve package-lock conflict * Revert "Bumped to node v12.13.0" This reverts commit 85cb734. * Fixed conflicting files * Revert "Merge branch 'clicky/ganacheIstanbul' into development" This reverts commit 654a1c3, reversing changes made to 6c0a21c. * Upgrade to ganache-cli@istanbul for testing with Istanbul fork (#482) * Added colour and newline to fork choice * Add node and npm version used * Removed eslint * Removed eslint * Got latest changes from Mike's PR * added colour and newline to fork choice * Fix coloring (I hope) * Changed BG to make text stand out * handle zero qty in get rate permission less reserve (#472) * handle zero qty in get rate permission less reserve * undo wrong change * Increase gas limit for 1 test due to Istanbul (#487) * [455] Enable unlist orderbook permissionless reserve (#485) * Add comments in FeeBurner for KNC decimals (#489) * Revert "Arbitrage check shifted to FeeBurner" (#488) * Revert "Arbitrage check shifted to FeeBurner (#433)" This reverts commit 80af391. * reverse un needed test changes for kyber NEtwork * Zero Src Qty Handling (#483) * Initial commit for zero rate handling. Need to rewrite tests * Improved comment to code * Modified getConversionRate function to require srcAmt > 0 * Removed tests, added one * Updated contracts as per feedback * Updated comment in test file * Added fix to expectedRateSmallQty * Remove extra = * Add additional section to check network.getExpectedRate doesn't revert as well * Removed redundant tests due to zero rate handling in network contract (#494) * update solidity coverage version (#491) * Added new deployment script and input files (#495) * Ilan/add rates getter (#493) * update solidity coverage version * add getter function for rates * update getter API and return values, add tests * Clicky/rename compile file (#496) * renamed compileOutput.js to compileContracts.js, added more contracts for reserveDeployer * Renamed compileOutput -> compileContracts * Minor spelling edit * update addresses for deployment (#497) * remove multi sig support. add admin functionality (#498) * remove multi sig support. add admin functionality * refactor wrapper contracts, update tests * update tests according to PR comments, add one missing check * Add some comments * Revert "[455] Enable unlist orderbook permissionless reserve (#485)" (#504) This reverts commit 8b63c0e. * Fix deployment script after staging deployment (#505) * Added new deployment script and input files * Remove old file * Added contract instantiation if deployer script breaks halfway through * avoid query to uniswap with zero rate (#510) * avoid query to uniswap with zero rate * test situation where trade result is 0 * enable setting min record resultion in conversion rates using wrapper (#513) * enable setting min record resultion in conversion rates using wrapper * linting, and add missing test * fix uniswap bridge for small amounts src qty (1) (#518) * fix uniswap bridge for smal amounts src qty (1) * avoid reading fee twice * fix uniswap deployer script * Ilan/deployment helper (#516) * some deployment changes, and helper contract" " * add kyber uniswap bridge to compilation. remove wrap conversion rate * small fixes in deployer * Add CR LF * Ilan/merge master (#519) * update some packages. fix solc version to 0.4.18 (#503) * fixed FPR deployment script (#508) * add smartdec audit for APR * use correct package-lock * Ilan/master to development (#525) * Development to Master. Istanbul changes. buidl. (#471) * Shifted arbitrage check to FeeBurner * Shifted relevant tests from expectedRate to feeBurner * Modified failing test, as it requires less gas to fail * Removed redundan >0 rate check, test for feeBurner * Added test for src==dest token * Istanbul fix, zero rate handling * Arbitrage check shifted to FeeBurner (#433) * Shifted arbitrage check to FeeBurner * Shifted relevant tests from expectedRate to feeBurner * Modified failing test, as it requires less gas to fail * Removed redundan >0 rate check, test for feeBurner * Added test for src==dest token * Revert "Merge branch 'clicky/zero_rate' into development" This reverts commit 795d2f1, reversing changes made to 80af391. * Clicky/buidl integration (#459) * Pulling stuff from KN/clicky/buidl-integration branch * Pull changes from expected rate and fee burner * Update kyberNetwork test file * Update kyberNetwork.js * Merge Mike's PR on enhanced step functions (#458) * [WIP] Add new conversion rate contract with step function enhancement (#416) * [WIP] Add new conversion rate contract with step function enhancement * Add more tests to kyberReserve for testing new conversion rate contract * Add more tests for kybetNetwork with new conversion rate contract * Add verify qty and imbalance step function amounts must be increasing * Add case set qty and balance step func with only one or zero element * Check for qty step function all amounts are non negative * Update conversion rate contract and kyber network tests * Fallback to old logics when qty is 0 * Update new conversion rate with inherite from old one, update logics for some edge cases and tests * Fixed typo * Add check for empty step function before executing step function * Fix issue when qty is negative and greater than all step qty * Fix issue with executing step func with 0 as qty (#417) * Remove unnecessary tests in conversionRate2, add more cases to test for new execute step function * Fix issue with execute step func for 0 qty * expose getImbalance function in conversionRate2.sol file (#419) * Update conversion rate contract 2 with new logics for executing step function * Resolve some comments, change name new conversion rates contract to EnhancedStepFunctions * Remove qty step function, compress data for imbalance steps * Resolve solhint max line length * Avoid modifying function params in execute step func * Update comments for each test case of getting correct bps for step func * Update with some comments, add more tests * Add await keyword * Update some comments, fix some old tests in kyber reserve * Fix wrong calculate imbalance of token in reserve test * Change default max imbalance to return when overflow, fix some tests with missing await keyword * Blocking setting qty step function in new conversion rate * Update execute step func with blocking rate, remove updatedRateBlock in getImbalance * Remove redundant stepX value in last step * Add check data when encoding step func, using MAX_VAL as default for last step X * Add checking for max value of step X and tests * Add check for max bps adjustment, check max_imbalance value test * Fixed Failing Tests (#462) * Istanbul fix * Fixed failing tests as per Ilan's feedback * Fixed failing tests * Minor fixes * Minor fixes * Sync enhanced step functions test file * Reduce getDecimals call in findBestRateTokenToToken in network contract (#467) * Reduce getDecimals call in findBestRateTokenToToken in network contract * Fix new line end file * Remove some tests in utils3 * Avoid decimals call when calling handleFees on FeeBurner (#468) * Ilan/merge master (#460) * reserve deployment scripts (#233) * reserve deployment scripts * read valid block duration from config file. (#423) * Weth Reserve Implementation (#304) * Weth Reserve Implementation Includes testing. Also includes an addition in helper to make throw catching more neat. * Add weth reserve test * Add deploy script for oasis reserve * Organize folder structure and update WrapConversionRate.sol #441 && #452 (#453) * Organize folder structure according to issue #441 this includes organizing audit folder and organizing reserves. Remove multisig support from WrapConversionRate.sol according to issue #452 * enable admin to call transfer admin of base contract * enable wrapConversionRate to control all adminOnly functions in conversionRate (#461) * Ilan/modify wrap conversion rate (#466) * enable wrapConversionRate to control all adminOnly functions in conversionRate * enable wrap conversion rate to control all admin functions in conversion rate contract * set admin as msg.sender. to follow base contract logic * remove redundant test. adopt to chainge in admin logic * add CR LF * rename folders. (#469) * rename folders. (remove contract from folder name.) * reset package - lock * move new mock contracts to mock folder. * create web3 object * Upgraded to ganache-cli@istanbul * Removed header * Istanbul fix (#457) * Return zero rate for same src and dest token (#475) * Return zero rate for same src & dest token * Removed unnecessary tests * Echo fork choice, fix broken logic * Try committing new package-lock.json * Added more info * Bumped to node v12.13.0 * Update package fix npm (#474) * Update package fix npm * Update package-lock file * Remove unused packages * Removed eslint * Add colour for better readability of fork choice * Add spacing between example commands * Resolve package-lock conflict * Revert "Bumped to node v12.13.0" This reverts commit 85cb734. * Fixed conflicting files * Revert "Merge branch 'clicky/ganacheIstanbul' into development" This reverts commit 654a1c3, reversing changes made to 6c0a21c. * Upgrade to ganache-cli@istanbul for testing with Istanbul fork (#482) * Added colour and newline to fork choice * Add node and npm version used * Removed eslint * Removed eslint * Got latest changes from Mike's PR * added colour and newline to fork choice * Fix coloring (I hope) * Changed BG to make text stand out * handle zero qty in get rate permission less reserve (#472) * handle zero qty in get rate permission less reserve * undo wrong change * Increase gas limit for 1 test due to Istanbul (#487) * [455] Enable unlist orderbook permissionless reserve (#485) * Add comments in FeeBurner for KNC decimals (#489) * Revert "Arbitrage check shifted to FeeBurner" (#488) * Revert "Arbitrage check shifted to FeeBurner (#433)" This reverts commit 80af391. * reverse un needed test changes for kyber NEtwork * Zero Src Qty Handling (#483) * Initial commit for zero rate handling. Need to rewrite tests * Improved comment to code * Modified getConversionRate function to require srcAmt > 0 * Removed tests, added one * Updated contracts as per feedback * Updated comment in test file * Added fix to expectedRateSmallQty * Remove extra = * Add additional section to check network.getExpectedRate doesn't revert as well * Removed redundant tests due to zero rate handling in network contract (#494) * update solidity coverage version (#491) * Added new deployment script and input files (#495) * Ilan/add rates getter (#493) * update solidity coverage version * add getter function for rates * update getter API and return values, add tests * Clicky/rename compile file (#496) * renamed compileOutput.js to compileContracts.js, added more contracts for reserveDeployer * Renamed compileOutput -> compileContracts * Minor spelling edit * update addresses for deployment (#497) * remove multi sig support. add admin functionality (#498) * remove multi sig support. add admin functionality * refactor wrapper contracts, update tests * update tests according to PR comments, add one missing check * Add some comments * Revert "[455] Enable unlist orderbook permissionless reserve (#485)" (#504) This reverts commit 8b63c0e. * Fix deployment script after staging deployment (#505) * Added new deployment script and input files * Remove old file * Added contract instantiation if deployer script breaks halfway through * avoid query to uniswap with zero rate (#510) * avoid query to uniswap with zero rate * test situation where trade result is 0 * enable setting min record resultion in conversion rates using wrapper (#513) * enable setting min record resultion in conversion rates using wrapper * linting, and add missing test * fix uniswap bridge for small amounts src qty (1) (#518) * fix uniswap bridge for smal amounts src qty (1) * avoid reading fee twice * fix uniswap deployer script * Ilan/deployment helper (#516) * some deployment changes, and helper contract" " * add kyber uniswap bridge to compilation. remove wrap conversion rate * small fixes in deployer * Add CR LF * Ilan/merge master (#519) * update some packages. fix solc version to 0.4.18 (#503) * fixed FPR deployment script (#508) * add smartdec audit for APR * use correct package-lock * Add Eth2Dai reserve and test (#523) * [WIP] Eth2Dai Reserve enhancement (#449) * Add eth2dai contract * Update Eth2Dai reserve * Update Eth2DaiReserve contract and tests * Update trade function tests for eth2dai reserve * Make single test can run on its own * Separate source codes, add some v5 contracts * Update contracts fixed some review comments, remove some unused functions * Update eth2dai contract with enable/disable internal inventory for token * Add test for set internal inventory enable/disable * Put enable/disable internal inventory into internal inventory data * Add internal inventory tests * Change overlow to out of range * Update v5 contracts * Update file name and folder structure * Update folder structure * Fix can not find artifacts for WethTokenV5 * Update test folder structure * Remove KyberOasisReserve source code + test, change WethTokenV5 to WethToken (#524) * remove duplicate files and v5 example * Add KyberBancorReserve and test * Fix some tests for kyber bancor reserve * Add more tests for kyber bancor reserve * Remove default value for bnt and bancor eth token * Add end of file line * Update contracts from review comments * Fix contract and test based on comments * Return 0 for rate and revert in trade when srcQty is 0 * Remove v5 example test file' Co-authored-by: Desmond <[email protected]> Co-authored-by: Ilan Doron <[email protected]> * Update bancor bridge with new fixed path (#548) * Quick fixes to liquidityReserveDeployer.js script for deployment of APR (#556) * update web3 helpers (#560) * fix web3 contract compilation * add helper to read conversion rate set data * enable more user friendly usage * add script to deploy enhanced step functions contract. * update SetStepFunctionWrapper.sol add it to compileContracts.js * indent fix deploy * update according to review comments * fix function signature for setImbalanceStepFunction (#576) * fix function signature for setImbalanceStepFunction * rename enhanced steps to ConversionRateEnhancedSteps * rename enhanced steps * review changes * fix contract name, require correct addresses on ctor (#585) * Revert change * Change name of mock conversion rate enhanced steps * Update bancor test + conversion rate enhanced file after merged from master * Revert change for conversion rate enhanced file Co-authored-by: Ilan Doron <[email protected]> Co-authored-by: Desmond <[email protected]> Co-authored-by: Anton Buenavista <[email protected]>
1 parent 23df8db commit ea3f268

File tree

3 files changed

+17
-17
lines changed

3 files changed

+17
-17
lines changed

contracts/mock/MockConversionRateEnhancedSteps.sol renamed to contracts/mock/MockEnhancedStepFunctions.sol

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@ pragma solidity ^0.4.18;
33
import "../reserves/fprConversionRate/ConversionRateEnhancedSteps.sol";
44

55

6-
contract MockConversionRateEnhancedSteps is ConversionRateEnhancedSteps {
6+
contract MockEnhancedStepFunctions is ConversionRateEnhancedSteps {
77

8-
function MockConversionRateEnhancedSteps(address admin) ConversionRateEnhancedSteps(admin) public {
8+
function MockEnhancedStepFunctions(address admin) ConversionRateEnhancedSteps(admin) public {
99

1010
}
1111

test/v4/conversionRateEnhancedSteps.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11

2-
let MockConversionRateEnhancedSteps = artifacts.require("./mockContracts/MockConversionRateEnhancedSteps.sol");
2+
let MockEnhancedStepFunctions = artifacts.require("./mockContracts/MockEnhancedStepFunctions.sol");
33
let TestToken = artifacts.require("./mockContracts/TestToken.sol");
44

55
let Helper = require("./helper.js");
@@ -58,7 +58,7 @@ contract('ConversionRateEnhancedSteps', function(accounts) {
5858

5959
it("should init ConversionRateEnhancedSteps Inst and set general parameters.", async function () {
6060
//init contracts
61-
convRatesInst = await MockConversionRateEnhancedSteps.new(admin);
61+
convRatesInst = await MockEnhancedStepFunctions.new(admin);
6262

6363
//set pricing general parameters
6464
convRatesInst.setValidRateDurationInBlocks(validRateDurationInBlocks);

test/v5/kyberBancorReserve.js

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,10 @@ let MockBancorNetwork = artifacts.require("MockBancorNetwork.sol");
55
let KyberBancorReserve = artifacts.require("KyberBancorReserve.sol");
66

77
let Helper = require("../v4/helper.js");
8-
const BN = web3.utils.BN;
8+
let BN = require('BN.js');
99

1010
const ethAddress = '0xEeeeeEeeeEeEeeEeEeEeeEEEeeeeEeeeeeeeEEeE';
11+
1112
const precision = new BN(10).pow(new BN(18));
1213
const feeBps = new BN(0);
1314

@@ -18,7 +19,7 @@ const initBntBalance = new BN(1000000).mul(new BN(10).pow(new BN(18)));
1819
const ethToBntRate = new BN(10).pow(new BN(18)).mul(new BN(500));
1920
const bntToEthRate = new BN(10).pow(new BN(18)).div(new BN(500));
2021

21-
const zeroAddress = "0x0000000000000000000000000000000000000000";
22+
const zeroAddress = '0x0000000000000000000000000000000000000000';
2223

2324
const tokenDecimal = 18;
2425

@@ -71,16 +72,15 @@ contract('KyberBancorNetwork', function(accounts) {
7172
await bancorBntToken.transfer(bancorNetwork.address, initBntBalance);
7273
let balance;
7374
balance = await bancorBntToken.balanceOf(bancorNetwork.address);
74-
Helper.assertEqual(balance, initBntBalance, "init balance bnt is not correct");
7575

7676
await Helper.sendEtherWithPromise(user, bancorNetwork.address, initEthBalance);
7777
balance = await Helper.getBalancePromise(bancorNetwork.address);
7878
Helper.assertEqual(balance, initEthBalance, "init balance eth is not correct");
7979
});
8080

8181
it("Should test getConversionRate returns 0 when token is not bnt", async function() {
82-
let token = await TestToken.new("test token", "tes", 18);
8382
let result;
83+
let token = await TestToken.new("test token", "TST", 18);
8484
result = await reserve.getConversionRate(ethAddress, token.address, precision, 0);
8585
Helper.assertEqual(result, 0, "rate should be 0 as token is not bnt");
8686

@@ -144,13 +144,13 @@ contract('KyberBancorNetwork', function(accounts) {
144144

145145
it("Should test getConversionRate returns 0 when path is not correct", async function() {
146146
let testNewToken = await TestToken.new("Test token", "TST", tokenDecimal);
147-
let testBancorNetwork = await MockBancorNetwork.new(testNewToken.address, bancorETHBNTToken.address, bancorBntToken.address);
147+
let testBancorNetwork = await MockBancorNetwork.new(testNewToken.address, bancorBntToken.address, bancorBntToken.address);
148148
let testReserve = await KyberBancorReserve.new(
149149
testBancorNetwork.address,
150150
network,
151151
feeBps,
152152
bancorEthToken.address,
153-
bancorETHBNTToken.address,
153+
bancorBntToken.address,
154154
bancorBntToken.address,
155155
admin
156156
);
@@ -163,13 +163,13 @@ contract('KyberBancorNetwork', function(accounts) {
163163
rate = await testReserve.getConversionRate(bancorBntToken.address, ethAddress, amount, 0);
164164
Helper.assertEqual(rate, 0, "rate should be 0 as path is incorrect");
165165

166-
testBancorNetwork = await MockBancorNetwork.new(bancorEthToken.address, bancorETHBNTToken.address, testNewToken.address);
166+
testBancorNetwork = await MockBancorNetwork.new(bancorEthToken.address, testNewToken.address, testNewToken.address);
167167
testReserve = await KyberBancorReserve.new(
168168
testBancorNetwork.address,
169169
network,
170170
feeBps,
171171
bancorEthToken.address,
172-
bancorETHBNTToken.address,
172+
testNewToken.address,
173173
bancorBntToken.address,
174174
admin
175175
);
@@ -233,8 +233,8 @@ contract('KyberBancorNetwork', function(accounts) {
233233
network,
234234
feeBps,
235235
bancorEthToken.address,
236+
bancorETHBNTToken.address,
236237
zeroAddress,
237-
bancorBntToken.address,
238238
admin
239239
);
240240
assert(false, "throw was expected in line above.")
@@ -248,8 +248,8 @@ contract('KyberBancorNetwork', function(accounts) {
248248
feeBps,
249249
bancorEthToken.address,
250250
bancorETHBNTToken.address,
251-
zeroAddress,
252-
admin
251+
bancorBntToken.address,
252+
zeroAddress
253253
);
254254
assert(false, "throw was expected in line above.")
255255
} catch (e) {
@@ -261,7 +261,7 @@ contract('KyberBancorNetwork', function(accounts) {
261261
network,
262262
feeBps,
263263
bancorEthToken.address,
264-
bancorETHBNTToken.address,
264+
zeroAddress,
265265
bancorBntToken.address,
266266
zeroAddress
267267
);
@@ -537,7 +537,7 @@ contract('KyberBancorNetwork', function(accounts) {
537537
let destAmount = Helper.calcDstQty(amountEth, tokenDecimal, tokenDecimal, ethToBntRate);
538538

539539
let expectedUserBalance = await bancorBntToken.balanceOf(user);
540-
expectedUserBalance = expectedUserBalance.add(destAmount);
540+
expectedUserBalance = expectedUserBalance.add(new BN(destAmount));
541541
await reserve.trade(ethAddress, amountEth, bancorBntToken.address, user, ethToBntRate, true, {from: network, value: amountEth});
542542
let userBalanceAfter = await bancorBntToken.balanceOf(user);
543543
Helper.assertEqual(expectedUserBalance, userBalanceAfter, "user balance must change as expected")

0 commit comments

Comments
 (0)