- #314: dry-run option

to choose identity among identities from wot/lookup
Display the uid, pubkey and the blockstamp to choose
Add tests on choose_identity()

Delete get_information_for_identity() based on top of the lookup

Allow to pass pubkey to the following commands:
The uid was the only identifier before
Adapt 'wot' and 'cert' commands to choose_identity()

Import silkaj.wot and click directly
Import directly bma otherwise there is a namespace conflict on 'wot'
on silkaj.wot and bma.wot

Harmonize by calling top-level click cmds group in CliRunner.invoke(cli.cli)
Merge the command in the same line
Directly monkeypatch the imported 'bma' module

Click https://click.palletsprojects.com/en/7.x/changelog/#version-7-1-2
 - Fix test: the error message changed from double to simple cotes
PyNaCl: https://github.com/pyca/pynacl/blob/master/CHANGELOG.rst#140-2020-05-25
Pytest: https://docs.pytest.org/en/latest/changelog.html#pytest-5-4-3-2020-06-02

* create a patched get_sources() function

    * use display_amount() to display pubkey's amounts

* modify test_transaction_confirmation() to match new behavior
    * add new test case
    * remove useless float()

* modify
    * handle_intpermediary_transaction(),
    * generate_and_send_transaction(),
    * generate_transaction_document()
* all will now use tx_amounts list

* create a unit test for generate_transaction_document()
* create a patched head_block() function.

/!\ the created patched function does not return a whole fake block, only relevant infos for now.

* transaction_amount() now returns the amounts list.

* modify tests in tests_tx.py for transaction_amount to match new behavior
* create unit tests in tests_unit_tx.py

* Uses a list of user amounts and a multiplicator (should be CENT_MULT_TO_UNIT or UD_Value).
* Multiplies the given amounts(UD) to get usable int() amounts for Duniter.
* Makes sure that each amountUD is superior to the minimal amount.
    * Absolute amounts are in FloatRange(MINIMAL_AMOUNT), so they have already been checked.
    * This check should be updated when solving issue #306 / DUBP v13.
* Returns list of amounts.

* both constants imported to tx.py
* MINIMAL_TX_AMOUNT used in --amount click option
* import silkaj.money and change related functions calls

* introduce one test case for send_transaction()

* Breaking change: Rename `--output` option to `--recipient`
* Add extra small options to ease passing multiple amounts and recipients:
  * `-a/--amount`
  * `-d/--amountUD`
  * `-r/--recipient`
* Add possibility to pass multiple options:
  * Breaking change: recipients public keys are no longer `:` separated: `-r A -r B`
* Store the options in variables names in plural for preciseness
* Fix: Handle the case when no option specifying the amount is passed
* Restrict `--allSources` option to a single recipient, case not handle for now
* Extend tests with two upper checks
* Improve help

- Integration test on the overall command:
  - Use click CliRunner().invoke()
  - https://click.palletsprojects.com/en/7.x/testing/#basic-testing
    - integration tests are requesting a node
    - If the node is down, the tests are broken
- Unit tests on all the functions to test most of the major corner cases:
  - valid and a non-valid blocks signatures:
    - Kind of already in DuniterPy tests…

Moul authored 5 years ago and matograine committed 5 years ago

- Kept for pyinstaller because unmaintained and lazy to search

* change the balance display to a table
* use display_amount() and tabulate in the balance display
* mod tests_end_to_end.py to match new display

* create tui.py with license statement
* move display_amount() and display_pubkey() to tui.py to prevent circular dependencies
* change display_amount() to prevent circular dependency with money.py.
    * Now display_amount() needs the ud_value parameter.
    * display_amount() is no more async
* mod tx.py and test_unit_tx.py to match new location of the two features.

* mock_ud_value set to 314. Former value 3.14159 shall never occur.

- 'poetry run' no longer needs `--`.

Revert "[fix] Call 'poetry run silkaj' tests with '--'"
Revert "[fix] 'poetry run' cli parsing changed in 1.0.0b3:"

This reverts commit f335a2bd.
This reverts commit 4f772671.

matograine authored 5 years ago and Moul committed 5 years ago

* add tests for display_pubkey(), display_amount() and transaction_confirmation() in tests/test_unit_tx.py.
* patch async functions.
* create the patched.py module containing patched/mocked functions.
* modify silkaj/tx.py to match patch requirements:
  * `from silkaj.wot import is_member` -> `from silkaj import wot`
  * `await is_member()` -> `await wot.is_member()`