Cover recursive tariff ledger payments

Author: Drefvelin

src/main/java/net/tfminecraft/simplefactions/guild/income/Ledger.java

@@ -784,6 +784,10 @@ public double getWarReparationsReceived() {
                     if (payerGuild == null) {
                         continue;
                     }
+                    Ledger payerLedger = payerGuild.getLedger();
+                    if (payerLedger == null || payerLedger.skipsMoneyMovement()) {
+                        continue;
+                    }
                     TradeBreakdown trade = payerGuild.getTradeBreakdown();
                     double grossTradeIncome = trade == null ? 0.0 : trade.getIncome();
                     total += grossTradeIncome * (obligation.getIncomePercent() / 100.0);

src/test/java/net/tfminecraft/simplefactions/war/resolution/WarReparationsServiceTest.java

@@ -12,9 +12,11 @@
 
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
+import org.mockito.MockedStatic;
 
 import net.tfminecraft.simplefactions.Cache;
 import net.tfminecraft.simplefactions.objects.Faction;
+import net.tfminecraft.simplefactions.managers.RelationManager;
 import net.tfminecraft.simplefactions.war.core.War;
 import net.tfminecraft.simplefactions.war.enums.WarGoalType;
 import net.tfminecraft.simplefactions.war.enums.WarType;
@@ -78,6 +80,36 @@ void applyFromWar_attackerPaysDefenderWithCacheValues() {
 		assertEquals(10, obligations.get(0).getDaysRemaining());
 	}
 
+	@Test
+	void apply_recursivelyAddsObligationToEveryVassalOnce() {
+		Faction child = mockFaction("child");
+		Faction grandchild = mockFaction("grandchild");
+		List<WarReparationsObligation> childObligations = new ArrayList<>();
+		List<WarReparationsObligation> grandchildObligations = new ArrayList<>();
+		when(child.getWarReparationsObligations()).thenReturn(childObligations);
+		when(grandchild.getWarReparationsObligations()).thenReturn(grandchildObligations);
+		doAnswer(invocation -> {
+			childObligations.add(invocation.getArgument(0));
+			return null;
+		}).when(child).addWarReparationsObligation(org.mockito.ArgumentMatchers.any());
+		doAnswer(invocation -> {
+			grandchildObligations.add(invocation.getArgument(0));
+			return null;
+		}).when(grandchild).addWarReparationsObligation(org.mockito.ArgumentMatchers.any());
+
+		try (MockedStatic<RelationManager> relations = org.mockito.Mockito.mockStatic(RelationManager.class)) {
+			relations.when(() -> RelationManager.getSubjects(payer)).thenReturn(List.of(child));
+			relations.when(() -> RelationManager.getSubjects(child)).thenReturn(List.of(grandchild));
+			relations.when(() -> RelationManager.getSubjects(grandchild)).thenReturn(List.of(payer));
+			assertTrue(WarReparationsService.apply(payer, payee));
+		}
+
+		assertEquals(1, obligations.size());
+		assertEquals(1, childObligations.size());
+		assertEquals(1, grandchildObligations.size());
+		assertEquals("def", grandchildObligations.get(0).getPayeeFactionId());
+	}
+
 	private static Faction mockFaction(String id) {
 		Faction faction = mock(Faction.class);
 		when(faction.getId()).thenReturn(id);