fflib-apex-common
fflib-apex-common copied to clipboard
Add support for upsert by external Id
I'm afraid that since there's no external id field in the standard data model I found no way to create a test for this.
Closes #269
@jonathanwiesel in your test you can register an IDML mock impl and assert on the parameters passed to its methods. I have seen this done in another PR recently.
@afawcett thank you! Will take a look to locate it and see how it goes
@afawcett Instead of using the isExternalId
from the DescribeFieldResult
class, I changed to use the isIdLookup
which is specifically for upsert
operations to be more consistent and also supports the standard Id, which allowed me to create a test for it.
Any feedbackcon this? Is been quite a while π
Any feedbackcon this? Is been quite a while π
Apologies, this fell off my radar. Let me take a deep review and give this some testing. Thank you very much for the work on this. :)