Skip to content

Commit

Permalink
chore(deps-dev): bump ruff from 0.8.4 to 0.9.4 in /functional-tests (#…
Browse files Browse the repository at this point in the history
…644)

* chore(deps-dev): bump ruff from 0.8.4 to 0.9.4 in /functional-tests

Bumps [ruff](https://github.com/astral-sh/ruff) from 0.8.4 to 0.9.4.
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@0.8.4...0.9.4)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* fmt: ruff 0.9.4

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jose Storopoli <[email protected]>
  • Loading branch information
dependabot[bot] and storopoli authored Feb 1, 2025
1 parent a5af03c commit cd459b3
Show file tree
Hide file tree
Showing 11 changed files with 46 additions and 46 deletions.
40 changes: 20 additions & 20 deletions functional-tests/poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion functional-tests/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ web3 = "^7.7.0"
py-solc-x = "^2.0.3"

[tool.poetry.group.dev.dependencies]
ruff = "0.8.4"
ruff = "0.9.4"
maturin = "1.8.1"

[tool.ruff]
Expand Down
6 changes: 3 additions & 3 deletions functional-tests/tests/bridge/bridge_deposit_happy.py
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,6 @@ def test_deposit(
assert new_balance > original_balance, "balance did not increase"

# Make sure that the balance is the default deposit amount of BTC in Strata "wei"
assert new_balance - original_balance == rollup_deposit_amount * (
10**10
), "balance is not the default rollup_deposit_amount"
assert new_balance - original_balance == rollup_deposit_amount * (10**10), (
"balance is not the default rollup_deposit_amount"
)
2 changes: 1 addition & 1 deletion functional-tests/tests/bridge/bridge_deposit_reclaim.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ def main(self, ctx: flexitest.RunContext):
blocks_to_generate = DEFAULT_TAKEBACK_TIMEOUT // chunks
self.debug(f"Generating {DEFAULT_TAKEBACK_TIMEOUT} blocks in {chunks} chunks")
for i in range(chunks):
self.debug(f"Generating {blocks_to_generate} blocks in chunk {i+1}/{chunks}")
self.debug(f"Generating {blocks_to_generate} blocks in chunk {i + 1}/{chunks}")
btcrpc.proxy.generatetoaddress(blocks_to_generate, UNSPENDABLE_ADDRESS)

# Make sure that the BTC refund address has the expected balance
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ def main(self, ctx: flexitest.RunContext):
blocks_to_generate = DEFAULT_TAKEBACK_TIMEOUT // chunks
self.debug(f"Generating {DEFAULT_TAKEBACK_TIMEOUT} blocks in {chunks} chunks")
for i in range(chunks):
self.debug(f"Generating {blocks_to_generate} blocks in chunk {i+1}/{chunks}")
self.debug(f"Generating {blocks_to_generate} blocks in chunk {i + 1}/{chunks}")
btcrpc.proxy.generatetoaddress(blocks_to_generate, UNSPENDABLE_ADDRESS)

# Make sure that the BTC refund address has the expected balance
Expand Down
6 changes: 3 additions & 3 deletions functional-tests/tests/bridge/bridge_duties.py
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,6 @@ def sorting_key(x) -> str:
assert len(duties) == len(expected_duties), "num duties must match"
assert duties == expected_duties, "duties in response should match expected ones"
assert duties_resp["start_index"] == start_index, "start index must match"
assert (
duties_resp["stop_index"] > start_index
), "stop_index must be greater than start_index"
assert duties_resp["stop_index"] > start_index, (
"stop_index must be greater than start_index"
)
6 changes: 3 additions & 3 deletions functional-tests/tests/btcio/btcio_inscriber.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ def main(self, ctx: flexitest.RunContext):
blobdata = "2c4253d512da5bb4223f10e8e6017ede69cc63d6e6126916f4b68a1830b7f805"
tx = submit_da_blob(btcrpc, seqrpc, blobdata)

assert any(
[blobdata in w.hex() for w in tx.inputs[0].witnesses]
), "Tx should have submitted blobdata in its witness"
assert any([blobdata in w.hex() for w in tx.inputs[0].witnesses]), (
"Tx should have submitted blobdata in its witness"
)

return True
10 changes: 5 additions & 5 deletions functional-tests/tests/btcio/btcio_read.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,11 +35,11 @@ def main(self, ctx: flexitest.RunContext):
cur_time = l1stat["last_update"] // 1000

# check if height on bitcoin is same as, it is seen in sequencer
self.debug(f'L1 stat curr height: {l1stat["cur_height"]}')
self.debug(f'Received from bitcoin: {received_block["height"]}')
assert (
l1stat["cur_height"] == received_block["height"]
), "sequencer height doesn't match the bitcoin node height"
self.debug(f"L1 stat curr height: {l1stat['cur_height']}")
self.debug(f"Received from bitcoin: {received_block['height']}")
assert l1stat["cur_height"] == received_block["height"], (
"sequencer height doesn't match the bitcoin node height"
)

# generate 2 more btc blocks
generate_n_blocks(btcrpc, 2)
Expand Down
4 changes: 2 additions & 2 deletions functional-tests/tests/client_status.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@ def main(self, ctx: flexitest.RunContext):
)

proto_ver = seqrpc.strata_protocolVersion()
self.debug(f"protocol version { proto_ver}")
self.debug(f"protocol version {proto_ver}")
assert proto_ver == 1, "query protocol version"

client_status = seqrpc.strata_clientStatus()
self.debug(f"client status { client_status}")
self.debug(f"client status {client_status}")

return True
2 changes: 1 addition & 1 deletion functional-tests/tests/sync/sync_genesis.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ def main(self, ctx: flexitest.RunContext):
stat = seqrpc.strata_clientStatus()
tip_slot = stat["chain_tip_slot"]
tip_blkid = stat["chain_tip"]
self.debug(f"cur tip slot {tip_slot} blkid { tip_blkid}")
self.debug(f"cur tip slot {tip_slot} blkid {tip_blkid}")
assert tip_slot >= last_slot, "cur slot went backwards"
assert tip_slot > last_slot, "seem to not be making progress"
last_slot = tip_slot
12 changes: 6 additions & 6 deletions functional-tests/utils/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -172,9 +172,9 @@ def check_nth_checkpoint_finalized(
timeout=3,
)

assert (
syncstat["finalized_block_id"] != batch_info["l2_blockid"]
), "Checkpoint block should not yet finalize"
assert syncstat["finalized_block_id"] != batch_info["l2_blockid"], (
"Checkpoint block should not yet finalize"
)
assert batch_info["idx"] == idx
checkpoint_info_next = seqrpc.strata_getCheckpointInfo(idx + 1)
assert checkpoint_info_next is None, f"There should be no checkpoint info for {idx + 1} index"
Expand Down Expand Up @@ -576,9 +576,9 @@ def confirm_btc_withdrawal(
debug_fn(f"BTC final balance: {btc_balance}")
debug_fn(f"Expected final balance: {original_balance + expected_increase}")

assert (
btc_balance == original_balance + expected_increase
), "BTC balance after withdrawal is not as expected"
assert btc_balance == original_balance + expected_increase, (
"BTC balance after withdrawal is not as expected"
)


def check_initial_eth_balance(rethrpc, address, debug_fn=print):
Expand Down

0 comments on commit cd459b3

Please sign in to comment.