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
Break v2_step.py into account_, application_, and other_ #341
Conversation
from algosdk import account, encoding, logic | ||
from algosdk.future import transaction | ||
|
||
import test.steps.other_v2_steps |
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.
This import is required to make MaybeString
and MaybeBool
which are defined in other_v2_steps
available
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.
Nice!
import test.steps.other_v2_steps | ||
|
||
|
||
def fund_account_address( |
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.
Move most helper functions to the top of the file
from test.steps.other_v2_steps import read_program | ||
|
||
|
||
def operation_string_to_enum(operation): |
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.
Move most helper functions to the top of the file
@@ -67,6 +57,52 @@ def parse_bool(value): | |||
register_type(MaybeBool=parse_bool) | |||
|
|||
|
|||
def validate_error(context, err): |
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.
Move most helper functions to the top of the file
Refactoring Cucumber Steps (V2 Only)
This PR is a NO-OP. Only code for testing is affected and the only changes amount to:
Details
v2_steps.py
has gotten ginormous, which makes it harder to work with.The PR breaks up the file into:
test/steps/account_v2_steps.py
- where most of the account-focused logic now residestest/steps/application_v2_steps.py
- where most of the app-centric logic now residestest/steps/other_v2_steps.py
- where miscellaneous other functionality including helper functions used by the other files residesTesting
All tests should continue passing