steemit / steem-python

The official Python (3) library for the Steem Blockchain.
https://steem.io
MIT License
154 stars 100 forks source link

assume appbase, downgrade on error #202

Closed roadscape closed 6 years ago

roadscape commented 6 years ago

Resolve #164

This PR changes exception flow -- it is more thorough and retries are more reliable.

roadscape commented 6 years ago

Potentially integrate #81 and #95.

roadscape commented 6 years ago

All tests are currently passing against appbase and non-appbase with the sole exception of production which still has that bug.

The diff is best viewed in split mode: https://github.com/steemit/steem-python/pull/202/files?diff=split&w=1

cyon1c commented 6 years ago

The test_broadcast suite fails in Python 2.7 due to #190.

cyon1c commented 6 years ago

When I set the node to https://steemd.steemitdev.com, all 71 tests pass on all three environments. Going to go ahead and approve.