PalladiumWallet

davide/PalladiumWallet

Fork 0

Commit Graph

Author	SHA1	Message	Date
davide	ee0b73dd52	test: expand coverage to 209 tests across all core modules Added tests for CoinAmount (all units, comma decimal, sub-satoshi rejection, overflow, unknown unit ArgumentException, roundtrip), Storage (nonce/salt uniqueness per encrypt, atomic write, WalletLock acquire/release/stale-file), WalletLoader (NewFromMnemonic, passphrase isolation, watch-only, invalid inputs, ProfileOf), SPV/Merkle (single tx, odd/even counts, altered proof, empty list), and Chain (unknown NetKind, distinct profiles, port constants).	2026-06-13 21:59:51 +02:00
davide	008e9c395a	fix(wallet): reject amounts with sub-satoshi precision Replace the silent (long) cast with an explicit fractional check: if value * factor has a non-zero remainder, TryParseIn/TryParseCoins return false. The caller already shows "Importo non valido." to the user, so no UI change is needed. Covers TryParseIn (all units) and TryParseCoins. Adds CoinAmountTests with valid and invalid cases including the triggering example (1.9 sat).	2026-06-13 20:39:08 +02:00

Author

SHA1

Message

Date

davide

ee0b73dd52

test: expand coverage to 209 tests across all core modules

Added tests for CoinAmount (all units, comma decimal, sub-satoshi rejection,
overflow, unknown unit ArgumentException, roundtrip), Storage (nonce/salt
uniqueness per encrypt, atomic write, WalletLock acquire/release/stale-file),
WalletLoader (NewFromMnemonic, passphrase isolation, watch-only, invalid
inputs, ProfileOf), SPV/Merkle (single tx, odd/even counts, altered proof,
empty list), and Chain (unknown NetKind, distinct profiles, port constants).

2026-06-13 21:59:51 +02:00

davide

008e9c395a

fix(wallet): reject amounts with sub-satoshi precision

Replace the silent (long) cast with an explicit fractional check:
if value * factor has a non-zero remainder, TryParseIn/TryParseCoins
return false. The caller already shows "Importo non valido." to the
user, so no UI change is needed.

Covers TryParseIn (all units) and TryParseCoins. Adds CoinAmountTests
with valid and invalid cases including the triggering example (1.9 sat).

2026-06-13 20:39:08 +02:00

2 Commits