@joshuaclayton Okay, so you're saying this PR as it exists is correct and can be merged?

> Still not sure why the suite would run twice - ideas for how to confirm FG is the culprit here?

I figured this out. It was how we define the task in the current `Rakefile`:

    RSpec::Core::RakeTask.new(:factory_specs)

That shouldn't be using RSpec's rake task constructor. It should be using Rake's normal DSL. So the Rakefile alternative to putting `FactoryGirl.lint` in `before(:suite)` would be:

```ruby
if defined?(FactoryGirl)
  desc 'Run factory specs.'
  task :factory_specs do
    FactoryGirl.lint
  end

  task spec: :factory_specs
end
```

We could also wrap that with the `DatabaseCleaner` bits if we wanted.

Which approach do you think is better?