http://ayende.comAyende @ RahienCopyright (C) Ayende Rahien 2004 - 2021 (c) 202660Any string syntax = The Horror! (use "some other" mocking framework) Any code that looks like it is not raising an event when it is = No Sorry for being a PIA, but readability is really high on my list. http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment10http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment10Mon, 25 Jun 2007 07:47:33 GMTI think this is pretty terrible and unintuitive syntax. It looks like the EventRaiser is being attached to the event. Of course it isn't. What's wrong with something like this: mockedView.Load += null; // don't actually add a handler EventRaiser eventRaiser = LastCall.GetEventRaiser(); // but do get the raiser for it. eventRaiser.Raise(...); Or the shorthand: mockedView.Load += null; LastCall.RaiseEvent(...); If it comes to it, I don't really mind the string version. At least its intent is clear and free of side-effects. Mocks.RaiseEvent(mockedView, "Load", ...); http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment9http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment9Sun, 24 Jun 2007 01:23:52 GMTI never tried using rhino mock with an event till now, I don’t know how it works now… Don’t know if it’s feasible or not, but adding an event handler to the event you want to raise seems a bit confusing to me… I know this uses strings: IEventRaiser loadRaiser = new EventRaiser((IMockedObject)mockedView, "Load"); But in my opinion expresses better the intent that you want the mockedView to raise the Load event then writing mockedView.Load += EventRaiser.Raise(this, EventArgs.Empty); http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment8http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment8Sun, 24 Jun 2007 00:45:22 GMTI find it confusing that adding an event handler also will raise the event. We really wish that object members were real, first class objects, don't we? How about: withCustomEvents.MyEvent += delegate { myEventCalled = true; }; withCustomEvents.MyEvent += EventRaiser.Create(out handler); handler.RaiseEvent(); Assert.IsTrue(myEventCalled); I left out the default parameters to the event handler, as I usually don't need them. http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment7http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment7Sun, 24 Jun 2007 00:01:36 GMTThe problem is that this is not valid syntax. How do I get this to work? http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment6http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment6Sat, 23 Jun 2007 23:49:55 GMTI don't like that you have to register an event handler for the event in order to force the mockObject to raise that event. It would have been great if it were possible to say EventRaiser.Raise(myEvent, myEventArgs), but you already said it's not. And what about EventRaiser.Create(mock, myEvent, myEventArgs)? is this possible? http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment5http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment5Sat, 23 Jun 2007 22:29:42 GMTthe intent is very unclear and you have to be extra careful to make sure you "get" the fact that an event is actually being thrown.. http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment4http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment4Sat, 23 Jun 2007 19:42:28 GMTIt isn't. http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment3http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment3Sat, 23 Jun 2007 18:23:17 GMTIt´s great, but it will be prittier to me you write something like this: EventRaiser.Raise(withCustomEvents.MyEvent); Of couse, I don´t know if this way is posible.. http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment2http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment2Sat, 23 Jun 2007 18:21:46 GMTThis is definitely a tricky problem to solve. I think this syntax works. It'd be nice to be able to setup an expectation for the event being raised rather than having to attach the event twice, but I think what you have here works for me. :) http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment1http://ayende.com/2588/natural-event-syntax-for-rhino-mocks#comment1Sat, 23 Jun 2007 16:45:46 GMT