fix: Fix processExpiration for pool 8 - Githubissues

NexusMutual / smart-contracts

GNU General Public License v3.0

170 stars 62 forks source link

fix: Fix processExpiration for pool 8 #1149

Closed rackstar closed 4 months ago

rackstar commented 5 months ago

Context

Closes #1151

See the above issue for detailed info

Changes proposed in this pull request

Add check in proccessExpirations for pool 8 and rollback lastAccNxmUpdate to prevent underflow

Test plan

fork test

Checklist

[x] Rebased the base branch
[x] Attached corresponding Github issue
[x] Prefixed the name with the type of change (i.e. feat, chore, test)
[x] Performed a self-review of my own code
[x] Followed the style guidelines of this project
[ ] Made corresponding changes to the documentation
[x] Didn't generate new warnings
[x] Didn't generate failures on existing tests
[x] Added tests that prove my fix is effective or that my feature works

Review

When reviewing a PR, please indicate intention in comments using the following emojis:

:cake: = Nice to have but not essential.
:bulb: = Suggestion or a comment based on personal opinion
:hammer: = I believe this should be changed.
:thinking: = I don’t understand something, do you mind giving me more context?
:rocket: = Feedback