Skip to content

Commit 42ebc13

Browse files
fix(ci): black + darglint on test_behaviours after round-3 refactor
Two lint failures on the caller-side tests from the round-3 refactor in 628d8a2: - black: fold the ``_state_response({}, performative=...)`` call onto one line now that the argument list fits. - darglint: multi-line docstring on ``_state_response`` needs ``:param body:``, ``:param performative:``, and ``:return:`` under ``strictness = short``. Fingerprint refresh via ``autonomy packages lock``. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 628d8a2 commit 42ebc13

11 files changed

Lines changed: 38 additions & 31 deletions

File tree

packages/packages.json

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,16 +10,16 @@
1010
"contract/valory/mech_prepaid_reader/0.1.0": "bafybeiaitnqi4kc2wiltj3k6a3iujxritwkh5w2a2xoff7dosoqy7p3sou",
1111
"connection/valory/polymarket_client/0.1.0": "bafybeid4eo37pka4tfhsjs6rb3knrugmokcvihust2dpfrgob56y77xd44",
1212
"skill/valory/market_manager_abci/0.1.0": "bafybeig3ak2eoimxwuupybde4napyi2jlf7pxk35se2zclikete2ni5hti",
13-
"skill/valory/decision_maker_abci/0.1.0": "bafybeiguwazbetsmoyfownoxgsgymiu43ggcso3pjdln32dviqlu3pto5u",
14-
"skill/valory/trader_abci/0.1.0": "bafybeihhd5srgyf7ndubboyg634mlf6vsk2wx44wjoxgwq5b6hdybvmxtu",
15-
"skill/valory/tx_settlement_multiplexer_abci/0.1.0": "bafybeia4ddgpu4bkru2yy4hemuqnpm22w2b7ve4lt6iurp6rpvc7oqxj4a",
13+
"skill/valory/decision_maker_abci/0.1.0": "bafybeih4csk2ah76a6vfss4wgtr2ojkxm3rl2wgrlmxwwb5t3tz4smgvnm",
14+
"skill/valory/trader_abci/0.1.0": "bafybeid72nchskckp766lc4567ysdaoqceefxetujin3ixxxibr3ffwlwa",
15+
"skill/valory/tx_settlement_multiplexer_abci/0.1.0": "bafybeiabdo7k6fh6g3443b66d2zzu7sbzixlmguhf55nv3uahaqzyennra",
1616
"skill/valory/staking_abci/0.1.0": "bafybeia6csjgex5zxdt6vymiaengu75lyzjdgfrnp65ewlghthiniquhje",
17-
"skill/valory/agent_performance_summary_abci/0.1.0": "bafybeie62542svnhafnrht6yonof4cpr4dfchjmvdwhxmujchqquwouujm",
18-
"skill/valory/check_stop_trading_abci/0.1.0": "bafybeif5mq7gdbkb6cgk53p3sgnjms7lg4kr6yrwcftj4krfezuodlv55u",
19-
"skill/valory/chatui_abci/0.1.0": "bafybeibpwoulbllkzmi5ez7agi3g2llg4sjwilxjsyydzilo3mldm6xl6e",
20-
"agent/valory/trader/0.1.0": "bafybeigcx6xppnfknareq3idxhwy6gcflxvviqziljyenvg4rhucgyzjze",
21-
"service/valory/trader_pearl/0.1.0": "bafybeic2yl66d4a6m77yl5wo5jqigbwmhv4nyfvy2mx6hiarzkc4hgdzki",
22-
"service/valory/polymarket_trader/0.1.0": "bafybeib6s6cmua3u5w6x37ohksmejpmpa5gat4qypcey2zrapc4aoe4fou"
17+
"skill/valory/agent_performance_summary_abci/0.1.0": "bafybeicvaeizglfuxwgcmg7i3wrbyxw5ojlmbeagrkvu5cw35ucmttcl4q",
18+
"skill/valory/check_stop_trading_abci/0.1.0": "bafybeifamiyvfif7vxuguz5lhqgdhqust74gzk2nsjv3neqbveo7mddzdy",
19+
"skill/valory/chatui_abci/0.1.0": "bafybeiguz3y7m4deaoabkqa3tbewtc5eukygaqlnc4dtc5qh45b4qsyhd4",
20+
"agent/valory/trader/0.1.0": "bafybeibmfd6rpavw3xsh4zj4weku4ernwpddcge74fra3v765koqkhowlm",
21+
"service/valory/trader_pearl/0.1.0": "bafybeihxfa2f64rwin5ujc27dvijk4zkiik2qyt6lnrjmb4c2zdxua3wau",
22+
"service/valory/polymarket_trader/0.1.0": "bafybeicg526ppbnsshhdlj4fp7xiop2ky3zhnb5fqlxaffaejudu4cendm"
2323
},
2424
"third_party": {
2525
"protocol/valory/acn_data_share/0.1.0": "bafybeieqisfr6lhaaesq37lk7vyaxza2h6nucshksgyo2djqfyf2saqo4i",

packages/valory/agents/trader/aea-config.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -70,15 +70,15 @@ skills:
7070
- valory/reset_pause_abci:0.1.0:bafybeifrnmlscfiki7wojs6zasppefupd2wkjyq6orhdllfppjud7isrxi
7171
- valory/termination_abci:0.1.0:bafybeih6sa2ttcvvjk7rcfn7ure2ws23kcfufssb4qmluzcfvednkfpvoi
7272
- valory/transaction_settlement_abci:0.1.0:bafybeihizaapype5sz7vlosms3nfo6jzfj4ujycuf5uthviq2otvmw6f6m
73-
- valory/tx_settlement_multiplexer_abci:0.1.0:bafybeia4ddgpu4bkru2yy4hemuqnpm22w2b7ve4lt6iurp6rpvc7oqxj4a
73+
- valory/tx_settlement_multiplexer_abci:0.1.0:bafybeiabdo7k6fh6g3443b66d2zzu7sbzixlmguhf55nv3uahaqzyennra
7474
- valory/market_manager_abci:0.1.0:bafybeig3ak2eoimxwuupybde4napyi2jlf7pxk35se2zclikete2ni5hti
75-
- valory/decision_maker_abci:0.1.0:bafybeiguwazbetsmoyfownoxgsgymiu43ggcso3pjdln32dviqlu3pto5u
76-
- valory/trader_abci:0.1.0:bafybeihhd5srgyf7ndubboyg634mlf6vsk2wx44wjoxgwq5b6hdybvmxtu
75+
- valory/decision_maker_abci:0.1.0:bafybeih4csk2ah76a6vfss4wgtr2ojkxm3rl2wgrlmxwwb5t3tz4smgvnm
76+
- valory/trader_abci:0.1.0:bafybeid72nchskckp766lc4567ysdaoqceefxetujin3ixxxibr3ffwlwa
7777
- valory/staking_abci:0.1.0:bafybeia6csjgex5zxdt6vymiaengu75lyzjdgfrnp65ewlghthiniquhje
78-
- valory/check_stop_trading_abci:0.1.0:bafybeif5mq7gdbkb6cgk53p3sgnjms7lg4kr6yrwcftj4krfezuodlv55u
78+
- valory/check_stop_trading_abci:0.1.0:bafybeifamiyvfif7vxuguz5lhqgdhqust74gzk2nsjv3neqbveo7mddzdy
7979
- valory/mech_interact_abci:0.1.0:bafybeici5tsgtlypnyrnp5colj7katnsyjfdzt6js4xdbnpdms4232muje
80-
- valory/chatui_abci:0.1.0:bafybeibpwoulbllkzmi5ez7agi3g2llg4sjwilxjsyydzilo3mldm6xl6e
81-
- valory/agent_performance_summary_abci:0.1.0:bafybeie62542svnhafnrht6yonof4cpr4dfchjmvdwhxmujchqquwouujm
80+
- valory/chatui_abci:0.1.0:bafybeiguz3y7m4deaoabkqa3tbewtc5eukygaqlnc4dtc5qh45b4qsyhd4
81+
- valory/agent_performance_summary_abci:0.1.0:bafybeicvaeizglfuxwgcmg7i3wrbyxw5ojlmbeagrkvu5cw35ucmttcl4q
8282
- valory/funds_manager:0.1.0:bafybeicfw7qjtnlcbw6awsvxdahqu42fxsfsd3wy7mbyhcbls4l5pee3by
8383
customs:
8484
- valory/kelly_criterion:0.1.0:bafybeidme2uybzyzagky3jtosheqmpgc6b6w4rum7cdxw2fc35nnpwblfe

packages/valory/services/polymarket_trader/service.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ license: Apache-2.0
88
fingerprint:
99
README.md: bafybeichoh5tnk4qctaazcjcrxvpwc5yasodleqoeapnfapwrrdh3tm5gu
1010
fingerprint_ignore_patterns: []
11-
agent: valory/trader:0.1.0:bafybeigcx6xppnfknareq3idxhwy6gcflxvviqziljyenvg4rhucgyzjze
11+
agent: valory/trader:0.1.0:bafybeibmfd6rpavw3xsh4zj4weku4ernwpddcge74fra3v765koqkhowlm
1212
number_of_agents: 1
1313
deployment:
1414
agent:

packages/valory/services/trader_pearl/service.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ license: Apache-2.0
88
fingerprint:
99
README.md: bafybeibg7bdqpioh4lmvknw3ygnllfku32oca4eq5pqtvdrdsgw6buko7e
1010
fingerprint_ignore_patterns: []
11-
agent: valory/trader:0.1.0:bafybeigcx6xppnfknareq3idxhwy6gcflxvviqziljyenvg4rhucgyzjze
11+
agent: valory/trader:0.1.0:bafybeibmfd6rpavw3xsh4zj4weku4ernwpddcge74fra3v765koqkhowlm
1212
number_of_agents: 1
1313
deployment:
1414
agent:

packages/valory/skills/agent_performance_summary_abci/skill.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ fingerprint:
3333
tests/graph_tooling/test_predictions_helper.py: bafybeidkmubpjfnxys74bjaqd7ptz4gc6ejfwrrmiodmdjwdovdclvdqhu
3434
tests/graph_tooling/test_queries.py: bafybeih4ybhkq5seb34eqgakfpfrtlijxju3w52cjwvlrbbx2afgai4jm4
3535
tests/graph_tooling/test_requests.py: bafybeic2dyqmf7uj42f2f7d3hjxguw4mmlesloci4uaioui2rx3wfjhouq
36-
tests/test_behaviours.py: bafybeiawkenruh6rwr4iinupjrn7c2lr6jr47g3s663ghmgxdubf3extqm
36+
tests/test_behaviours.py: bafybeihiu4wiv5jlzep7grhftuzt5x3blnvmosrcwhtxvhl4k7qa3dqvxe
3737
tests/test_dialogues.py: bafybeigezi53b2jukm5ju6z6zvecfjkjtzxcge3ehnzxryuhpambzknc3y
3838
tests/test_handlers.py: bafybeigybc6zkku7ldtawjcjjdkndiz3alvckxbeyjj23rsmny5ql7leku
3939
tests/test_models.py: bafybeibeos6fm67nb3w6d7nb2yrygjttz7zmrwlnjogzp77rj4ttp23wgi

packages/valory/skills/agent_performance_summary_abci/tests/test_behaviours.py

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2278,6 +2278,15 @@ def _state_response(
22782278
Defaults to the contract-API performative for the incremental-scan
22792279
callers; pass ``LedgerApiMessage.Performative.STATE`` for the
22802280
head-block ledger read.
2281+
2282+
:param body: value to place on ``response.state.body`` (usually a
2283+
dict, sometimes an empty ``{}`` for malformed-response tests).
2284+
:param performative: response performative; defaults to the
2285+
contract-API STATE, override to the ledger-API STATE for head
2286+
block reads.
2287+
:return: fully-wired ``MagicMock`` response ready to feed into
2288+
``get_contract_api_response`` / ``get_ledger_api_response``
2289+
side_effects.
22812290
"""
22822291
response = MagicMock()
22832292
response.performative = performative
@@ -2504,9 +2513,7 @@ def test_first_run_malformed_seed_response_leaves_checkpoint_unset(self) -> None
25042513
def _ledger_api(*_a: Any, **_kw: Any) -> Generator:
25052514
# STATE-performative but body has no ``get_block_number_result`` key.
25062515
return _return_gen(
2507-
_state_response(
2508-
{}, performative=LedgerApiMessage.Performative.STATE
2509-
)
2516+
_state_response({}, performative=LedgerApiMessage.Performative.STATE)
25102517
)()
25112518

25122519
with (

packages/valory/skills/chatui_abci/skill.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ protocols:
3232
contracts: []
3333
skills:
3434
- valory/abstract_round_abci:0.1.0:bafybeifkxnvvgsldkb4rgejsoon2mvrmrnl7asy2nhenwss7hpwg3myflu
35-
- valory/agent_performance_summary_abci:0.1.0:bafybeie62542svnhafnrht6yonof4cpr4dfchjmvdwhxmujchqquwouujm
35+
- valory/agent_performance_summary_abci:0.1.0:bafybeicvaeizglfuxwgcmg7i3wrbyxw5ojlmbeagrkvu5cw35ucmttcl4q
3636
handlers:
3737
abci:
3838
args: {}

packages/valory/skills/check_stop_trading_abci/skill.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ contracts:
3030
protocols: []
3131
skills:
3232
- valory/abstract_round_abci:0.1.0:bafybeifkxnvvgsldkb4rgejsoon2mvrmrnl7asy2nhenwss7hpwg3myflu
33-
- valory/chatui_abci:0.1.0:bafybeibpwoulbllkzmi5ez7agi3g2llg4sjwilxjsyydzilo3mldm6xl6e
33+
- valory/chatui_abci:0.1.0:bafybeiguz3y7m4deaoabkqa3tbewtc5eukygaqlnc4dtc5qh45b4qsyhd4
3434
- valory/mech_interact_abci:0.1.0:bafybeici5tsgtlypnyrnp5colj7katnsyjfdzt6js4xdbnpdms4232muje
3535
- valory/staking_abci:0.1.0:bafybeia6csjgex5zxdt6vymiaengu75lyzjdgfrnp65ewlghthiniquhje
3636
behaviours:

packages/valory/skills/decision_maker_abci/skill.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -180,8 +180,8 @@ skills:
180180
- valory/transaction_settlement_abci:0.1.0:bafybeihizaapype5sz7vlosms3nfo6jzfj4ujycuf5uthviq2otvmw6f6m
181181
- valory/mech_interact_abci:0.1.0:bafybeici5tsgtlypnyrnp5colj7katnsyjfdzt6js4xdbnpdms4232muje
182182
- valory/staking_abci:0.1.0:bafybeia6csjgex5zxdt6vymiaengu75lyzjdgfrnp65ewlghthiniquhje
183-
- valory/agent_performance_summary_abci:0.1.0:bafybeie62542svnhafnrht6yonof4cpr4dfchjmvdwhxmujchqquwouujm
184-
- valory/chatui_abci:0.1.0:bafybeibpwoulbllkzmi5ez7agi3g2llg4sjwilxjsyydzilo3mldm6xl6e
183+
- valory/agent_performance_summary_abci:0.1.0:bafybeicvaeizglfuxwgcmg7i3wrbyxw5ojlmbeagrkvu5cw35ucmttcl4q
184+
- valory/chatui_abci:0.1.0:bafybeiguz3y7m4deaoabkqa3tbewtc5eukygaqlnc4dtc5qh45b4qsyhd4
185185
behaviours:
186186
main:
187187
args: {}

packages/valory/skills/trader_abci/skill.yaml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -57,13 +57,13 @@ skills:
5757
- valory/transaction_settlement_abci:0.1.0:bafybeihizaapype5sz7vlosms3nfo6jzfj4ujycuf5uthviq2otvmw6f6m
5858
- valory/termination_abci:0.1.0:bafybeih6sa2ttcvvjk7rcfn7ure2ws23kcfufssb4qmluzcfvednkfpvoi
5959
- valory/market_manager_abci:0.1.0:bafybeig3ak2eoimxwuupybde4napyi2jlf7pxk35se2zclikete2ni5hti
60-
- valory/decision_maker_abci:0.1.0:bafybeiguwazbetsmoyfownoxgsgymiu43ggcso3pjdln32dviqlu3pto5u
61-
- valory/tx_settlement_multiplexer_abci:0.1.0:bafybeia4ddgpu4bkru2yy4hemuqnpm22w2b7ve4lt6iurp6rpvc7oqxj4a
60+
- valory/decision_maker_abci:0.1.0:bafybeih4csk2ah76a6vfss4wgtr2ojkxm3rl2wgrlmxwwb5t3tz4smgvnm
61+
- valory/tx_settlement_multiplexer_abci:0.1.0:bafybeiabdo7k6fh6g3443b66d2zzu7sbzixlmguhf55nv3uahaqzyennra
6262
- valory/staking_abci:0.1.0:bafybeia6csjgex5zxdt6vymiaengu75lyzjdgfrnp65ewlghthiniquhje
63-
- valory/check_stop_trading_abci:0.1.0:bafybeif5mq7gdbkb6cgk53p3sgnjms7lg4kr6yrwcftj4krfezuodlv55u
63+
- valory/check_stop_trading_abci:0.1.0:bafybeifamiyvfif7vxuguz5lhqgdhqust74gzk2nsjv3neqbveo7mddzdy
6464
- valory/mech_interact_abci:0.1.0:bafybeici5tsgtlypnyrnp5colj7katnsyjfdzt6js4xdbnpdms4232muje
65-
- valory/chatui_abci:0.1.0:bafybeibpwoulbllkzmi5ez7agi3g2llg4sjwilxjsyydzilo3mldm6xl6e
66-
- valory/agent_performance_summary_abci:0.1.0:bafybeie62542svnhafnrht6yonof4cpr4dfchjmvdwhxmujchqquwouujm
65+
- valory/chatui_abci:0.1.0:bafybeiguz3y7m4deaoabkqa3tbewtc5eukygaqlnc4dtc5qh45b4qsyhd4
66+
- valory/agent_performance_summary_abci:0.1.0:bafybeicvaeizglfuxwgcmg7i3wrbyxw5ojlmbeagrkvu5cw35ucmttcl4q
6767
- valory/funds_manager:0.1.0:bafybeicfw7qjtnlcbw6awsvxdahqu42fxsfsd3wy7mbyhcbls4l5pee3by
6868
behaviours:
6969
main:

0 commit comments

Comments
 (0)