Why is this trigger creating only 200 records?

Question

This trigger creates an entitlement for every account that meets certain criteria.

I have read the triggers process records in chunks of 200 but I am not understanding why the trigger below only creates 200 entitlements records even when I am inserting 250 accounts.

It seems like only the first chunk is processed by the trigger. I don't see any errors in the logs. Thanks in advance.

This is part of the code.

UPDATE:I have added the 2 lines of code at the beginning that are causing the issue.

 if(util1.accountEntitlementAssignment) return;
     util1.accountEntitlementAssignment=true;

for(Entitlement ent: [Select Id,AccountId from Entitlement Where Status='Active']){
    AccsWithEnt.add(ent.AccountId);
}


for (Account acc : trigger.new) {
    if(acc.test_Subscription_Type__c=='testEnterprise'){
        //Only create entitlements for accounts that don't have one
        if(!AccsWithEnt.contains(acc.id)){
            createentitlement.add(new Entitlement (
                Name='Enterprise Entitlement',
                AccountId=acc.Id,
                StartDate=todayDate,
                EndDate=todayDate.addYears(5),
                SlaProcessId=slaMap.get('Enterprise Entitlement').Id)); 
        }           
    }
}

Insert createentitlement;    

This is how I am testing it

List<Account> accs=new List<Account>();

for(Integer i=0;i<250;i++){

    accs.add(new Account(
        name='test'+i,
        test_Subscription_Type__c='testEnterprise'));           
    }   

    insert accs;
}

Answer 1

Your trigger isn't bulk-safe, although that has little to do with the problem at hand. Your trigger should certainly create 250 brand new entitlements; I think you might be missing something. It's true that your trigger will be batched into sets of 200 (so, 200 on the first loop, plus 50 on the second), but all 250 entitlements should be created.

Make sure you're not just reading the logs wrong. You should query the entitlements back afterwards and assert that there are indeed 250 entitlements:

Entitlement[] entitlements = [SELECT Id FROM Entitlement WHERE AccountId IN :accs];
System.assertEquals(250, entitlements.size());

Here's a better way to approach your trigger:

Entitlement[] newEntitlements = new Entitlement[0];
Set<Id> newAcctIds = Trigger.newMap.keySet().clone();
for(Entitlement record:[SELECT AccountId FROM Entitlement WHERE Status='Active' AND AccountId IN :Trigger.new])
    newAcctIds.remove(record.AccountId);
for(Id recordId:newAcctIds)
    newEntitlements.add(
        new Entitlement(
            Name='Enterprise Entitlement', 
            AccountId=recordId, 
            StartDate=Date.Today(), 
            Date.Today().addYears(5), 
            SLAProcessId=slaMap.get('Enterprise Entitlement').Id));
insert newEntitlements;