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

Added tests for subscription messages #102

Merged
merged 2 commits into from
Jul 2, 2021

Conversation

jay-dee7
Copy link
Contributor

Signed-off-by: jay-dee7 [email protected]

Added test cases for the following message types:

  • MsgAddQuotaRequest
  • MsgUpdateQuotaRequest
  • MsgCancelRequest
  • MsgSubscribeToNodeRequest
  • MsgSubscribeToPlanRequest

@jay-dee7 jay-dee7 requested a review from bsrinivas8687 April 29, 2021 06:39
@jay-dee7 jay-dee7 self-assigned this Apr 29, 2021
@jay-dee7 jay-dee7 force-pushed the jay-dee7/subscription-messgage-tests branch from 70e1982 to 58ab148 Compare May 3, 2021 10:35
@bsrinivas8687 bsrinivas8687 changed the base branch from development to testcases June 29, 2021 12:16
@bsrinivas8687
Copy link
Contributor

@jay-dee7 can you please fix conflicts

Signed-off-by: jay-dee7 <[email protected]>
Signed-off-by: jay-dee7 <[email protected]>
@jay-dee7 jay-dee7 force-pushed the jay-dee7/subscription-messgage-tests branch from 58ab148 to 9f90d63 Compare June 29, 2021 12:47
@jay-dee7
Copy link
Contributor Author

@bsrinivas8687 i've removed the decoder case for ActiveSessionAtKeyPrefix since there is no key in keys.go for session module. It compiles but im not sure about the change which caused the key to be removed in the first place

@jay-dee7 jay-dee7 force-pushed the jay-dee7/subscription-messgage-tests branch from 9f90d63 to 809d973 Compare June 29, 2021 13:00
Signed-off-by: jay-dee7 <[email protected]>
@jay-dee7 jay-dee7 force-pushed the jay-dee7/subscription-messgage-tests branch from 809d973 to a2f5ac8 Compare June 29, 2021 13:02
@bsrinivas8687 bsrinivas8687 merged commit 9ef0929 into testcases Jul 2, 2021
@ironman0x7b2 ironman0x7b2 deleted the jay-dee7/subscription-messgage-tests branch July 19, 2021 18:12
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.

2 participants