Silence warnings for common leaf -> tree expansion scenario

Closes #242

Signed-off-by: Aleksandrs Ļedovskis <[email protected]>

lib/i18n/tasks/data/tree/siblings.rb

@@ -10,6 +10,9 @@ module I18n::Tasks::Data::Tree
   # in case of an empty parent (nil) it represents a forest
   # siblings' keys are unique
   class Siblings < Nodes # rubocop:disable Metrics/ClassLength
+    # Ref: http://cldr.unicode.org/index/cldr-spec/plural-rules
+    CLDR_CATEGORY_KEYS = %w[zero one two few many other].freeze
+
     include ::I18n::Tasks::SplitKey
 
     attr_reader :parent, :key_to_node
@@ -100,12 +103,12 @@ def set(full_key, node)
             key: key_part,
             parent: parent,
             children: [],
-            warn_about_add_children_to_leaf: @warn_add_children_to_leaf
+            warn_about_add_children_to_leaf: @warn_about_add_children_to_leaf
           )
           append! child
         end
         unless child.children
-          warn_add_children_to_leaf child if @warn_about_add_children_to_leaf
+          conditionally_warn_add_children_to_leaf(child, [])
           child.children = []
         end
         child.children.set rest, node
@@ -187,7 +190,7 @@ def merge_node!(node, on_leaves_merge: nil) # rubocop:disable Metrics/AbcSize,Me
           if our.children
             our.children.merge!(node.children)
           else
-            warn_add_children_to_leaf our
+            conditionally_warn_add_children_to_leaf(our, node.children)
             our.children = node.children
           end
         elsif on_leaves_merge
@@ -228,6 +231,14 @@ def add_ancestors_that_only_contain_nodes!(nodes)
       end
     end
 
+    def conditionally_warn_add_children_to_leaf(node, children)
+      return unless @warn_about_add_children_to_leaf
+      return if !children.empty? &&
+                (children.map(&:key) - CLDR_CATEGORY_KEYS).empty?
+
+      warn_add_children_to_leaf(node)
+    end
+
     def warn_add_children_to_leaf(node)
       ::I18n::Tasks::Logging.log_warn "'#{node.full_key}' was a leaf, now has children (value <- scope conflict)"
     end

spec/locale_tree/siblings_spec.rb

@@ -75,6 +75,14 @@
       a = build_tree(a: 1)
       b = build_tree(a: { b: 1 })
       expect { silence_stderr { a.merge(b) } }.to_not raise_error
+      expect(capture_stderr { a.merge(b) })
+        .to include("[WARN] 'a' was a leaf, now has children (value <- scope conflict)")
+    end
+
+    it '#merge does not warn about conflict with Unicode CLDR category keys' do
+      a = build_tree(a: 1)
+      b = build_tree(a: { zero: 0, one: 1, two: 2, few: 7, many: 88, other: 'ok' })
+      expect(capture_stderr { a.merge(b) }).to be_empty
     end
 
     it '#set conflict value <- scope' do