VoucherKernel.getTotalDeposits expected that tokenIdSupply is passed in, but there was a mistake and the voucherId was passed in.
I renamed the function to getTotalDepositsForVoucher to be clear it depends on voucher, not on voucher set and modified it to return correct values.
A consequence of bug was, that LogFundsReleased was not emitted when it should be. I noticed that this was not properly tested so I also added missing checks to 3_withdrawals and 3_1_withdrawals_per_entity
VoucherKernel.getTotalDepositsexpected thattokenIdSupplyis passed in, but there was a mistake and thevoucherIdwas passed in.I renamed the function to
getTotalDepositsForVoucherto be clear it depends on voucher, not on voucher set and modified it to return correct values.A consequence of bug was, that
LogFundsReleasedwas not emitted when it should be. I noticed that this was not properly tested so I also added missing checks to3_withdrawalsand3_1_withdrawals_per_entity