In Apex Test Execution, email notifications and DML operations in the same transaction can cause System.CalloutException if callouts are attempted

Apex , Workflow

Last updated 2017-04-14 ·Reference W-3171280 ·Reported By 157 users

Fixed - Winter '17 Patch 12.0

Summary
In apex test implementing mock callout, if a transaction performs a DML operation and an email notification (e.g. email alert) is involved, the following exception will be thrown on attempting an Apex callout:

System.CalloutException: "You have uncommitted work pending. Please commit or rollback before calling out"

Repro
1) Create the following Apex class:

public class CalloutService {
@future (callout=true)
public static void docall() {
HttpRequest req = new HttpRequest();
req.setMethod('GET');
req.setEndpoint('http://requestb.in/x2lw0xx2');
Http http = new Http();
HTTPResponse res = http.send(req);
}
}

2) Create the following test class:

@isTest
public class CalloutTest{

public class CalloutServiceMock implements HttpCalloutMock {
HttpRequest originalRequest;
HttpRequest getRequest() {
return originalRequest;
}
public HTTPResponse respond(HTTPRequest req) {
HTTPResponse response = new Httpresponse();
response.setBody('OK');
response.setStatusCode(200);
return response;
}
}
static testMethod void myMethod() {
Account Acc = new Account(name = 'SFDC');
insert Acc;
Test.startTest();
Test.setMock(HttpCalloutMock.class, new CalloutServiceMock());
CalloutService.docall();
Test.stopTest();
}
}

3) Create an email alert. Email template or sender fields are not relevant.
4) Create a workflow rule to fire on account with criteria: Name contains “SFDC”. Have WF rule fire the email alert action created in step 3. Activate WF rule.

5) Open CalloutTest Apex class, and hit Run Test. The test will fail:

“System.CalloutException: You have uncommitted work pending. Please commit or rollback before calling out”.

Notes:
a) If the workflow rule is deactivated, the test will pass.
b) If instead of an email alert, the workflow rule performs a field update that changes record ownership and notifies assignee, the issue will also be reproducible.

Workaround
a) Separate DML and email notification in different transactions

OR

b) Deactivate email alerts/field update notifications

Reported By (157)

Michele McMahon Marcel Santos Vijay AdvApex Trey Ward Michele McMahon Simon Parker Nancy Lane Donald Martin Brenda Finn Pruthvi Raj Doug Ayers Tech Cobone Carl Huseby Fosli Pedro Espada Manuel Candace Fovel Mike Jones Doug Kells Jonathan Romley Harald Proksch Vijay Swamidass Mateusz Przywara Eric Alexander Amit Jain GOCODE Ltda Mohandaas Rangaswamy Dave Buch Eric Kintzer Jonathan Broquist Batch One Cloud Nerd Pankaj Mehra Jared Neufer Derek Hansen Taylor Kingsbury Guy Clairbois Meredith Ready Fabian Manzano Mark Schelluch Adam Gallinat Ernesto Rodriguez William Garcia Jose Rodriguez Daymel Marrero Rene Raul Garcia Marcel Gomez Osmara Ramos Jose Daniel Inclan Michel Carrillo Rodney Farach Bin Ye David Griffiths Michal Krawczyk Nathen Drees Brian Chong Crystal Rochlitz James Allen Dylan Gerow Thomas Gagne Chris Dundon Eoin O'Neill Richa Midha Tim Purdie John Lewis Daniel Tutila Jim Rae Marc Smale Gennadiy Pervukhin Jeff Battles Andrea Borsani ADMIN ATLANTIC Peter Yao Tre'Von Mckay Austin Gelber Sebastien Colladon Kyle Gossman Diego Silva Eugene Gervacio Ben Naylor Codrin Morhan Kalyan Lanka S Morgan Robertson Michael Smith Yakup Kaygusuz System Administrator Bradley Pierce Rob Craven Joel Anderson Miranda Ragland James Snyder Thomas Thankachan Katharine Louie Chad Meyer Shiran Fernando Anthony Spicciati Hayk Avagyan Pankaj D Pankaj Dhapke Neeraj Soni Neeraj Soni Erin Lamb Nathan Banas John Sung Patrick Watkins Philip Clark Eugene Orr Martha Seneta Avanxo LTE Peter Thomas Jobright Peter Percolator Consulting Adam Ith Pablo Venturino Michal Lachowski Maciej Gunia Priya Marupudi Deborah Orth Aditi Deshpande Ramesh Ale Nathan Pilkington David Inniss Sonia Cook-Broen Rohit Macherla Chris Huang Zac Owens David Sokoloff Or Weissler Elias Dawson Danny Smart Brian Cline Mohit Pant CRM-plus Jur van Oerle Kevin Bruss Chad Henson Huy Nguyen Allan Hotchkiss Brian Siebert Support Innoveer Aidan Harding Dorottyad Dvorak Michael Slawnik David Limer Alex Tennant Rajkumar Venkatachalam Bohdan Dovhan Julfy Patlatus Bohdan Dovhan Roman Skurchak Paul Rice Travis Paxton Julian Virguez Ian Sidle Caitlin Pfeiffer Nathen Drees Ross Talbot Eli Kloswick Andrew Monshizadeh

Is it Fixed?

AP0 AP3 AP4 AP5 AP6 AP7 AP8 AP9 AP10 AP11 AP12 AP13 AP14 AP15 AP16 AP17 AP18 AP19 AP20 AP21 AP22 AP28 CS1 CS2 CS3 CS4 CS5 CS6 CS7 CS8 CS9 CS10 CS109 CS108 CS107 CS106 CS105 CS102 CS101 CS100 CS115 CS110 CS117 CS114 CS113 CS112 CS111 CS11 CS116 CS12 CS129 CS128 CS125 CS124 CS123 CS137 CS138 CS13 CS133 CS132 CS14 CS148 CS142 CS15 CS152 CS151 CS16 CS17 CS18 CS19 CS20 CS21 CS22 CS23 CS24 CS25 CS26 CS27 CS28 CS29 CS30 CS31 CS32 CS33 CS34 CS35 CS36 CS37 CS40 CS41 CS42 CS43 CS44 CS45 CS47 CS50 CS51 CS52 CS53 CS54 CS57 CS58 CS59 CS60 CS61 CS62 CS63 CS64 CS65 CS66 CS67 CS68 CS69 CS70 CS71 CS72 CS73 CS74 CS75 CS76 CS77 CS78 CS79 CS80 CS81 CS82 CS83 CS84 CS85 CS86 CS87 CS88 CS89 CS90 CS91 CS92 CS93 CS94 CS95 CS96 CS97 CS98 CS99 EU7 EU8 EU10 EU12 EU13 EU14 EU15 EU16 EU17 EU18 EU19 EU25 EU26 EU27 EU28 EU29 EU30 EU31 EU32 NA104 NA107 NA109 NA100 NA101 NA103 NA102 NA105 NA119 NA116 NA110 NA118 NA112 NA111 NA115 NA114 NA113 NA117 NA121 NA124 NA122 NA120 NA126 NA129 NA130 NA134 NA136 NA135 NA132 NA131 NA146 NA142 NA141 NA154 NA155 NA171 NA174 NA196 NA21 NA32 NA37 NA39 NA44 NA45 NA46 NA47 NA49 NA52 NA53 NA54 NA57 NA58 NA59 NA60 NA61 NA62 NA64 NA65 NA66 NA67 NA68 NA69 NA70 NA71 NA72 NA73 NA74 NA75 NA76 NA77 NA79 NA80 NA81 NA82 NA83 NA84 NA85 NA86 NA87 NA88 NA89 NA90 NA91 NA92 NA93 NA94 NA95 NA96 NA97 NA98 NA99 UM1 UM2 UM3 UM4 UM5

Any unreleased services, features, statuses, or dates referenced in this or other public statements are not currently available and may not be delivered on time or at all. Customers who purchase our services should make their purchase decisions based upon features that are currently available.