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] Identity: Add tests for TooManyFields #290

Open
3 tasks
vatsa287 opened this issue Feb 5, 2024 · 5 comments
Open
3 tasks

[C4GT] Identity: Add tests for TooManyFields #290

vatsa287 opened this issue Feb 5, 2024 · 5 comments
Assignees
Labels

Comments

@vatsa287
Copy link
Member

vatsa287 commented Feb 5, 2024

Description

Subtask under : cord-network/community#7

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

Goals

  • Add tests for TooManyFields for pallet/identity

Expected Outcome

  • Test should assert for TooManyFields 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: uninvited_judgement_should_work which validates error code RegistrarNotFound

Mockups / Wireframes

NA


Product Name

CORD

Organization Name

Dhiway

Domain

Blockchain

Tech Skills Needed

Rust

Mentor(s)

@amarts

Complexity

[Medium]

Category

[Test]

Sub Category

[Beginner friendly]

@amarts amarts added the good first issue Good for newcomers label Feb 6, 2024
@sahil-1729
Copy link

I am interested in working in this issue

@HarshitaJaiswal5
Copy link

Hello @vatsa287 I have gone through this issue and I'm confident about solving it. Can you assign this task to me?

@PayalKumari10
Copy link

Hello @vatsa287 @amarts, I want to work on this project
Can you assign this issue to me?

@VedantKhairnar
Copy link

Hello @HarshitaJaiswal5 @PayalKumari10
Are you currently open to contributing to the issue?
Thanks.

@prshntrajput
Copy link

Hi @vatsa287 , I am interested in working on this issue it would be great if you share some details that will help fixing this issue.

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