Some classes/triggers added after enabling Chatter Answers don't have code coverage
Trailblazer Community

Some classes/triggers added after enabling Chatter Answers don't have code coverage

Chatter

Last updated 2015-09-03 ·Reference W-2177244 ·Reported By 18 users

Fixed - Spring '15

Summary
Some classes/triggers, which are added when Chatter Answers are enabled, do not have test cases. This leads to lower test coverage.

Repro
- Enable Chatter Answers
- Some triggers/classes are added
- Not all of those have a corresponding test class

Workaround
You can add following test cases to get the code coverage for missing test class

1. If you ARE NOT USING Auth provider based registration for Chatter answers, replace ChatterAnswersAuthProviderRegistration with this code :

global class ChatterAnswersAuthProviderRegistration implements Auth.RegistrationHandler{
global User createUser(Id portalId, Auth.UserData data){
return null;
}
global void updateUser(Id userId, Id portalId, Auth.UserData data){
}
}

Create the following test class
@isTest
private class ChatterAnswersAuthProviderRegTest {
static testMethod void validateCreateUpdateUser() {
Auth.UserData userData = new Auth.UserData('testId', 'testFirst', 'testLast',
'testFirst testLast', 'no-reply@salesforce.com', null, 'testuserlong', 'en_US', 'facebook',
null, new Map<String, String>{'language' => 'en_US'});
ChatterAnswersAuthProviderRegistration reg = new ChatterAnswersAuthProviderRegistration();
User newUser = reg.createUser(null, userData);
System.assert(newUser == null);


Auth.UserData updateUserData = new Auth.UserData('testId', 'updatedFirst', 'updatedLast',
'updatedFirst updatedLast', 'no-reply@new.salesforce.com', null, 'testuserlong', 'en_US', 'facebook',
null, new Map<String, String>{'language' => 'en_US'});
reg.updateUser(newUser.Id, null, updateUserData);
}
}

2. If you ARE USING Auth provider based registration for Chatter answers, replace your existing apex class named ChatterAnswersAuthProviderRegistration

global class ChatterAnswersAuthProviderRegistration implements Auth.RegistrationHandler{
global User createUser(Id portalId, Auth.UserData data){
//If your org does not use Chatter Answers Auth provider based registration,
// comment out the code of this method and just return null from the method.
Savepoint sp = Database.setSavepoint();
String accountId = null;
try {
String adminId = Site.getAdminId();
if (adminId ==null && Test.isRunningTest()) {
Profile[] p = [SELECT Id FROM Profile WHERE Name = 'System Administrator'];
User[] adminUser = [SELECT Id, Firstname, Lastname FROM User WHERE IsActive = true and ProfileId =: p[0].Id LIMIT 1];
adminId = adminUser[0].Id;
}
accountId = new ChatterAnswersRegistration().createAccount(data.firstname, data.lastname, adminId);
} catch (Exception e) {
Database.rollback(sp);
return null;
}
User u = new User();
u.FirstName = data.firstname;
u.LastName = data.lastname;
u.Username = accountId + '@facebook.com';
u.Email = data.email;
u.CommunityNickname = data.email.substring(0, data.email.indexOf('@')) + System.currentTimeMillis();
u.UserPermissionsChatterAnswersUser = true;
//If using this registration handler with Salesforce Community registration then specify the profileid to associated with the new user.
//This profile is used only for Salesforce Community site registration and not for standalone Force.com site registration.
u.profileId = '';
Site.setPortalUserAsAuthProvider(u, accountId);
return u;
}

global void updateUser(Id userId, Id portalId, Auth.UserData data){
//If your org does not use Chatter Answers Auth provider based registration,
// comment out the code of this method and just do nothing inside the method.
Savepoint sp = Database.setSavepoint();
User u = [SELECT Contact.email, Contact.firstName, Contact.lastName FROM User WHERE id=:userId];
u.Contact.email = data.email;
u.Contact.firstName = data.firstName;
u.Contact.lastName = data.lastName;
u.firstName = data.firstName;
u.lastName = data.lastName;
u.email = data.email;
try {
update u;
update u.Contact;
} catch (Exception e) {
Database.rollback(sp);
}
}
}


Create the following test class :
@isTest
private class ChatterAnswersAuthProviderRegTest {
static testMethod void validateCreateUpdateUser() {
User thisUser = [ select Id from User where Id = :UserInfo.getUserId() ];
System.runAs ( thisUser ) {
Auth.UserData userData = new Auth.UserData('testId', 'testFirst', 'testLast',
'testFirst testLast', 'no-reply@salesforce.com', null, 'testuserlong', 'en_US', 'facebook',
null, new Map<String, String>{'language' => 'en_US'});
ChatterAnswersAuthProviderRegistration reg = new ChatterAnswersAuthProviderRegistration();
User newUser = reg.createUser(null, userData);
System.assert(newUser != null, 'A new user should have been created');
System.assertEquals(newUser.Firstname, 'testFirst', 'First name should have been same');
System.assertEquals(newUser.Lastname, 'testLast', 'Last name should have been same');
System.assertEquals(newUser.Email, 'no-reply@salesforce.com', 'Email should have been same');

Contact c = new Contact();
c.AccountId = (newUser.Username.split('@'))[0];
c.LastName = 'contactLast';
insert(c);

newUser.Alias = 'firstusr';
newUser.TimeZoneSidKey = 'America/Los_Angeles';
newUser.LocaleSidKey = 'en_US';
newUser.EmailEncodingKey = 'UTF-8';
newUser.LanguageLocaleKey = 'en_US';
newUser.ContactId = c.Id;

insert(newUser);


Auth.UserData updateUserData = new Auth.UserData('testId', 'updatedFirst', 'updatedLast',
'updatedFirst updatedLast', 'no-reply@new.salesforce.com', null, 'testuserlong', 'en_US', 'facebook',
null, new Map<String, String>{'language' => 'en_US'});
reg.updateUser(newUser.Id, null, updateUserData);

User dbUser = [SELECT Id, Firstname, Lastname, Email FROM User WHERE Id = :newUser.Id];
System.assertEquals(dbUser.Firstname, 'updatedFirst', 'First name should have been updated');
System.assertEquals(dbUser.Lastname, 'updatedLast', 'Last name should have been updated');
System.assertEquals(dbUser.Email, 'no-reply@new.salesforce.com', 'Email should have been updated');
}
}
}

Make sure you put the right profile id (i.e the one that you use for actual Chatter answers user registration) in the ChatterAnswersAuthProviderRegistration.createUser() (line #28) method before you run the test.

3. To get code coverage for "chatter_answers_question_escalation_to_case_trigger" trigger :

Notice that the uncovered code coverage is from the Exception block of the trigger. This is a pretty hard to test code path since the exception has to be thrown from inside the trigger code.
This will happen only by forcing a metadata change from the test code which isn't really worth it.

The code in the exception block sends email in case of an error during case creation.
There are two options to get code coverage in this scenario :
1. If you don't want to send out emails, you can simply comment out the logic.
2. If you want the emails, move the code inside the exception logic to a helper class. And you can easily write a test for the helper method.
So the trigger would look like this :

trigger chatter_answers_question_escalation_to_case_trigger on Question (after update) {
for (Question q: Trigger.new) {
try {
if (q.Priority == 'high' && (q.Cases == null || q.Cases.size() == 0) && Trigger.oldMap.get(q.id).Priority != 'high') {
q = [select Id, Title, Body, CommunityId, createdById, createdBy.AccountId, createdBy.ContactId from Question where Id = :q.Id];
Case newCase = new Case(Origin='Chatter Answers', OwnerId=q.CreatedById, QuestionId=q.Id, CommunityId=q.CommunityId, Subject=q.Title, Description = (q.Body == null? null: q.Body.stripHtmlTags()), AccountId=q.CreatedBy.AccountId, ContactId=q.CreatedBy.ContactId);
insert newCase;
}
} catch (Exception e) {
TriggerMailer.sendEmail(Site.getAdminEmail(), q.Id, e.getMessage(), e.getStacktraceString());
}
}
}

public class TriggerMailer {
public static void sendEmail(String toAddress, String questionId, String errorMessage, String stackTrace) {
String subjectText = 'Case Escalation exception in site ' + Site.getName();
String bodyText = 'Case Escalation on Question having ID: ' + questionId + ' has failed with the following message: ' + errorMessage +
'\n\nStacktrace: ' + stackTrace;

Messaging.SingleEmailMessage mail = new Messaging.SingleEmailMessage();
String[] toAddresses = new String[] { toAddress };
mail.setReplyTo('no-reply@salesforce.com');
mail.setSenderDisplayName('Salesforce Chatter Answers User');

// The default sender is the portal user causing this trigger to run, to change this, set an organization-wide address for
// the portal user profile, and set the ID in the following line.
// mail.setOrgWideEmailAddressId(orgWideEmailAddressId);
mail.setToAddresses(toAddresses);
mail.setSubject(subjectText);
mail.setPlainTextBody(bodyText);
Messaging.sendEmail(new Messaging.SingleEmailMessage[] { mail });
}
}

@isTest
private class TriggerMailerTest {
static testMethod void validateSendEmail() {
TriggerMailer.sendEmail('no-reply@salesforce.com', '906000000000000', 'Error message', 'Error stack trace');
}
}

Is it Fixed?

AP0 AP3 AP4 AP5 AP6 AP7 AP8 AP9 AP10 AP11 AP12 AP13 AP14 AP15 AP16 AP17 AP18 AP19 AP20 AP21 AP22 AP24 AP25 AP26 AP27 AP28 AUS1 AUS11 AUS23 AUS26S AUS24S AUS22S AUS2S AUS27 AUS25 AUS3 AUS43 AUS4S AUS5 AUS6S AUS7 AUS9 BRA1 BRA2S BRA4S CAN1 CAN17 CAN11 CAN2S CAN4S CAN6S CAN8S CS1 CS2 CS4 CS5 CS6 CS7 CS8 CS9 CS10 CS109 CS108 CS107 CS106 CS105 CS102 CS101 CS100 CS115 CS119 CS110 CS117 CS114 CS113 CS112 CS111 CS11 CS116 CS123 CS122 CS121 CS126 CS127 CS129 CS128 CS125 CS124 CS137 CS138 CS133 CS132 CS14 CS148 CS142 CS159 CS152 CS151 CS15 CS162 CS167 CS166 CS16 CS169 CS165 CS160 CS173 CS17 CS172 CS170 CS171 CS174 CS18 CS189 CS194 CS192 CS193 CS190 CS191 CS199 CS197 CS19 CS198 CS196 CS195 CS20 CS209 CS200 CS202 CS201 CS203 CS21 CS219 CS218 CS217 CS216 CS215 CS214 CS213 CS212 CS211 CS210 CS22 CS220 CS23 CS234 CS24 CS25 CS26 CS27 CS28 CS29 CS31 CS32 CS33 CS34 CS35 CS36 CS37 CS40 CS41 CS42 CS43 CS44 CS45 CS47 CS53 CS57 CS58 CS59 CS60 CS61 CS62 CS63 CS64 CS65 CS66 CS67 CS68 CS69 CS72 CS73 CS74 CS75 CS76 CS77 CS78 CS79 CS80 CS81 CS84 CS86 CS87 CS88 CS89 CS90 CS91 CS92 CS94 CS95 CS96 CS97 CS98 CS99 DEU1 DEU2S DEU4S EU16 EU17 EU18 EU19 EU25 EU26 EU27 EU28 EU29 EU30 EU31 EU32 EU33 EU34 EU35 EU36 EU37 EU38 EU39 EU40 EU41 EU42 EU43 EU44 EU45 EU46 EU47 EU48 FRA1 FRA2S FRA4S IND1 IND18S IND16S IND15 IND13 IND11 IND23 IND2S IND3S IND4S IND5 IND6S IND7 IND9 JPN1 JPN12S JPN10S JPN2S JPN3 JPN4S JPN5 JPN6S JPN7 JPN8S NA104 NA107 NA109 NA100 NA101 NA103 NA102 NA105 NA119 NA116 NA110 NA118 NA112 NA111 NA115 NA114 NA113 NA117 NA125 NA124 NA122 NA120 NA126 NA127 NA123 NA129 NA121 NA128 NA138 NA134 NA133 NA136 NA135 NA132 NA131 NA130 NA137 NA139 NA140 NA142 NA141 NA149 NA146 NA147 NA148 NA154 NA158 NA159 NA153 NA151 NA155 NA152 NA150 NA156 NA161 NA163 NA167 NA160 NA166 NA165 NA169 NA164 NA168 NA162 NA172 NA170 NA174 NA171 NA173 NA196 NA207 NA204 NA206 NA208 NA209 NA21 NA218 NA214 NA217 NA215 NA211 NA212 NA213 NA210 NA65 NA70 NA71 NA75 NA80 NA81 NA82 NA83 NA84 NA85 NA89 NA90 NA91 NA92 NA93 NA94 NA95 NA96 NA97 NA98 NA99 SGP2S SGP4S UM1 UM2 UM3 UM4 UM5 UM6 UM7 UM8 UM9 USA1 USA18S USA16S USA10S USA2S USA3S USA7 USA8S

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.