Saturday, November 8, 2008

RubyConf 2008 Day 3 - Code Review

There is not much I can say about this. I actually felt a little uncomfortable because it was poking fun at some of my customers. Here are some of the problems they pointed out:
  • Leave no broken windows
  • Fixtures are brittle
  • Do not test "validate" using invalid data in your fixtures
  • Use VI as your editor
  • If you need default values for your Model, encode them in a class method
  • Use "comment out code" while writing characterization tests
  • hook methods ... check if it applies to you.. make sure you call super in another if you are implementing decoration using inheritance

No comments: