Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding service account #57

Merged
merged 8 commits into from
Dec 13, 2023
Merged

Adding service account #57

merged 8 commits into from
Dec 13, 2023

Conversation

ihoroleksiienko
Copy link
Contributor

@ihoroleksiienko ihoroleksiienko commented Dec 12, 2023

Main Changes

This pull request adds temporary solution for testing that will be removed in the future. It adds a service testing account to BalanceTracker in the getDailyBalances() function with conditions:

  • if the day index is less or equal 19703 (12 Dec 2023 BRT), the function provides 10 000 BRLC as the balance of the account;
  • if the day index is higher than 19703, the function provides the actual balance of the account.

These changes lead to increased yield for the service account in the YieldStreamer contract.

Test coverage

File % Stmts % Branch % Funcs % Lines
contracts/ 94.37 66.67 87.88 94.37
    BRLCToken.sol 100 66.67 100 100
    BRLCTokenBridgeable.sol 86.67 66.67 71.43 86.67
    InfinitePointsToken.sol 100 66.67 100 100
    LightningBitcoin.sol 100 66.67 100 100
     USJimToken.sol 86.67 66.67 71.43 86.67
contracts/base/ 99.35 100 98.11 99.49
     ERC20Base.sol 100 100 100 100
     ERC20Bridgeable.sol 100 100 100 100
     ERC20Freezable.sol 95.83 100 88.89 96.88
     ERC20Hookable.sol 100 100 100 100
     ERC20Mintable.sol 100 100 100 100
     ERC20Restrictable.sol 100 100 100 100
contracts/base/common/ 100 100 100 100
     BlacklistableUpgradeable.sol 100 100 100 100
     PausableExtUpgradeable.sol 100 100 100 100
     RescuableUpgradeable.sol 100 100 100 100
contracts/periphery/ 100 97.18 100 100
     BalanceTracker.sol 100 94.74 100 100
     YieldStreamer.sol 100 98.08 100 100
contracts/base/interfaces/ 100 100 100 100
contracts/base/interfaces/periphery/ 100 100 100 100
contracts/mocks/ 93.18 83.33 100 95.83
contracts/mocks/base/ 100 100 100 100
contracts/mocks/base/common/ 100 100 100 100
-------------------------------------- ---------- ---------- ---------- ----------
All files 91.35 87.55 87.35 91.74
-------------------------------------- ---------- ---------- ---------- ----------

@ihoroleksiienko ihoroleksiienko marked this pull request as ready for review December 12, 2023 10:26
@igorsenych-cw igorsenych-cw merged commit cb1f136 into main Dec 13, 2023
6 of 7 checks passed
@igorsenych-cw igorsenych-cw deleted the add_service_account branch December 13, 2023 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants