From b00a930e8d62511a02d9af91dc7db921b495cebf Mon Sep 17 00:00:00 2001 From: Paul Huliganga Date: Wed, 25 Feb 2026 09:39:18 -0500 Subject: [PATCH] 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 --- .../DocusignCompositeEnvelopeBuilderTest.cls | 31 ------------------- 1 file changed, 31 deletions(-) diff --git a/composite-envelope-builder/force-app/main/default/classes/DocusignCompositeEnvelopeBuilderTest.cls b/composite-envelope-builder/force-app/main/default/classes/DocusignCompositeEnvelopeBuilderTest.cls index 2dca219..95b21c2 100644 --- a/composite-envelope-builder/force-app/main/default/classes/DocusignCompositeEnvelopeBuilderTest.cls +++ b/composite-envelope-builder/force-app/main/default/classes/DocusignCompositeEnvelopeBuilderTest.cls @@ -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 templateIds = new List{'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'); - } }