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 InvalidAssetType #374

Open
3 tasks
vatsa287 opened this issue Mar 12, 2024 · 3 comments · May be fixed by #427
Open
3 tasks

[C4GT] Asset: Add tests for InvalidAssetType #374

vatsa287 opened this issue Mar 12, 2024 · 3 comments · May be fixed by #427
Assignees
Labels

Comments

@vatsa287
Copy link
Member

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 InvalidAssetType for pallet/asset

Expected Outcome

  • Test should assert for InvalidAssetType 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

[Low]

Category

[Test]

Sub Category

[Beginner friendly]

@aishalxgupta
Copy link

Hi @vatsa287
I'm confident in my understanding of the problem and possess the necessary skills. I'd be grateful for the opportunity to be assigned to this project

@hardik-pratap-singh
Copy link
Contributor

Hey @vatsa287 @amarts !!
I want to take this issue.
Kindly assign it to me if possible.

@hardik-pratap-singh
Copy link
Contributor

Hey @amarts @vatsa287 !
Is this issue still needs to be worked upon ?

If yes, I would love to work on it !
Can you please assign it to me ?

hardik-pratap-singh added a commit to hardik-pratap-singh/cord that referenced this issue Apr 10, 2024
Signed-off-by: hardik-pratap-singh <21bcs090@iiitdmj.ac.in>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants