* 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

Moul authored 5 years ago and matograine committed 5 years ago

- Pubkey pattern defined in constants
- Merge search and compline on the same line

Moul authored 5 years ago and matograine committed 5 years ago

- 're.' is not too long

* use Duniterpy's regexp for authfile checking
    * former regexp would fail on legit privkeys)
    * makes Silkaj compatible with Cesiums PubSec authentication files.
* document the function auth_by_auth_file()

- 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…

- Based on DuniterPy 0.56.0 which implements block signature
  verification
- #183: Click progress bar
- Handle BMA anti-spam protection whith an exception
- Possibility to pass starting and ending block numbers to verify them
- Default to 0,0(head) for the full blockchain
- Get rid of the client singleton, in order to have full control over it

- Use caret requirement:
- https://python-poetry.org/docs/dependency-specification/#caret-requirements
- Remove now useless black prereleases statement

- Build and tests jobs weren't triggered when the code was changing

Moul authored 5 years ago and matograine committed 5 years ago

- Kept for pyinstaller because unmaintained and lazy to search

- Change bma imports for more clarity

- Update the documentation accordingly

* 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.

- for tx and cert commands

- Try using same only files block changes to trigger tests-3.7 and pages jobs
- Otherwise, pages job could think tests-3.7 does not exists

* 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()`

matograine authored 5 years ago and Moul committed 5 years ago

* add display_pubkey() and display_amount() functions to match the proposal
* modify transaction_confirmation()

- '--' is necessary to pass options
- Update documentation and CI
- See https://github.com/sdispater/poetry/issues/1500

- in particular pytest to have tests-3.8 job succeed
- new look of poetry.lock