Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Integration Test #271

Closed
brianjohnson5972 opened this issue Aug 30, 2017 · 9 comments
Closed

Integration Test #271

brianjohnson5972 opened this issue Aug 30, 2017 · 9 comments
Assignees

Comments

@brianjohnson5972
Copy link
Contributor

Add a script to run eosd and launch eosc to exercise the api (create account, transfer, get info, transactions,...) and verify the output. If the test can be run sufficiently fast, then it should be made part of Travis, or look into adding flags to run a limited set for Travis.

@brianjohnson5972 brianjohnson5972 self-assigned this Aug 30, 2017
@heifner heifner assigned heifner and unassigned brianjohnson5972 Sep 1, 2017
@heifner
Copy link
Contributor

heifner commented Sep 1, 2017

Taking this one from Brian so he can concentrate on #129.

@brianjohnson5972
Copy link
Contributor Author

Needs to create multiple accounts and transfer funds
Then needs to exercise the Account History API using ./eosc commands:
transaction
transactions
accounts
servants

Also needs to set filter_on_accounts in config.ini for eosd to verify that transactions are filtered if its scope doesn't contain at least 1 filter_on_accounts entries.

@brianjohnson5972
Copy link
Contributor Author

My suggestion is to make several commits on this, making sure that at least initial integration testing is occurring and then improving it. Since we are coming up on the wallet demo and the September release and the sooner we find out that an integration problem has occurred the better.

@heifner
Copy link
Contributor

heifner commented Sep 1, 2017

Would like to get this working with eos-walletd as well.

@heifner
Copy link
Contributor

heifner commented Sep 5, 2017

I think it will be useful to get something simple in place to start. We can then add to it over time. Thinking something like the following for the first pass.

  • Bash script that launches eosd, walletd.
  • Create keys, import into wallet
  • Create new account w/ signed transaction
  • Transfer amount w/ signed transaction

@heifner
Copy link
Contributor

heifner commented Sep 5, 2017

I think this gives us a good first cut at a simple integration test. At this point, maybe we should create new issues for additional functionality.

@heifner
Copy link
Contributor

heifner commented Sep 5, 2017

@bytemaster Do you want me to add additional tests or submit a pull request?

@heifner
Copy link
Contributor

heifner commented Sep 6, 2017

  • Add check of return code of eosc to script
  • Add check for exception in json returned to eosc and return an error code
    • This already is done via http error codes

@heifner
Copy link
Contributor

heifner commented Sep 12, 2017

Integration test was merged. Closing.

@heifner heifner closed this as completed Sep 12, 2017
taokayan pushed a commit to taokayan/eos that referenced this issue May 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants