Skip to content
This repository has been archived by the owner on Jun 16, 2024. It is now read-only.

Added DeleteRecordChangeHistoryRequest Fake executor for Shared project. #542

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

victorsolaya
Copy link

Title

Added DeleteRecordChangeHistoryRequestTest Fake executor for Test Shared project.

Description

It has been included a quick fake DeleteRecordChangeHistory to fake the message in the Shared Projects.
It has been created a DeleteRecordChangeHistory_Tests as well in the SharedTest project.

Victor Sanchez added 2 commits January 8, 2021 21:43
@victorsolaya
Copy link
Author

Hi @jordimontana82 I am having some errors because DeleteRecordChangeHistory was added in the v9.0.1.0

9.0.1.0: UR1 (Update Rollup 1)
   Added DeleteRecordChangeHistory message to delete audit change history for a particular record

How could I say to my tests not to include this if it doesn't exist? Do you have any example?
CC @BetimBeja just in case you had the same issue before and you could help me.

@BetimBeja
Copy link
Contributor

I made a PR victorsolaya#1 to your repository which will update this PR if merged.
Each build of FakeXrmEasy defines a symbol for the dependency it has on the specific version of the CrmSdk. When there are braking changes we can use those symbols in the #if directives to handle the missing dependencies.

…istory

Added preprocessor directive to exclude older version
@jordimontana82
Copy link
Owner

Thanks both for this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants