Skip to content
This repository has been archived by the owner on Nov 27, 2024. It is now read-only.

test: filling in test coverage post audit #90

Merged
merged 15 commits into from
Jan 2, 2024

Conversation

dd0sxx
Copy link
Contributor

@dd0sxx dd0sxx commented Dec 29, 2023

Motivation:

final test coverage bump to token voting post-audit.

Modifications:

  • Added tests for missing lines, specifically all of the checkpoint getters in the governor contract

Result:

improved test coverage from 90.87% to 100%

Copy link

Coverage after merging theo/post-audit-test-coverage into main will be

100.00%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src/token-voting
   LlamaTokenGovernor.sol99.68%98.78%100%100%830
   LlamaTokenVotingFactory.sol88.89%66.67%100%100%67–68
src/token-voting/token-adapters
   LlamaTokenAdapterVotesTimestamp.sol100%100%100%100%

@AustinGreen AustinGreen merged commit 29c0dba into main Jan 2, 2024
5 checks passed
@AustinGreen AustinGreen deleted the theo/post-audit-test-coverage branch January 2, 2024 05:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants