Fix bug where the gem removes validation errors on relational fields

.rspec

@@ -0,0 +1,3 @@
+--color
+--require spec_helper
+--format documentation

lib/mongoid-embedded-errors.rb

@@ -16,9 +16,10 @@ def errors_with_embedded_errors
       embedded_relations.each do |name, metadata|
         # name is something like pages or sections
         # if there is an 'is invalid' message for the relation then let's work it:
-        next unless errs[name]
+        next unless Array(public_send(name)).any?(&:invalid?)
         # first delete the unless 'is invalid' error for the relation
-        errs.delete(name.to_sym)
+        errs[name].delete 'is invalid'
+        errs.delete name.to_sym if errs[name].empty?
         # next, loop through each of the relations (pages, sections, etc...)
         [send(name)].flatten.reject(&:nil?).each_with_index do |rel, i|
           # get each of their individual message and add them to the parent's errors:

spec/embedded_errors_spec.rb

@@ -1,8 +1,7 @@
-require 'spec_helper'
-
 describe Mongoid::EmbeddedErrors do
   let(:article) { Article.new }
   let(:invalid_page) { Page.new }
+  let(:valid_page) { Page.new title: 'test' }
   let(:invalid_section) { Section.new }
   let(:valid_section) { Section.new(header: 'My Header') }
   let(:invalid_annotation) { Annotation.new }
@@ -24,6 +23,7 @@
     end
 
     it 'save works as before' do
+      article.pages << valid_page
       article.save.should be_false
       article.should_not be_persisted
       article.errors.messages.should eql(
@@ -33,8 +33,14 @@
     it 'handles errors on the main object' do
       article.should_not be_valid
       article.errors.messages.should eql(
-        name: ["can't be blank"], summary: ["can't be blank"]
+        name: ["can't be blank"],
+        summary: ["can't be blank"],
+        pages: ["can't be blank"]
       )
     end
+    it 'does not remove other validation errors from relational fields' do
+      article.validate
+      expect(article.errors[:pages]).to include "can't be blank"
+    end
   end
 end

spec/support/models.rb

@@ -3,14 +3,15 @@ class Article
   include Mongoid::Timestamps
   include Mongoid::EmbeddedErrors
 
+  embeds_many :pages
+  embeds_one :annotation
+
   field :name, type: String
   field :summary, type: String
 
   validates :name, presence: true
   validates :summary, presence: true
-
-  embeds_many :pages
-  embeds_one :annotation
+  validates :pages, presence: true
 end
 
 class Page