killbill-memoizeit

Details

diff --git a/beatrix/src/test/java/com/ning/billing/beatrix/util/SubscriptionChecker.java b/beatrix/src/test/java/com/ning/billing/beatrix/util/SubscriptionChecker.java
index ffaa2a1..eea4f02 100644
--- a/beatrix/src/test/java/com/ning/billing/beatrix/util/SubscriptionChecker.java
+++ b/beatrix/src/test/java/com/ning/billing/beatrix/util/SubscriptionChecker.java
@@ -76,9 +76,6 @@ public class SubscriptionChecker {
     }
 
     private List<SubscriptionBaseTransition> getSubscriptionEvents(final SubscriptionBase subscription) {
-        // STEPH_ENT
         return subscription.getAllTransitions();
     }
-
-
 }
diff --git a/entitlement/src/main/java/com/ning/billing/entitlement/api/svcs/DefaultInternalBlockingApi.java b/entitlement/src/main/java/com/ning/billing/entitlement/api/svcs/DefaultInternalBlockingApi.java
index 7b17850..4c34c63 100644
--- a/entitlement/src/main/java/com/ning/billing/entitlement/api/svcs/DefaultInternalBlockingApi.java
+++ b/entitlement/src/main/java/com/ning/billing/entitlement/api/svcs/DefaultInternalBlockingApi.java
@@ -47,7 +47,7 @@ public class DefaultInternalBlockingApi implements BlockingInternalApi {
     public BlockingState getBlockingStateForService(final Blockable overdueable, final String serviceName, final InternalTenantContext context) {
         BlockingState state = dao.getBlockingStateForService(overdueable.getId(), serviceName, context);
         if (state == null) {
-            state = DefaultBlockingState.getClearState(getBlockingStateType(overdueable), clock);
+            state = DefaultBlockingState.getClearState(getBlockingStateType(overdueable), serviceName, clock);
         }
         return state;
     }
diff --git a/util/src/main/java/com/ning/billing/util/svcapi/junction/DefaultBlockingState.java b/util/src/main/java/com/ning/billing/util/svcapi/junction/DefaultBlockingState.java
index 54e7a02..fe7e83f 100644
--- a/util/src/main/java/com/ning/billing/util/svcapi/junction/DefaultBlockingState.java
+++ b/util/src/main/java/com/ning/billing/util/svcapi/junction/DefaultBlockingState.java
@@ -41,10 +41,9 @@ public class DefaultBlockingState extends EntityBase implements BlockingState {
     private final DateTime effectiveDate;
     private final BlockingStateType type;
 
-    public static BlockingState getClearState(final BlockingStateType type, final Clock clock) {
+    public static BlockingState getClearState(final BlockingStateType type, final String serviceName, final Clock clock) {
         if (clearState == null) {
-            // STEPH_ENT should we not always have a service name?
-            clearState = new DefaultBlockingState(null, type, CLEAR_STATE_NAME, null, false, false, false, clock.getUTCNow());
+            clearState = new DefaultBlockingState(null, type, CLEAR_STATE_NAME, serviceName, false, false, false, clock.getUTCNow());
         }
         return clearState;
     }
diff --git a/util/src/test/java/com/ning/billing/util/audit/api/TestDefaultAuditUserApi.java b/util/src/test/java/com/ning/billing/util/audit/api/TestDefaultAuditUserApi.java
index b4e18b0..0861ac9 100644
--- a/util/src/test/java/com/ning/billing/util/audit/api/TestDefaultAuditUserApi.java
+++ b/util/src/test/java/com/ning/billing/util/audit/api/TestDefaultAuditUserApi.java
@@ -27,6 +27,7 @@ import org.testng.annotations.BeforeClass;
 import org.testng.annotations.Test;
 
 import com.ning.billing.ObjectType;
+import com.ning.billing.entitlement.api.SubscriptionBundle;
 import com.ning.billing.subscription.api.timeline.BundleBaseTimeline;
 import com.ning.billing.invoice.api.Invoice;
 import com.ning.billing.invoice.api.InvoiceItem;
@@ -35,6 +36,7 @@ import com.ning.billing.payment.api.Payment;
 import com.ning.billing.payment.api.Refund;
 import com.ning.billing.util.api.AuditLevel;
 import com.ning.billing.util.audit.AuditLog;
+import com.ning.billing.util.audit.AuditLogsForBundles;
 import com.ning.billing.util.audit.AuditLogsForInvoicePayments;
 import com.ning.billing.util.audit.AuditLogsForInvoices;
 import com.ning.billing.util.audit.AuditLogsForPayments;
@@ -59,7 +61,6 @@ public class TestDefaultAuditUserApi extends AuditLogsTestBase {
         auditLogs = ImmutableList.<AuditLog>of(createAuditLog(), createAuditLog(), createAuditLog(), createAuditLog());
         objectIds = ImmutableList.<UUID>of(UUID.randomUUID(), UUID.randomUUID(), UUID.randomUUID(), UUID.randomUUID());
 
-//        super.setup();
 
         for (final TableName tableName : TableName.values()) {
             for (final UUID objectId : objectIds) {
@@ -72,17 +73,16 @@ public class TestDefaultAuditUserApi extends AuditLogsTestBase {
 
     @Test(groups = "fast")
     public void testForBundles() throws Exception {
-        final List<BundleBaseTimeline> bundles = new ArrayList<BundleBaseTimeline>();
+        final List<SubscriptionBundle> bundles = new ArrayList<SubscriptionBundle>();
         for (final UUID objectId : objectIds) {
-            final BundleBaseTimeline entity = Mockito.mock(BundleBaseTimeline.class);
+            final SubscriptionBundle entity = Mockito.mock(SubscriptionBundle.class);
             Mockito.when(entity.getId()).thenReturn(objectId);
             bundles.add(entity);
         }
 
         for (final AuditLevel level : AuditLevel.values()) {
-            // STEPH_ENT
-            //final AuditLogsForBundles auditLogsForBundles = auditUserApi.getAuditLogsForBundles(bundles, level, callContext);
-            //verifyAuditLogs(auditLogsForBundles.getBundlesAuditLogs(), level);
+            final AuditLogsForBundles auditLogsForBundles = auditUserApi.getAuditLogsForBundles(bundles, level, callContext);
+            verifyAuditLogs(auditLogsForBundles.getBundlesAuditLogs(), level);
         }
     }