added test for AssetIdNotFound #375 in pallet-asset #397
Conversation
This fil
8000
e 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
|
@aishalxgupta Any update on this PR. Pleas rebase and update.
|
* Registries: Remove registry-id as a argument for create (dhiway#559) Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * pallet-asset: Added new test-case for DistributionLimitExceeded. (dhiway#522) This issue has been structured similarly to the `asset_create_should_succeed` test. This test will ensure that an asset can be issued successfully and will include coverage for reaching and exceeding the `MaxAssetDistribution` limit. - Test Initialization: The test sets up a space, approves it, and creates an asset similarly to the `asset_create_should_succeed` test. - Asset Issuance: The loop issues the asset up to the `MaxAssetDistribution` limit (set to 25 for testing). Each iteration simulates issuing the asset to a different recipient. - Error Handling: After the limit is reached, the test tries to issue the asset to one more recipient, which should trigger the `DistributionLimitExceeded` error. This ensures that the pallet's logic correctly handles and enforces the distribution limit. This implementation checks that the asset issuance process is valid up to the `MaxAssetDistribution` limit and correctly throws an error when the `limit` is exceeded. * Namespace: Add pallet to have namespace feature (dhiway#560) Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * namespace: Remove approval requirement for usage (dhiway#563) Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * docs: Update readme for PR validation requirements. (dhiway#540) * network-score : Add Test Cases for SpaceMismatch error Fixes: dhiway#304 * network-score: Add Test Cases for SpaceMismatch Error in pallet-network-score Signed-off-by: Aman Raj <113578582+huamanraj@users.noreply.github.com> * namespace: Add digest and blob for namespace create Fixes: dhiway#564 Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * namespace: Identifier updates (dhiway#565) Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * namespace: Integrate with Registries framework Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * registries: Integration testing with Namespace Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * registries: Benchmarking updates for Namespace Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * entries: Integration testing with updated Registries & Namespace Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * Update to use new identifiers Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * namespace, registries: Update lookup table on create (dhiway#569) Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * Update the CORD version to 0.9.5 (dhiway#570) Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * Registries: Add tests for AuthorizationNotFound (dhiway#566) * registries: add tests for AuthorizationNotFound --------- Co-authored-by: Mudit Sarda <muditsarda@Mudits-MacBook-Pro.local> * Fixes: dhiway#310 Add test cases for SchemaNotFound error (dhiway#551) Signed-off-by: Vaibhav <sahusv4527@gmail.com> * fix upstream merge conflicts --------- Signed-off-by: Shreevatsa N <vatsa@dhiway.com> Signed-off-by: Aman Raj <113578582+huamanraj@users.noreply.github.com> Signed-off-by: Vaibhav <sahusv4527@gmail.com> Co-authored-by: Shree Vatsa N <vatsa@dhiway.com> Co-authored-by: RITANKAR SAHA <ritankar.saha786@gmail.com> Co-authored-by: Aman Raj <113578582+huamanraj@users.noreply.github.com> Co-authored-by: muditDhiway <mudit.sarda@dhiway.com> Co-authored-by: Mudit Sarda <muditsarda@Mudits-MacBook-Pro.local> Co-authored-by: VAIBHAV SAHU <154520239+Vaibhavsahu2810@users.noreply.github.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
* CORD: Remove sp-std usage and dependencies Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * remove sp-std from packages Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * Remove sp-std from runtimes & utils Signed-off-by: Shreevatsa N <vatsa@dhiway.com> * Replace scale_info dependency Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
Signed-off-by: Shreevatsa N <vatsa@dhiway.com>
* sdk update -> 1.18.1 * origin infrastructure * origin - remove claims and vesting * origin - remove crowdloans and auction * origin - xcm and governance * origin - container constants and coretime * identifier updates to support runtime prefix and chain modes * element - multi-data container * pallet-entity - identity with elements * pallet-entity - integrate with cord runtimes * entity container chain with cord-origin
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 test case creates a space, authorization ID, and an asset definition.
It then creates an issue_entry with a non-existent asset_id using a different digest.
The test calls Asset::issue with this invalid issue_entry and asserts that the result is an error containing AssetIdNotFound.