Madiator2011/lbry-sdk
1
0
Fork 0
forked from LBRYCommunity/lbry-sdk
Code Pull requests Activity

fix tests

Browse source
This commit is contained in:
Victor Shyba 2020-09-04 01:25:56 -03:00
parent d89b074615
commit 8c525b6dfc
1 changed files with 8 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
tests/unit/service/test_parser.py
View file

@ -45,9 +45,6 @@ class FakeAPI:
) -> Paginated[Wallet]: # list of wallets ) -> Paginated[Wallet]: # list of wallets
"""list command doc""" """list command doc"""
def thing_save(self, **another_test_kwargs):
"""save command doc"""
def thing_update(self, value1: str) -> Wallet: # updated wallet def thing_update(self, value1: str) -> Wallet: # updated wallet
"""update command doc""" """update command doc"""
@ -89,18 +86,23 @@ class BadAPI(FakeAPI):
""" """
class BadContinuationAPI(FakeAPI):
def thing_save(self, **another_test_kwargs):
"""save command doc"""
class TestParser(TestCase): class TestParser(TestCase):
maxDiff = None maxDiff = None
def test_parse_does_not_allow_duplicate_arguments(self): def test_parse_does_not_allow_duplicate_arguments(self):
with self.assertRaises(Exception) as exc: with self.assertRaises(Exception) as exc:
parse_method(BadAPI.thing_search, get_expanders()) parse_method(BadAPI.thing_search, get_expanders())
self.assertEqual(exc.exception.args[0], "Expander 'another_test' argument repeated: thing_search.") self.assertEqual(exc.exception.args[0], "Expander 'another_test' argument repeated: somevalue.")
def test_parse_does_not_allow_line_break_with_two_dashes(self): def test_parse_does_not_allow_line_break_with_two_dashes(self):
# breaking with two dashes breaks docopt parsing # breaking with two dashes breaks docopt parsing
with self.assertRaises(Exception) as exc: with self.assertRaises(Exception) as exc:
get_api_definitions(FakeAPI) get_api_definitions(BadContinuationAPI)
self.assertEqual( self.assertEqual(
exc.exception.args[0], exc.exception.args[0],
"Continuation line starts with -- on thing save: \"--angry [default: 3]\"" "Continuation line starts with -- on thing save: \"--angry [default: 3]\""
@ -241,13 +243,12 @@ class TestGenerator(TestCase):
Usage: Usage:
thing delete <value1> thing delete <value1>
[--wallet_id=<wallet_id>] [--change_account_id=<change_account_id>] [--change_account_id=<change_account_id>]
[--fund_account_id=<fund_account_id>...] [--preview] [--no_wait] [--fund_account_id=<fund_account_id>...] [--preview] [--no_wait]
[--page=<page>] [--page_size=<page_size>] [--include_total] [--page=<page>] [--page_size=<page_size>] [--include_total]
Options: Options:
--value1=<value1> : (str) --value1=<value1> : (str)
--wallet_id=<wallet_id> : (str) restrict operation to specific wallet
--change_account_id=<change_account_id> : (str) account to send excess change (LBC) --change_account_id=<change_account_id> : (str) account to send excess change (LBC)
--fund_account_id=<fund_account_id> : (str, list) accounts to fund the --fund_account_id=<fund_account_id> : (str, list) accounts to fund the
transaction transaction