Ayende @ Rahien

pb commented on Recursive Mocking

Sun, 12 Oct 2008 15:48:50 GMT

In case anyone else looks at this, the first one passes, the second one doesn't. So either the replay is coincidentally returning the same mocked instance or that is the intent and it just needs to work when not in replay mode. I'll take another look during the week sometime. [Test] public void WillGetSameInstanceOfRecursedMockInReplayMode() { MockRepository mocks = new MockRepository(); var mock = mocks.DynamicMock

Ayende Rahien commented on Recursive Mocking

Sat, 11 Oct 2008 19:47:27 GMT

pb, Yes, that is the main thing. Of course, trying to add more tests will likely create additional failure points.

pb commented on Recursive Mocking

Sat, 11 Oct 2008 19:44:30 GMT

Ok, that clears things up. So the only thing remaining in your request is to make sure that the auto mocked instances are the same object then?

Ayende Rahien commented on Recursive Mocking

Sat, 11 Oct 2008 12:30:51 GMT

pb, Yes, sorry. I posted about what I wanted, but that was fuzzy at the edges. Ideally, I _would_ want to get that to work. The problem is that this is not really possible, I don't have the seam to be able to tell when Expect.Call( expression ) is called, so I don't know how to differentiate between that at just expression

pb commented on Recursive Mocking

Sat, 11 Oct 2008 12:28:34 GMT

In your first post, the example you posted as needing development in order to work was the second one. The first test is what now works after getting the latest code after your mod. So have you revised the requirements that you expect it to work only in the first scenario but not the second? Because my understanding was that you wanted the second to work.

Ayende Rahien commented on Recursive Mocking

Sat, 11 Oct 2008 05:12:38 GMT

pb, Yes, that is the expected behavior. Expect.Call() can only be used with mocks in record mode. the static methods on MockRepository are generating mocks in replay mode. If you want to deal with them, you need to use the Expect() or Stub()

pb commented on Recursive Mocking

Sat, 11 Oct 2008 04:56:21 GMT

The first test passes and the second doesn't. is that as you expect? [Test] public void CanCallSomeRandomInterfacePropertyWithReturn() { var mockService = MockRepository.GenerateMock x.Identity.Name).Return("foo"); Assert.AreEqual("foo", mockService.Identity.Name); } [Test] public void CanCallSomeRandomInterfacePropertyWithReturn2() { var mockService = MockRepository.GenerateMock

Ayende Rahien commented on Recursive Mocking

Fri, 10 Oct 2008 14:59:08 GMT

thanks, the wiki would be great

Chad Myers commented on Recursive Mocking

Fri, 10 Oct 2008 14:54:17 GMT

@Ayende: I promised Roy I'd do it for him, I suppose I should afford you the same courtesy. Should I put it in the wiki, or do you have something in SVN?

Vladan Strigo commented on Recursive Mocking

Fri, 10 Oct 2008 14:47:26 GMT

Just one small comment about my specs in that docs... they are based on an older version of Rhino Mocks (pre AAA style version) and the fancy usings are just wrapping up the Record/Replay model. Cheers! Vladan

Vladan Strigo commented on Recursive Mocking

Fri, 10 Oct 2008 14:44:08 GMT

Ayende, Is that something like this? [akuaproject.googlecode.com/.../...odChainMocker.cs](http://akuaproject.googlecode.com/svn/trunk/framework/trunk/src/Specifications/Mocking/MethodChainMocker.cs) [akuaproject.googlecode.com/.../...nsionsGateway.cs](http://akuaproject.googlecode.com/svn/trunk/framework/trunk/src/Specifications/Mocking/MockRepositoryExtensionsGateway.cs) I've got a couple of specs in place for it here: [akuaproject.googlecode.com/.../...MockerConcern.cs](http://akuaproject.googlecode.com/svn/trunk/framework/trunk/src/Specifications.Specs/Mocking/MethodChainMockerConcern.cs) Is that something similar to your intent? ;) Vladan

Ayende Rahien commented on Recursive Mocking

Fri, 10 Oct 2008 14:23:29 GMT

Chad, No argument about that from me. Will you be willing to write that guidance for Rhino Mocks' documentation?

Chad Myers commented on Recursive Mocking

Fri, 10 Oct 2008 14:20:52 GMT

@Ayende: I agree you should not take it away. I'm appreciative of you and Roy for taking the time to add this. It will definitely be useful in certain circumstances. But, like I was telling Roy, I believe there should be some heavy guidance around this type of feature warning people that this should be used only in special circumstances.

Ayende Rahien commented on Recursive Mocking

Fri, 10 Oct 2008 14:17:43 GMT

The reasoning is as follows: a/ I get a lot of questions about that b/ It is a neat feature c/ It helps greatly in some scenarios. There are plenty of things in Rhino Mocks that can be abused, that doesn't mean that they aren't valuable. I don't really believe in taking away options.

Chad Myers commented on Recursive Mocking

Fri, 10 Oct 2008 14:12:40 GMT

Ayende, thank you for your contribution here (and Roy's in his project), but I still think that, while this is a neat technical feature, it will encourage/allow people to get themselves into some big problems. I follow the idea that mocking large, complicated API's like ICriteria, ADO.NET, HttpContext, etc is wrong. And not just because it was a big hassle (which you and Roy have already fixed), but because I feel large API's should be wrapped very tightly and tested via integration (since unit testing tightly-wrapped APIs is ultimately just testing the underlying API which I'm not interested in). In my practice, I'm going to advise people to not use this feature unless there's some special circumstance that warrants it for some reason. Was there a specific use case that has driven this feature in both TM and RM that I'm perhaps not thinking of? Or was it just friendly competition among colleagues (Roy & Ayende)? :) :) :)

Torkel commented on Recursive Mocking

Fri, 10 Oct 2008 14:12:17 GMT

Instead of: svn diff [rhino-tools.svn.sourceforge.net/.../trunk](https://rhino-tools.svn.sourceforge.net/svnroot/rhino-tools/trunk) -r 1682:1683 view this: [demo.codesaga.com/history/RhinoTools?cs=1683](http://demo.codesaga.com/history/RhinoTools?cs=1683) sorry, just had to :)

John Chapman commented on Recursive Mocking

Fri, 10 Oct 2008 12:37:09 GMT

Ayende, This is truly awesome. This will be a godsend for a lot of my work. I almost wish I was unemployed right now so that I had the time to finish this up for you. Maybe if nobody gets to it by next weekend, I'll have a shot at it. Any idea what the planned release schedule is to include this feature? John Chapman