Skip to content

Commit 209bcbc

Browse files
authored
Rename OpportunityTriggerHandler as OpportunitiesTriggerHandler (#39)
On branch Rename-OpportunityTriggerHandler-as-OpportunitiesTriggerHandler Changes to be committed: renamed: sfdx-source/apex-common-samplecode/main/classes/triggerHandlers/OpportunityTriggerHandler.cls -> sfdx-source/apex-common-samplecode/main/classes/triggerHandlers/OpportunitiesTriggerHandler.cls renamed: sfdx-source/apex-common-samplecode/main/classes/triggerHandlers/OpportunityTriggerHandler.cls-meta.xml -> sfdx-source/apex-common-samplecode/main/classes/triggerHandlers/OpportunitiesTriggerHandler.cls-meta.xml modified: sfdx-source/apex-common-samplecode/main/triggers/Opportunities.trigger renamed: sfdx-source/apex-common-samplecode/test/classes/triggerHandlers/OpportunityTriggerHandlerTest.cls -> sfdx-source/apex-common-samplecode/test/classes/triggerHandlers/OpportunitiesTriggerHandlerTest.cls renamed: sfdx-source/apex-common-samplecode/test/classes/triggerHandlers/OpportunityTriggerHandlerTest.cls-meta.xml -> sfdx-source/apex-common-samplecode/test/classes/triggerHandlers/OpportunitiesTriggerHandlerTest.cls-meta.xml
1 parent 13b9243 commit 209bcbc

File tree

5 files changed

+9
-9
lines changed

5 files changed

+9
-9
lines changed

sfdx-source/apex-common-samplecode/main/classes/triggerHandlers/OpportunityTriggerHandler.cls renamed to sfdx-source/apex-common-samplecode/main/classes/triggerHandlers/OpportunitiesTriggerHandler.cls

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,9 @@
2424
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
2525
**/
2626

27-
public class OpportunityTriggerHandler extends fflib_SObjectDomain
27+
public class OpportunitiesTriggerHandler extends fflib_SObjectDomain
2828
{
29-
public OpportunityTriggerHandler(List<Opportunity> sObjectList)
29+
public OpportunitiesTriggerHandler(List<Opportunity> sObjectList)
3030
{
3131
// Domain classes are initialised with lists to enforce bulkification throughout
3232
super(sObjectList);
@@ -109,7 +109,7 @@ public class OpportunityTriggerHandler extends fflib_SObjectDomain
109109
{
110110
public fflib_SObjectDomain construct(List<SObject> sObjectList)
111111
{
112-
return new OpportunityTriggerHandler(sObjectList);
112+
return new OpportunitiesTriggerHandler(sObjectList);
113113
}
114114
}
115115
}

sfdx-source/apex-common-samplecode/main/triggers/Opportunities.trigger

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,5 +28,5 @@ trigger Opportunities on Opportunity (
2828
after delete, after insert, after update, before delete, before insert, before update)
2929
{
3030
// Creates Domain class instance and calls apprpoprite overideable methods according to Trigger state
31-
fflib_SObjectDomain.triggerHandler(OpportunityTriggerHandler.class);
31+
fflib_SObjectDomain.triggerHandler(OpportunitiesTriggerHandler.class);
3232
}

sfdx-source/apex-common-samplecode/test/classes/triggerHandlers/OpportunityTriggerHandlerTest.cls renamed to sfdx-source/apex-common-samplecode/test/classes/triggerHandlers/OpportunitiesTriggerHandlerTest.cls

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,13 @@
2828
* This test class demonstrates two approaches to unit testing, with and without DML, each has its own pros and cons
2929
**/
3030
@IsTest
31-
private class OpportunityTriggerHandlerTest
31+
private class OpportunitiesTriggerHandlerTest
3232
{
3333

3434
@IsTest
3535
private static void testValidationWithoutDML()
3636
{
37-
OpportunityTriggerHandler opps = new OpportunityTriggerHandler(new Opportunity[] { new Opportunity ( Name = 'Test', Type = 'Existing Account' ) } );
37+
OpportunitiesTriggerHandler opps = new OpportunitiesTriggerHandler(new Opportunity[] { new Opportunity ( Name = 'Test', Type = 'Existing Account' ) } );
3838
opps.onValidate();
3939
System.assertEquals(1, fflib_SObjectDomain.Errors.getAll().size());
4040
System.assertEquals('You must provide an Account for OpportunityTriggerHandler for existing Customers.', fflib_SObjectDomain.Errors.getAll()[0].message);
@@ -46,7 +46,7 @@ private class OpportunityTriggerHandlerTest
4646
{
4747
Opportunity opp = new Opportunity ( Name = 'Test', Type = 'Existing Account' );
4848
fflib_SObjectDomain.Test.Database.onInsert(new Opportunity[] { opp } );
49-
fflib_SObjectDomain.triggerHandler(OpportunityTriggerHandler.class);
49+
fflib_SObjectDomain.triggerHandler(OpportunitiesTriggerHandler.class);
5050
System.assertEquals(1, fflib_SObjectDomain.Errors.getAll().size());
5151
System.assertEquals('You must provide an Account for OpportunityTriggerHandler for existing Customers.', fflib_SObjectDomain.Errors.getAll()[0].message);
5252
System.assertEquals(Opportunity.AccountId, ((fflib_SObjectDomain.FieldError)fflib_SObjectDomain.Errors.getAll()[0]).field);
@@ -58,7 +58,7 @@ private class OpportunityTriggerHandlerTest
5858
Opportunity oldOpp = new Opportunity ( Name = 'Test', Type = 'Existing Account' );
5959
Opportunity newOpp = new Opportunity ( Name = 'Test', Type = 'New Account' );
6060
fflib_SObjectDomain.Test.Database.onUpdate(new Opportunity[] { newOpp }, new Map<Id, SObject> { newOpp.Id => oldOpp } );
61-
fflib_SObjectDomain.triggerHandler(OpportunityTriggerHandler.class);
61+
fflib_SObjectDomain.triggerHandler(OpportunitiesTriggerHandler.class);
6262
System.assertEquals(1, fflib_SObjectDomain.Errors.getAll().size());
6363
System.assertEquals('You cannot change the Opportunity type once it has been created.', fflib_SObjectDomain.Errors.getAll()[0].message);
6464
System.assertEquals(Opportunity.Type, ((fflib_SObjectDomain.FieldError)fflib_SObjectDomain.Errors.getAll()[0]).field);
@@ -69,7 +69,7 @@ private class OpportunityTriggerHandlerTest
6969
{
7070
Opportunity opp = new Opportunity ( Name = 'Test' );
7171
fflib_SObjectDomain.Test.Database.onInsert(new Opportunity[] { opp } );
72-
fflib_SObjectDomain.triggerHandler(OpportunityTriggerHandler.class);
72+
fflib_SObjectDomain.triggerHandler(OpportunitiesTriggerHandler.class);
7373
System.assertEquals(fflib_SObjectDomain.Errors.getAll().size(), 0);
7474
}
7575

0 commit comments

Comments
 (0)