Bugfixes in the triggered instances

* Fill in missing fields in the data provided through the `Trigger`
  context, to include (among others) the `nameField` field
* Map the "object updated" webhook trigger to the correct test
* Rename the "object updated" trigger test template to match the
  pattern of the rest of them

Author: jverce

resources/templates/apex/src/DeletedSObject.trigger.handlebars

@@ -1,8 +1,29 @@
-trigger {{triggerName}} on {{sObjectType}} (after delete) {
-    for ({{sObjectType}} item : Trigger.Old) {
+trigger {{triggerName}} on {{sObjectType}} (before delete) {
+    final List<SObjectField> fields = {{sObjectType}}.getSObjectType()
+        .getDescribe()
+        .fields
+        .getMap()
+        .values();
+    final List<String> fieldNames = new List<String>();
+    for (SObjectField f : fields) {
+        final String fieldName = f.getDescribe().getName();
+        fieldNames.add(fieldName);
+    }
+    final String joinedFieldNames = String.join(fieldNames, ', ');
+    for (SObject item : Trigger.Old) {
+        final String query = String.format(
+            'SELECT {0} FROM {{sObjectType}} WHERE Id = \'\'{1}\'\'',
+            new List<String> {
+                joinedFieldNames,
+                String.valueOf(item.Id)
+            }
+        );
+        final {{sObjectType}} itemFull = ({{sObjectType}}) Database.query(query);
+
         final Map<String, {{sObjectType}}> eventData = new Map<String, {{sObjectType}}>();
-        eventData.put('Old', item);
-        String content = {{webhookCalloutName}}.jsonContent(eventData);
+        eventData.put('Old', itemFull);
+
+        final String content = {{webhookCalloutName}}.jsonContent(eventData);
         {{webhookCalloutName}}.callout('{{endpointUrl}}', content);
     }
 }

resources/templates/apex/src/NewSObject.trigger.handlebars

@@ -1,8 +1,29 @@
 trigger {{triggerName}} on {{sObjectType}} (after insert) {
-    for ({{sObjectType}} item : Trigger.New) {
+    final List<SObjectField> fields = {{sObjectType}}.getSObjectType()
+        .getDescribe()
+        .fields
+        .getMap()
+        .values();
+    final List<String> fieldNames = new List<String>();
+    for (SObjectField f : fields) {
+        final String fieldName = f.getDescribe().getName();
+        fieldNames.add(fieldName);
+    }
+    final String joinedFieldNames = String.join(fieldNames, ', ');
+    for (SObject item : Trigger.New) {
+        final String query = String.format(
+            'SELECT {0} FROM {{sObjectType}} WHERE Id = \'\'{1}\'\'',
+            new List<String> {
+                joinedFieldNames,
+                String.valueOf(item.Id)
+            }
+        );
+        final {{sObjectType}} itemFull = ({{sObjectType}}) Database.query(query);
+
         final Map<String, {{sObjectType}}> eventData = new Map<String, {{sObjectType}}>();
-        eventData.put('New', item);
-        String content = {{webhookCalloutName}}.jsonContent(eventData);
+        eventData.put('New', itemFull);
+
+        final String content = {{webhookCalloutName}}.jsonContent(eventData);
         {{webhookCalloutName}}.callout('{{endpointUrl}}', content);
     }
 }

resources/templates/apex/src/UpdatedSObject.trigger.handlebars

@@ -1,13 +1,34 @@
 trigger {{triggerName}} on {{sObjectType}} (after update) {
-    for ({{sObjectType}} item : Trigger.New) {
+    final List<SObjectField> fields = {{sObjectType}}.getSObjectType()
+        .getDescribe()
+        .fields
+        .getMap()
+        .values();
+    final List<String> fieldNames = new List<String>();
+    for (SObjectField f : fields) {
+        final String fieldName = f.getDescribe().getName();
+        fieldNames.add(fieldName);
+    }
+    final String joinedFieldNames = String.join(fieldNames, ', ');
+    for (SObject item : Trigger.New) {
+        final String query = String.format(
+            'SELECT {0} FROM {{sObjectType}} WHERE Id = \'\'{1}\'\'',
+            new List<String> {
+                joinedFieldNames,
+                String.valueOf(item.Id)
+            }
+        );
+        final {{sObjectType}} itemFull = ({{sObjectType}}) Database.query(query);
+
         final Map<String, {{sObjectType}}> eventData = new Map<String, {{sObjectType}}>();
-        eventData.put('New', item);
+        eventData.put('New', itemFull);
 
         if (Trigger.OldMap != null) {
-            final {{sObjectType}} oldItem = Trigger.OldMap.get(item.Id);
+            final {{sObjectType}} oldItem = ({{sObjectType}}) Trigger.OldMap.get(item.Id);
             eventData.put('Old', oldItem);
         }
-        String content = {{webhookCalloutName}}.jsonContent(eventData);
+
+        final String content = {{webhookCalloutName}}.jsonContent(eventData);
         {{webhookCalloutName}}.callout('{{endpointUrl}}', content);
     }
 }

resources/templates/apex/test/UpdatedObjectTriggerTest.cls.handlebars renamed to resources/templates/apex/test/UpdatedSObjectTriggerTest.cls.handlebars

@@ -13,6 +13,6 @@ public class {{testClassName}} {
         Test.setMock(HttpCalloutMock.class, new {{webhookCalloutMockName}}('{{endpointUrl}}'));
         {{sObjectType}} a = ({{sObjectType}}) {{sObjectFactoryName}}.createSObject('{{sObjectType}}', true);
         insert a;
-        insert a;
+        update a;
     }
 }

src/client.js

@@ -361,7 +361,7 @@ export class SalesforceClient {
   async createWebhookUpdated(opts) {
     this._validateCreateWebhookOpts(opts);
     const triggerTemplate = require('../resources/templates/apex/src/UpdatedSObject.trigger.handlebars');
-    const triggerTestTemplate = require('../resources/templates/apex/test/DeletedSObjectTriggerTest.cls.handlebars');
+    const triggerTestTemplate = require('../resources/templates/apex/test/UpdatedSObjectTriggerTest.cls.handlebars');
     return this._createWebhookWorkflow(
       triggerTemplate,
       triggerTestTemplate,