Unit test review: Dynamic block rewards

[vjaiman]

module.spec.ts

• In LIP we use integer division//. Line 157, 222, 300 has /.
• line 222: BigInt(5) should be properly formulated considering (bftWeight * stakeRewardActiveValidators)//bftWeightSum. Also would be better to use 'ratioReward' as stakeRewardActiveValidators for terminology alignment.
• Line 271: I think isSeedRevealValid() and impliesMaximalPrevotes() checks should be done in beforeTransactionsExecute().
• Not sure if it is true but CONTEXT_STORE_KEY_BLOCK_REWARD is set in "afterTransactionsExecute()"

Missing Tests

• Should not store timestamp when it is not end of round
• When every validator in a round miss the blocks, check endofRoundTimestamp and blockreward.
• Check should be done similar to Reward module for rewardAtHeight.