[SalesForce] Error using Database.Savepoint in Apex Test class

I have a test class with the following function in it:

@isTest
private static void testDequeue() {
    ... // a LOT of setup code here

    Test.startTest();
    Savepoint sp = Database.setSavepoint();
    String query = ...; // some query

    // CODE PART A
    JobA job = new JobA(query, false);
    Database.executeBatch(job, 10);
    List<ObjectA> records = [ ... ];
    System.assert(records.size() > 0);
    ... // a LOT more assertions
    Database.rollback(sp);

    // CODE PART B
    JobB job = new JobB(query);
    Database.executeBatch(job, 10);
    List<ObjectA> records = [ ... ];
    System.assert(records.size() > 0);
    ... // the same assertions as before
    Test.stopTest();
}

I'm testing two Jobs, JobA and JobB which both do the same thing. Both require the same setup code, which is why it is easier for me to test them in the same function. So, I'm using savepoints to make JobA do something, do some tests, then undo what JobA did, make JobB do the same thing, and then run the same tests.

The issue is, this code throws an AssertionError for the first assert statement:

System.AssertException: Assertion Failed

I've confirmed the Savepoint code is the reason, because when I remove it (as well as the PART B code), the error goes.

What's even more annoying is that, I've a lot of scenarios like this, and this code works for some, and fails for others. I'm not able to figure out why that happens. Is it a bug? Or is there something about savepoints I should know?

Best Answer

This wont work for a few reasons

Without putting your batch invocations in separate startTest/stopTest blocks, you have no guarantee they will finish one after the other (they are two independent asynchronous transactions)
Savepoint works per transaction and you can't rollback changes across transaction. In other words, each batch execute invocation can only role its own transaction back. There is an excellent post on this here: Rollback whole Batch

You should just make an @setup method and break your 2 tests into 2 methods with 2 startTest/stopTest blocks.

Related Solutions

[SalesForce] Issues with calling 2 setTest methods in apex test class

As it turns out, I overlooked something very simple. When calling the setTest() methods you need to make sure that variables maintain their state once set and therefore, because I was not declaring my variables as static, they were not maintaining their state and thus the error was being thrown.

Here's the modified controller code:

public class CanNewsChatterFeedController {
    //declare necessary static variables
    static ConnectApi.ChatterGroupPage chatterGrpPg;
    static String theGroupId;

    //empty constructor
    public CanNewsChatterFeedController(){}

    // get first page of news feed
    public static ConnectApi.FeedItemPage getNewsFeed() {
        //get the proper Chatter Group Page
        chatterGrpPg = getCanNewsChatterGrpPg();
       //get the groupId
       theGroupId = chatterGrpPg.groups[0].id;
       //use the groupId to get the feedItems
       ConnectApi.FeedItemPage feedPage = ConnectApi.ChatterFeeds.getFeedItemsFromFeed(null, ConnectApi.FeedType.Record, theGroupId);
       return feedPage;
    }

    //The Id of the ChatterGroup we want to display will be different in the sandbox from what it is in production
    //(production id: 0F9D0000000UGqS in sandbox id:0F9c0000000Chu1), in order to get the feed items we want, 
    //we have to search for the ChatterGroup by name. Since Group Name must be unique,
    //the searchGroups method should only return one ChatterGroupPage with one ChatterGroup
    public static ConnectApi.ChatterGroupPage getCanNewsChatterGrpPg(){
        return ConnectApi.ChatterGroups.searchGroups(null, 'Canada - Corby Scoreboard', null, 1);
    }    

    // build list of wrapped feed items for display in VisualForce
    public static List<FeedItemInfo> getNewsFeedForDisplay() {
        ConnectApi.FeedItemPage feed = getNewsFeed();      
        List<FeedItemInfo> result = new List<FeedItemInfo>();
        for (ConnectApi.FeedItem item : feed.items) {
            result.add(new FeedItemInfo(item));
        }

        return result;
    }


    public PageReference refreshFeed(){
        return null;
    }
}

I hope this helps someone else experiencing the same issue.

[SalesForce] Platform Event Testing: Unexpected Exception: Error processing messages

Platform events in test class are unitested differently.

Instead of EventBus.publish(exev); you have to use Test.getEventBus().deliver();

Test.startTest();
// Create test events
// ...
// Publish test events with EventBus.publish()
// ...
// Deliver test events
Test.getEventBus().deliver();
// Perform validation 
// ...
Test.stopTest();

Make sure you put Test.getEventBus().deliver(); between start and stop test.

https://releasenotes.docs.salesforce.com/en-us/summer18/release-notes/rn_platform_events_apex_enhancements_deliver_events.htm This method is only Available after Summer 18. or API v43

For Version 42 and Below:

Create a blank Trigger for your Platform Event Object, SRC:.Unable to unit test code that publishes platform events

Related Topic