One of the biggest mistakes I see in Rails testing (especially model testing) is testing implementation instead of behavior.
Let’s say I run a factory that makes cars. I want to make sure that each car I make actually works. How should I go about making sure each car works?
Here are two possible methods.
One way I could try to make sure each car works is to check to see if it has all the right stuff. I could check for the presence of an engine, an ignition, a brake system, wheels, and everything else that’s needed in order to get from point A to point B. If the car has all this stuff, then it’s all set.
Another way I could try to make sure the car works is to start it up and try to drive it somewhere.
The first way is bad and the second way is good. Here’s why.
Why testing behavior is better
If I “verify” that my car works by checking for the presence of various parts, then I haven’t really actually verified anything. I haven’t demonstrated that the system under test (the car) actually meets spec (can drive).
If I test the car by actually driving it, then the questions of whether the car has various components become moot. If for example the car can travel down the road, we don’t need to ask if the car has wheels. If it didn’t have wheels it wouldn’t be moving.
All of our “implementation” questions can be translated into more meaningful “behavior” questions.
- Does it have an ignition? -> Can it start up?
- Does it have an engine and wheels? -> Can it drive?
- Does it have brakes? -> Can it stop?
Lastly, behavior tests are better than implementation tests because behavior tests are more loosely coupled to the implementation. I ask “Can it start up?” instead of “Does it have an engine?” then I’m free to, for example, change my car from a gasoline-powered car to an electric car without having to change the set of tests that I perform. In other words, behavior tests enable refactoring.
Different granularities of behavior testing
I want to be sure that my argument isn’t misconstrued as a discussion between unit tests and integration tests. The implementation tests being performed weren’t actually verifying the proper behavior of the various components (engine, ignition, etc.) in isolation. The implementation tests were merely verifying the presence of components.
In fact, performing some “unit” tests and some “integration” tests in my car factory would probably be a good idea. It would be smart to make sure each part works individually before I combine them all. I’d just want to make sure my unit and integration tests were testing behavior, not implementation.
Verifying that my car can drive is a test at the highest granularity. At a lower level of granularity, I might want to verify that the engine works, even before the engine is connected to an actual car. To test my engine, I wouldn’t want to check to see whether the engine has six cylinders, each with a piston inside. Instead I would want to give my engine some fuel and actually see if it runs.
So please don’t confuse the implementation vs. behavior question with the unit test vs. integration test question. We sometimes want to perform unit tests and we sometimes want to perform integration tests, but we pretty much always want to test behavior instead of implementation.
How this relates to Rails
I sometimes come across examples of Rails model tests that I consider pointless. These tests verify the presence of validations, associations, callbacks and other implementation details. These tests are pointless because they don’t actually test anything. Having these tests is like checking for the presence of your engine and brakes rather than actually just driving your car.
It’s also pointless to write system specs that e.g. check for the presence of various form fields and buttons, for the same exact reason.
Test behavior, not implementation.