Skip to content
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

feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member #343

Conversation

jgutierrezre
Copy link
Contributor

sorry for the delay, felt a tad sick

@jgutierrezre jgutierrezre force-pushed the pallet-communities-benchmarking-runtime-parameters branch from fe72e07 to 8c838da Compare March 5, 2024 06:39
@jgutierrezre
Copy link
Contributor Author

forgot to format

@pandres95 pandres95 force-pushed the pallet-communities-benchmarking-runtime-parameters branch from cca96ff to 607cb61 Compare March 6, 2024 05:17
@pandres95 pandres95 force-pushed the pallet-communities-benchmarking-runtime-parameters branch from 8c838da to 898e729 Compare March 8, 2024 23:38
@pandres95 pandres95 self-requested a review March 8, 2024 23:38
@jgutierrezre jgutierrezre force-pushed the pallet-communities-benchmarking-runtime-parameters branch from 898e729 to 8c838da Compare March 8, 2024 23:45
…o jgutierrezre-pallet-communities-benchmarking-runtime-parameters
@pandres95 pandres95 force-pushed the pallet-communities-benchmarking-runtime-parameters branch from f15d67c to 8cfca86 Compare March 8, 2024 23:58
@pandres95 pandres95 merged commit f0a1fb8 into virto-network:pallet-communities-benchmarking-runtime-parameters Mar 9, 2024
2 checks passed
pandres95 added a commit that referenced this pull request Mar 9, 2024
…er, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>
pandres95 added a commit that referenced this pull request Mar 12, 2024
…er, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>
pandres95 added a commit that referenced this pull request Mar 12, 2024
…er, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>
pandres95 added a commit that referenced this pull request Mar 12, 2024
…er, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>
pandres95 added a commit that referenced this pull request Mar 12, 2024
…er, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>
olanod pushed a commit that referenced this pull request Mar 15, 2024
* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* change(pallet-communities): prepare

- configure benchmark helper for pallet-referenda-tracks
- fix: missing imports for vec-like structures on benchmarking

* change(kreivo-runtime): setup benchmark helper for community tracks pallet

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmark method for unlock

* feat(kreivo-runtime): configure benchmark helper for pallet-communities

* change(pallet-communities): ensure benchmarks run with default externalities

* chore(cargo): update dependencies

* fix(kreivo-runtime): set MembershipCollectionId to 0

* fix(pallet-commujnities): remove mock out of tests module

* fix(zombienet): add parachain inclusion to relay genesis

* feat(pallet-communities): receive community accountId as a valid origin for CommunityMemberMgmt

- Create and implement EnsureCommunityAccountId.
- Add a benchmarking method to it.
- Setup parameter for mock.
- Setup parameter for runtime.

* change(kreivo-runtime): setup communities-related origins

* change(kreivo-runtime): remove Nfts pallet from runtime

- This is because it's not being used initially

* fix(kreivo-runtime): add EnsureRootWithSuccess for SubmitOrigin on CommunityReferenda

---------

Co-authored-by: Juan Pablo Gutiérrez Restrepo <[email protected]>
pandres95 added a commit that referenced this pull request Mar 17, 2024
* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* change(pallet-communities): prepare

- configure benchmark helper for pallet-referenda-tracks
- fix: missing imports for vec-like structures on benchmarking

* change(kreivo-runtime): setup benchmark helper for community tracks pallet

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmark method for unlock

* feat(kreivo-runtime): configure benchmark helper for pallet-communities

* change(pallet-communities): ensure benchmarks run with default externalities

* chore(cargo): update dependencies

* fix(kreivo-runtime): set MembershipCollectionId to 0

* fix(pallet-commujnities): remove mock out of tests module

* fix(zombienet): add parachain inclusion to relay genesis

* feat(pallet-communities): receive community accountId as a valid origin for CommunityMemberMgmt

- Create and implement EnsureCommunityAccountId.
- Add a benchmarking method to it.
- Setup parameter for mock.
- Setup parameter for runtime.

* change(kreivo-runtime): setup communities-related origins

* change(kreivo-runtime): remove Nfts pallet from runtime

- This is because it's not being used initially

* fix(kreivo-runtime): add EnsureRootWithSuccess for SubmitOrigin on CommunityReferenda

---------

Co-authored-by: Juan Pablo Gutiérrez Restrepo <[email protected]>
@jgutierrezre jgutierrezre deleted the pallet-communities-benchmarking-runtime-parameters branch March 18, 2024 22:15
pandres95 added a commit that referenced this pull request Mar 21, 2024
* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* change(pallet-communities): prepare

- configure benchmark helper for pallet-referenda-tracks
- fix: missing imports for vec-like structures on benchmarking

* change(kreivo-runtime): setup benchmark helper for community tracks pallet

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmark method for unlock

* feat(kreivo-runtime): configure benchmark helper for pallet-communities

* change(pallet-communities): ensure benchmarks run with default externalities

* chore(cargo): update dependencies

* fix(kreivo-runtime): set MembershipCollectionId to 0

* fix(pallet-commujnities): remove mock out of tests module

* fix(zombienet): add parachain inclusion to relay genesis

* feat(pallet-communities): receive community accountId as a valid origin for CommunityMemberMgmt

- Create and implement EnsureCommunityAccountId.
- Add a benchmarking method to it.
- Setup parameter for mock.
- Setup parameter for runtime.

* change(kreivo-runtime): setup communities-related origins

* change(kreivo-runtime): remove Nfts pallet from runtime

- This is because it's not being used initially

* fix(kreivo-runtime): add EnsureRootWithSuccess for SubmitOrigin on CommunityReferenda

---------

Co-authored-by: Juan Pablo Gutiérrez Restrepo <[email protected]>
pandres95 added a commit that referenced this pull request Mar 29, 2024
* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* change(pallet-communities): prepare

- configure benchmark helper for pallet-referenda-tracks
- fix: missing imports for vec-like structures on benchmarking

* change(kreivo-runtime): setup benchmark helper for community tracks pallet

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmark method for unlock

* feat(kreivo-runtime): configure benchmark helper for pallet-communities

* change(pallet-communities): ensure benchmarks run with default externalities

* chore(cargo): update dependencies

* fix(kreivo-runtime): set MembershipCollectionId to 0

* fix(pallet-commujnities): remove mock out of tests module

* fix(zombienet): add parachain inclusion to relay genesis

* feat(pallet-communities): receive community accountId as a valid origin for CommunityMemberMgmt

- Create and implement EnsureCommunityAccountId.
- Add a benchmarking method to it.
- Setup parameter for mock.
- Setup parameter for runtime.

* change(kreivo-runtime): setup communities-related origins

* change(kreivo-runtime): remove Nfts pallet from runtime

- This is because it's not being used initially

* fix(kreivo-runtime): add EnsureRootWithSuccess for SubmitOrigin on CommunityReferenda

---------

Co-authored-by: Juan Pablo Gutiérrez Restrepo <[email protected]>
pandres95 added a commit that referenced this pull request Mar 29, 2024
* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* change(pallet-communities): prepare

- configure benchmark helper for pallet-referenda-tracks
- fix: missing imports for vec-like structures on benchmarking

* change(kreivo-runtime): setup benchmark helper for community tracks pallet

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmark method for unlock

* feat(kreivo-runtime): configure benchmark helper for pallet-communities

* change(pallet-communities): ensure benchmarks run with default externalities

* chore(cargo): update dependencies

* fix(kreivo-runtime): set MembershipCollectionId to 0

* fix(pallet-commujnities): remove mock out of tests module

* fix(zombienet): add parachain inclusion to relay genesis

* feat(pallet-communities): receive community accountId as a valid origin for CommunityMemberMgmt

- Create and implement EnsureCommunityAccountId.
- Add a benchmarking method to it.
- Setup parameter for mock.
- Setup parameter for runtime.

* change(kreivo-runtime): setup communities-related origins

* change(kreivo-runtime): remove Nfts pallet from runtime

- This is because it's not being used initially

* fix(kreivo-runtime): add EnsureRootWithSuccess for SubmitOrigin on CommunityReferenda

---------

Co-authored-by: Juan Pablo Gutiérrez Restrepo <[email protected]>
olanod pushed a commit that referenced this pull request Apr 4, 2024
* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* change(pallet-communities): prepare

- configure benchmark helper for pallet-referenda-tracks
- fix: missing imports for vec-like structures on benchmarking

* change(kreivo-runtime): setup benchmark helper for community tracks pallet

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member (#343)

* change(runtime/kreivo): include pallet-communities on dependencies

* chore(cargo): update Cargo.lock

* feat(pallet-communities): implement structures and storage for governance

* feat(pallet-communities): add set_decision_method extrinsic

* feat(pallet-communities): implement vote extrinsic

* chore(pallet-communities): add initial set of tests for governance section

* fix(pallet-communities): lint

* chore(cargo): update lockfile

* fix(pallet-communities:tests): typo

* feat(pallet-communities): better handle type bounds for T::Assets and T::Polls

* chore(pallet-communities): refine usage of type aliases

* change(pallet-communities:tests): support including multiple communities

* feat(pallet-communities): implement unlock call

* feat(pallet-communities:tests): extend dynamic communities creation on TestEnvBuilder to create tracks

* chore(pallet-communities): test when casting vote on an invalid track / test tie-breaking

* fix(pallet-communities): lint

* chore(pallet-communities): coverage testing for AssetBalance and NativeBalance voting methods

* chore(pallet-communities): add tests for unlock

* feat(pallet-communities): implement remove_vote

* feat(pallet-communities): implement voting for DecisionMethod::Rank

* fix(pallet-communities): lint

* chore(kreivo-runtime): subdependencies to pallet-communities

* feat(kreivo-runtime): initial parameters for pallet-communities

* chore(env): update zombienet configuration files

* chore(cargo): update lockfile

* wip(pallet-communities): initial parmeters for benchmarks

* feat(pallet-communities): bench test for create

* fix(runtime/kreivo): set FreezeIdentifier to RuntimeHoldReason on pallet_balance

* feat(pallet-communities): benchmark for set_metadata / set_decision_method

* feat(pallet-communities): preparing benchmarking helpers for loading memberships and subsequent testings

* fix(pallet-communities): check/lint

* fix(kreivo-runtime)[wip]: prepare for building with runtime-benchmarks

* change(pallet-communities): Added event emitter for removing votes. (#341)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmarking test for vote

* fix(pallet-communities): lint

* Pallet communities benchmarking runtime parameters (#342)

* change(pallet-communities): Added event emitter for removing votes.

* fix(runtime-kreivo): increase MaxHolds / increase MaxFreezes

- Increased MaxHolds: This is because we included pallet_communities::HoldReasons::Voting to the available HoldReasons
- Increase MaxFreezes: This is since you now have funds freezed if you're voting on a community.

* feat(pallet-communities): benchmarking test for add_member.

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* change(pallet-communities): sort benchmarks by call order

* feat(pallet-communities): benchmarking test for remove_vote

* refactor(pallet-communities): externalize factory dependencies from benchmarking

* fix(pallet-communities): remove unnecessary imports on benchmarking

* refactor(pallet-communities): move BenchmarkHelper to types module

* change(kreivo-runtime): configure benchmarking helpers for community related pallets

* fix(pallet-communities): gate exporting types::BenchmarkHelper behind feature flag

* fix(kreivo-runtime): missing dependencies / missing parameters

* feat(pallet-communities): finished benchmarking tests for remove_member, promote_member, and demote_member

* fix(kreivo-runtime): lint

---------

Co-authored-by: Pablo Andrés Dorado Suárez <[email protected]>

* feat(pallet-communities): benchmark method for unlock

* feat(kreivo-runtime): configure benchmark helper for pallet-communities

* change(pallet-communities): ensure benchmarks run with default externalities

* chore(cargo): update dependencies

* fix(kreivo-runtime): set MembershipCollectionId to 0

* fix(pallet-commujnities): remove mock out of tests module

* fix(zombienet): add parachain inclusion to relay genesis

* feat(pallet-communities): receive community accountId as a valid origin for CommunityMemberMgmt

- Create and implement EnsureCommunityAccountId.
- Add a benchmarking method to it.
- Setup parameter for mock.
- Setup parameter for runtime.

* change(kreivo-runtime): setup communities-related origins

* change(kreivo-runtime): remove Nfts pallet from runtime

- This is because it's not being used initially

* fix(kreivo-runtime): add EnsureRootWithSuccess for SubmitOrigin on CommunityReferenda

---------

Co-authored-by: Juan Pablo Gutiérrez Restrepo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants