Skip to content
This repository was archived by the owner on Nov 15, 2021. It is now read-only.

Fix getcontractstate to match neo-cli #750

Merged
merged 30 commits into from
Dec 12, 2018

Conversation

jseagrave21
Copy link
Contributor

What current issue(s) does this address, or what feature is it adding?

How did you solve this problem?
Trial and Error

How did you make sure your solution works?
Manual Testing and Unittest

Are there any special changes in the code that we should be aware of?
No

Please check the following, if applicable:

  • Did you add any tests? (I updated tests)
  • Did you run make lint?
  • Did you run make test?
  • Are you making a PR to a feature branch or development rather than master?
  • Did you add an entry to CHANGELOG.rst? (if not, please do)

jseagrave21 and others added 30 commits October 9, 2018 20:38
Merge CoZ Development into jseagrave21 Development
Merge CoZ Development into jseagrave21 Development
Merge CoZ Development into jseagrave21 Development
* Update ExtendedJsonRpcApi.py

- add fix provided by @localhuman so original methods are returned as well as extended methods
…yOfZion#661)

* Add the option -u (unittest-net) to prompt.py

* Add unittest guildeline and add the smart contract source codes (UnitTest-SM.zip) to the fixtures package
for compatibility
* Fix ExtendedJsonRpcApi (CityOfZion#662)

* Update ExtendedJsonRpcApi.py

- add fix provided by @localhuman so original methods are returned as well as extended methods

* Mute expected test stacktrace and clearly identify why an exception is thrown. (CityOfZion#663)

* Add guideline for adding tests to the neo-privnet-unittest image (CityOfZion#661)

* Add the option -u (unittest-net) to prompt.py

* Add unittest guildeline and add the smart contract source codes (UnitTest-SM.zip) to the fixtures package

* Add raw transaction building examples (CityOfZion#665)

* Update neo-boa version to fix core building test
Merge CoZ Development into jseagrave21 Development
Merge CoZ Development into jseagrave21 Development
Merge CoZ Development into jseagrave21 Development
Merge CoZ Development into jseagrave21 development
Merge CoZ development into jseagrave21 development
- add tests for "showallaccounts" and "showallaccountstates"
Merge neo-python development into jseagrave21 development
Merge neo-python development into jseagrave21 development
Merge neo-python development into jseagrave21 development
Merge CoZ development into jseagrave21 development
Gracefully handle NEP-5 balance query failures (CityOfZion#744)
- update for parody with neo-cli 2.9.2
- update for parody with neo-cli 2.9.2
- fix so ToName works
- update so test passes
- update so test passes
- update for integration with previous changes and so tests pass
- consolidate code
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 83.969% when pulling 57ebadd on jseagrave21:fix-getcontractstate into f076936 on CityOfZion:development.

@ixje
Copy link
Member

ixje commented Dec 12, 2018

Thanks for the quick fix! 🥇

@ixje ixje merged commit 633095f into CityOfZion:development Dec 12, 2018
@jseagrave21 jseagrave21 deleted the fix-getcontractstate branch December 12, 2018 15:17
jseagrave21 added a commit to jseagrave21/neo-python that referenced this pull request Dec 12, 2018
jseagrave21 added a commit to jseagrave21/neo-python that referenced this pull request Dec 12, 2018
jseagrave21 added a commit to jseagrave21/neo-python that referenced this pull request Dec 12, 2018
jseagrave21 added a commit to jseagrave21/neo-python that referenced this pull request Dec 12, 2018
jseagrave21 added a commit to jseagrave21/neo-python that referenced this pull request Dec 12, 2018
jseagrave21 added a commit to jseagrave21/neo-python that referenced this pull request Dec 12, 2018
jseagrave21 added a commit to jseagrave21/neo-python that referenced this pull request Dec 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants