test: fix sending from subaddresses in single transaction
What the developer wrote
Authored by woodser
test: fix sending from subaddresses in single transaction
What changed, and why it matters
This commit changes one line in a test file. It fixes a test that checks sending Monero from multiple subaddresses in a single, unsplit transaction. Previously the test passed null (likely allowing default split behavior); now it explicitly disables transaction splitting. There is no change to production wallet code, no security fix, and no indication of a vulnerability.
No security action needed. Treat as a normal test fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In src/test/java/TestMoneroWalletCommon.java, the testSendFromSubaddresses() test now calls testSendFromMultiple(new MoneroTxConfig().setCanSplit(false)) instead of testSendFromMultiple(null). This aligns the test with its documented purpose of sending from subaddresses in a single transaction. The change is purely test-configuration and does not alter library behavior or address any security defect.
Changed components
src/test/java/TestMoneroWalletCommon.javaInspect captured patch +1 / −1
diff --git a/src/test/java/TestMoneroWalletCommon.java b/src/test/java/TestMoneroWalletCommon.java
index 30a8e80..e96b2b2 100644
--- a/src/test/java/TestMoneroWalletCommon.java
+++ b/src/test/java/TestMoneroWalletCommon.java
@@ -3212,7 +3212,7 @@ public abstract class TestMoneroWalletCommon {
@Test
public void testSendFromSubaddresses() {
assumeTrue(TEST_RELAYS);
- testSendFromMultiple(null);
+ testSendFromMultiple(new MoneroTxConfig().setCanSplit(false));
}
// Can send from multiple subaddresses in split transactions
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.