ansible_spec 0.2.19 → 0.2.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 661e661ae85944e236e4758161ba33e90935f00a
-  data.tar.gz: 57b656bca472d717d93e2fa1db6126e49d6afdbe
+  metadata.gz: 739a7ec257b6767370c7919f2cbf8a5a0508f73c
+  data.tar.gz: d391ada2f510a29d259a5d7f6a51585857ffd5cf
 SHA512:
-  metadata.gz: 18a2dc21930270208b66f522beb6b5458363950c88b96bcb4426ce66f0b77fee3ae3ba4ab0c35579b268c0692df0af8ac9241604d3b9730c829951a2f92258a5
-  data.tar.gz: c64b901c495a6df54342263d15472606b04b0b6833a7a31f378a1c3ff4200bb6526745231648d83046fa9919236465f2c03cffae87907708061b4d6e098c10ba
+  metadata.gz: ebe5c892c76fb3a1e30e06be59f3b88a94da0521ac92a64705acd20077a392cb98067b38c9393abc0e1b60ea91547240b160a3521ba96c7bb9208ae5998bc3db
+  data.tar.gz: a9f50b89fbba2ce8553517f3bba0033131b378af1760ec14153cc1069c8848cc2b0314de729833a7a7c1001ef32e468b923436b4b770c5c62f9fb45daf1a4da8

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+# v0.2.20
+- Merge [#101 Added feature set path to group_vars and host_vars](https://github.com/volanja/ansible_spec/pull/101) by [rbramwell](https://github.com/rbramwell)
+- Merge [#99 Unbreak 'serverspec:all' if groups with no hosts exist](https://github.com/volanja/ansible_spec/pull/99) by [agx](https://github.com/agx)
+
+
 # v0.2.19
 - Merge [#98 delete directory('roles','more_roles') after execute rpsec](https://github.com/volanja/ansible_spec/pull/98) by volanja
 - Merge [#97 Add local connection test path](https://github.com/volanja/ansible_spec/pull/97) by [mtoriumi](https://github.com/mtoriumi)

data/README.md

@@ -48,6 +48,7 @@ customize the playbook and inventory using an `.ansiblespec` file.
 -
   playbook: site.yml
   inventory: hosts
+  vars_dirs_path: inventories/staging
   hash_behaviour: merge
 ```
 
@@ -55,9 +56,10 @@ customize the playbook and inventory using an `.ansiblespec` file.
 
 You can use environment variables with the `rake` command. They are listed below.
 
-- `PLAYBOOK`       -- playbook name                                (e.g. `site.yml`)
-- `INVENTORY`      -- inventory file name                          (e.g. `hosts`)
-- `HASH_BEHAVIOUR` -- hash behaviour when duplicate hash variables (e.g. `merge`)
+- `PLAYBOOK`        -- playbook name                                (e.g. `site.yml`)
+- `INVENTORY`       -- inventory file name                          (e.g. `hosts`)
+- `VARS_DIRS_PATH`  -- directory path containing the group_vars and host_vars directories (e.g. `inventories/staging`)
+- `HASH_BEHAVIOUR`  -- hash behaviour when duplicate hash variables (e.g. `merge`)
 - `SSH_CONFIG_FILE` -- ssh configuration file path (e.g. `ssh_config`)  
 `SSH_CONFIG_FILE` take precedence over the path at ssh_args( -F "filename") in [ssh_connection] section of ansible.cfg
 
@@ -73,6 +75,8 @@ $ PLAYBOOK=site.yml rake serverspec:Ansible-Sample-TDD
 or
 $ INVENTORY=hosts rake serverspec:Ansible-Sample-TDD
 or
+$ VARS_DIRS_PATH=inventories/staging rake serverspec:Ansible-Sample-TDD
+or
 $ HASH_BEHAVIOUR=merge rake serverspec:Ansible-Sample-TDD
 ```
 
@@ -148,7 +152,7 @@ Ansible variables supported by following condition.
 * Playbook's variables are supported. If same variable is defined in different places,
   priority follows [Ansible order](http://docs.ansible.com/ansible/playbooks_variables.html#variable-precedence-where-should-i-put-a-variable).
 * Variables defined main.yml in role's tasks are not supported.
-* Inventry variables are not supported.
+* Inventory variables are not supported.
 * Facts are not supported.
 
 ### Sample
@@ -174,7 +178,46 @@ Support variables are in site.yml, group_vars, host_vars, roles.
 
 ```
 
-**Note:** Parse roledirs from ansible.cfg. This allows us to find specs that are not under ./roles.
+and (e.g. when `vars_dirs_path: inventories/staging`)
+
+```
+├── site.yml
+├── inventories
+│   ├── development
+|   |   ├── group_vars
+|   |   │   ├── all.yml
+|   |   │   ├── dbserver.yml
+|   |   │   └── webserver.yml
+|   |   └── host_vars
+|   |       ├── 192.168.1.1.yml
+|   |       └── 192.168.1.2.yml
+│   ├── production
+|   |   ├── group_vars
+|   |   │   ├── all.yml
+|   |   │   ├── dbserver.yml
+|   |   │   └── webserver.yml
+|   |   └── host_vars
+|   |       ├── 192.168.10.1.yml
+|   |       └── 192.168.10.2.yml
+│   └── staging
+|       ├── group_vars
+|       │   ├── all.yml
+|       │   ├── dbserver.yml
+|       │   └── webserver.yml
+|       └── host_vars
+|           ├── 192.168.20.1.yml
+|           └── 192.168.20.2.yml
+└── roles
+    ├── apaches
+    │   └── vars
+    │       └── main.yml
+    └── mariadb
+        └── vars
+            └── main.yml
+
+```
+
+**Note:** Parse role dirs from ansible.cfg. This allows us to find specs that are not under ./roles.
 
 ```
 # ansible.cfg

data/lib/ansible_spec/load_ansible.rb

@@ -205,7 +205,7 @@ module AnsibleSpec
 
       if File.exist?(path)
         dependencies = YAML.load_file(path).fetch("dependencies", [])
-        unless dependencies.nil? 
+        unless dependencies.nil?
           new_deps = dependencies.map { |h|
             h["role"] || h
           }
@@ -387,8 +387,8 @@ module AnsibleSpec
             hosts["#{v}"].map {|target_server| target_server["hosts"] = v}
             var["hosts"].concat hosts["#{v}"]
           end
-        end 
-        if var["hosts"].size == 0 
+        end
+        if var["hosts"].size == 0
           properties = properties.compact.reject{|e| e["hosts"].length == 0}
           #puts "#{var["name"]} roles no hosts matched for #{var["hosts"]}"
         end
@@ -400,6 +400,24 @@ module AnsibleSpec
     return properties
   end
 
+  # param: none
+  # return: vars_dirs_path
+  def self.get_vars_dirs_path()
+    f = '.ansiblespec'
+    y = nil
+    if File.exist?(f)
+      y = YAML.load_file(f)
+    end
+    if ENV["VARS_DIRS_PATH"]
+      vars_dirs_path = ENV["VARS_DIRS_PATH"]
+    elsif y.is_a?(Array) && y[0]['vars_dirs_path']
+      vars_dirs_path = y[0]['vars_dirs_path']
+    else
+      vars_dirs_path = ''
+    end
+    return vars_dirs_path
+  end
+
   def self.find_group_vars_file(hosts_childrens, hosts)
       target_host = hosts_childrens.select { |key, value|
         value["children"].include?(hosts)
@@ -416,16 +434,22 @@ module AnsibleSpec
       vars = load_vars_file(vars ,"roles/#{role}/defaults/main.yml")
     end
 
+    # get parent directory of group_vars and host_vars directories
+    vars_dirs_path = get_vars_dirs_path
+    if vars_dirs_path != ''
+      vars_dirs_path = "#{vars_dirs_path}/"
+    end
+
     # all group
-    vars = load_vars_file(vars ,'group_vars/all', true)
+    vars = load_vars_file(vars ,"#{vars_dirs_path}group_vars/all", true)
 
     # each group vars
     if p[group_idx].has_key?('group')
-      vars = load_vars_file(vars ,"group_vars/#{p[group_idx]['group']}", true)
+      vars = load_vars_file(vars ,"#{vars_dirs_path}group_vars/#{p[group_idx]['group']}", true)
     end
 
     # each host vars
-    vars = load_vars_file(vars ,"host_vars/#{host}", true)
+    vars = load_vars_file(vars ,"#{vars_dirs_path}host_vars/#{host}", true)
 
     # site vars
     if p[group_idx].has_key?('vars')
@@ -442,7 +466,7 @@ module AnsibleSpec
       hosts_childrens = p[group_idx]["hosts_childrens"]
       next_find_target = hosts
       while(!next_find_target.nil? && hosts_childrens.size > 0)
-        vars = load_vars_file(vars ,"group_vars/#{next_find_target}", true)
+        vars = load_vars_file(vars ,"#{vars_dirs_path}group_vars/#{next_find_target}", true)
         group_vars_file = find_group_vars_file(hosts_childrens,next_find_target)
         next_find_target = group_vars_file
         hosts_childrens.delete(group_vars_file)

data/lib/ansible_spec/version.rb

@@ -1,3 +1,3 @@
 module AnsibleSpec
-  VERSION = "0.2.19"
+  VERSION = "0.2.20"
 end

data/lib/src/Rakefile

@@ -12,8 +12,8 @@ desc "Run serverspec to all test"
 task :all => "serverspec:all"
 
 namespace :serverspec do
-  task :all => properties.map {|v| 'serverspec:' + v["name"] }
   properties = properties.compact.reject{|e| e["hosts"].length == 0}
+  task :all => properties.map {|v| 'serverspec:' + v["name"] }
   properties.each_with_index.map do |property, index|
     property["hosts"].each do |host|
       desc "Run serverspec for #{property["name"]}"

data/spec/case/get_variable/inventories_env_group_all/inventories/development/group_vars/all.yml

@@ -0,0 +1,6 @@
+---
+role_var:  "group all"
+site_var:  "group all"
+host_var:  "group all"
+group_var: "group all"
+group_all_var: "group all"

data/spec/case/get_variable/inventories_env_group_all/inventories/development/hosts

@@ -0,0 +1,10 @@
+[group1]
+192.168.1.1
+
+[group2]
+192.168.1.2
+
+[all]
+192.168.1.1
+192.168.1.2
+

data/spec/case/get_variable/inventories_env_group_all/inventories/production/group_vars/all.yml

@@ -0,0 +1,6 @@
+---
+role_var:  "group all"
+site_var:  "group all"
+host_var:  "group all"
+group_var: "group all"
+group_all_var: "group all"

data/spec/case/get_variable/inventories_env_group_all/inventories/production/hosts

@@ -0,0 +1,9 @@
+[group1]
+192.168.3.1
+
+[group2]
+192.168.3.2
+
+[all]
+192.168.3.1
+192.168.3.2

data/spec/case/get_variable/inventories_env_group_all/inventories/staging/group_vars/all.yml

@@ -0,0 +1,6 @@
+---
+role_var:  "group all"
+site_var:  "group all"
+host_var:  "group all"
+group_var: "group all"
+group_all_var: "group all"

data/spec/case/get_variable/inventories_env_group_all/inventories/staging/hosts

@@ -0,0 +1,9 @@
+[group1]
+192.168.2.1
+
+[group2]
+192.168.2.2
+
+[all]
+192.168.2.1
+192.168.2.2

data/spec/case/get_variable/inventories_env_group_all/roles/test/defaults/main.yml

@@ -0,0 +1,4 @@
+---
+role_var: 'role_var in roles default'
+site_var: 'site_var in roles default'
+host_var: 'host_var in roles default'

data/spec/case/get_variable/inventories_env_group_all/site.yml

@@ -0,0 +1,13 @@
+- name: group1
+  hosts: group1
+  connection: local
+  roles:
+  - test
+  - test/nested
+
+- name: group2
+  hosts: group2
+  connection: local
+  roles:
+  - test
+  - test/nested

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/development/group_vars/all.yml

@@ -0,0 +1,6 @@
+---
+role_var:  "group all"
+site_var:  "group all"
+host_var:  "group all"
+group_var: "group all"
+group_all_var: "group all"

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/development/group_vars/group1.yml

@@ -0,0 +1,4 @@
+role_var:  'group1'
+site_var:  'group1'
+host_var:  'group1'
+group_var: 'group1'

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/development/hosts

@@ -0,0 +1,10 @@
+[group1]
+192.168.1.1
+
+[group2]
+192.168.1.2
+
+[all]
+192.168.1.1
+192.168.1.2
+

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/production/group_vars/all.yml

@@ -0,0 +1,6 @@
+---
+role_var:  "group all"
+site_var:  "group all"
+host_var:  "group all"
+group_var: "group all"
+group_all_var: "group all"

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/production/group_vars/group1.yml

@@ -0,0 +1,4 @@
+role_var:  'group1'
+site_var:  'group1'
+host_var:  'group1'
+group_var: 'group1'

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/production/hosts

@@ -0,0 +1,9 @@
+[group1]
+192.168.3.1
+
+[group2]
+192.168.3.2
+
+[all]
+192.168.3.1
+192.168.3.2

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/group_vars/all.yml

@@ -0,0 +1,6 @@
+---
+role_var:  "group all"
+site_var:  "group all"
+host_var:  "group all"
+group_var: "group all"
+group_all_var: "group all"

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/group_vars/group1.yml

@@ -0,0 +1,4 @@
+role_var:  'group1'
+site_var:  'group1'
+host_var:  'group1'
+group_var: 'group1'

data/spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/hosts

@@ -0,0 +1,9 @@
+[group1]
+192.168.2.1
+
+[group2]
+192.168.2.2
+
+[all]
+192.168.2.1
+192.168.2.2

data/spec/case/get_variable/inventories_env_group_each_vars/site.yml

@@ -0,0 +1,13 @@
+- name: group1
+  hosts: group1
+  connection: local
+  roles:
+  - test
+  - test/nested
+
+- name: group2
+  hosts: group2
+  connection: local
+  roles:
+  - test
+  - test/nested

data/spec/case/get_variable/inventories_env_host_vars/inventories/development/host_vars/192.168.1.1.yml

@@ -0,0 +1,3 @@
+role_var: "host role_var"
+site_var: "host site_var"
+host_var: "host 192.168.1.1"

data/spec/case/get_variable/inventories_env_host_vars/inventories/development/host_vars/192.168.1.2.yml

@@ -0,0 +1,3 @@
+role_var: "host role_var"
+site_var: "host site_var"
+host_var: "host 192.168.1.2"

data/spec/case/get_variable/inventories_env_host_vars/inventories/development/hosts

@@ -0,0 +1,10 @@
+[group1]
+192.168.1.1
+
+[group2]
+192.168.1.2
+
+[all]
+192.168.1.1
+192.168.1.2
+

data/spec/case/get_variable/inventories_env_host_vars/inventories/production/host_vars/192.168.3.1.yml

@@ -0,0 +1,3 @@
+role_var: "host role_var"
+site_var: "host site_var"
+host_var: "host 192.168.3.1"

data/spec/case/get_variable/inventories_env_host_vars/inventories/production/host_vars/192.168.3.2.yml

@@ -0,0 +1,3 @@
+role_var: "host role_var"
+site_var: "host site_var"
+host_var: "host 192.168.3.2"

data/spec/case/get_variable/inventories_env_host_vars/inventories/production/hosts

@@ -0,0 +1,9 @@
+[group1]
+192.168.3.1
+
+[group2]
+192.168.3.2
+
+[all]
+192.168.3.1
+192.168.3.2

data/spec/case/get_variable/inventories_env_host_vars/inventories/staging/host_vars/192.168.2.1.yml

@@ -0,0 +1,3 @@
+role_var: "host role_var"
+site_var: "host site_var"
+host_var: "host 192.168.2.1"

data/spec/case/get_variable/inventories_env_host_vars/inventories/staging/host_vars/192.168.2.2.yml

@@ -0,0 +1,3 @@
+role_var: "host role_var"
+site_var: "host site_var"
+host_var: "host 192.168.2.2"

data/spec/case/get_variable/inventories_env_host_vars/inventories/staging/hosts

@@ -0,0 +1,9 @@
+[group1]
+192.168.2.1
+
+[group2]
+192.168.2.2
+
+[all]
+192.168.2.1
+192.168.2.2

data/spec/case/get_variable/inventories_env_host_vars/site.yml

@@ -0,0 +1,13 @@
+- name: group1
+  hosts: group1
+  connection: local
+  roles:
+  - test
+  - test/nested
+
+- name: group2
+  hosts: group2
+  connection: local
+  roles:
+  - test
+  - test/nested

data/spec/get_variable_vars_dirs_path_spec.rb

@@ -0,0 +1,321 @@
+# coding: utf-8
+require 'fileutils'
+require 'ansible_spec'
+
+describe "Run get_variables" do
+  context 'Correct operation : env VARS_DIRS_PATH=inventories/staging - all groups' do
+    require 'yaml'
+    tmp_playbook = 'site.yml'
+    tmp_inventory_file = 'inventories/staging/hosts'
+    tmp_vars_dirs_path = 'inventories/staging'
+
+    before do
+      @current_dir = Dir.pwd()
+      Dir.chdir('spec/case/get_variable/inventories_env_group_all/')
+      ENV["PLAYBOOK"] = tmp_playbook
+      ENV["INVENTORY"] = tmp_inventory_file
+      ENV["VARS_DIRS_PATH"] = tmp_vars_dirs_path
+      @res = AnsibleSpec.get_variables('192.168.2.1', 0)
+    end
+
+    it 'res is hash' do
+      expect(@res.instance_of?(Hash)).to be_truthy
+    end
+
+    it 'exist 5 pair in Hash' do
+      expect(@res.length).to eq 5
+    end
+
+    it 'exist each pair' do
+      expect(@res).to include({'role_var' => 'group all'},
+                              {'site_var' => 'group all'},
+                              {'host_var' => 'group all'},
+                              {'group_var' => 'group all'},
+                              {'group_all_var' => 'group all'}
+                             )
+    end
+
+    after do
+      ENV.delete('PLAYBOOK')
+      ENV.delete('INVENTORY')
+      ENV.delete('VARS_DIRS_PATH')
+      Dir.chdir(@current_dir)
+    end
+  end
+
+  context 'Correct operation : env VARS_DIRS_PATH=inventories/staging - each group vars' do
+    require 'yaml'
+    tmp_playbook = 'site.yml'
+    tmp_inventory_file = 'inventories/staging/hosts'
+    tmp_vars_dirs_path = 'inventories/staging'
+
+    before do
+      @current_dir = Dir.pwd()
+      Dir.chdir('spec/case/get_variable/inventories_env_group_each_vars/')
+      ENV["PLAYBOOK"] = tmp_playbook
+      ENV["INVENTORY"] = tmp_inventory_file
+      ENV["VARS_DIRS_PATH"] = tmp_vars_dirs_path
+      @res = AnsibleSpec.get_variables('192.168.2.1', 0)
+    end
+
+    it 'res is hash' do
+      expect(@res.instance_of?(Hash)).to be_truthy
+    end
+
+    it 'exist 5 pair in Hash' do
+      expect(@res.length).to eq 5
+    end
+
+    it 'exist each pair' do
+      expect(@res).to include({'role_var' => 'group1'},
+                              {'site_var' => 'group1'},
+                              {'host_var' => 'group1'},
+                              {'group_var' => 'group1'},
+                              {'group_all_var' => 'group all'}
+                             )
+    end
+
+    after do
+      ENV.delete('PLAYBOOK')
+      ENV.delete('INVENTORY')
+      ENV.delete('VARS_DIRS_PATH')
+      Dir.chdir(@current_dir)
+    end
+  end
+
+  context 'Correct operation : env VARS_DIRS_PATH=inventories/staging - host_vars' do
+    require 'yaml'
+    tmp_playbook = 'site.yml'
+    tmp_inventory_file = 'inventories/staging/hosts'
+    tmp_vars_dirs_path = 'inventories/staging'
+
+    before do
+      @current_dir = Dir.pwd()
+      Dir.chdir('spec/case/get_variable/inventories_env_host_vars/')
+      ENV["PLAYBOOK"] = tmp_playbook
+      ENV["INVENTORY"] = tmp_inventory_file
+      ENV["VARS_DIRS_PATH"] = tmp_vars_dirs_path
+      @res = AnsibleSpec.get_variables('192.168.2.1', 0)
+    end
+
+    it 'res is hash' do
+      expect(@res.instance_of?(Hash)).to be_truthy
+    end
+
+    it 'exist 3 pair in Hash' do
+      expect(@res.length).to eq 3
+    end
+
+    it 'exist each pair' do
+      expect(@res).to include({'role_var' => 'host role_var'},   # in host_var/192.168.2.1.yml
+                              {'site_var' => 'host site_var'},   # in host_var/192.168.2.1.yml
+                              {'host_var' => 'host 192.168.2.1'} # in host_var/192.168.2.1.yml
+                             )
+    end
+
+    after do
+      ENV.delete('PLAYBOOK')
+      ENV.delete('INVENTORY')
+      ENV.delete('VARS_DIRS_PATH')
+      Dir.chdir(@current_dir)
+    end
+  end
+
+  context 'Correct operation : env VARS_DIRS_PATH=inventories/staging - host_vars' do
+    require 'yaml'
+    tmp_playbook = 'site.yml'
+    tmp_inventory_file = 'inventories/staging/hosts'
+    tmp_vars_dirs_path = 'inventories/staging'
+
+    before do
+      @current_dir = Dir.pwd()
+      Dir.chdir('spec/case/get_variable/inventories_env_host_vars/')
+      ENV["PLAYBOOK"] = tmp_playbook
+      ENV["INVENTORY"] = tmp_inventory_file
+      ENV["VARS_DIRS_PATH"] = tmp_vars_dirs_path
+      @res = AnsibleSpec.get_variables('192.168.2.2', 1)
+    end
+
+    it 'res is hash' do
+      expect(@res.instance_of?(Hash)).to be_truthy
+    end
+
+    it 'exist 3 pair in Hash' do
+      expect(@res.length).to eq 3
+    end
+
+    it 'exist each pair' do
+      expect(@res).to include({'role_var' => 'host role_var'},   # in host_var/192.168.2.2.yml
+                              {'site_var' => 'host site_var'},   # in host_var/192.168.2.2.yml
+                              {'host_var' => 'host 192.168.2.2'} # in host_var/192.168.2.2.yml
+                             )
+    end
+
+    after do
+      ENV.delete('PLAYBOOK')
+      ENV.delete('INVENTORY')
+      ENV.delete('VARS_DIRS_PATH')
+      Dir.chdir(@current_dir)
+    end
+  end
+
+  context 'Correct operation : .ansiblespec "vars_dirs_path: inventories/staging" - all groups' do
+    require 'yaml'
+    tmp_ansiblespec = '.ansiblespec'
+
+    before do
+      @current_dir = Dir.pwd()
+      Dir.chdir('spec/case/get_variable/inventories_env_group_all/')
+      content = <<'EOF'
+---
+-
+  playbook: 'site.yml'
+  inventory: 'inventories/staging/hosts'
+  vars_dirs_path: 'inventories/staging'
+EOF
+      create_file(tmp_ansiblespec,content)
+      @res = AnsibleSpec.get_variables('192.168.2.1', 0)
+    end
+
+    it 'res is hash' do
+      expect(@res.instance_of?(Hash)).to be_truthy
+    end
+
+    it 'exist 5 pair in Hash' do
+      expect(@res.length).to eq 5
+    end
+
+    it 'exist each pair' do
+      expect(@res).to include({'role_var' => 'group all'},
+                              {'site_var' => 'group all'},
+                              {'host_var' => 'group all'},
+                              {'group_var' => 'group all'},
+                              {'group_all_var' => 'group all'}
+                             )
+    end
+
+    after do
+      File.delete(tmp_ansiblespec)
+      Dir.chdir(@current_dir)
+    end
+  end
+
+  context 'Correct operation : .ansiblespec "vars_dirs_path: inventories/staging" - each group vars' do
+    require 'yaml'
+    tmp_ansiblespec = '.ansiblespec'
+
+    before do
+      @current_dir = Dir.pwd()
+      Dir.chdir('spec/case/get_variable/inventories_env_group_each_vars/')
+      content = <<'EOF'
+---
+-
+  playbook: 'site.yml'
+  inventory: 'inventories/staging/hosts'
+  vars_dirs_path: 'inventories/staging'
+EOF
+      create_file(tmp_ansiblespec,content)
+      @res = AnsibleSpec.get_variables('192.168.2.1', 0)
+    end
+
+    it 'res is hash' do
+      expect(@res.instance_of?(Hash)).to be_truthy
+    end
+
+    it 'exist 5 pair in Hash' do
+      expect(@res.length).to eq 5
+    end
+
+    it 'exist each pair' do
+      expect(@res).to include({'role_var' => 'group1'},
+                              {'site_var' => 'group1'},
+                              {'host_var' => 'group1'},
+                              {'group_var' => 'group1'},
+                              {'group_all_var' => 'group all'}
+                             )
+    end
+
+    after do
+      File.delete(tmp_ansiblespec)
+      Dir.chdir(@current_dir)
+    end
+  end
+
+  context 'Correct operation : .ansiblespec "vars_dirs_path: inventories/staging" - host_vars' do
+    require 'yaml'
+    tmp_ansiblespec = '.ansiblespec'
+
+    before do
+      @current_dir = Dir.pwd()
+      Dir.chdir('spec/case/get_variable/inventories_env_host_vars/')
+      content = <<'EOF'
+---
+-
+  playbook: 'site.yml'
+  inventory: 'inventories/staging/hosts'
+  vars_dirs_path: 'inventories/staging'
+EOF
+      create_file(tmp_ansiblespec,content)
+      @res = AnsibleSpec.get_variables('192.168.2.1', 0)
+    end
+
+    it 'res is hash' do
+      expect(@res.instance_of?(Hash)).to be_truthy
+    end
+
+    it 'exist 3 pair in Hash' do
+      expect(@res.length).to eq 3
+    end
+
+    it 'exist each pair' do
+      expect(@res).to include({'role_var' => 'host role_var'},   # in host_var/192.168.2.1.yml
+                              {'site_var' => 'host site_var'},   # in host_var/192.168.2.1.yml
+                              {'host_var' => 'host 192.168.2.1'} # in host_var/192.168.2.1.yml
+                             )
+    end
+
+    after do
+      File.delete(tmp_ansiblespec)
+      Dir.chdir(@current_dir)
+    end
+  end
+
+  context 'Correct operation : .ansiblespec "vars_dirs_path: inventories/staging" - host_vars' do
+    require 'yaml'
+    tmp_ansiblespec = '.ansiblespec'
+
+    before do
+      @current_dir = Dir.pwd()
+      Dir.chdir('spec/case/get_variable/inventories_env_host_vars/')
+      content = <<'EOF'
+---
+-
+  playbook: 'site.yml'
+  inventory: 'inventories/staging/hosts'
+  vars_dirs_path: 'inventories/staging'
+EOF
+      create_file(tmp_ansiblespec,content)
+      @res = AnsibleSpec.get_variables('192.168.2.2', 1)
+    end
+
+    it 'res is hash' do
+      expect(@res.instance_of?(Hash)).to be_truthy
+    end
+
+    it 'exist 3 pair in Hash' do
+      expect(@res.length).to eq 3
+    end
+
+    it 'exist each pair' do
+      expect(@res).to include({'role_var' => 'host role_var'},   # in host_var/192.168.2.2.yml
+                              {'site_var' => 'host site_var'},   # in host_var/192.168.2.2.yml
+                              {'host_var' => 'host 192.168.2.2'} # in host_var/192.168.2.2.yml
+                             )
+    end
+
+    after do
+      File.delete(tmp_ansiblespec)
+      Dir.chdir(@current_dir)
+    end
+  end
+end

data/spec/get_vars_dirs_path_spec.rb

@@ -0,0 +1,157 @@
+# coding: utf-8
+require 'fileutils'
+require 'ansible_spec'
+
+def create_file(name,content)
+  dir = File.dirname(name)
+  unless File.directory?(dir)
+    FileUtils.mkdir_p(dir)
+  end
+  File.open(name, 'w') do |f|
+    f.puts content
+  end
+end
+
+describe "Run get_vars_dirs_path" do
+  context 'Normal system (environment variable VARS_DIRS_PATH not specified and vars_dirs_path not specified in .ansiblespec)' do
+    require 'yaml'
+    expected_default_vars_dirs_path = ''
+
+    before do
+      @vars_dirs_path = AnsibleSpec.get_vars_dirs_path()
+    end
+
+    it "vars_dirs_path is '#{expected_default_vars_dirs_path}'" do
+      expect(@vars_dirs_path).to eq expected_default_vars_dirs_path
+    end
+  end
+
+  context 'Normal system (environment variable VARS_DIRS_PATH specified as current directory and vars_dirs_path not specified in .ansiblespec)' do
+    require 'yaml'
+    tmp_vars_dirs_path = ''
+
+    before do
+      ENV['VARS_DIRS_PATH'] = tmp_vars_dirs_path
+      @vars_dirs_path = AnsibleSpec.get_vars_dirs_path()
+    end
+
+    it "vars_dirs_path is #{tmp_vars_dirs_path}" do
+      expect(@vars_dirs_path).to eq tmp_vars_dirs_path
+    end
+
+    after do
+      ENV.delete('VARS_DIRS_PATH')
+    end
+  end
+
+  context 'Normal system (environment variable VARS_DIRS_PATH specified relative to playbook base directory and vars_dirs_path not specified in .ansiblespec)' do
+    require 'yaml'
+    tmp_vars_dirs_path = 'inventories/staging'
+
+    before do
+      ENV['VARS_DIRS_PATH'] = tmp_vars_dirs_path
+      @vars_dirs_path = AnsibleSpec.get_vars_dirs_path()
+    end
+
+    it "vars_dirs_path is #{tmp_vars_dirs_path}" do
+      expect(@vars_dirs_path).to eq tmp_vars_dirs_path
+    end
+
+    after do
+      ENV.delete('VARS_DIRS_PATH')
+    end
+  end
+
+  context 'Normal system (environment variable VARS_DIRS_PATH specified as absolute path and vars_dirs_path not specified in .ansiblespec)' do
+    require 'yaml'
+    tmp_vars_dirs_path = '/etc/ansible/inventories/staging'
+
+    before do
+      ENV['VARS_DIRS_PATH'] = tmp_vars_dirs_path
+      @vars_dirs_path = AnsibleSpec.get_vars_dirs_path()
+    end
+
+    it "vars_dirs_path is #{tmp_vars_dirs_path}" do
+      expect(@vars_dirs_path).to eq tmp_vars_dirs_path
+    end
+
+    after do
+      ENV.delete('VARS_DIRS_PATH')
+    end
+  end
+
+  context 'Normal line (vars_dirs_path specified relative to playbook base directory in .ansiblespec and environment variable VARS_DIRS_PATH not specified)' do
+    require 'yaml'
+    tmp_ansiblespec = '.ansiblespec'
+    tmp_vars_dirs_path = 'inventories/staging'
+
+    before do
+
+      content = <<'EOF'
+---
+-
+  vars_dirs_path: 'inventories/staging'
+EOF
+      create_file(tmp_ansiblespec,content)
+      @vars_dirs_path = AnsibleSpec.get_vars_dirs_path()
+    end
+
+    it "vars_dirs_path is #{tmp_vars_dirs_path}" do
+      expect(@vars_dirs_path).to eq tmp_vars_dirs_path
+    end
+
+    after do
+      File.delete(tmp_ansiblespec)
+    end
+  end
+
+  context 'Normal line (vars_dirs_path specified as absolute path in .ansiblespec and environment variable VARS_DIRS_PATH not specified)' do
+    require 'yaml'
+    tmp_ansiblespec = '.ansiblespec'
+    tmp_vars_dirs_path = '/etc/ansible/inventories/staging'
+
+    before do
+
+      content = <<'EOF'
+---
+-
+  vars_dirs_path: '/etc/ansible/inventories/staging'
+EOF
+      create_file(tmp_ansiblespec,content)
+      @vars_dirs_path = AnsibleSpec.get_vars_dirs_path()
+    end
+
+    it "vars_dirs_path is #{tmp_vars_dirs_path}" do
+      expect(@vars_dirs_path).to eq tmp_vars_dirs_path
+    end
+
+    after do
+      File.delete(tmp_ansiblespec)
+    end
+  end
+
+  context 'Normal line (ENV VAR higher precedence check: vars_dirs_path specified in .ansiblespec and environment variable VARS_DIRS_PATH specified)' do
+    require 'yaml'
+    tmp_ansiblespec = '.ansiblespec'
+    tmp_vars_dirs_path = 'inventories/staging'
+
+    before do
+      ENV['VARS_DIRS_PATH'] = tmp_vars_dirs_path
+      content = <<'EOF'
+---
+-
+  vars_dirs_path: 'inventories/development'
+EOF
+      create_file(tmp_ansiblespec,content)
+      @vars_dirs_path = AnsibleSpec.get_vars_dirs_path()
+    end
+
+    it "vars_dirs_path is #{tmp_vars_dirs_path}" do
+      expect(@vars_dirs_path).to eq tmp_vars_dirs_path
+    end
+
+    after do
+      File.delete(tmp_ansiblespec)
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ansible_spec
 version: !ruby/object:Gem::Version
-  version: 0.2.19
+  version: 0.2.20
 platform: ruby
 authors:
 - volanja
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-18 00:00:00.000000000 Z
+date: 2017-04-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -258,6 +258,34 @@ files:
 - spec/case/get_variable/host_vars/host_vars/192.168.1.2.yml
 - spec/case/get_variable/host_vars/hosts
 - spec/case/get_variable/host_vars/site.yml
+- spec/case/get_variable/inventories_env_group_all/inventories/development/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_all/inventories/development/hosts
+- spec/case/get_variable/inventories_env_group_all/inventories/production/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_all/inventories/production/hosts
+- spec/case/get_variable/inventories_env_group_all/inventories/staging/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_all/inventories/staging/hosts
+- spec/case/get_variable/inventories_env_group_all/roles/test/defaults/main.yml
+- spec/case/get_variable/inventories_env_group_all/site.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/development/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/development/group_vars/group1.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/development/hosts
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/production/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/production/group_vars/group1.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/production/hosts
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/group_vars/group1.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/hosts
+- spec/case/get_variable/inventories_env_group_each_vars/site.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/development/host_vars/192.168.1.1.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/development/host_vars/192.168.1.2.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/development/hosts
+- spec/case/get_variable/inventories_env_host_vars/inventories/production/host_vars/192.168.3.1.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/production/host_vars/192.168.3.2.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/production/hosts
+- spec/case/get_variable/inventories_env_host_vars/inventories/staging/host_vars/192.168.2.1.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/staging/host_vars/192.168.2.2.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/staging/hosts
+- spec/case/get_variable/inventories_env_host_vars/site.yml
 - spec/case/get_variable/playbook_only/hosts
 - spec/case/get_variable/playbook_only/site.yml
 - spec/case/get_variable/roles_default/hosts
@@ -277,6 +305,8 @@ files:
 - spec/ec2_dynamic_inventory_spec.rb
 - spec/get_variable_hosts_childrens_spec.rb
 - spec/get_variable_spec.rb
+- spec/get_variable_vars_dirs_path_spec.rb
+- spec/get_vars_dirs_path_spec.rb
 - spec/inventory_parameters_spec.rb
 - spec/load_ansible_cfg_spec.rb
 - spec/load_ansible_cfg_ssh_spec.rb
@@ -403,6 +433,34 @@ test_files:
 - spec/case/get_variable/host_vars/host_vars/192.168.1.2.yml
 - spec/case/get_variable/host_vars/hosts
 - spec/case/get_variable/host_vars/site.yml
+- spec/case/get_variable/inventories_env_group_all/inventories/development/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_all/inventories/development/hosts
+- spec/case/get_variable/inventories_env_group_all/inventories/production/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_all/inventories/production/hosts
+- spec/case/get_variable/inventories_env_group_all/inventories/staging/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_all/inventories/staging/hosts
+- spec/case/get_variable/inventories_env_group_all/roles/test/defaults/main.yml
+- spec/case/get_variable/inventories_env_group_all/site.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/development/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/development/group_vars/group1.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/development/hosts
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/production/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/production/group_vars/group1.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/production/hosts
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/group_vars/all.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/group_vars/group1.yml
+- spec/case/get_variable/inventories_env_group_each_vars/inventories/staging/hosts
+- spec/case/get_variable/inventories_env_group_each_vars/site.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/development/host_vars/192.168.1.1.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/development/host_vars/192.168.1.2.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/development/hosts
+- spec/case/get_variable/inventories_env_host_vars/inventories/production/host_vars/192.168.3.1.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/production/host_vars/192.168.3.2.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/production/hosts
+- spec/case/get_variable/inventories_env_host_vars/inventories/staging/host_vars/192.168.2.1.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/staging/host_vars/192.168.2.2.yml
+- spec/case/get_variable/inventories_env_host_vars/inventories/staging/hosts
+- spec/case/get_variable/inventories_env_host_vars/site.yml
 - spec/case/get_variable/playbook_only/hosts
 - spec/case/get_variable/playbook_only/site.yml
 - spec/case/get_variable/roles_default/hosts
@@ -422,6 +480,8 @@ test_files:
 - spec/ec2_dynamic_inventory_spec.rb
 - spec/get_variable_hosts_childrens_spec.rb
 - spec/get_variable_spec.rb
+- spec/get_variable_vars_dirs_path_spec.rb
+- spec/get_vars_dirs_path_spec.rb
 - spec/inventory_parameters_spec.rb
 - spec/load_ansible_cfg_spec.rb
 - spec/load_ansible_cfg_ssh_spec.rb