-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add more tests #60
add more tests #60
Conversation
Methods
Deployments
Solidity and Network Config
|
Slither reportTHIS CHECKLIST IS NOT COMPLETE. Use
reentrancy-no-ethImpact: Medium
BNBParty.Factory/contracts/BNBPartyLiquidity.sol Lines 14 to 56 in c127abf
unused-returnImpact: Medium
BNBParty.Factory/contracts/BNBPartyLiquidity.sol Lines 14 to 56 in c127abf
BNBParty.Factory/contracts/BNBPartyCreation.sol Lines 45 to 79 in c127abf
BNBParty.Factory/contracts/BNBPartyFee.sol Lines 68 to 80 in c127abf
BNBParty.Factory/contracts/BNBPartyLiquidityHelper.sol Lines 16 to 40 in c127abf
BNBParty.Factory/contracts/BNBPartyManageable.sol Lines 93 to 105 in c127abf
BNBParty.Factory/contracts/BNBPartySwaps.sol Lines 47 to 53 in c127abf
reentrancy-benignImpact: Low
BNBParty.Factory/contracts/BNBPartyCreation.sol Lines 45 to 79 in c127abf
BNBParty.Factory/contracts/BNBPartyCreation.sol Lines 16 to 33 in c127abf
BNBParty.Factory/contracts/BNBPartyLiquidity.sol Lines 14 to 56 in c127abf
reentrancy-eventsImpact: Low
BNBParty.Factory/contracts/BNBPartySwaps.sol Lines 13 to 22 in c127abf
BNBParty.Factory/contracts/BNBPartySwaps.sol Lines 13 to 22 in c127abf
BNBParty.Factory/contracts/BNBPartySwaps.sol Lines 13 to 22 in c127abf
BNBParty.Factory/contracts/BNBPartyLiquidity.sol Lines 14 to 56 in c127abf
naming-conventionImpact: Informational
reentrancy-unlimited-gasImpact: Informational
BNBParty.Factory/contracts/BNBPartySwaps.sol Lines 13 to 22 in c127abf
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ ❗ Your organization needs to install the Codecov GitHub app to enable full functionality. |
closes #59