Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

Scaffolded functional tests should test for failed create/update

#626

Currently, the functional tests generated by the scaffold generator do not test the parts of the create and update actions which handle failed creates/updates. The tests should be updated to cover the entire create and update actions.

This is difficult to accomplish in a way that is both automatic and elegant. We could create new tests passing invalid parameters to the create/update actions, but we do not define any validations at the time that the generator is run, and so we have no way of knowing what constitutes invalid parameters. We could mock the .save and .update_attributes methods to fail, but we cannot depend on the developer having an appropriate mocking library installed. We could dike out the save/update methods with sneaky uses of the private define_method call, but that approach is ugly.

The attached patch simply defines two new tests, one for create and one for update, with the same parameters as the original tests, but expecting the operation to fail instead of to succeed. It also includes FIXME notes stating that these tests will fail until validations are added to the model and the tests are updated with parameters that fail those validations.

I expect this change to be controversial, in that scaffold-generated tests will now fail out of the box. My rationale is that if we consider scaffolding to be an educational tool (see http://groups.google.com/group/r..., doing it this way will help the developer learn to write tests; besides, the sort of people who are inclined to not run tests won't notice the failure anyway (and those who are so inclined will either fix the tests to pass, or remove them entirely.)

Reported by Mark Cornick · July 15th, 2008 @ 07:28 PM

State: stale
Milestone: 2.x
Assigned to: nobody
Importance: none

Activity

  1. josh
    josh
    • State changed from new to stale

    Staling out, please reopen if this is still a problem.

    October 28th, 2008 @ 04:25 PM