Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_QueryFactory.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_QueryFactoryTest.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SObjectDescribe.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
5 changes: 3 additions & 2 deletions fflib/src/classes/fflib_SObjectDescribeTest.cls
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ private class fflib_SObjectDescribeTest {
for(integer i = 0; i < 10; i++){
fields = d.getFieldsMap();
}
System.assertEquals(1, Limits.getFieldsDescribes() );

System.assertEquals(false,fields.isEmpty());
}

Expand All @@ -109,7 +109,8 @@ private class fflib_SObjectDescribeTest {
for(integer i = 0; i < 10; i++){
fields = d.getFieldSetsMap();
}
System.assertEquals(1, Limits.getFieldSetsDescribes() );

// We need to assert something here... but what?
//no asserts on result size to avoid a requirement on field sets existing
}

Expand Down
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SObjectDescribeTest.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SObjectDomain.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SObjectDomainTest.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SObjectSelector.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SObjectSelectorTest.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SObjectUnitOfWork.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
27 changes: 11 additions & 16 deletions fflib/src/classes/fflib_SObjectUnitOfWorkTest.cls
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@
@IsTest
private with sharing class fflib_SObjectUnitOfWorkTest
{

// SObjects (in order of dependency) used by UnitOfWork in tests bellow
private static List<Schema.SObjectType> MY_SOBJECTS =
new Schema.SObjectType[] {
Expand All @@ -36,12 +35,9 @@ private with sharing class fflib_SObjectUnitOfWorkTest
Opportunity.SObjectType,
OpportunityLineItem.SObjectType };

@IsTest(seeAllData=true)
@isTest
private static void testUnitOfWorkNewDirtyDelete()
{
// Grab the Standard Pricebook (cannot create these via Apex!?!)
Pricebook2 pb = [select Id from Pricebook2 where IsStandard = true];

// Insert Opporunities with UnitOfWork
{
fflib_SObjectUnitOfWork uow = new fflib_SObjectUnitOfWork(MY_SOBJECTS);
Expand All @@ -61,7 +57,7 @@ private with sharing class fflib_SObjectUnitOfWorkTest
pbe.UnitPrice = 10;
pbe.IsActive = true;
pbe.UseStandardPrice = false;
pbe.Pricebook2Id = pb.Id;
pbe.Pricebook2Id = Test.getStandardPricebookId();
uow.registerNew(pbe, PricebookEntry.Product2Id, product);
OpportunityLineItem oppLineItem = new OpportunityLineItem();
oppLineItem.Quantity = 1;
Expand Down Expand Up @@ -95,7 +91,7 @@ private with sharing class fflib_SObjectUnitOfWorkTest
pbe.UnitPrice = 10;
pbe.IsActive = true;
pbe.UseStandardPrice = false;
pbe.Pricebook2Id = pb.Id;
pbe.Pricebook2Id = Test.getStandardPricebookId();
uow.registerNew(pbe, PricebookEntry.Product2Id, product);
OpportunityLineItem newOppLineItem = new OpportunityLineItem();
newOppLineItem.Quantity = 1;
Expand Down Expand Up @@ -140,12 +136,9 @@ private with sharing class fflib_SObjectUnitOfWorkTest
System.assertEquals(0, prods.size()); // Should have deleted Product added above
}

@IsTest(seeAllData=true)
@isTest
private static void testUnitOfWorkOverhead()
{
// Grab the Standard Pricebook (cannot create these via Apex!?!)
Pricebook2 pb = [select Id from Pricebook2 where IsStandard = true];

// Insert Opporunities with UnitOfWork
{
fflib_SObjectUnitOfWork uow = new fflib_SObjectUnitOfWork(MY_SOBJECTS);
Expand All @@ -165,7 +158,7 @@ private with sharing class fflib_SObjectUnitOfWorkTest
pbe.UnitPrice = 10;
pbe.IsActive = true;
pbe.UseStandardPrice = false;
pbe.Pricebook2Id = pb.Id;
pbe.Pricebook2Id = Test.getStandardPricebookId();
uow.registerNew(pbe, PricebookEntry.Product2Id, product);
OpportunityLineItem oppLineItem = new OpportunityLineItem();
oppLineItem.Quantity = 1;
Expand All @@ -178,7 +171,7 @@ private with sharing class fflib_SObjectUnitOfWorkTest
}

// Assert Results from using UnitOfWork
Integer costToCommitUoW = Limits.getScriptStatements();
Integer costToCommitUoW = Limits.getLimitCpuTime();
// TODO: Need to re-instate this check with a better approach, as it is not possible when
// product triggers contribute to DML (e.g. in sample app Opportunity trigger)
// System.assertEquals(5, Limits.getDmlStatements());
Expand Down Expand Up @@ -209,7 +202,7 @@ private with sharing class fflib_SObjectUnitOfWorkTest
pbe.UnitPrice = 10;
pbe.IsActive = true;
pbe.UseStandardPrice = false;
pbe.Pricebook2Id = pb.Id;
pbe.Pricebook2Id = Test.getStandardPricebookId();
pricebookEntries.add(pbe);
OpportunityLineItem oppLineItem = new OpportunityLineItem();
oppLineItem.Quantity = 1;
Expand Down Expand Up @@ -262,13 +255,15 @@ private with sharing class fflib_SObjectUnitOfWorkTest
}

// Assert Results from not using UnitOfWork
Integer costToCommitNoUoW = Limits.getScriptStatements() - costToCommitUoW;
Integer costToCommitNoUoW = Limits.getLimitCpuTime() - costToCommitUoW;
System.debug('costToCommitUoW ' + costToCommitUoW);
System.debug('costToCommitNoUoW ' + costToCommitNoUoW);
assertResults('NoUoW');

// Assert an acceptable statement % cost for using the UnitOfWork approach
System.assert((costToCommitUoW - costToCommitNoUoW) <= 3000, 'Overhead of using UnitOfWork has grown above 3000 statements.');
// Increased to 10,000 due to switch to getLimitCpuTime() and to allow overhead for enhancements,
// this is a *very* volatile test and needs a better implementation
System.assert((costToCommitUoW - costToCommitNoUoW) <= 10000, 'Overhead of using UnitOfWork has grown above 10000 statements.');
}

private static void assertResults(String prefix)
Expand Down
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SObjectUnitOfWorkTest.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SecurityUtils.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_SecurityUtilsTest.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_StringBuilder.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/classes/fflib_StringBuilderTest.cls-meta.xml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<?xml version="1.0" encoding="UTF-8"?>
<ApexClass xmlns="http://soap.sforce.com/2006/04/metadata">
<apiVersion>30.0</apiVersion>
<apiVersion>31.0</apiVersion>
<status>Active</status>
</ApexClass>
2 changes: 1 addition & 1 deletion fflib/src/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -21,5 +21,5 @@
<members>*</members>
<name>CustomLabels</name>
</types>
<version>24.0</version>
<version>31.0</version>
</Package>