paulh
/
salesforce-composite-envelope-builder
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: remove duplicate JSON builder test from main class 

- Removed testJSONBuilder from DocusignCompositeEnvelopeBuilderTest
- JSON building logic tests now handled by DocusignEnvelopeRequestHandlerTest (10 comprehensive tests)
- Fixes deployment error: method buildCompositeEnvelopeJSON no longer exists in main class
- Eliminates code duplication between test classes
This commit is contained in:
Paul Huliganga 2026-02-25 09:39:18 -05:00
parent 1ff6bb7d76
commit b00a930e8d
1 changed files with 0 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

31
composite-envelope-builder/force-app/main/default/classes/DocusignCompositeEnvelopeBuilderTest.cls
View File

@ -278,35 +278,4 @@ private class DocusignCompositeEnvelopeBuilderTest {
System.assertNotEquals(null, results[0].errorMessage, 'Should have error message');
System.assert(String.isNotBlank(results[0].errorMessage), 'Error message should not be blank');
}
/**
* @description Test JSON builder with all parameters
*/
@isTest
static void testJSONBuilder() {
// Arrange
List<String> templateIds = new List<String>{'template-A', 'template-B'};
String recordId = '001000000ABC123';
String language = 'es';
String emailSubject = 'Custom Subject';
// Act
Test.startTest();
String json = DocusignCompositeEnvelopeBuilder.buildCompositeEnvelopeJSON(
templateIds,
recordId,
language,
emailSubject,
null // customFields - not supported in Phase 1
);
Test.stopTest();
// Assert
System.assertNotEquals(null, json, 'JSON should be generated');
System.assert(json.contains('template-A'), 'Should contain first template');
System.assert(json.contains('template-B'), 'Should contain second template');
System.assert(json.contains(emailSubject), 'Should contain email subject');
System.assert(json.contains('SalesforceRecordId'), 'Should contain record ID field');
System.assert(json.contains(language), 'Should contain language');
}
}