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] Statement: Add tests for PresentationDigestAlreadyAnchored #318

Open
3 tasks
vatsa287 opened this issue Feb 6, 2024 · 8 comments · May be fixed by #360
Open
3 tasks

[C4GT] Statement: Add tests for PresentationDigestAlreadyAnchored #318

vatsa287 opened this issue Feb 6, 2024 · 8 comments · May be fixed by #360
Assignees
Labels

Comments

@vatsa287
Copy link
Member

vatsa287 commented Feb 6, 2024

Description

Subtask under : cord-network/community#7

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

Goals

  • Add tests for PresentationDigestAlreadyAnchored for pallet/statement

Expected Outcome

  • Test should assert for PresentationDigestAlreadyAnchored 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 trying_to_register_statement_by_a_non_delegate_should_fail which validates error code UnauthorizedOperation

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]

@Sayanjones
Copy link

@vatsa287 I'm interested on this project. Can we discuss further?

@rising0raj
Copy link

Hey, @vatsa287 I would like to work on this . I have gone through the code base can you assign this to me if it is open??

@Sohajawdekar
Copy link

Hello @vatsa287 !
I am interested in contributing to this project, can you please assign me this ticket ?

@summerpun2803
Copy link

@vatsa287 I would like to contribute to this project

@VishnuAmit
Copy link

Hey @vatsa287 I would like to take on this project. Kindly assign it to me.
Regards

@PayalKumari10
Copy link

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

@VedantKhairnar
Copy link

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

@Vaibhavsahu2810
Copy link

Hello @vatsa287 @VedantKhairnar i see that this issue has no activity for quite a while. If it's alright, Can you assign this issue to me?

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.

9 participants