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

[C4GT] Asset: Add tests for DistributionLimitExceeded #379

Open
3 tasks
vatsa287 opened this issue Mar 12, 2024 · 7 comments
Open
3 tasks

[C4GT] Asset: Add tests for DistributionLimitExceeded #379

vatsa287 opened this issue Mar 12, 2024 · 7 comments
Labels

Comments

@vatsa287
Copy link
Member

vatsa287 commented Mar 12, 2024

Description

Subtask under : cord-network/community#7

New testcase can be tested under cargo test -p pallet-asset after adding it in the code.

Goals

  • Add tests for DistributionLimitExceeded for pallet/asset

Expected Outcome

  • Test should assert for DistributionLimitExceeded being returned properly in all the possible calls.
  • If there are more than one function returning this error code, all of them should be validated either as one test case or as multiple test cases.

Acceptance Criteria

NA

Implementation Details

Look at other test cases, and add a test case for the same.
Ex: check method asset_create_should_succeed which validates creation of asset.

Mockups / Wireframes

NA


Product Name

CORD

Organization Name

Dhiway

Domain

Blockchain

Tech Skills Needed

Rust

Mentor(s)

@amarts @vatsa287

Complexity

[Medium]

Category

[Test]

Sub Category

[Beginner friendly]

@vnitin08
Copy link

Hey @vatsa287 , I would like to work on this issue.

@zeel991
Copy link
Contributor

zeel991 commented Mar 29, 2024

Hey @vatsa287 , I would like to work on this issue.

Hey Vinit , you can raise the PR , if everything seems right , you will be assigned .

@kartikeshwar156
Copy link

hey @vatsa287 I would like to work on this issue

@PriyaD17
Copy link

Hey @vatsa287 @amarts I have read the code and understood the idea to approach the issue, and I would like to work on this issue

@VedantKhairnar
Copy link

Hello @PriyaD17
Any update on the contribution?
Pls do let us know if you need any assistance.
Thanks.

@PriyaD17
Copy link

Hey @VedantKhairnar Thanks for checking in, just got busy in exams and and all, will def reach out if I get stuck anywhere, once again thanks for asking

@hp77-creator
Copy link

Hi @vatsa287
Would like to work on this, but needed to understand, what does Distribution mean? what are assets? Is there any design doc for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

7 participants