-
Notifications
You must be signed in to change notification settings - Fork 366
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
INDY-2067: Implement GET_TXN_AUTHOR_AGREEMENT #1204
Conversation
…EEMENT Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
…EMENT Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
|
||
# TODO: Change txnPoolNodeSet to nodeSetWithOneNodeResponding after | ||
# correct state proof checking is implemented in Indy SDK | ||
def test_state_proof_returned_for_get_txn_author_agreement( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we need to have multiple tests (and hence multiple sdk_get_txn_author_agreement
) for each variant of GET_TAA: latest, by digest, by ts.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Of course they are planned. Also tests checking for consistency proofs for absent keys are planned as well.
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
Signed-off-by: Sergey Khoroshavin <sergey.khoroshavin@dsr-corporation.com>
@@ -75,6 +76,15 @@ class ClientTxnAuthorAgreementOperation(MessageValidator): | |||
) | |||
|
|||
|
|||
# TODO: Add more fields along with implementation | |||
class ClientGetTxnAuthorAgreementOperation(MessageValidator): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe we need a specific validation that either GET_TXN_AUTHOR_AGREEMENT_VERSION
or GET_TXN_AUTHOR_AGREEMENT_DIGEST
can be specified, not both.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for pointing at it
Looks good, so I think we can merge it.
|
No description provided.