moose-inventory 1.0.7 → 1.0.9

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

@@ -9,7 +9,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
     @mockarg_parts = {
       config:  File.join(spec_root, 'config/config.yml'),
       format:  'yaml',
-      env:     'test'
+      env:     'test',
     }
 
     @mockargs = []
@@ -22,7 +22,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
     @config.init(@mockargs)
 
     @console = Moose::Inventory::Cli::Formatter
-    
+
     @db = Moose::Inventory::DB
     @db.init if @db.db.nil?
 
@@ -52,18 +52,17 @@ RSpec.describe Moose::Inventory::Cli::Group do
       desired[:STDERR] = "ERROR: Wrong number of arguments, 0 for 1 or more.\n"
       expected(actual, desired)
     end
-    
+
     # --------------------
     it 'ungrouped ... should abort with an error' do
-      actual = runner { @app.start(%W(group rm ungrouped)) }
+      actual = runner { @app.start(%w(group rm ungrouped)) }
 
       # Check output
-      desired = {aborted: true}
+      desired = { aborted: true }
       desired[:STDERR] =
         "Cannot manually manipulate the automatic group 'ungrouped'\n"
       expected(actual, desired)
-    end    
-    
+    end
 
     #---------------
     it '<non-existent group> ... should warn about non-existent groups' do
@@ -73,10 +72,10 @@ RSpec.describe Moose::Inventory::Cli::Group do
       # already exists.
 
       # no items in the db
-      group_name = "fake"
-      actual = runner {  @app.start(%W(group rm #{group_name})) }
- 
-      #@console.out(actual,'y')
+      group_name = 'fake'
+      actual = runner { @app.start(%W(group rm #{group_name})) }
+
+      # @console.out(actual,'y')
       desired = {}
       desired[:STDOUT] =
         "Remove group '#{group_name}':\n"\
@@ -114,26 +113,26 @@ RSpec.describe Moose::Inventory::Cli::Group do
       group = @db.models[:group].find(name: group_name)
       expect(group).to be_nil
     end
-    
+
     #---------------
     it "GROUP ... should handle the automatic 'ungrouped' group for associated hosts" do
-       host_name  = 'test-host1'
-       group_name  = 'test-group1'
-
-       tmp = runner { @app.start(%W(group add #{group_name} --hosts #{host_name})) }
-       expect(tmp[:unexpected]).to eq(false)
-       expect(tmp[:aborted]).to eq(false)  
-       host = @db.models[:host].find(name: host_name)
-       groups_ds = host.groups_dataset
-       expect(groups_ds).not_to be_nil
-       expect(groups_ds[name: 'ungrouped']).to be_nil # Shouldn't be ungrouped 
-
-       # Now do the rm       
-       actual = runner { @app.start(%W(group rm #{group_name})) }
- 
-       # @console.out(actual)
-
-      # Check output 
+      host_name = 'test-host1'
+      group_name = 'test-group1'
+
+      tmp = runner { @app.start(%W(group add #{group_name} --hosts #{host_name})) }
+      expect(tmp[:unexpected]).to eq(false)
+      expect(tmp[:aborted]).to eq(false)
+      host = @db.models[:host].find(name: host_name)
+      groups_ds = host.groups_dataset
+      expect(groups_ds).not_to be_nil
+      expect(groups_ds[name: 'ungrouped']).to be_nil # Shouldn't be ungrouped
+
+      # Now do the rm
+      actual = runner { @app.start(%W(group rm #{group_name})) }
+
+      # @console.out(actual)
+
+      # Check output
       desired = {}
       desired[:STDOUT] =
         "Remove group '#{group_name}':\n"\
@@ -147,16 +146,16 @@ RSpec.describe Moose::Inventory::Cli::Group do
         "Succeeded.\n"
       expected(actual, desired)
 
-       # Check db
-       group = @db.models[:group].find(name: group_name)
-       expect(group).to be_nil
-       
-       host = @db.models[:host].find(name: host_name)
-       expect(host).not_to be_nil
-       groups_ds = host.groups_dataset
-       expect(groups_ds).not_to be_nil
-       expect(groups_ds[name: 'ungrouped']).not_to be_nil
-     end    
+      # Check db
+      group = @db.models[:group].find(name: group_name)
+      expect(group).to be_nil
+
+      host = @db.models[:host].find(name: host_name)
+      expect(host).not_to be_nil
+      groups_ds = host.groups_dataset
+      expect(groups_ds).not_to be_nil
+      expect(groups_ds[name: 'ungrouped']).not_to be_nil
+    end
 
     #---------------
     it 'GROUP1 GROUP2 ... should remove multiple groups' do
@@ -168,24 +167,79 @@ RSpec.describe Moose::Inventory::Cli::Group do
       actual = runner { @app.start(%w(group rm) + names) }
 
       # Check output
-      desired = {STDOUT: ''}
+      desired = { STDOUT: '' }
       names.each do |name|
         # Check output
         desired[:STDOUT] = desired[:STDOUT] +
-          "Remove group '#{name}':\n"\
-          "  - Retrieve group '#{name}'...\n"\
-          "    - OK\n"\
-          "  - Destroy group '#{name}'...\n"\
-          "    - OK\n"\
-          "  - All OK\n"
+                           "Remove group '#{name}':\n"\
+                           "  - Retrieve group '#{name}'...\n"\
+                           "    - OK\n"\
+                           "  - Destroy group '#{name}'...\n"\
+                           "    - OK\n"\
+                           "  - All OK\n"
       end
       desired[:STDOUT] = desired[:STDOUT] + "Succeeded.\n"
       expected(actual, desired)
-      
 
       # Check db
-      hosts = @db.models[:host].all
-      expect(hosts.count).to eq(0)
+      groups = @db.models[:group].all
+      expect(groups.count).to eq(0)
+    end
+
+    #---------------
+    it 'GROUP ... should remove GROUP, where GROUP has an associated parent.' do
+      @db.models[:group].create(name: 'parent')
+      runner { @app.start(%w(group addchild parent child)) }
+
+      actual = runner { @app.start(%w(group rm child)) }
+
+      # Check output
+      desired = { STDOUT: '' }
+      # Check output
+      desired[:STDOUT] = desired[:STDOUT] +
+                         "Remove group 'child':\n"\
+                         "  - Retrieve group 'child'...\n"\
+                         "    - OK\n"\
+                         "  - Remove association {group:child <-> group:parent}...\n"\
+                         "    - OK\n"\
+                         "  - Destroy group 'child'...\n"\
+                         "    - OK\n"\
+                         "  - All OK\n"
+
+      desired[:STDOUT] = desired[:STDOUT] + "Succeeded.\n"
+      expected(actual, desired)
+
+      # Check db
+      groups = @db.models[:group].all
+      expect(groups.count).to eq(1)
+    end
+
+    #---------------
+    it 'GROUP ... should remove GROUP, where GROUP has an associated child.' do
+      @db.models[:group].create(name: 'parent')
+      runner { @app.start(%w(group addchild parent child)) }
+
+      actual = runner { @app.start(%w(group rm parent)) }
+
+      # Check output
+      desired = { STDOUT: '' }
+      # Check output
+      desired[:STDOUT] = desired[:STDOUT] +
+                         "Remove group 'parent':\n"\
+                         "  - Retrieve group 'parent'...\n"\
+                         "    - OK\n"\
+                         "  - Remove association {group:parent <-> group:child}...\n"\
+                         "    - OK\n"\
+                         "  - Destroy group 'parent'...\n"\
+                         "    - OK\n"\
+                         "  - All OK\n"
+
+      desired[:STDOUT] = desired[:STDOUT] + "Succeeded.\n"
+      expected(actual, desired)
+
+      # Check db
+      groups = @db.models[:group].all
+      expect(groups.count).to eq(1)
     end
   end
 end

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

@@ -9,7 +9,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
     @mockarg_parts = {
       config:  File.join(spec_root, 'config/config.yml'),
       format:  'yaml',
-      env:     'test'
+      env:     'test',
     }
 
     @mockargs = []
@@ -19,7 +19,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
     end
 
     @console = Moose::Inventory::Cli::Formatter
-    
+
     @config = Moose::Inventory::Config
     @config.init(@mockargs)
 
@@ -49,65 +49,63 @@ RSpec.describe Moose::Inventory::Cli::Group do
         @app.start(%w(group addchild))
       end
 
-      #@console.out(actual, 'y')
-      
+      # @console.out(actual, 'y')
+
       # Check output
-      desired = { aborted: true}
+      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
+      parent_name = 'ungrouped'
+      child_name = 'fake'
+
+      actual = runner do
         @app.start(%W(group addchild #{parent_name} #{child_name}))
       end
 
-      #@console.out(actual, 'y')
-      
+      # @console.out(actual, 'y')
+
       # Check output
-      desired = { aborted: true}
+      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
+      # 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')
-      
+      # @console.out(actual, 'y')
+
       # Check output
-      desired = { aborted: true}
+      desired = { aborted: true }
       desired[:STDERR] = "ERROR: Cannot manually manipulate the automatic group 'ungrouped'.\n"
       expected(actual, desired)
-      
-    end    
+    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')
+      # @console.out(actual, 'y')
       # Check output
-      desired = { aborted: true}
-      desired[:STDOUT] = 
+      desired = { aborted: true }
+      desired[:STDOUT] =
         "Dissociate parent group '#{pname}' from child group(s) '#{cname}':\n"\
         "  - retrieve group '#{pname}'...\n"
-      desired[:STDERR] = 
+      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)
@@ -115,21 +113,20 @@ RSpec.describe Moose::Inventory::Cli::Group do
 
     #------------------------
     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')
-      
+      # @console.out(actual, 'y')
+
       # Check output
       desired = {}
-      desired[:STDOUT] = 
+      desired[:STDOUT] =
         "Dissociate parent group '#{pname}' from child group(s) '#{cname}':\n"\
         "  - retrieve group '#{pname}'...\n"\
         "    - OK\n"\
@@ -137,9 +134,9 @@ RSpec.describe Moose::Inventory::Cli::Group do
         "    - doesn't exist, skipping.\n"\
         "    - OK\n"\
         "  - all OK\n"\
-        "Succeeded, with warnings.\n"  
-        
-      desired[:STDERR] = 
+        "Succeeded, with warnings.\n"
+
+      desired[:STDERR] =
         "WARNING: Association {group:#{pname} <-> group:#{cname}} does not exist, skipping.\n"
 
       expected(actual, desired)
@@ -147,30 +144,29 @@ RSpec.describe Moose::Inventory::Cli::Group do
 
     #------------------------
     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')
-      
+      # @console.out(actual, 'y')
+
       # Check output
       desired = {}
-      desired[:STDOUT] = 
+      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"  
-        
+        "Succeeded.\n"
+
       expected(actual, desired)
     end
   end

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

@@ -9,7 +9,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
     @mockarg_parts = {
       config:  File.join(spec_root, 'config/config.yml'),
       format:  'yaml',
-      env:     'test'
+      env:     'test',
     }
 
     @mockargs = []
@@ -41,7 +41,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
       result = @group.instance_methods(false).include?(:rmhost)
       expect(result).to eq(true)
     end
-    
+
     #----------------
 
     #------------------------
@@ -51,7 +51,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
       end
 
       # Check output
-      desired = { aborted: true}
+      desired = { aborted: true }
       desired[:STDERR] = "ERROR: Wrong number of arguments, 0 for 2 or more.\n"
       expected(actual, desired)
     end
@@ -65,16 +65,16 @@ RSpec.describe Moose::Inventory::Cli::Group do
       end
 
       # Check output
-      desired = { aborted: true}
-      desired[:STDOUT] = 
+      desired = { aborted: true }
+      desired[:STDOUT] =
         "Dissociate group '#{group_name}' from host(s) '#{host_name}':\n"\
         "  - retrieve group '#{group_name}'...\n"
-      desired[:STDERR] = 
+      desired[:STDERR] =
         "ERROR: The group '#{group_name}' does not exist.\n"\
         "An error occurred during a transaction, any changes have been rolled back.\n"
       expected(actual, desired)
     end
- 
+
     #------------------------
     it 'GROUP HOST ... should dissociate the group from an existing group' do
       host_name = 'test1'
@@ -82,22 +82,22 @@ RSpec.describe Moose::Inventory::Cli::Group do
 
       runner { @app.start(%W(host add #{host_name})) }
       runner { @app.start(%W(group add #{group_name})) }
-      runner { @app.start(%W(group addhost #{group_name} #{host_name} )) }
+      runner { @app.start(%W(group addhost #{group_name} #{host_name})) }
 
       #
       # Dissociate the host
       # 1. expect that the group association is removed
       # 2. expect that no association with ungrouped is made.
-      
+
       actual = runner do
-        @app.start(%W(group rmhost #{group_name} #{host_name} )) 
+        @app.start(%W(group rmhost #{group_name} #{host_name}))
       end
-      
-      #@console.dump(actual, 'y')
-      
+
+      # @console.dump(actual, 'y')
+
       # rubocop:disable Metrics/LineLength
-      desired = { aborted: false}
-      desired[:STDOUT] = 
+      desired = { aborted: false }
+      desired[:STDOUT] =
         "Dissociate group '#{group_name}' from host(s) '#{host_name}':\n"\
         "  - retrieve group '#{group_name}'...\n"\
         "    - OK\n"\
@@ -115,56 +115,55 @@ RSpec.describe Moose::Inventory::Cli::Group do
       hosts = group.hosts_dataset
       expect(hosts.count).to eq(0)
     end
-    
+
     #------------------------
     it 'GROUP HOST ... should warn about non-existing associations' do
       # 1. Should warn that the association doesn't exist.
-      # 2. Should complete with success. (desired state == actual state) 
-      
+      # 2. Should complete with success. (desired state == actual state)
+
       host_name = 'test_host'
       group_name = 'test_group'
       runner { @app.start(%W(host add #{host_name})) }
       runner { @app.start(%W(group add #{group_name})) }
       runner { @app.start(%W(group addhost #{host_name})) }
-    
+
       actual = runner do
-        @app.start(%W(group rmhost #{group_name} #{host_name})) 
+        @app.start(%W(group rmhost #{group_name} #{host_name}))
       end
-     
+
       # rubocop:disable Metrics/LineLength
-      desired = { aborted: false}
-      desired[:STDOUT] = 
+      desired = { aborted: false }
+      desired[:STDOUT] =
         "Dissociate group '#{group_name}' from host(s) '#{host_name}':\n"\
         "  - retrieve group \'#{group_name}\'...\n"\
         "    - OK\n"\
-        "  - remove association {group:#{group_name } <-> host:#{host_name}}...\n"\
+        "  - remove association {group:#{group_name} <-> host:#{host_name}}...\n"\
         "    - doesn't exist, skipping.\n"\
         "    - OK\n"\
         "  - all OK\n"\
         "Succeeded, with warnings.\n"
-      desired[:STDERR] = 
+      desired[:STDERR] =
         "WARNING: Association {group:#{group_name} <-> host:#{host_name}} "\
-          "doesn't exist, skipping.\n" 
+          "doesn't exist, skipping.\n"
 
       expected(actual, desired)
-    end    
+    end
 
     #------------------------
     it '\'ungrouped\' HOST ... should abort with an error' do
-
       host_name = 'test_host'
       group_name = 'ungrouped'
 
       runner { @app.start(%W(host add #{name})) } # <- auto creates the association with ungrouped
 
-      actual = runner { @app.start(%W(group rmhost #{group_name} #{host_name} )) }
+      actual = runner { @app.start(%W(group rmhost #{group_name} #{host_name})) }
 
-      desired = { aborted: true}
+      desired = { aborted: true }
       desired[:STDERR] =
         "ERROR: Cannot manually manipulate the automatic group 'ungrouped'.\n"
       expected(actual, desired)
     end
-    
+
     #------------------------
     it 'GROUP HOST1 HOST2 ... should dissociate the group from'\
       ' multiple hosts at once' do
@@ -173,33 +172,31 @@ RSpec.describe Moose::Inventory::Cli::Group do
       #    if it has no other groups.
 
       group_name = 'test_group'
-      host_names = %W( test_host1 test_host2 test_host3 )
+      host_names = %w(test_host1 test_host2 test_host3)
+
+      runner { @app.start(%W(group add #{group_name})) }
+      runner { @app.start(%W(group addhost #{group_name}) + host_names) }
 
-      runner { @app.start(%W(group add #{group_name} )) }
-      runner {  @app.start(%W(group addhost #{group_name}) + host_names)  }
-      
       actual = runner do
         @app.start(%W(group rmhost #{group_name}) + host_names)
       end
 
-      #@console.out(actual, 'y')
-
-      # rubocop:disable Metrics/LineLength
-      desired = { aborted: false}
+      # @console.out(actual, 'y')
+      desired = { aborted: false }
       desired[:STDOUT] =
         "Dissociate group '#{group_name}' from host(s) '#{host_names.join(',')}':\n"\
         "  - retrieve group \'#{group_name}\'...\n"\
         "    - OK\n"
       host_names.each do |host|
         desired[:STDOUT] = desired[:STDOUT] +
-        "  - remove association {group:#{group_name} <-> host:#{host}}...\n"\
-        "    - OK\n"\
-        "  - add automatic association {group:ungrouped <-> host:#{host}}...\n"\
-        "    - OK\n"\
+                           "  - remove association {group:#{group_name} <-> host:#{host}}...\n"\
+                           "    - OK\n"\
+                           "  - add automatic association {group:ungrouped <-> host:#{host}}...\n"\
+                           "    - OK\n"\
       end
       desired[:STDOUT] = desired[:STDOUT] +
-        "  - all OK\n"\
-        "Succeeded.\n"
+                         "  - all OK\n"\
+                         "Succeeded.\n"
       expected(actual, desired)
       # rubocop:enable Metrics/LineLength
 
@@ -210,6 +207,6 @@ RSpec.describe Moose::Inventory::Cli::Group do
       host_names.each do |host|
         expect(hosts[name: host]).to be_nil
       end
-    end         
+    end
   end
 end