-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Три проблемы с Typer #77
Comments
Сейчас внедрено решение по 1, 2 и 3, но решение по 3 выглядит наиболее громоздким. Возможно, если решить 1, по и тестирование в 3 будет единообразным. |
python mult.py --deposit ru 100 The amount of 100 (ru) to be added |
|
Проблема в Typer начинается, когда мы несколько таких ключей делаем: в тестах есть bx post --debit ar 120 --credit sales 100 --credit vat 20 Реализация в click: https://github.com/epogrebnyak/abacus/blob/main/abacus%2Ftyper_cli%2Fpost.py#L34-L36 На типе list[tuple[str, str]] Typer валится |
https://github.com/epogrebnyak/abacus/blob/main/tests%2Ftest_typer_cli.py#L125-L134 Вот тест который на Typer не получается сделать |
Typer не дает сделать ключ с двумя значениями, как например
--debit ar 100
. Текущее решение: сделана вставка Click в приложенте Typer.Вставку с Click нельзя повесить на команду второго уровня типа
bx ledger post
. Текущее решение - повесили команду Click какbx post
.Даже вставку Click первого уровня нельзя тестировать с CliRunner, он падает: CliRunner fails on app with added Click command fastapi/typer#732. Текущее решение: тестировать именно эту команду с через
subprocess.call
, что делает тесты более кривыми.The text was updated successfully, but these errors were encountered: