diff --git a/invoice/src/main/java/org/killbill/billing/invoice/dao/InvoiceDaoHelper.java b/invoice/src/main/java/org/killbill/billing/invoice/dao/InvoiceDaoHelper.java
index e80ffa1..0680144 100644
--- a/invoice/src/main/java/org/killbill/billing/invoice/dao/InvoiceDaoHelper.java
+++ b/invoice/src/main/java/org/killbill/billing/invoice/dao/InvoiceDaoHelper.java
@@ -81,10 +81,6 @@ public class InvoiceDaoHelper {
final InternalTenantContext context) throws InvoiceApiException {
// Populate the missing amounts for individual items, if needed
final Map<UUID, BigDecimal> outputItemIdsWithAmounts = new HashMap<UUID, BigDecimal>();
- if (invoiceItemIdsWithNullAmounts.size() == 0) {
- return outputItemIdsWithAmounts;
- }
-
// Retrieve invoice before the Refund
final InvoiceModelDao invoice = entitySqlDaoWrapperFactory.become(InvoiceSqlDao.class).getById(invoiceId, context);
if (invoice != null) {
diff --git a/invoice/src/test/java/org/killbill/billing/invoice/api/invoice/TestDefaultInvoicePaymentApi.java b/invoice/src/test/java/org/killbill/billing/invoice/api/invoice/TestDefaultInvoicePaymentApi.java
index 0af1018..728fa97 100644
--- a/invoice/src/test/java/org/killbill/billing/invoice/api/invoice/TestDefaultInvoicePaymentApi.java
+++ b/invoice/src/test/java/org/killbill/billing/invoice/api/invoice/TestDefaultInvoicePaymentApi.java
@@ -51,15 +51,6 @@ public class TestDefaultInvoicePaymentApi extends InvoiceTestSuiteWithEmbeddedDB
verifyRefund(THIRTY, BigDecimal.TEN, BigDecimal.TEN, false, ImmutableMap.<UUID, BigDecimal>of());
}
- @Test(groups = "slow")
- public void testFullRefundWithInvoiceAdjustment() throws Exception {
- verifyRefund(THIRTY, THIRTY, BigDecimal.ZERO, true, ImmutableMap.<UUID, BigDecimal>of());
- }
-
- @Test(groups = "slow")
- public void testPartialRefundWithInvoiceAdjustment() throws Exception {
- verifyRefund(THIRTY, BigDecimal.TEN, BigDecimal.ZERO, true, ImmutableMap.<UUID, BigDecimal>of());
- }
@Test(groups = "slow")
public void testFullRefundWithBothInvoiceItemAdjustments() throws Exception {
diff --git a/invoice/src/test/java/org/killbill/billing/invoice/dao/TestInvoiceDao.java b/invoice/src/test/java/org/killbill/billing/invoice/dao/TestInvoiceDao.java
index 6ceaaa2..1779b19 100644
--- a/invoice/src/test/java/org/killbill/billing/invoice/dao/TestInvoiceDao.java
+++ b/invoice/src/test/java/org/killbill/billing/invoice/dao/TestInvoiceDao.java
@@ -598,11 +598,6 @@ public class TestInvoiceDao extends InvoiceTestSuiteWithEmbeddedDB {
testAccountBalanceWithRefundInternal(false);
}
- @Test(groups = "slow")
- public void testAccountBalanceWithRefundAndAdj() throws InvoiceApiException, EntityPersistenceException {
- testAccountBalanceWithRefundInternal(true);
- }
-
private void testAccountBalanceWithRefundInternal(final boolean withAdjustment) throws InvoiceApiException, EntityPersistenceException {
final UUID accountId = account.getId();
@@ -735,20 +730,6 @@ public class TestInvoiceDao extends InvoiceTestSuiteWithEmbeddedDB {
testAccountBalanceWithRefundAndCBAInternal(true, refundAmount, expectedBalance);
}
- @Test(groups = "slow")
- public void testAccountBalanceWithLargeRefundAndCBANoAdj() throws InvoiceApiException, EntityPersistenceException {
- final BigDecimal refundAmount = new BigDecimal("20.00");
- final BigDecimal expectedBalance = new BigDecimal("10.00");
- testAccountBalanceWithRefundAndCBAInternal(false, refundAmount, expectedBalance);
- }
-
- @Test(groups = "slow")
- public void testAccountBalanceWithLargeRefundAndCBAWithAdj() throws InvoiceApiException, EntityPersistenceException {
- final BigDecimal refundAmount = new BigDecimal("20.00");
- final BigDecimal expectedBalance = new BigDecimal("-10.00");
- testAccountBalanceWithRefundAndCBAInternal(true, refundAmount, expectedBalance);
- }
-
private void testAccountBalanceWithRefundAndCBAInternal(final boolean withAdjustment, final BigDecimal refundAmount, final BigDecimal expectedFinalBalance) throws InvoiceApiException, EntityPersistenceException {
final UUID accountId = account.getId();
final UUID bundleId = UUID.randomUUID();
@@ -804,7 +785,7 @@ public class TestInvoiceDao extends InvoiceTestSuiteWithEmbeddedDB {
assertEquals(cba.compareTo(new BigDecimal("10.00")), 0);
// PARTIAL REFUND on the payment
- invoiceDao.createRefund(paymentId, refundAmount, withAdjustment, ImmutableMap.<UUID, BigDecimal>of(), UUID.randomUUID().toString(), context);
+ invoiceDao.createRefund(paymentId, refundAmount, withAdjustment, ImmutableMap.<UUID, BigDecimal>of(item2Replace.getId(), refundAmount), UUID.randomUUID().toString(), context);
balance = invoiceDao.getAccountBalance(accountId, context);
assertEquals(balance.compareTo(expectedFinalBalance), 0);