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

Addition of Conway Tests related to delegation changes, drep multiple registration, deposit returns. #2444

Merged
merged 2 commits into from
Jul 11, 2024

Conversation

nabinpkl
Copy link
Contributor

@nabinpkl nabinpkl commented Jun 12, 2024

The Conway tests which are not already covered on existing test topics are:

  • DRep Id is blake2b_224 of DRep Vkey
  • Delegation change
  • No multiple Vote delegation
  • Vote Delegation without stake registration
  • DRep no multiple registrations
  • DRep no retirement before register
  • Deposit return for committee
  • Info deposit return after expiry
  • Update Protocol Params action's deposit return after expiry and enactment

@nabinpkl
Copy link
Contributor Author

nabinpkl commented Jun 12, 2024

Link to User stories PR : IntersectMBO/cardano-test-plans#45

After the review and merge of User Stories, This PR should be updated to facilitate the test status on the Cardano-node-tests dashboard.

@nabinpkl nabinpkl force-pushed the extra-conway-tests branch 2 times, most recently from a2370b4 to e9724ac Compare June 18, 2024 04:57
@nabinpkl nabinpkl marked this pull request as ready for review June 18, 2024 05:00
@nabinpkl nabinpkl force-pushed the extra-conway-tests branch from e9724ac to 8518bab Compare June 18, 2024 18:19
@saratomaz
Copy link
Collaborator

It would be nice to have the tests in the correspondent files, like drep tests should be in test_drep.py file

@nabinpkl nabinpkl force-pushed the extra-conway-tests branch 2 times, most recently from abbfd28 to 87b3301 Compare June 19, 2024 23:19
@nabinpkl nabinpkl changed the title Adds Extra Conway Tests such as delegation changes, drep multiple registration, deposit return tests. Addition of Conway Tests such as delegation changes, drep multiple registration, deposit return tests. Jun 19, 2024
@nabinpkl
Copy link
Contributor Author

It would be nice to have the tests in the correspondent files, like drep tests should be in test_drep.py file

Thank you for the review. Respective changes are added to respective files in the latest update.

@nabinpkl nabinpkl requested a review from saratomaz June 19, 2024 23:26
@nabinpkl nabinpkl force-pushed the extra-conway-tests branch 2 times, most recently from 698f343 to 5de871e Compare June 25, 2024 04:35
@nabinpkl nabinpkl changed the title Addition of Conway Tests such as delegation changes, drep multiple registration, deposit return tests. Addition of Conway Tests related to delegation changes, drep multiple registration, deposit return tests. Jun 25, 2024
@nabinpkl nabinpkl changed the title Addition of Conway Tests related to delegation changes, drep multiple registration, deposit return tests. Addition of Conway Tests related to delegation changes, drep multiple registration, deposit returns. Jun 25, 2024
@nabinpkl nabinpkl force-pushed the extra-conway-tests branch 3 times, most recently from 402f591 to 4825d80 Compare July 5, 2024 03:18
@nabinpkl nabinpkl force-pushed the extra-conway-tests branch from 4825d80 to 89de998 Compare July 7, 2024 23:43
@nabinpkl
Copy link
Contributor Author

nabinpkl commented Jul 8, 2024

@saratomaz

This PR has been updated and rebased with the latest changes. User stories also have been merged. Can you have a look at it whether it can be merged?

nabinpkl added 2 commits July 11, 2024 13:38
- blake2b_224 of drep id
- delegation change
- no multiple delegation
- delegation without stake registration
- drep no multiple registration
- drep no retirement before register
@nabinpkl nabinpkl force-pushed the extra-conway-tests branch from 89de998 to ab2c117 Compare July 11, 2024 08:01
@nabinpkl nabinpkl requested a review from mkoura July 11, 2024 08:18
@mkoura mkoura merged commit 4b2c0e9 into IntersectMBO:master Jul 11, 2024
3 checks passed
@nabinpkl nabinpkl deleted the extra-conway-tests branch July 16, 2024 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants