moose-inventory 0.1.6 → 0.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7694cb8a3c33618943fcc8bd0542b466c1fc2661
-  data.tar.gz: 057335d70625fca40ffc58f0e509b80466e5868c
+  metadata.gz: c371e17bbc7c89eb31612771e665eb281d7f0c4c
+  data.tar.gz: 169b4a227e853a706db3ae6a2831b9ad6adbe546
 SHA512:
-  metadata.gz: 394c749d2d70e32e19514de5b00b60ccb45bc1829f8071804ec4f05b542e2fcbff0a1f27e28ba299d162a6c86ab4dc99bab9c180b31c4d38db18b102ec225a71
-  data.tar.gz: 5598e22deade66147638d2f94d4453f75dfcb01797c70fb37047b5b5c3a5099f078388cc57f9967fab333e53e1b5c1191f1359d2542bc9c759427d760b2c6394
+  metadata.gz: a30eec0b29edcd53ac7e8df397e6adb48c2877b9f867ffdc5f9113abe04dbcaebe8f8d5cbe64e260c95e234ba645695942a70dafaa0c87dbf8229ca1a4def965
+  data.tar.gz: cc8adf6c82178b9d662dfed5399cb0a42121dee21d948aed1e8c14e9ce83443561a9e60d427f595d57f377b346f98ced9299bfd0c1b6fd8e11cca1639444f83b

data/lib/moose_inventory/cli/group.rb

@@ -11,8 +11,8 @@ module Moose
         require_relative 'group_get'
         require_relative 'group_list'
         require_relative 'group_rm'
-        #require_relative 'group_addchild'
-        #require_relative 'group_rmchild'
+        require_relative 'group_addchild'
+        require_relative 'group_rmchild'
         require_relative 'group_addhost'
         require_relative 'group_rmhost'
         require_relative 'group_addvar'

data/lib/moose_inventory/cli/group_addchild.rb

@@ -8,14 +8,85 @@ module Moose
       # Implemention of the "group addchild" methods of the CLI
       class Group < Thor # rubocop:disable ClassLength
         #==========================
-        desc 'addchild [options] NAME CHILDNAME',
-             'Associate a child-group CHILDNAME with the group NAME'
-        option :allowcreate, type: :boolean
-        def addchild
-          abort("The 'groups addchild GROUP' method is not yet implemented")
-          puts 'group addchild'
+        desc 'addchild PARENTGROUP CHILDGROUP_1 [CHILDGROUP_2 ... ]',
+             'Associate one or more child-groups CHILDGROUP_n with PARENTGROUP'
+        def addchild(*argv)
+          
+          # Sanity check
+          if args.length < 2
+            abort("ERROR: Wrong number of arguments, #{args.length} "\
+              "for 2 or more.")
+          end
+          
+          # Arguments
+          pname = args[0].downcase
+          cnames = args.slice(1, args.length - 1).uniq.map(&:downcase)
+           
+          # Sanity
+          if pname == 'ungrouped' || cnames.include?('ungrouped')
+            abort("ERROR: Cannot manually manipulate the automatic group 'ungrouped'.")
+          end
+          
+          # Convenience
+          db = Moose::Inventory::DB
+          fmt = Moose::Inventory::Cli::Formatter
+
+          # Transaction
+          warn_count = 0
+          begin 
+            db.transaction do # Transaction start
+              puts "Associate parent group '#{pname}' with child group(s) '#{cnames.join(',')}':"
+              # Get the target group
+              fmt.puts 2, "- retrieve group '#{pname}'..."
+              pgroup = db.models[:group].find(name: pname)
+              if pgroup.nil?
+                abort("ERROR: The group '#{pname}' does not exist.")
+              end
+              fmt.puts 4, "- OK"
+              
+               # Associate parent group with the child groups
+
+               groups_ds = pgroup.children_dataset
+               cnames.each do |cname|
+                 fmt.puts 2, "- add association {group:#{pname} <-> group:#{cname}}..."
+                  
+                 # Check against existing associations
+                 unless groups_ds[name: cname].nil?
+                    warn_count += 1
+                    fmt.warn "Association {group:#{pname} <-> group:#{cname}}}"\
+                      " already exists, skipping.\n" 
+                    fmt.puts 4, '- already exists, skipping.'
+                    fmt.puts 4, '- OK'
+                    next
+                  end
+                  
+                  # Add new association
+                  cgroup = db.models[:group].find(name: cname)
+                  if cgroup.nil?
+                    warn_count += 1
+                    fmt.warn "Group '#{cname}' does not exist and will be created.\n"
+                    fmt.puts 4, '- child group does not exist, creating now...'
+                    cgroup = db.models[:group].create(name: cname)
+                    fmt.puts 6, '- OK'
+                  end
+                  pgroup.add_child(cgroup)
+                  fmt.puts 4, '- OK'
+                end
+                fmt.puts 2, '- all OK'
+            end # Transaction end
+          rescue db.exceptions[:moose] => e
+              abort("ERROR: #{e}")
+          end
+          if warn_count == 0
+            puts 'Succeeded.'
+          else
+            puts 'Succeeded, with warnings.'
+          end
+                    
         end
       end
     end
   end
 end
+
+

data/lib/moose_inventory/cli/group_addhost.rb

@@ -13,7 +13,8 @@ module Moose
         def addhost(*args) # rubocop:disable Metrics/AbcSize
           # Sanity
           if args.length < 2
-            abort("ERROR: Wrong number of arguments, #{args.length} for 2 or more.")
+            abort("ERROR: Wrong number of arguments, #{args.length} "\
+              "for 2 or more.")
           end
 
           # Arguments

data/lib/moose_inventory/cli/group_get.rb

@@ -29,6 +29,8 @@ module Moose
             unless group.nil?
               hosts = group.hosts_dataset.map(:name)
             
+              children = group.children_dataset.map(:name)
+              
               groupvars = {}
               group.groupvars_dataset.each do |gv|
                 groupvars[gv[:name].to_sym] = gv[:value]
@@ -38,6 +40,11 @@ module Moose
               unless hosts.length == 0
                 results[group[:name].to_sym][:hosts]     = hosts
               end
+
+              unless children.length == 0
+                results[group[:name].to_sym][:children] = children
+              end
+              
               unless groupvars.length == 0
                 results[group[:name].to_sym][:groupvars] = groupvars
               end

data/lib/moose_inventory/cli/group_list.rb

@@ -23,16 +23,24 @@ module Moose
 
             # Hide the automatic ungrouped group, if it's empty
             next if group[:name] == 'ungrouped' && hosts.length == 0
-            
+
+            children = group.children_dataset.map(:name)
+                          
             groupvars = {}
             group.groupvars_dataset.each do |gv|
               groupvars[gv[:name].to_sym] = gv[:value]
             end
 
+              
             results[group[:name].to_sym] = {}
             unless hosts.length == 0
               results[group[:name].to_sym][:hosts] = hosts
             end
+
+            unless children.length == 0
+              results[group[:name].to_sym][:children] = children
+            end
+            
             unless groupvars.length == 0
               if confopts[:ansible] == true
                 results[group[:name].to_sym][:vars] = groupvars

data/lib/moose_inventory/cli/group_rmchild.rb

@@ -1,4 +1,5 @@
 require 'thor'
+
 require_relative './formatter.rb'
 
 module Moose
@@ -8,11 +9,72 @@ module Moose
       # Implemention of the "group rmchild" methods of the CLI
       class Group < Thor # rubocop:disable ClassLength
         #==========================
-        desc 'rmchild NAME CHILDNAME',
-             'Dissociate a child-group CHILDNAME from the group NAME'
-        def rmchild
-          abort("The 'groups rmchild GROUP' method is not yet implemented")
-          puts 'group rmchild'
+        desc 'rmchild PARENTGROUP CHILDGROUP_1 [CHILDGROUP_2 ... ]',
+        'Dissociate one or more child-groups CHILDGROUP_n from PARENTGROUP'
+        def rmchild(*argv)
+
+          # Sanity check
+          if args.length < 2
+            abort("ERROR: Wrong number of arguments, #{args.length} "\
+              "for 2 or more.")
+          end
+
+          # Arguments
+          pname = args[0].downcase
+          cnames = args.slice(1, args.length - 1).uniq.map(&:downcase)
+
+          # Sanity
+          if pname == 'ungrouped' || cnames.include?('ungrouped')
+            abort("ERROR: Cannot manually manipulate the automatic group 'ungrouped'.")
+          end
+
+          # Convenience
+          db = Moose::Inventory::DB
+          fmt = Moose::Inventory::Cli::Formatter
+
+          # Transaction
+          warn_count = 0
+          begin
+            db.transaction do # Transaction start
+              puts "Dissociate parent group '#{pname}' from child group(s) '#{cnames.join(',')}':"
+              # Get the target group
+              fmt.puts 2, "- retrieve group '#{pname}'..."
+              pgroup = db.models[:group].find(name: pname)
+              if pgroup.nil?
+                abort("ERROR: The group '#{pname}' does not exist.")
+              end
+              fmt.puts 4, "- OK"
+
+              # Dissociate parent group from the child groups
+              groups_ds = pgroup.children_dataset
+              cnames.each do |cname|
+                fmt.puts 2, "- remove association {group:#{pname} <-> group:#{cname}}..."
+
+                # Check against existing associations
+                if groups_ds[name: cname].nil?
+                  warn_count += 1
+                  fmt.warn "Association {group:#{pname} <-> group:#{cname}}"\
+                      " does not exist, skipping.\n"
+                  fmt.puts 4, "- doesn't exist, skipping."
+                  fmt.puts 4, '- OK'
+                  next
+                end
+
+                # remove association
+                cgroup = db.models[:group].find(name: cname)
+                pgroup.remove_child(cgroup)
+                fmt.puts 4, '- OK'
+              end
+              fmt.puts 2, '- all OK'
+            end # Transaction end
+          rescue db.exceptions[:moose] => e
+            abort("ERROR: #{e}")
+          end
+          if warn_count == 0
+            puts 'Succeeded.'
+          else
+            puts 'Succeeded, with warnings.'
+          end
         end
       end
     end

data/lib/moose_inventory/db/db.rb

@@ -34,9 +34,9 @@ module Moose
         # long-running code. Personally, I don't like this pooling regime - 
         # perhaps I'm not understanding how it's supposed to be used? 
         #
-        # TODO: can the models be refreshed, to make then again valid? What if
+        # QUESTION: can the models be refreshed, to make then again valid? What if
         #       we "load" instead of "require" the models?
-        # UPDATE: Nope, still borks even if we use a load.
+        # ANSWER: Nope, still borks even if we use a load.
         #
         # @db = nil            # <- fails for unit tests
         return unless @db.nil? # <- works for unit tests
@@ -113,6 +113,7 @@ module Moose
           Group.all.each do |g|
             g.remove_all_hosts
             g.remove_all_groupvars
+            g.remove_all_children
             g.destroy
           end
 
@@ -153,6 +154,7 @@ module Moose
         unless @db.table_exists? :groups
           @db.create_table(:groups) do
             primary_key :id
+            foreign_key :parent_id
             column :name, :text, unique: true
           end
         end

data/lib/moose_inventory/db/models.rb

@@ -11,7 +11,8 @@ module Moose
       ##
       # Model for the groups table
       class Group < Sequel::Model
-        # TODO: Groups of groups? (i.e. a group with children?)
+        many_to_one :parent, :class=>self
+        one_to_many :children, :key=>:parent_id, :class=>self
         many_to_many :hosts
         one_to_many :groupvars
       end

data/lib/moose_inventory/version.rb

@@ -2,6 +2,6 @@ module Moose
   ##
   # The Moose-Tools dynamic inventory management library
   module Inventory
-    VERSION = '0.1.6'
+    VERSION = '0.1.7'
   end
 end

data/spec/lib/moose_inventory/cli/group_addchild_spec.rb

@@ -0,0 +1,181 @@
+require 'spec_helper'
+
+# TODO: the usual respond_to? method doesn't seem to work on Thor objects.
+# Why not? For now, we'll check against instance_methods.
+
+RSpec.describe Moose::Inventory::Cli::Group do
+  before(:all) do
+    # Set up the configuration object
+    @mockarg_parts = {
+      config:  File.join(spec_root, 'config/config.yml'),
+      format:  'yaml',
+      env:     'test'
+    }
+
+    @mockargs = []
+    @mockarg_parts.each do |key, val|
+      @mockargs << "--#{key}"
+      @mockargs << val
+    end
+
+    @console = Moose::Inventory::Cli::Formatter
+    
+    @config = Moose::Inventory::Config
+    @config.init(@mockargs)
+
+    @db = Moose::Inventory::DB
+    @db.init if @db.db.nil?
+
+    @group = Moose::Inventory::Cli::Group
+    @host  = Moose::Inventory::Cli::Host
+    @app   = Moose::Inventory::Cli::Application
+  end
+
+  before(:each) do
+    @db.reset
+  end
+
+  #=======================
+  describe 'addchild' do
+    #------------------------
+    it 'Group.addchild() should be responsive' do
+      result = @group.instance_methods(false).include?(:addchild)
+      expect(result).to eq(true)
+    end
+
+    #------------------------
+    it '<missing args> ... should abort with an error' do
+      actual = runner  do
+        @app.start(%w(group addchild))
+      end
+
+      #@console.out(actual, 'y')
+      
+      # Check output
+      desired = { aborted: true}
+      desired[:STDERR] = "ERROR: Wrong number of arguments, 0 for 2 or more.\n"
+      expected(actual, desired)
+    end
+    
+    #------------------------
+    it 'ungrouped ... should abort with an error' do
+      parent_name = "ungrouped"
+      child_name = "fake"
+      
+      actual = runner  do
+        @app.start(%W(group addchild #{parent_name} #{child_name}))
+      end
+
+      #@console.out(actual, 'y')
+      
+      # Check output
+      desired = { aborted: true}
+      desired[:STDERR] = "ERROR: Cannot manually manipulate the automatic group 'ungrouped'.\n"
+      expected(actual, desired)
+      
+      ############################
+      # Should work the other way round too, when the child in the ungrouped item      
+      parent_name = "fake"
+      child_name = "ungrouped"
+      
+      actual = runner  do
+        @app.start(%W(group addchild #{parent_name} #{child_name}))
+      end
+
+      #@console.out(actual, 'y')
+      
+      # Check output
+      desired = { aborted: true}
+      desired[:STDERR] = "ERROR: Cannot manually manipulate the automatic group 'ungrouped'.\n"
+      expected(actual, desired)
+      
+    end    
+
+    #------------------------
+    it 'GROUP CHILDGROUP  ... should abort if GROUP does not exist' do
+      # TODO: Why don't we just create GROUP?  Likewise for all similar functions?
+      
+      pname = 'parent_group'
+      cname = 'child group'
+      
+      actual = runner do
+        @app.start(%W(group addchild #{pname} #{cname}))
+      end
+
+      #@console.out(actual, 'y')
+      # Check output
+      desired = { aborted: true}
+      desired[:STDOUT] = 
+        "Associate parent group '#{pname}' with child group(s) '#{cname}':\n"\
+        "  - retrieve group '#{pname}'...\n"
+      desired[:STDERR] = 
+        "ERROR: The group '#{pname}' does not exist.\n"\
+        "An error occurred during a transaction, any changes have been rolled back.\n"
+      expected(actual, desired)
+    end
+
+    #------------------------
+    it 'GROUP CHILDGROUP... should associate GROUP with an existing CHILDGROUP' do
+      pname = 'parent_group'
+      cname = 'child_group'
+
+      runner { @app.start(%W(group add #{pname} #{cname})) }
+
+      actual = runner { @app.start(%W(group addchild #{pname} #{cname} )) }
+
+      #@console.out(actual, 'y')
+       
+      desired = { aborted: false}
+      desired[:STDOUT] = 
+        "Associate parent group '#{pname}' with child group(s) '#{cname}':\n"\
+        "  - retrieve group '#{pname}'...\n"\
+        "    - OK\n"\
+        "  - add association {group:#{pname} <-> group:#{cname}}...\n"\
+        "    - OK\n"\
+        "  - all OK\n"\
+        "Succeeded.\n"
+      expected(actual, desired)
+
+      # We should have the correct group associations
+      pgroup = @db.models[:group].find(name: pname)
+      cgroups = pgroup.children_dataset 
+      expect(cgroups.count).to eq(1)
+      expect(cgroups[name: cname]).not_to be_nil
+    end
+    
+    #------------------------
+    it 'GROUP CHILDGROUP... should associate GROUP with a CHILDGROUP '\
+      'creating it if necessary' do
+      #  
+      pname = 'parent_group'
+      cname = 'child_group' 
+  
+      runner { @app.start(%W(group add #{pname})) } # <- don't pre-create the child
+  
+      actual = runner { @app.start(%W(group addchild #{pname} #{cname} )) }
+  
+      #@console.out(actual, 'y')
+       
+      desired = { aborted: false}
+      desired[:STDOUT] = 
+        "Associate parent group '#{pname}' with child group(s) '#{cname}':\n"\
+        "  - retrieve group '#{pname}'...\n"\
+        "    - OK\n"\
+        "  - add association {group:#{pname} <-> group:#{cname}}...\n"\
+        "    - child group does not exist, creating now...\n"\
+        "      - OK\n"\
+        "    - OK\n"\
+        "  - all OK\n"\
+        "Succeeded, with warnings.\n"
+      desired[:STDERR] = "WARNING: Group '#{cname}' does not exist and will be created.\n" 
+      expected(actual, desired)
+  
+      # We should have the correct group associations
+      pgroup = @db.models[:group].find(name: pname)
+      cgroups = pgroup.children_dataset 
+      expect(cgroups.count).to eq(1)
+      expect(cgroups[name: cname]).not_to be_nil
+    end
+    
+  end
+end

data/spec/lib/moose_inventory/cli/group_rmchild_spec.rb

@@ -0,0 +1,177 @@
+require 'spec_helper'
+
+# TODO: the usual respond_to? method doesn't seem to work on Thor objects.
+# Why not? For now, we'll check against instance_methods.
+
+RSpec.describe Moose::Inventory::Cli::Group do
+  before(:all) do
+    # Set up the configuration object
+    @mockarg_parts = {
+      config:  File.join(spec_root, 'config/config.yml'),
+      format:  'yaml',
+      env:     'test'
+    }
+
+    @mockargs = []
+    @mockarg_parts.each do |key, val|
+      @mockargs << "--#{key}"
+      @mockargs << val
+    end
+
+    @console = Moose::Inventory::Cli::Formatter
+    
+    @config = Moose::Inventory::Config
+    @config.init(@mockargs)
+
+    @db = Moose::Inventory::DB
+    @db.init if @db.db.nil?
+
+    @group = Moose::Inventory::Cli::Group
+    @host  = Moose::Inventory::Cli::Host
+    @app   = Moose::Inventory::Cli::Application
+  end
+
+  before(:each) do
+    @db.reset
+  end
+
+  #=======================
+  describe 'rmchild' do
+    #------------------------
+    it 'Group.rmchild() should be responsive' do
+      result = @group.instance_methods(false).include?(:rmchild)
+      expect(result).to eq(true)
+    end
+
+    #------------------------
+    it '<missing args> ... should abort with an error' do
+      actual = runner  do
+        @app.start(%w(group addchild))
+      end
+
+      #@console.out(actual, 'y')
+      
+      # Check output
+      desired = { aborted: true}
+      desired[:STDERR] = "ERROR: Wrong number of arguments, 0 for 2 or more.\n"
+      expected(actual, desired)
+    end
+    
+    #------------------------
+    it 'ungrouped ... should abort with an error' do
+      parent_name = "ungrouped"
+      child_name = "fake"
+      
+      actual = runner  do
+        @app.start(%W(group addchild #{parent_name} #{child_name}))
+      end
+
+      #@console.out(actual, 'y')
+      
+      # Check output
+      desired = { aborted: true}
+      desired[:STDERR] = "ERROR: Cannot manually manipulate the automatic group 'ungrouped'.\n"
+      expected(actual, desired)
+      
+      ############################
+      # Should work the other way round too, when the child in the ungrouped item      
+      parent_name = "fake"
+      child_name = "ungrouped"
+      
+      actual = runner  do
+        @app.start(%W(group rmchild #{parent_name} #{child_name}))
+      end
+
+      #@console.out(actual, 'y')
+      
+      # Check output
+      desired = { aborted: true}
+      desired[:STDERR] = "ERROR: Cannot manually manipulate the automatic group 'ungrouped'.\n"
+      expected(actual, desired)
+      
+    end    
+
+    #------------------------
+    it 'GROUP CHILDGROUP  ... should abort if GROUP does not exist' do
+      
+      pname = 'parent_group'
+      cname = 'child group'
+      
+      actual = runner do
+        @app.start(%W(group rmchild #{pname} #{cname}))
+      end
+
+      #@console.out(actual, 'y')
+      # Check output
+      desired = { aborted: true}
+      desired[:STDOUT] = 
+        "Dissociate parent group '#{pname}' from child group(s) '#{cname}':\n"\
+        "  - retrieve group '#{pname}'...\n"
+      desired[:STDERR] = 
+        "ERROR: The group '#{pname}' does not exist.\n"\
+        "An error occurred during a transaction, any changes have been rolled back.\n"
+      expected(actual, desired)
+    end
+
+    #------------------------
+    it 'GROUP CHILDGROUP  ... should succeed with warnings if CHILDGROUP is not associated' do
+      
+      pname = 'parent_group'
+      cname = 'child group'
+
+      runner { @app.start(%W(group add #{pname} #{cname})) }
+            
+      actual = runner do
+        @app.start(%W(group rmchild #{pname} #{cname}))
+      end
+
+      #@console.out(actual, 'y')
+      
+      # Check output
+      desired = {}
+      desired[:STDOUT] = 
+        "Dissociate parent group '#{pname}' from child group(s) '#{cname}':\n"\
+        "  - retrieve group '#{pname}'...\n"\
+        "    - OK\n"\
+        "  - remove association {group:#{pname} <-> group:#{cname}}...\n"\
+        "    - doesn't exist, skipping.\n"\
+        "    - OK\n"\
+        "  - all OK\n"\
+        "Succeeded, with warnings.\n"  
+        
+      desired[:STDERR] = 
+        "WARNING: Association {group:#{pname} <-> group:#{cname}} does not exist, skipping.\n"
+
+      expected(actual, desired)
+    end
+
+    #------------------------
+    it 'GROUP CHILDGROUP  ... should succeed without warnings if CHILDGROUP is associated' do
+      
+      pname = 'parent_group'
+      cname = 'child group'
+
+      runner { @app.start(%W(group add #{pname} #{cname})) }
+      runner { @app.start(%W(group addchild #{pname} #{cname})) }
+            
+      actual = runner do
+        @app.start(%W(group rmchild #{pname} #{cname}))
+      end
+
+      #@console.out(actual, 'y')
+      
+      # Check output
+      desired = {}
+      desired[:STDOUT] = 
+        "Dissociate parent group '#{pname}' from child group(s) '#{cname}':\n"\
+        "  - retrieve group '#{pname}'...\n"\
+        "    - OK\n"\
+        "  - remove association {group:#{pname} <-> group:#{cname}}...\n"\
+        "    - OK\n"\
+        "  - all OK\n"\
+        "Succeeded.\n"  
+        
+      expected(actual, desired)
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: moose-inventory
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.7
 platform: ruby
 authors:
 - Russell Davies
@@ -309,12 +309,14 @@ files:
 - spec/lib/moose_inventory/cli/cli_spec.rb
 - spec/lib/moose_inventory/cli/formatter_spec.rb
 - spec/lib/moose_inventory/cli/group_add_spec.rb
+- spec/lib/moose_inventory/cli/group_addchild_spec.rb
 - spec/lib/moose_inventory/cli/group_addhost_spec.rb
 - spec/lib/moose_inventory/cli/group_addvar_spec.rb
 - spec/lib/moose_inventory/cli/group_get_spec.rb
 - spec/lib/moose_inventory/cli/group_list_spec.rb
 - spec/lib/moose_inventory/cli/group_listvar_spec.rb
 - spec/lib/moose_inventory/cli/group_rm_spec.rb
+- spec/lib/moose_inventory/cli/group_rmchild_spec.rb
 - spec/lib/moose_inventory/cli/group_rmhost_spec.rb
 - spec/lib/moose_inventory/cli/group_rmvar_spec.rb
 - spec/lib/moose_inventory/cli/group_spec.rb
@@ -363,12 +365,14 @@ test_files:
 - spec/lib/moose_inventory/cli/cli_spec.rb
 - spec/lib/moose_inventory/cli/formatter_spec.rb
 - spec/lib/moose_inventory/cli/group_add_spec.rb
+- spec/lib/moose_inventory/cli/group_addchild_spec.rb
 - spec/lib/moose_inventory/cli/group_addhost_spec.rb
 - spec/lib/moose_inventory/cli/group_addvar_spec.rb
 - spec/lib/moose_inventory/cli/group_get_spec.rb
 - spec/lib/moose_inventory/cli/group_list_spec.rb
 - spec/lib/moose_inventory/cli/group_listvar_spec.rb
 - spec/lib/moose_inventory/cli/group_rm_spec.rb
+- spec/lib/moose_inventory/cli/group_rmchild_spec.rb
 - spec/lib/moose_inventory/cli/group_rmhost_spec.rb
 - spec/lib/moose_inventory/cli/group_rmvar_spec.rb
 - spec/lib/moose_inventory/cli/group_spec.rb