-
Notifications
You must be signed in to change notification settings - Fork 141
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
Enhancement: Boxes Indexer Integration test #382
Enhancement: Boxes Indexer Integration test #382
Conversation
a357521
to
8538701
Compare
8538701
to
d1c9b1a
Compare
…nd/py-algorand-sdk into ahangsu/indexer-integration-boxes
ae14d9a
to
11dc4bb
Compare
11dc4bb
to
ce374b8
Compare
48ff94d
to
aff3e6a
Compare
aff3e6a
to
b517f1d
Compare
@@ -1023,6 +1023,12 @@ def algod_v2_client(context): | |||
context.app_acl = algod.AlgodClient(daemon_token, algod_address) | |||
|
|||
|
|||
@given("an indexer v2 client") |
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.
👶 - indexer is reborn!
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.
nit: might be nice to put right next to where algod
is defined.
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.
Very tempted to approve, but put up some small nits worth discussing.
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.
LGTM
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.
Minor formatting comments, looks good!
Non-urgent thought but I wonder if long term we aim to write indexer integration tests for all existing features and we want to split the indexer functionality into another file.
Co-authored-by: algochoi <[email protected]>
Co-authored-by: algochoi <[email protected]>
3f4dd69
to
cb70c71
Compare
908b7f7
to
ab55587
Compare
@@ -415,14 +419,19 @@ def remember_app_id(context): | |||
context.app_ids.append(app_id) | |||
|
|||
|
|||
def wait_for_algod_transaction_processing_to_complete(): | |||
@then( | |||
"I sleep for {millisecond_num} milliseconds for indexer to digest things down." |
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.
👍
No description provided.