fix: don't add custom key if it's null

rollbar · Jun 5, 2020 · 42cbd4d · 42cbd4d
1 parent 8da5e23
commit 42cbd4d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/force-app/main/default/classes/DataBuilder.cls b/force-app/main/default/classes/DataBuilder.cls
@@ -81,7 +81,10 @@ public with sharing class DataBuilder {
         structure.put('environment', environment);
         structure.put('framework', 'apex');
         structure.put('body', body);
-        structure.put('custom', custom);
+
+        if (custom != null) {
+            structure.put('custom', custom);
+        }
 
         return structure;
     }

diff --git a/force-app/main/default/tests/DataBuilderTest.cls b/force-app/main/default/tests/DataBuilderTest.cls
@@ -11,6 +11,8 @@ public class DataBuilderTest
 
         Map<String, Object> data = (Map<String, Object>)result.get('data');
 
+        System.assertEquals(data.containsKey('custom'), false);
+
         Map<String, Object> notifierMap = (Map<String, Object>)data.get('notifier');
 
         System.assertEquals(Notifier.NAME, (String)notifierMap.get('name'));
@@ -87,6 +89,8 @@ public class DataBuilderTest
 
             Map<String, Object> data = (Map<String, Object>)result.get('data');
 
+            System.assertEquals(data.containsKey('custom'), false);
+
             Map<String, Object> notifierMap = (Map<String, Object>)data.get('notifier');
             System.assertEquals(Notifier.NAME, (String)notifierMap.get('name'));
             System.assertEquals(Notifier.VERSION, (String)notifierMap.get('version'));