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

PartCover Integration with JustMock #54

Open
mehfuzh opened this issue Sep 7, 2011 · 8 comments
Open

PartCover Integration with JustMock #54

mehfuzh opened this issue Sep 7, 2011 · 8 comments
Assignees

Comments

@mehfuzh
Copy link

mehfuzh commented Sep 7, 2011

Hi Shaun,
How are you doing ? I am creating this issue as a part of JustMock integration with your tool. We support PartCover but that requires some code changes in the PartCover itself described in the following online reference:

http://www.telerik.com/help/justmock/integration-partcover.html

I am sending this issue to ask you to make the changes in the official branch of the product so that users get more seamless experience using JustMock with Partcover.

Cheers,
Mehfuz

@sawilde
Copy link
Owner

sawilde commented Sep 7, 2011

I see what you are doing but surely this environment variable should be set by the host process or even set it in the environment variables (user/system) of the process i.e. in the command line before you call PartCover via the console or the browser; the same way you would do so with Moles or TypeMock.

I am wary of adding an environment variable that is specific to a commercial product. Would you expect me to do the same to OpenCover (which handles 32 and 64 bit processes and also does branch coverage?). Have you made the same request to NCover; I see from your site that you don't need NCover to make changes to integrate - why is that - could the same approach be used for PartCover/OpenCover?

@ghost ghost assigned sawilde Sep 7, 2011
@mehfuzh
Copy link
Author

mehfuzh commented Sep 7, 2011

Hi Shaun,

Thanks again for the quick reply. Integrating the same way we did it for NCover is one way. However I will talk with the team and keep you posted on any issues regarding it.

Cheers,
Mehfuz.

@sawilde
Copy link
Owner

sawilde commented Sep 7, 2011

You might want to use a consistent approach for all profilers like NCover/PartCover/OpenCover in much the same way as Moles (or TypeMock) take as there may be other profilers out there that may need to work with your mocking tool which you are not aware currently aware of.

@mehfuzh
Copy link
Author

mehfuzh commented Sep 9, 2011

Hi Shaun,
Thanks for the info, We are also looking for a generic way to the problem but not using an intermediate tool (JustMockRunner.exe /TMockRunner.exe).

Will also keep you posted on that.

Cheers,
Mehfuz

@sawilde
Copy link
Owner

sawilde commented Sep 9, 2011

The simplest way is to just have the user register the GUID/CLASSID of the chained profiler before calling your mock runner. A simple environment variable would suffice. Your users (who are developers) should be quite capable of this small step if they are looking to chain a profiler in line with the mocking tool. TypeMock I think uses a file and lists all the profilers they are compatible with but it is still user editable (see http://stackoverflow.com/questions/6698290/can-opencover-be-used-with-typemock-isolator) . Personally I'd keep it simple, developers aren't stupid, sometimes daft, but not stupid.

@sawilde
Copy link
Owner

sawilde commented May 17, 2012

Did you progress further with TypeMock and PartCover integration?

@mehfuzh
Copy link
Author

mehfuzh commented May 22, 2012

You mean JustMock and PartCover ?

@sawilde
Copy link
Owner

sawilde commented May 22, 2012

Yup - sorry yes JustMock and PartCover (or OpenCover)

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

No branches or pull requests

2 participants