mirror of
https://github.com/bisq-network/bisq.git
synced 2025-03-03 18:56:59 +01:00
Fix FeeReceiverSelectorTest
Split test into sub-tests in order to avoid mockito UnnecessaryStubbingExceptions. These exceptions occurred because of 3 ambiguous uses of "when().thenReturn()" in the same test. These calls were ambiguous, because the when() argument was identical and mockito did not check the thenReturn() arguments, which would have differentiated the calls. Splitting this test method into 4 test methods achieved this call differentiation.
This commit is contained in:
parent
7a73fea622
commit
e55594d1e3
1 changed files with 16 additions and 1 deletions
|
@ -67,17 +67,32 @@ public class FeeReceiverSelectorTest {
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void testGetAddress_noValidReceivers() {
|
public void testGetAddress_noValidReceivers_nullFilter() {
|
||||||
when(daoFacade.getParamValue(Param.RECIPIENT_BTC_ADDRESS)).thenReturn("default");
|
when(daoFacade.getParamValue(Param.RECIPIENT_BTC_ADDRESS)).thenReturn("default");
|
||||||
|
|
||||||
when(filterManager.getFilter()).thenReturn(null);
|
when(filterManager.getFilter()).thenReturn(null);
|
||||||
assertEquals("default", FeeReceiverSelector.getAddress(daoFacade, filterManager));
|
assertEquals("default", FeeReceiverSelector.getAddress(daoFacade, filterManager));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void testGetAddress_noValidReceivers_filterWithNullList() {
|
||||||
|
when(daoFacade.getParamValue(Param.RECIPIENT_BTC_ADDRESS)).thenReturn("default");
|
||||||
|
|
||||||
when(filterManager.getFilter()).thenReturn(filterWithReceivers(null));
|
when(filterManager.getFilter()).thenReturn(filterWithReceivers(null));
|
||||||
assertEquals("default", FeeReceiverSelector.getAddress(daoFacade, filterManager));
|
assertEquals("default", FeeReceiverSelector.getAddress(daoFacade, filterManager));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void testGetAddress_noValidReceivers_filterWithEmptyList() {
|
||||||
|
when(daoFacade.getParamValue(Param.RECIPIENT_BTC_ADDRESS)).thenReturn("default");
|
||||||
|
|
||||||
when(filterManager.getFilter()).thenReturn(filterWithReceivers(List.of()));
|
when(filterManager.getFilter()).thenReturn(filterWithReceivers(List.of()));
|
||||||
assertEquals("default", FeeReceiverSelector.getAddress(daoFacade, filterManager));
|
assertEquals("default", FeeReceiverSelector.getAddress(daoFacade, filterManager));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void testGetAddress_noValidReceivers_filterWithIllFormedList() {
|
||||||
|
when(daoFacade.getParamValue(Param.RECIPIENT_BTC_ADDRESS)).thenReturn("default");
|
||||||
|
|
||||||
when(filterManager.getFilter()).thenReturn(filterWithReceivers(List.of("ill-formed")));
|
when(filterManager.getFilter()).thenReturn(filterWithReceivers(List.of("ill-formed")));
|
||||||
assertEquals("default", FeeReceiverSelector.getAddress(daoFacade, filterManager));
|
assertEquals("default", FeeReceiverSelector.getAddress(daoFacade, filterManager));
|
||||||
|
|
Loading…
Add table
Reference in a new issue