Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggested refactoring of XrmFakedContext classes into a single base test class at the project level #83

Closed
jordimontana82 opened this issue May 5, 2021 · 2 comments

Comments

@jordimontana82
Copy link

Hi Mark,

Congrats on the MVP award 😄.

I've seen you're using FakeXrmEasy in this repo which is great!

Since I'm working on a rewrite of it for the next major version, just thought that pointing you to this pattern would be useful and will make transition to v2 smoother:

https://github.com/DynamicsValue/announcements

@MarkMpn
Copy link
Owner

MarkMpn commented May 8, 2021

Thanks @jordimontana82!

I don't see the XrmFakedContextFactory class in the package I'm using (FakeXrmEasy.9) - do I need to bring in another package to get that?

@jordimontana82
Copy link
Author

Hi @MarkMpn ,

Nope, that's new in v2.x, I've updated the announcements repo to make it a bit more clear, I was referring to the fact that having XrmFakedContext declared at the class level will be easier, it'll also make the individual tests smaller too.

https://github.com/DynamicsValue/announcements

Thanks!

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

No branches or pull requests

2 participants