Refactor cli_test

[abitmore]

@oxarbitrage wrote in #1177 (review):

one thing we should maybe consider(in a separated issue) is to add a function in cli_wallet tests to make a few of the common things that we need to do pretty much in all the tests.
connect to the server
set wallet password
import Nathan account
upgrade nathan
verify that the upgrade was successful
create a new account
save the private key for this new account in the wallet file
...
this can be in a test case that can be invoked, a function or even maybe a "database_fixture" version file for the cli.

CORE TEAM TASK LIST

• Evaluate / Prioritize Feature Request
• Refine User Stories / Requirements
• Define Test Cases
• Design / Develop Solution
  • Assigned: @cogutvalera
  • Estimated: 10 hours (Remittance Weeks 34-35)
• Perform QA/Testing
• Update Documentation

[abitmore]

I think this issue is a good place for adding clean shutdown tests.

Update: perhaps not a good place? There are quite some logic in the main function in programs/cli_wallet/main.cpp, perhaps hard to test.

[cogutvalera]

I want to claim this issue

[cogutvalera]

Thanks !

[oxarbitrage]

this was closed by #1243