Skip to content

Conversation

rackstar
Copy link
Contributor

Context

Add the remaining main functions unit tests - loadState, storeState, removeBudget

NOTE: this PR is behind #943, once that is merge I will point this PR to #935

Changes proposed in this pull request

  • loadState unit test
  • storeState unit test
  • removeBudget unit test

Test plan

Please describe the tests cases that you ran to verify your changes. Add further instructions on
how to run them if needed (i.e. migration / deployment scripts, env vars, etc).

Checklist

  • Rebased the base branch
  • Attached corresponding Github issue
  • Prefixed the name with the type of change (i.e. feat, chore, test)
  • Performed a self-review of my own code
  • Followed the style guidelines of this project
  • Made corresponding changes to the documentation
  • Didn't generate new warnings
  • Didn't generate failures on existing tests
  • 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:

  • 🍰 = Nice to have but not essential.
  • 💡 = Suggestion or a comment based on personal opinion
  • 🔨 = I believe this should be changed.
  • 🤔 = I don’t understand something, do you mind giving me more context?
  • 🚀 = Feedback

@rackstar rackstar self-assigned this Oct 11, 2023
@rackstar rackstar requested a review from shark0der October 11, 2023 15:28
@rackstar rackstar mentioned this pull request Oct 11, 2023
16 tasks
Base automatically changed from feature/tokenomics-leftovers-getReserves-unit-test to feature/tokenomics-leftovers October 12, 2023 08:33
@shark0der shark0der force-pushed the feature/tokenomics-leftovers-main-unit-tests branch from f964c14 to fe7833e Compare October 12, 2023 10:21
@rackstar rackstar merged commit fe7833e into feature/tokenomics-leftovers Oct 12, 2023
@rackstar rackstar deleted the feature/tokenomics-leftovers-main-unit-tests branch October 12, 2023 10:28
@roxdanila roxdanila linked an issue Oct 17, 2023 that may be closed by this pull request
16 tasks
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.

Tokenomics: Ramm Contract Unit Tests
2 participants