Adding error messages to lint output

lib/factory_girl/linter.rb

@@ -6,34 +6,34 @@ def self.lint!(factories_to_lint)
 
     def initialize(factories_to_lint)
       @factories_to_lint = factories_to_lint
-      @invalid_factories = calculate_invalid_factories
+      @invalid_factory_messages = calculate_invalid_factory_messages
     end
 
     def lint!
-      if invalid_factories.any?
+      if invalid_factory_messages.any?
         raise InvalidFactoryError, error_message
       end
     end
 
     private
 
-    attr_reader :factories_to_lint, :invalid_factories
+    attr_reader :factories_to_lint, :invalid_factory_messages
 
-    def calculate_invalid_factories
-      factories_to_lint.select do |factory|
+    def calculate_invalid_factory_messages
+      factories_to_lint.map do |factory|
         built_factory = FactoryGirl.build(factory.name)
 
-        if built_factory.respond_to?(:valid?)
-          !built_factory.valid?
+        if built_factory.respond_to?(:valid?) && !built_factory.valid?
+          "* #{factory.name} -- #{built_factory.errors.full_messages.join('; ')}"
         end
-      end
+      end.compact
     end
 
     def error_message
       <<-ERROR_MESSAGE.strip
 The following factories are invalid:
 
-#{invalid_factories.map {|factory| "* #{factory.name}" }.join("\n")}
+#{invalid_factory_messages.join("\n")}
       ERROR_MESSAGE
     end
   end

spec/acceptance/lint_spec.rb

@@ -19,8 +19,30 @@
     error_message = <<-ERROR_MESSAGE.strip
 The following factories are invalid:
 
-* user
-* admin_user
+* user -- Name can't be blank
+* admin_user -- Name can't be blank
+    ERROR_MESSAGE
+
+    expect do
+      FactoryGirl.lint
+    end.to raise_error FactoryGirl::InvalidFactoryError, error_message
+  end
+
+  it 'lists all errors when multiple are present' do
+    define_model 'Person', first_name: :string, last_name: :string do
+      validates :first_name, presence: true
+      validates :last_name, presence: true
+    end
+
+    FactoryGirl.define do
+      factory :person do
+      end
+    end
+
+    error_message = <<-ERROR_MESSAGE.strip
+The following factories are invalid:
+
+* person -- First name can't be blank; Last name can't be blank
     ERROR_MESSAGE
 
     expect do