Merge d94134390af082adaa628ceb8ee919c8286f1cce into 95d55a9b81145fcbc0a09023680b96b684f57b64
e6ce065a
pull/598/merge
2/40 ++ 1 --
Commits and releases over time
No repositories match that filter.
Per the documented contract, delete_key should return True only when the account was deleted. Check membership via list_keys first so a missing address returns False instead of always reporting success. Closes #472
chore: remove dryrun
Fix GitHub issue #472: Wallet.delete_key() and Wallet.delete_multisig() now correctly return False when the specified address does not exist in the wallet, instead of always returning True. The fix adds a pre-check using list_keys() / list_multisig() before calling the KMD delete endpoint. This prevents callers from incorrectly assuming a deletion succeeded when the key/multisig was never present. Changes: - algosdk/wallet.py: Add existence check in delete_key() and delete_multisig() - algosdk/wallet.py: Update docstrings to document False return value - tests/unit_tests/test_wallet.py: New test file with 6 unit tests