Mocking correctly test_membership doesn't worth it

monkeypatch transfer_file

Adapt tests

Revocation tests are failing since
test_transfer::test_generate_and_send_transaction
runs first. network.send_document() patching issue?

Used to work before when they use to run in the other order:
test_revocation/test_tx -> wot.test_revocation/money.test_transaction

Skipping test_generate_and_send_transaction
which anyway will be removed in next Substrate milestone %0.20.0

Closer to money commands usage

Would avoid import cycle issue

In order to avoid as much as possible circural import

Also fits better with other pubkey tools

Split balance cmd in a file, money_tools in an other

Fix types
Improve method
Set value types in blocks cells in order to get the proper format

matograine authored 2 years ago and Moul committed 2 years ago

generate_table does not add table headers any more.

matograine authored 2 years ago and Moul committed 2 years ago

Import entire tui module

Adapt balance test
In the test environment, the checksum gets to the line below
In the test environment and the runtime env,
shutil.get_terminal_size().columns is of same value
No idea why this is happening
Check if checksum present in output

matograine authored 2 years ago and Moul committed 2 years ago

update related tests

matograine authored 2 years ago and Moul committed 2 years ago

- create related tests

To avoid following report:
W0622: Redefining built-in 'license' (redefined-builtin)

To be compatible with mypy, pylint, pytest
Returning in case of dry-run
Not returning a value since mypy and pylint complain
that every parts of the functions should return an integer

It was wrongly done before
Defining an attribute of a function

Fix OutputSource(amount=str(X) -> int)