The usefulness of interaction tests, or “How to question the method”
[26/02/10]
More and more often these days, I’ve been having a crisis of faith conscience when it comes to TDD, or rather, interaction testing in general. It happens more so on projects where I’m the only developer or there is a very small team of strong developers. And with the latest project being in GWT, where you are practically beaten down by all the guidance espousing separation of concerns, it’s even more pronounced.
Consider a screen that display a list of pet names for your truck. When you click the “add” button, it displays a dialogue box with a blank form. If you’re using an MVP pattern, this is likely what would happen when the page is loaded, from a testing point of view:
- Load a list of data from the datastore
- When it’s retrieved, send the list to the view
- Add a click handler to the “add” button that does something
In this case, I’m using gwt-dispatch, which is a command pattern implementation for GWT. So the first point is executed by creating a command and dispatching it like so:
dispatcher.execute( new GetPetNamesCommand(), new GetPetNamesCallback( ));
The GetPetNamesCallback class looks like so:
public class GetPetNamesCallback extends AsyncCallback<GetPetNamesResult> { @Override public void callback( GetPetNamesResult result ) { display.showList( result.getPetNames( ) ); } }
And the code to add the click handler when the page is loaded:
display.getAddNewLink().addClickHandler(new AddNewClickHandler( ));
(This is a passive view, I think, so the presenter handles all the event wire-up.)
Going to the testing side of things, I have a test context class called When_initializing_pet_names_list_presenter and within it, I have these tests:
- should_request_data_from_data_store
- should_wire_up_add_new_button
In each of these tests, I dutifully go about my business mocking out a view and a dispatcher and creating a presenter. I set up expectations and the real meat of the test is in this line (using Mockito syntax):
verify( dispatcher ).execute( isA( GetPetNamesCommand.class ), isA( GetPetNamesCallback.class ) );
That is, we verify that we executed the dispatcher with appropriate arguments. There’s a similar test for the second method and in another class, When_pet_names_have_been_retrieved, there’s a method that tests the GetPetNamesCallback class to ensure, also dutifully, that display.showList was called.
When creating this code, I did it the other way around, writing the tests first and fleshing out the details. That makes no never mind here. The end result is the same no matter which way you go about it.
Where my crisis of conscience comes in is when I review the code I’ve written based on these tests. This screen isn’t very complicated. If I were to bang out the code for it without tests, I can’t imagine doing it any other way.
But someone else could, of course. And maybe they aren’t as disciplined as I am about keeping concerns separated. Fair enough. At the moment, they aren’t working on my project though. It’s just me and it will be me for the foreseeable future. Besides which, I’ve done enough thinking on how smart or dumb the next guy is going to be.
Furthermore, these tests feel too granular. Like they’re too focused on implementation and that they’d break at the slightest refactoring. Like say, if we switched to an architecture where the view wires up its click handler and raises events. Now I have to go and fix all these tests.
Maybe that’s short-term thinking and it will lead to a lapse in judgment one day where I write untestable code to get something out the door.
Maybe that’s true but maybe it’s not. I don’t think I’m advocating against interaction testing in any case. Just wonder what value it holds once you’ve been doing it long enough that you can’t write untestable code if you tried, even if there aren’t any actual tests.
In any case, this is just interaction testing. There are plenty of cases where unit testing can be useful in verifying complex business rules (like whether it’s okay to have duplicate pet names for both your truck and your sister-wife).
What I see more value in is a higher level integration tests. Having recently taking Selenium for a spell, it seems more useful. What happens when the user lands on the “Pet Names List” page? Why, it should have this text showing in its HTML somewhere. What about when you click the “add” button? Well, then the page should include a form in it.
In this case, if anything changes in the bowels of the application, these tests remain untouched and, more importantly, still very relevant. If you have to change these tests, it’s as the result of a user interaction change and I think that warrants a change in your tests more than which flavour of MVP you decide to use.
This isn’t really a position I’ll defend to the death, or even to the slightly injured. As it is, I’m not altogether happy about how I’ve explained my position. Should probably let it sit for a bit but I don’t have that kind of patience. In any case, more and more when I run my test suite and see everything running green, I feel a little bit Pavlovian.
Kyle the Trained
The usefulness of interaction tests, or “How to question the method”
Archives
Web Forms Routing in ASP.NET 4
[28/02/10]
At our first Sarasota Web Developer Group meeting we discussed several of the new enhancements in ASP.NET 4 Web Forms. One of my favorite enhancements is the new routing features which are very similar to the ones I have enjoyed so much in ASP.NET MVC.
Register Routes
This is old hat for those using ASP.NET MVC. Just register your routes at application startup. Rather than your endpoint being a controller, however, you associate a physical page as the handler of the request.
public class Global : System.Web.HttpApplication { void Application_Start(object sender, EventArgs e) { RegisterRoutes(RouteTable.Routes); } void RegisterRoutes(RouteCollection routes) { routes.MapPageRoute( "Contact_Details", // Route Name "Contacts/Details/{id}", // Url and Parameters "~/Contacts/Details.aspx" // Page Handling Request ); } }
In this case we are telling the routing engine 3 things:
- Name of the Route: Contact_Details
- The Route: Contacts/Details/{id}
- The Physical Page Handling the Request: Details.aspx
Notice the id parameter ( route value ) which will be the id of the contact to display in the details page.
Expression Builders for Creating HyperLinks, etc.
With ASP.NET MVC we have strongly-typed View Helpers to help generate links. With ASP.NET 4 Web Forms you utilize Expression Builders to create links such as:
<asp:HyperLink NavigateUrl="<%$RouteUrl:RouteName=Contact_Details, id=1 %>" runat="server">John Doe </asp:HyperLink> <asp:HyperLink NavigateUrl="<%$RouteUrl:id=1%>" runat="server">John Doe </asp:HyperLink>
We can explicitly specify the name of the route or let the routing engine figure out the correct route based on the parameters.
Getting RouteData from the Page
You can access the RouteData from the Page by accessing the Page.RouteData Property, which is just a convenient access point to RequestContext.RouteData. Here are a couple of ways to get the id from the route to display the proper contact given its id:
public partial class Details : System.Web.UI.Page { protected void Page_Load(object sender, EventArgs e) { var id = Page.RouteData.GetRequiredString("id"); var id2 = Page.RouteData.Values["id"]; } }
RouteParameter for use with DataSources
If you are displaying the contact in a DetailsView, for example, you can use the new RouteParameter with your DataSource to get values from the route as such:
<asp:ObjectDataSource ID="ObjectDataSource1" runat="server" SelectMethod="FindById" TypeName="Contact"> <SelectParameters> <asp:RouteParameter Name="id" RouteKey="id" Type="Int32" /> </SelectParameters> </asp:ObjectDataSource>
Binding your DetailsView to the ObjectDataSource will now cause the contact to be displayed appropriately.
Response.RedirectToRoute and RedirectToRoutePermanent
A lot has been mentioned about using Response.RedirectPermanent for SEO, but even cooler is Response.RedirectToRoute and Response.RedirectToRoutePermanent for working with the new routing engine. Below I am specifying the route name and passing in any route values where necessary when redirecting:
Response.RedirectToRoute("Contact_Details", new { id = 1 }); Response.RedirectToRoutePermanent("Contact_Details", new { id = 1 });
Conclusion
Lots of really neat things in ASP.NET 4 and ASP.NET 4 Web Forms. I am going to continue to post a few more we discussed during the first meeting.
For those interested, the second meeting of the Sarasota Web Developer Group will discuss a number of interesting topics: Leveraging ASP.NET MVC - Web Forms - DynamicData - Castle ActiveRecord.
Introduction to the Reactive Extensions for JavaScript – Creating Observers
[27/02/10]
Looking back to the previous post, we covered how we create observable sequences, the producers of our data. We have quite a number of ways of creating these outside of events which we covered earlier. Now that we have these observable sequences, now what? We need to address the consumer side of this producer/consumer story in the form of an observer.
Before we get started, let’s get caught up to where we are today:
Creating Observers
Let’s go back to the Observer pattern definition once again before we get started. The idea here is that we have an object, called the Observable (or Subject) which keeps a list of its dependents, the observers, and notifies each of them automatically of any state changes. In the case of the Reactive Extensions for JavaScript, we’re talking more about observable sequences. As we discussed last time, the Observer has three parts:
- OnNext – when a new value is produced
- OnError – when an exception occurs
- OnCompleted – when the observable sequence terminates
When creating an observer, we should take all three into account and how we’re going to handle them.
In order to attach these observers to our observable sequence, we can invoke the Subscribe method on our observable while passing in our observer. And when we’re no longer interested in the subscription to the observable sequence, we can detach by calling Dispose on the result of the Subscribe method.
New Observer via Create
Let’s get started in creating an Observer by looking at the Observer.Create method. This method takes in three functions, one for the OnNext, one for the OnError and finally one for the OnCompleted. This function returns to us an Observer which we can then use for subscribing.
Rx.Observer.Create( function(next) { ... }, // OnNext function(err) { ... }, // OnError function() { ... }) // OnCompleted );
Once we have an Observer, we can then attach to the Observable using the Subscribe method which takes our Observer. When we call Subscribe, we get back a disposable object with a single Dispose method which allows us to detach from the Observable.
Observable { Subscribe : function(observer) { ... } }
One of the best ways I find to explore a new API is to write tests to show the expected behavior. By writing these, I have a comprehensive view of what each method does, especially if the code didn’t come with the tests already. So, let’s create a few tests to show the behavior of creating an Observer and then subscribing to an Observable sequence. I’ll use QUnit to write my tests, and in particular, the asynchronous test feature due because we are testing asynchronous callbacks.
The first test will be to check the OnNext function parameter on Observable.Create. In this case, I’ll assert at the single value in my observable sequence is the value I receive when OnNext is invoked.
asyncTest("Observer should observe OnNext", function() { var observable = Rx.Observable.Return(0); var observer = Rx.Observer.Create( function(next) { equals(0, next); start(); }, function(err) { }, function() {}); observable.Subscribe(observer); });
The next test, I will make an example on how my OnError function parameter will work. In this case, I’ll have an Observer throw an exception via the Throw method and my OnError function check the message and assert that it’s the same as my error that I threw.
asyncTest("Observer should observe OnError", function() { var someError = "FAIL!"; var observable = Rx.Observable.Throw(someError); var observer = Rx.Observer.Create( function(next) { }, function(err) { equals(someError, err); start(); }, function() {}); observable.Subscribe(observer); });
Finally, in my last example, let’s create a simple test to show off the OnCompleted behavior. In order to do so we’ll create an empty observable which should not yield any values and instead only invoke the OnCompleted. Then we’ll create an Observer which has the test logic in the OnCompleted function parameter.
asyncTest("Observer should observe OnCompleted", function() { var observable = Rx.Observable.Empty(); var observer = Rx.Observer.Create( function(next) { }, function(err) { }, function() { ok(true, "True when invoked on complete"); start(); }); observable.Subscribe(observer); });
Creating Observers this way is good for reusability, especially if you wish to attach to any number of observable sequences. But, we’re not tied to having to create them via Create, there are other ways.
Overloading Subscribe
In addition to creating an Observer via the Create method, we also have shortcuts which allow us to create an Observer on the fly with the Subscribe method. In addition to the Subscribe which takes an Observer, we have three other overloads which can take functions for our OnNext, OnError and OnCompleted. The first overload takes a function for OnNext, where the next takes a function for OnNext and OnError, and finally the last overload takes functions for all three, the OnNext, OnError and OnCompleted.
Observable { Subscribe : function( function(next) { ... }) Subscribe : function( function(next) { ... }, function(err) { ... }) Subscribe : function( function(next) { ... }, function(err) { ... }, function() { ... }) }
This function, just as above, will create a disposable object for us which allows us to unsubscribe at any time via the Dispose method.
Unsubscribing
As I’ve stated earlier, one of the great things about the design of Rx for JavaScript is that it’s quite easy to both subscribe and unsubscribe from an observer. The design of Rx for JavaScript follows very closely to the design of Rx for .NET including subscribing and unsubscribing. Let’s step through an example of how we can use the Dispose method on our subscription. In this instance, we’ll have two observers, and after the first value has been produced, we unhook the first observer and continue listening on the second. We’ll assert that indeed the first has been unhooked while the second continues to listen.
asyncTest("Dispose should unhook observer", function() { var nextValue = 0; var observable = Rx.Observable.FromArray([1,2,3]); var disp1 = observable.Subscribe( function(next) { nextValue = next; }); var disp2 = observable.Subscribe( function(next) { disp1.Dispose(); equals(1, nextValue); start(); }); });
Such scenarios could be quite helpful in unhooking events when others happen, such as mouse events, keyboard or even AJAX requests. We’ll cover some of those scenarios in upcoming posts.
Conclusion
So, now we’ve covered the basics of creating Observable sequences and Observers and subscriptions. Now that we have some of the basics what else can we do with it? That’s where some of the LINQ combinators come in handy and we’ll pick that up next time.
This of course is only scratching the surface of what capabilities this library has and there is much more yet left to cover. The question you’re probably asking now is where can I get it? Well, for that you’ll have to stay tuned. I hope to have more announcements soon about its general availability.
What can I say? I love JavaScript and very much looking forward to the upcoming JSConf 2010 here in Washington, DC where the Reactive Extensions for JavaScript will be shown in its full glory with Jeffrey Van Gogh. For too many times, we’ve looked for the abstractions over the natural language of the web (HTML, CSS and JavaScript) and created monstrosities instead of embracing the web for what it is. With libraries such as jQuery and indeed the Reactive Extensions for JavaScript gives us better tools for dealing with the troubled child that is DOM manipulation and especially events.
Certified Scrum Product Owner Training class on May 24, 2010 in Boulder.
[26/02/10]
Certified Scrum Product Owner Training class on May 24, 2010 in Boulder.
[26/02/10] - The usefulness of interaction tests, or “How to question the method”
[26/02/10] - Castle Project code organization
[26/02/10] - Scrum Challenge #1 OVER: Scrum is…
[25/02/10] - A Vision for FubuMVC’s Component Model (gems, Nu, engines, slices, oh my…)
[25/02/10] - The Exec Problem
[25/02/10] - 3 Days To Go!
[25/02/10] - Random Thought Scrum Challenge – #1
[24/02/10] - DevTeach Toronto 2010 Ultimate Edition
[23/02/10] - Introduction to the Reactive Extensions for JavaScript – Creating Observables
[23/02/10] - Sarasota Web Developer Group - MVC and ASP.NET 4 From Scratch
[22/02/10] - How I Stole an Office and Fixed Our Daily Scrum
[22/02/10] - CST Application Process Improvement Community
[22/02/10] - Scrum Alliance National, USA: Managing Director
[22/02/10] - 10 Things I Hate About Agile Development!
[21/02/10] - XP Day Suisse, Geneva, 29 March 2010
[21/02/10] - In Praise of Middle Management
[20/02/10] - Why use Event Sourcing?