Add more tests for the semaphore groups contract - Githubissues

semaphore-protocol / semaphore

A zero-knowledge protocol for anonymous interactions.

https://semaphore.pse.dev

MIT License

908 stars 198 forks source link

Add more tests for the semaphore groups contract #710

Closed vplasencia closed 7 months ago

vplasencia commented 7 months ago

Description

This PR adds more test to contracts. Now all the SemaphoreGroups.sol contract lines are covered after merging https://github.com/semaphore-protocol/semaphore/pull/709.

Related Issue(s)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have run yarn prettier and yarn lint without getting any errors
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules