Skip to content

Conversation

@belane
Copy link
Collaborator

@belane belane commented May 20, 2024

This pull request initiates the environment for unit testing of the governance contracts.

  • Initialize Hardhat project.
  • Add OpenZeppelin dependencies and remove ERC1967Proxy.
  • Include a workaround for the unique deployment characteristics of these contracts to enable testing.
  • Create some basic unit tests for the Governance contract.

@txhsl txhsl changed the title Governance Unit Tests systemcontract: governance unit tests May 21, 2024
@txhsl txhsl merged commit ca4ba57 into bane-main May 21, 2024
@txhsl txhsl deleted the governance-ut branch May 21, 2024 02:46
txhsl added a commit that referenced this pull request May 21, 2024
* index some event parameters

* update solidity compiler version

* systemcontract: governance unit tests (#210)

* update gitignore

* init hardhat project

* first ut

* systemcontract: add candidate management ut and getter check (#211)

* systemcontract: use customized error library (#209)

* systemcontract: use customized error library

* update existing ut

---------

Co-authored-by: belane <[email protected]>
txhsl added a commit that referenced this pull request May 30, 2024
* index some event parameters

* update solidity compiler version

* systemcontract: governance unit tests (#210)

* update gitignore

* init hardhat project

* first ut

* systemcontract: add candidate management ut and getter check (#211)

* systemcontract: use customized error library (#209)

* systemcontract: use customized error library

* update existing ut

* systemcontract: add policy unit tests (#214)

* systemcontract: add ut about reward distribution and fix several ut (#215)

* systemcontract: optimize gas cost (#195)

* systemcontract: optimize gas cost

---------

Co-authored-by: belane <[email protected]>
@AnnaShaleva AnnaShaleva added this to the v0.1.2 milestone May 30, 2024
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.

4 participants