Beginner FriendlyFoundryNFT
100 EXP
View results
Submission Details
Severity: low
Invalid

Low Test Coverage is bad for protocol and might cause unintended behavior of the protocol

Summary

The test coverage of the tests are below 90%. This often means that there are parts of the code that are not tested.

Vulnerability Details

Test coverage -

| File | % Lines | % Statements | % Branches | % Funcs |
|---------------------|----------------|----------------|----------------|----------------|
| src/ChoosingRam.sol | 61.36% (27/44) | 66.00% (33/50) | 39.47% (15/38) | 80.00% (4/5) |
| src/Dussehra.sol | 76.67% (23/30) | 78.79% (26/33) | 75.00% (15/20) | 85.71% (6/7) |
| src/RamNFT.sol | 84.62% (11/13) | 85.71% (12/14) | 50.00% (2/4) | 100.00% (8/8) |
| Total | 70.11% (61/87) | 73.20% (71/97) | 51.61% (32/62) | 90.00% (18/20) |

Impact

Can cause unintended behavior of the protocol

Recommendations

Increase test coverage to 90% or higher, especially for the Branches column.

Updates

Lead Judging Commences

bube Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Info/Gas/Invalid according to docs

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.