Advice from Ben

Ben.md

Don't load db.seeds in test runs

Add factory_girl (read the GETTING_STARTED.md file, at least the first half)

Build the data that each test needs in that test.

Use the 4-phase test plan: setup, exercise, verify, teardown

You can use two numbers in code: 1 and 0. The rest are constants.

Add database_cleaner gem

Actually comment in GitHub

Don't merge commented-out code

If you don't understand someone's pull, it is their fault