moose-inventory 1.0.8 → 1.0.9

data/spec/lib/moose_inventory/cli/group_addvar_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 = []
@@ -40,96 +40,96 @@ RSpec.describe Moose::Inventory::Cli::Group do
       result = @group.instance_methods(false).include?(:addvar)
       expect(result).to eq(true)
     end
-    
+
     #-----------------
     it '<missing args> ... should abort with an error' do
       actual = runner  do
         @app.start(%w(group addvar)) # <- no group given
       end
       # @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 'GROUP key=value ... should abort if the group does not exist' do
-      group_name ='not-a-group'
-      group_var = "foo=bar"
-      
+      group_name = 'not-a-group'
+      group_var = 'foo=bar'
+
       actual = runner do
         @app.start(%W(group addvar #{group_name} #{group_var}))
       end
 
       # Check output
-      desired = { aborted: true}
-      desired[:STDOUT] =  
+      desired = { aborted: true }
+      desired[:STDOUT] =
         "Add variables '#{group_var}' to group '#{group_name}':\n"\
         "  - retrieve group '#{group_name}'...\n"
-      desired[:STDERR] = 
+      desired[:STDERR] =
         "An error occurred during a transaction, any changes have been rolled back.\n"\
         "ERROR: The group '#{group_name}' does not exist.\n"
       expected(actual, desired)
     end
 
     #------------------------
-     it 'GROUP <malformed> ... should abort with an error' do
-       # 1. Should add the var to the db
-       # 2. Should associate the host with the var
-       
-       group_name = 'test_group'
-       @db.models[:group].create(name: group_name)
- 
-       var  = {name: 'var1', value: "testval"}
-       cases = %W(
-         testvar 
-         testvar= 
-         =testval 
-         testvar=testval= 
-         =testvar=testval 
-         testvar=testval=extra
-       )      
-     
-       cases.each do |args| 
-         actual = runner do
-           @app.start(%W(group addvar #{group_name} #{args} ))
-         end
-         #@console.out(actual,'p')
-   
-         desired = { aborted: true}
-         desired[:STDOUT] = 
-         "Add variables '#{args}' to group '#{group_name}':\n"\
-         "  - retrieve group '#{group_name}'...\n"\
-         "    - OK\n"\
-         "  - add variable '#{args}'...\n"
-         
-         desired[:STDERR] = 
-           "An error occurred during a transaction, any changes have been rolled back.\n"\
-           "ERROR: Incorrect format in '{#{args}}'. Expected 'key=value'.\n"
-           
-         expected(actual, desired)
-       end
-     end
-    
+    it 'GROUP <malformed> ... should abort with an error' do
+      # 1. Should add the var to the db
+      # 2. Should associate the host with the var
+
+      group_name = 'test_group'
+      @db.models[:group].create(name: group_name)
+
+      var = { name: 'var1', value: 'testval' }
+      cases = %w(
+        testvar
+        testvar=
+        =testval
+        testvar=testval=
+        =testvar=testval
+        testvar=testval=extra
+      )
+
+      cases.each do |args|
+        actual = runner do
+          @app.start(%W(group addvar #{group_name} #{args}))
+        end
+        # @console.out(actual,'p')
+
+        desired = { aborted: true }
+        desired[:STDOUT] =
+          "Add variables '#{args}' to group '#{group_name}':\n"\
+          "  - retrieve group '#{group_name}'...\n"\
+          "    - OK\n"\
+          "  - add variable '#{args}'...\n"
+
+        desired[:STDERR] =
+          "An error occurred during a transaction, any changes have been rolled back.\n"\
+          "ERROR: Incorrect format in '{#{args}}'. Expected 'key=value'.\n"
+
+        expected(actual, desired)
+      end
+    end
+
     #------------------------
     it 'GROUP key=value ... should associate the group with the key/value pair' do
       # 1. Should add the var to the db
       # 2. Should associate the host with the var
-      
+
       group_name = 'test1'
-      var  = {name: 'var1', value: "testval"}
+      var = { name: 'var1', value: 'testval' }
 
       @db.models[:group].create(name: group_name)
-      
+
       actual = runner do
-        @app.start(%W(group addvar #{group_name} #{var[:name]}=#{var[:value]} ))
+        @app.start(%W(group addvar #{group_name} #{var[:name]}=#{var[:value]}))
       end
-      #@console.out(actual,'p')
-      
-      desired = { aborted: false}
-      desired[:STDOUT] = 
+      # @console.out(actual,'p')
+
+      desired = { aborted: false }
+      desired[:STDOUT] =
         "Add variables '#{var[:name]}=#{var[:value]}' to group '#{group_name}':\n"\
         "  - retrieve group '#{group_name}'...\n"\
         "    - OK\n"\
@@ -146,71 +146,71 @@ RSpec.describe Moose::Inventory::Cli::Group do
       expect(groupvars[name: var[:name]]).not_to be_nil
       expect(groupvars[name: var[:name]][:value]).to eq(var[:value])
     end
-     
+
     #------------------------
     it 'GROUP key1=value1 key2=value2 ... should associate the group with multiple key/value pairs' do
       # 1. Should add the var to the db
       # 2. Should associate the host with the var
-      
+
       group_name = 'test1'
       varsarray  = [
-        {name: 'var1', value: "val1"},
-        {name: 'var2', value: "val2"}
+        { name: 'var1', value: 'val1' },
+        { name: 'var2', value: 'val2' },
       ]
-    
+
       vars = []
       varsarray.each do |var|
         vars << "#{var[:name]}=#{var[:value]}"
       end
-        
+
       @db.models[:group].create(name: group_name)
-      
+
       actual = runner do
-        @app.start(%W(group addvar #{group_name}) + vars )
+        @app.start(%W(group addvar #{group_name}) + vars)
       end
-      
-      #@console.out(actual,'y')
-      
-      desired = { aborted: false} 
-      desired[:STDOUT] = 
+
+      # @console.out(actual,'y')
+
+      desired = { aborted: false }
+      desired[:STDOUT] =
         "Add variables '#{vars.join(',')}' to group '#{group_name}':\n"\
         "  - retrieve group '#{group_name}'...\n"\
         "    - OK\n"
       vars.each do |var|
-        desired[:STDOUT] =  desired[:STDOUT] + 
-          "  - add variable '#{var}'...\n"\
-          "    - OK\n"
+        desired[:STDOUT] =  desired[:STDOUT] +
+                            "  - add variable '#{var}'...\n"\
+                            "    - OK\n"
       end
-      desired[:STDOUT] =  desired[:STDOUT] + 
-        "  - all OK\n"\
-        "Succeeded.\n"
+      desired[:STDOUT] = desired[:STDOUT] +
+                         "  - all OK\n"\
+                         "Succeeded.\n"
       expected(actual, desired)
-    
+
       # We should have the correct hostvar associations
       group = @db.models[:group].find(name: group_name)
       groupvars = group.groupvars_dataset
       expect(vars.count).to eq(vars.length)
-    end    
+    end
 
     #------------------------
     it 'GROUP key=value ... should update an already existing association' do
       # 1. Should add the var to the db
       # 2. Should associate the host with the var
-      
+
       group_name = 'test1'
-      var  = {name: 'var1', value: "testval"}
-    
+      var = { name: 'var1', value: 'testval' }
+
       @db.models[:group].create(name: group_name)
-      runner { @app.start(%W(group addvar #{group_name} #{var[:name]}=#{var[:value]} )) }
-    
-      var[:value]  = "newtestval"
+      runner { @app.start(%W(group addvar #{group_name} #{var[:name]}=#{var[:value]})) }
+
+      var[:value] = 'newtestval'
       actual = runner do
-        @app.start(%W(group addvar #{group_name} #{var[:name]}=#{var[:value]} ))
+        @app.start(%W(group addvar #{group_name} #{var[:name]}=#{var[:value]}))
       end
-      #@console.out(actual,'y')
-      
-      desired = { aborted: false}
-      desired[:STDOUT] = 
+      # @console.out(actual,'y')
+
+      desired = { aborted: false }
+      desired[:STDOUT] =
         "Add variables '#{var[:name]}=#{var[:value]}' to group '#{group_name}':\n"\
         "  - retrieve group '#{group_name}'...\n"\
         "    - OK\n"\
@@ -220,16 +220,16 @@ RSpec.describe Moose::Inventory::Cli::Group do
         "  - all OK\n"\
         "Succeeded.\n"
       expected(actual, desired)
-    
+
       # We should have the correct hostvar associations
       group = @db.models[:group].find(name: group_name)
       groupvars = group.groupvars_dataset
       expect(groupvars.count).to eq(1)
       expect(groupvars[name: var[:name]]).not_to be_nil
       expect(groupvars[name: var[:name]][:value]).to eq(var[:value])
-        
+
       groupvars = @db.models[:groupvar].all
-      expect(groupvars.count).to eq(1)   
+      expect(groupvars.count).to eq(1)
     end
   end
 end

data/spec/lib/moose_inventory/cli/group_get_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 = []
@@ -21,7 +21,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
     @config = Moose::Inventory::Config
     @config.init(@mockargs)
     @console = Moose::Inventory::Cli::Formatter
-    
+
     @db = Moose::Inventory::DB
     @db.init if @db.db.nil?
 
@@ -43,23 +43,23 @@ RSpec.describe Moose::Inventory::Cli::Group do
 
     #---------------------
     it '<missing args> ... should abort with an error' do
-      actual = runner { @app.start(%W(group get)) }
+      actual = runner { @app.start(%w(group get)) }
+
+      # @console.out(actual,'y')
 
-      #@console.out(actual,'y')
-      
-      desired = {aborted: true}
+      desired = { aborted: true }
       desired[:STDERR] = "ERROR: Wrong number of arguments, 0 for 1 or more\n"
 
       expected(actual, desired)
     end
-    
+
     #---------------------
     it "GROUP ... should return an empty set when GROUP doesn't exist" do
       group_name = 'does-not-exist'
-      actual = runner { @app.start(%W(group get #{ group_name })) }
+      actual = runner { @app.start(%W(group get #{group_name})) }
+
+      # @console.out(actual, 'y')
 
-      #@console.out(actual, 'y')
-      
       desired = {}
       desired[:STDOUT] = {}.to_yaml
 
@@ -69,14 +69,14 @@ RSpec.describe Moose::Inventory::Cli::Group do
     #---------------------
     it 'GROUP ... should get a group from the db' do
       name = 'test_group'
-      runner { @app.start(%W(group add #{ name })) }
+      runner { @app.start(%W(group add #{name})) }
 
-      actual = runner { @app.start(%W(group get #{ name })) }
+      actual = runner { @app.start(%W(group get #{name})) }
 
       mock = {}
       mock[name.to_sym] = {}
-      # mock[name.to_sym][:hosts] = [] # TODO: Should this be present or not? 
-        
+      # mock[name.to_sym][:hosts] = [] # TODO: Should this be present or not?
+
       desired = { aborted: false, STDOUT: '', STDERR: '' }
       desired[:STDOUT] = mock.to_yaml
 
@@ -87,21 +87,20 @@ RSpec.describe Moose::Inventory::Cli::Group do
     it 'GROUP ... should display groupvars, if any are set' do
       name = 'test_group'
       var = 'foo=bar'
-      tmp = runner { @app.start(%W(group add #{ name })) }
-      tmp = runner { @app.start(%W(group addvar #{ name } #{ var })) }
+      tmp = runner { @app.start(%W(group add #{name})) }
+      tmp = runner { @app.start(%W(group addvar #{name} #{var})) }
+
+      actual = runner { @app.start(%W(group get #{name})) }
+      # @console.out(actual, 'y')
 
-      actual = runner { @app.start(%W(group get #{ name })) }
-      #@console.out(actual, 'y')
-      
       mock = {}
       mock[name.to_sym] = {}
-      mock[name.to_sym][:groupvars] = {foo: 'bar'}
-        
+      mock[name.to_sym][:groupvars] = { foo: 'bar' }
+
       desired = {}
       desired[:STDOUT] = mock.to_yaml
-       
+
       expected(actual, desired)
     end
-        
   end
 end

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

@@ -6,7 +6,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 = []
@@ -23,7 +23,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
 
     @console = Moose::Inventory::Cli::Formatter
     @group = Moose::Inventory::Cli::Group
-    @cli = Moose::Inventory::Cli 
+    @cli = Moose::Inventory::Cli
     @app = Moose::Inventory::Cli::Application
   end
 
@@ -42,7 +42,7 @@ RSpec.describe Moose::Inventory::Cli::Group do
     #---------------------
     it 'should return an empty set when no results' do
       # no items in the db
-      actual = runner { @app.start(%W(group list)) }
+      actual = runner { @app.start(%w(group list)) }
 
       desired = { aborted: false, STDOUT: '', STDERR: '' }
       desired[:STDOUT] = {}.to_yaml
@@ -59,14 +59,14 @@ RSpec.describe Moose::Inventory::Cli::Group do
       groups = %w(group1 group2 group3)
       groups.each do |name|
         runner { @app.start(%W(group add #{name})) }
-        runner { @app.start(%W(group addvar #{ name } #{var})) }
+        runner { @app.start(%W(group addvar #{name} #{var})) }
         mock[name.to_sym] = {}
-        mock[name.to_sym][:groupvars] = {foo: 'bar'}
+        mock[name.to_sym][:groupvars] = { foo: 'bar' }
       end
 
       # items should now be in the db
-      actual = runner{ @app.start(%w(group list)) }
-        
+      actual = runner { @app.start(%w(group list)) }
+
       desired = { aborted: false, STDOUT: '', STDERR: '' }
       desired[:STDOUT] = mock.to_yaml
 
@@ -75,28 +75,27 @@ RSpec.describe Moose::Inventory::Cli::Group do
 
     #---------------------
     it 'should be an alias of --list (i.e. Ansible parameter)' do
-      
       host_name = 'test_host'
 
       mock = {}
       groups = %w(group1 group2 group3)
       groups.each do |name|
         runner { @app.start(%W(group add #{name})) }
-        mock[name.to_sym] = {}
+        mock[name.to_sym] = { hosts: [] }
       end
-    
+
       args = @mockargs.clone
-      args << "--list"
-    
-      actual = runner{ @cli.start(args) }
-        
-      #@console.out(actual, 'y')
-       
+      args << '--list'
+
+      actual = runner { @cli.start(args) }
+
+      # @console.out(actual, 'y')
+
       desired = { aborted: false, STDOUT: '', STDERR: '' }
       desired[:STDOUT] = mock.to_json + "\n"
-    
+
       expected(actual, desired)
-    end    
-    
+    end
+    end
   end
-end
+

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

@@ -6,7 +6,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 = []
@@ -23,12 +23,12 @@ RSpec.describe Moose::Inventory::Cli::Group do
 
     @console = Moose::Inventory::Cli::Formatter
     @group = Moose::Inventory::Cli::Group
-    @cli = Moose::Inventory::Cli 
+    @cli = Moose::Inventory::Cli
     @app = Moose::Inventory::Cli::Application
   end
 
   before(:each) do
-    # We make some @cli calls, which changes config, 
+    # We make some @cli calls, which changes config,
     # so we must reset config on each pass
     @config.init(@mockargs)
     @db.reset
@@ -41,79 +41,78 @@ RSpec.describe Moose::Inventory::Cli::Group do
       result = @group.instance_methods(false).include?(:listvars)
       expect(result).to eq(true)
     end
-     
+
     #-----------------
     it '<missing args> ... should abort with an error' do
-      actual = runner  {  @app.start(%w(group listvars))  }
+      actual = runner  { @app.start(%w(group listvars)) }
 
       # Check output
-      desired = { aborted: true}
+      desired = { aborted: true }
       desired[:STDERR] = "ERROR: Wrong number of arguments, 0 for 1 or more.\n"
       expected(actual, desired)
     end
-    
+
     #-----------------
     it '--ansible <missing args> ... should abort with an error' do
-      
       args = @mockargs.clone
-      args.concat( %w(--ansible group listvars) ).flatten
-    
-      actual = runner{ @cli.start(args) }
- 
+      args.concat(%w(--ansible group listvars)).flatten
+
+      actual = runner { @cli.start(args) }
+
       # Check output
-      desired = { aborted: true}
+      desired = { aborted: true }
       desired[:STDERR] = "ERROR: Wrong number of arguments for Ansible mode, 0 for 1.\n"
       expected(actual, desired)
     end
-    
+
     #------------------------
     it 'GROUP ... should return a list of group variables grouped by group' do
-      group_name ='test_group'
+      group_name = 'test_group'
       group_vars = %w(foo=bar cow=chicken)
-      
-      tmp = runner {  @app.start(%W(group add #{group_name} )) }
+
+      tmp = runner {  @app.start(%W(group add #{group_name})) }
       tmp = runner {  @app.start(%W(group addvar #{group_name} #{group_vars[0]} #{group_vars[1]})) }
-      
+
       actual = runner do
         @app.start(%W(group listvars #{group_name}))
       end
 
-      #@console.out(actual, 'y')
-      
+      # @console.out(actual, 'y')
+
       # Check output
       mock = {}
-      mock[group_name.to_sym] = {} 
+      mock[group_name.to_sym] = {}
       group_vars.each do |hv|
-        hv_array = hv.split('=')  
-        mock[group_name.to_sym][hv_array[0].to_sym] = hv_array[1] 
+        hv_array = hv.split('=')
+        mock[group_name.to_sym][hv_array[0].to_sym] = hv_array[1]
       end
-      
+
       desired = {}
       desired[:STDOUT] = mock.to_yaml
       expected(actual, desired)
     end
-    
+
     #------------------------
     it '--ansible GROUP ... should return a list of group variables, in a  style akin to Ansible\'s \'--host HOSTNAME\'' do
-      group_name ='test_group'
+      group_name = 'test_group'
       group_vars = %w(foo=bar cow=chicken)
-      
-      tmp = runner {  @app.start(%W(group add #{group_name} )) }
+
+      tmp = runner {  @app.start(%W(group add #{group_name})) }
       tmp = runner {  @app.start(%W(group addvar #{group_name} #{group_vars[0]} #{group_vars[1]})) }
-      
+
       actual = runner do
-        @cli.start(%W(--ansible group listvars #{group_name}))
+        @cli.start(%W(--config #{@mockarg_parts[:config]} --ansible group listvars #{group_name}))
       end
-    
-      #@console.out(actual, 'y')
-      
+
+      # @console.out(actual, 'y')
+
       # Check output
       mock = {}
       group_vars.each do |hv|
-        hv_array = hv.split('=')  
-        mock[hv_array[0].to_sym] = hv_array[1] 
+        hv_array = hv.split('=')
+        mock[hv_array[0].to_sym] = hv_array[1]
       end
-      
+
       desired = {}
       desired[:STDOUT] = mock.to_json + "\n"
       expected(actual, desired)