-
Notifications
You must be signed in to change notification settings - Fork 483
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
Android tests refactoring #940
Conversation
@jackrobison Updated raised errors and tests as per your comments in Slack. |
if not int(time.time()) - ts < 600: | ||
raise AssertionError() | ||
if not spot > 0: | ||
raise AssertionError() | ||
self.currency_pair = (market[0:3], market[3:6]) |
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.
might as well make these of type ValueError for clarity (first is because timestamp is too dated, second is because spot must be more than 0)
Is there a reason android must be compiled with PYTHONOPTIMIZE flag? |
I'm not sure what the exact reason is as it is a setting from the default kivy / python for android app launcher build. I tried to remove it but that resulted in errors launching the Python modules when trying to run the app. |
3ee6a9f
to
e522e7e
Compare
LGTM |
PYTHONOPTIMIZE
flag set for the Android build.__doc__
attribute isNone
in cases where the docstring isn't present (also due to thePYTHONOPTIMIZE
flag)