Happy path tests for CCP core, treasury and utility contracts#7
Merged
rayedsikder merged 125 commits intodevfrom Oct 26, 2023
Merged
Happy path tests for CCP core, treasury and utility contracts#7rayedsikder merged 125 commits intodevfrom
rayedsikder merged 125 commits intodevfrom
Conversation
- Add `@nomicfoundation/hardhat-foundry` package - Upgrade `hardhat` from `^2.17.0` to `^2.17.3`
- CampaignInfoFactory.ts - SampleCampaign.ts
- `enlistPlatform` function test - `addBytecode` function test - `enlistBytecode` function test
- update `currentTimeIsGreater` to `currentTimeIsLess`
This reverts commit 2f3e47e.
- _checkIfCurrentTimeIsLess - _checkIfCurrentTimeIsGreater
- Update - protocol percentage - platform percentage - claimRefund - disburseFees - Add - withdraw
- add 2 new backer address
- add new user creation - use contract owner to deploy all the contracts - initialize `campaignInfoFactory` - mint token to the backer - warp realistic block time
- add comment - update protocolFeePercent variable
- use contract owner for minting - add comments
- implement all the helper functions for AllOrNothing test - enlistPlatform - addBytecode - enlistBytecode - createCampaign - deploy - addReward - pledgeOnPreLaunch - pledgeForAReward - pledgeWithoutAReward - claimRefund - disburseFees - withdraw
- Test functions for happy cases:- - test_addReward - test_pledgeOnPreLaunch - test_pledgeForAReward - test_pledgeWithoutAReward - test_claimRefund - test_disburseFees - test_withdraw
Add tests for Openzeppelin dependencies of feat/v3-implementation
Add tests for `AllOrNothing` Model
rayedsikder
changed the title
rayedsikder
pushed a commit
that referenced
this pull request
Dec 9, 2025
* Fix incorrect description of the `claimFund` function (#6) (immunefi)(issue#06) - Updated the NatSpec comment for claimFund() * Refactor `platformDataKey` parameter validation optimization (#7) (immunefi)(issue#09) - Move `platformDataKey` parameter validation to an earlier stage of the `createCampaign` flow * Add configuring platform data during `updateSelectedPlatform` (#8) (immunefi)(issue#10) * Add creator's non-zero validation during `createCampaign` (#9) (immunefi)(issue#16) * Add zero validation for `platformDataValue` in `createCampaign` (#10) (immunefi)(issue#12) * Add reward value zero validation in pledge (#11) (immunefi)(issue#14) * Fix `updateDeadline` allowing past deadline that blocks `claimRefund` (#12) (immunefi)(issue#05) - Added check to ensure new deadline is after current block timestamp * Fix blocking KeepWhatsRaised pledge functions via front-running (#13) (immunefi)(issue#04) - Add internal pledge ID generation using msg.sender and pledgeId * Add fee configuration via configure treasury (#14) (immunefi)(issue#11) - Update configure treasury to support fee values - Add getter function for fee value * Add campaign data validation in configure treasury (#15) (immunefi)(issue#20) - Update fee values js doc - Update custom error * Fix Gateway fee bypass (#16) (immunefi)(issue#19) - When `setFeeAndPledge` is called, tokens are transferred from the admin's wallet (msg.sender) - When `pledgeWithoutAReward` or `pledgeForAReward` is called directly, tokens are transferred from the backer's wallet * Add expected fee description in create campaign jsdoc (#17) (immunefi)(issue#03) * Refactor withdrawal and pledge calculation (#19) (immunefi)(issue#15) (immunefi)(issue#18) - Restrict access to the withdrawal function so that only the campaign owner and platform administrator can use it. - Move the protocol fee calculation from the withdrawal process to the pledge stage. - For withdrawals: - Partial Withdrawals: - Conditions: amount > 0 and amount + fees ≤ available balance. - The creator must specify the exact withdrawal amount, and the system will ensure there are sufficient funds to cover both the withdrawal and the fees. - Final Withdrawals: - Conditions: available balance > 0 and fees ≤ available balance. - The creator can withdraw the entire remaining balance. The system will check if there are enough funds to cover the fees and will then provide the remainder to the creator. --------- Co-authored-by: mahabubAlahi <mahabub@ccprotocol.xyz>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds the Foundry test suites for contracts added in PR #6
The detailed documentation of test suites are archived in their respective PRs.
Dependency contract tests - #4
AllOrNothing contract tests - #5