Skip to content

chore: fix some minor issues in comment #19980

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.

Sign up for GitHub

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
+4 −4
Open

chore: fix some minor issues in comment #19980

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion chia/_tests/environments/wallet.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -347,7 +347,7 @@ async def process_pending_states(
1) Ensures all pending transactions have entered the mempool
2) Checks that all balances have changed properly prior to a block being farmed
3) Farms a block (to no one in particular)
4) Chacks that all balances have changed properly after the block was farmed
4) Checks that all balances have changed properly after the block was farmed
5) Checks that all pending transactions that were gathered in step 1 are now confirmed
6) Checks that if `reuse_puzhash` was set, no new derivations were created
7) Ensures the wallet is in a synced state before progressing to the rest of the test
Expand Down
4 changes: 2 additions & 2 deletions chia/_tests/wallet/test_wallet_node.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -611,7 +611,7 @@ async def test_transaction_send_cache(
) -> None:
"""
The purpose of this test is to test that calling _resend_queue on the wallet node does not result in resending a
spend to a peer that has already recieved that spend and is currently processing it. It also tests that once we
spend to a peer that has already received that spend and is currently processing it. It also tests that once we
have heard that the peer is done processing the spend, we _do_ properly resend it.
"""
[full_node_api], [(wallet_node, wallet_server)], _ = simulator_and_wallet
Expand Down Expand Up @@ -649,7 +649,7 @@ def logged_spends_len() -> int:
with pytest.raises(AssertionError):
await time_out_assert(5, logged_spends_len, 2)

# Tell the wallet that we recieved the spend (but failed to process it so it should send again)
# Tell the wallet that we received the spend (but failed to process it so it should send again)
msg = make_msg(
ProtocolMessageTypes.transaction_ack,
wallet_protocol.TransactionAck(
Expand Down
2 changes: 1 addition & 1 deletion chia/wallet/conditions.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -516,7 +516,7 @@ def from_mode_and_maybe_args(
amount_committed: Optional[uint64] = None
# This loop probably looks a little strange
# It's trying to account for the fact that the arguments may be any 1 or 2 of these arguments in this order
# Not sure of a more elgant way to do it
# Not sure of a more elegant way to do it
original_mode = mode
for arg in args:
if mode & 0b100:
Expand Down
Loading