Skip to content

Cli wallet tests framework#98

Merged
bobinson merged 3 commits intodevelopfrom
cli_wallet_tests
Sep 6, 2019
Merged

Cli wallet tests framework#98
bobinson merged 3 commits intodevelopfrom
cli_wallet_tests

Conversation

@oxarbitrage
Copy link

The first commit removes the cache from wallet call get_account that do more harm than good and was removed from bitshares since a while(bitshares/bitshares-core#640).

Second commit actually add the cli test framework with some test cases examples, more can be found in the bitshares version.

oxarbitrage@oxarbitrage-Lenovo-ideapad-320S-14IKB:~/CLionProjects/pp_cli_wallet_tests/peerplays$ ./tests/cli_test -l all
...
*** No errors detected
oxarbitrage@oxarbitrage-Lenovo-ideapad-320S-14IKB:~/CLionProjects/pp_cli_wallet_tests/peerplays$ 

Sorry the delay on this, had a hard time making it work.

@oxarbitrage
Copy link
Author

This is for #59

@bobinson bobinson added the enhancement New feature or request label Sep 2, 2019
con.wallet_api_ptr->save_wallet_file(con.wallet_filename);

// attempt to give jmjatlanta some CORE
BOOST_TEST_MESSAGE("Transferring bitshares from Nathan to jmjatlanta");

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say core tokens instead of bitshares in the message.

Copy link
Author

@oxarbitrage oxarbitrage Sep 3, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, changed at 1d1193e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants