Skip to content

Update tests for after upgrade#116

Merged
tsudmi merged 16 commits intomasterfrom
update-tests
Sep 16, 2022
Merged

Update tests for after upgrade#116
tsudmi merged 16 commits intomasterfrom
update-tests

Conversation

@tsudmi
Copy link
Member

@tsudmi tsudmi commented Sep 15, 2022

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2022

Codecov Report

Merging #116 (54cd360) into master (29a802e) will decrease coverage by 27.11%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #116       +/-   ##
==========================================
- Coverage   35.89%   8.78%   -27.12%     
==========================================
  Files          16      16               
  Lines         560     558        -2     
  Branches      115     114        -1     
==========================================
- Hits          201      49      -152     
- Misses        359     509      +150     
Impacted Files Coverage Δ
contracts/tokens/RewardEthToken.sol 0.00% <0.00%> (-91.21%) ⬇️
contracts/pool/Pool.sol 0.00% <0.00%> (-98.51%) ⬇️
contracts/presets/OwnablePausableUpgradeable.sol 90.00% <0.00%> (-10.00%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tsudmi tsudmi merged commit 0e51a35 into master Sep 16, 2022
@tsudmi tsudmi deleted the update-tests branch September 16, 2022 09:13
cliddlebd pushed a commit to cliddlebd/contracts that referenced this pull request Sep 21, 2023
* Update tests for after upgrade

* Update transaction for mainnet

* Fix tests

* replace getBalance with balance.current

* Fix redundant import

* Update FeesEscrow test

* Update RewardEthToken totalSupply

* Fix tests for updated total supply

* Remove redundant import

* Remove upgrade function from RewardEthToken

* Remove RewardEthToken upgrade test

* Fix incorrect file removal

* Deploy RewardEthToken 2.1.1

* Upgrade RewardEthToken proxy

* Upload updated Pessimistic report

* Update proxies test
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