crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix tests that assume touching stake will finalize pool

Browse Source
This commit is contained in:
Amir Bandeali
2019-10-19 19:26:06 -07:00
parent a66ea2bf74
commit 44dac2cd80
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
contracts/staking/test/unit_tests/delegator_reward_test.ts
View File

@@ -207,7 +207,7 @@ blockchainTests.resets('Delegator rewards unit tests', env => {
}
async function finalizePoolAsync(poolId: string): Promise<ResultWithTransfers<{}>> {
const receipt = await testContract.originalFinalizePool.awaitTransactionSuccessAsync(poolId);
const receipt = await testContract.finalizePool.awaitTransactionSuccessAsync(poolId);
const delegatorTransfers = getTransfersFromLogs(receipt.logs, poolId);
return {
delegatorTransfers,
@@ -273,8 +273,7 @@ blockchainTests.resets('Delegator rewards unit tests', env => {
it('nothing once rewards are finalized', async () => {
const { poolId } = await setUnfinalizedPoolRewardAsync();
// Delegate some stake to trigger finalization.
await delegateStakeAsync(poolId);
await finalizePoolAsync(poolId);
const operatorReward = await getOperatorRewardBalanceAsync(poolId);
expect(operatorReward).to.bignumber.eq(0);
});
@@ -751,7 +750,7 @@ blockchainTests.resets('Delegator rewards unit tests', env => {
membersStake: totalStake,
});
const totalRewards = BigNumber.sum(prevReward, unfinalizedReward);
// delegator A will finalize and collect rewards by touching stake.
await finalizePoolAsync(poolId);
const { delegatorTransfers: withdrawalA } = await touchStakeAsync(poolId, delegatorA);
assertRoughlyEquals(withdrawalA, computeDelegatorRewards(totalRewards, stakeA, totalStake));
// delegator B will collect rewards by touching stake