nugrant 1.1.0 → 1.2.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NTBjZWZmNDZmODZhZjM5YWQ2ZmVkNWI5YjRkZDUwZjc2YTdlNWY2YQ==
+    Y2U0MTBmOTMyYzhmYzk5NDBjOTRlZmNhMWYyMDkyYzM3ZTBkOGRmZA==
   data.tar.gz: !binary |-
-    MTI2MWU0MWJmNTEwZjI5ZWQxZTRkYWZhMDAwODM2N2I3NGQ1NWFhMQ==
+    OGFkNTY2MjRjZTQ1ODhhM2FjNDBlNzVkZDE4NTA1NDdiODExNTExZA==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    N2VjMzZkZGI4MTE3MDdlYjM1YTY5MzY4ZjIwZjQ3ZmM4Y2VmMjMzMzJlODE2
-    Mjk4ODFiNjNhNDdjM2U1ZWViMGYwMmE3OWU3NTI1ZmY2MTJkY2M5ZWI3MzNj
-    ZjM5NmM4NjY1OGI2NWY4OTY3YWMxMWEwMjdjYzRiODI0OTYxNTE=
+    MjNmODVmZGQ1YjFlYjAxMjFjZWRmMWQyMDE0N2ZhNTlkM2RjNjM2NDJjYjVj
+    NGE1NGE0NzE0MzNmMDc5MTNmODZiMWI3MWZmMDA4ZWM3NzI4ZmYxOTMzMTI5
+    Njc0Zjg5ZTAwZTFmMzg5MTJhMjBiNzhlNzFmNzRhYTVlNmZlYzk=
   data.tar.gz: !binary |-
-    Nzc0NzFiZTE1MDQ3MTE2ZDhlMWMzMTk3ZGYzMzQ3NDFlNWQ2ZDJiYWJjYzRi
-    OGRhYmIzODg3N2U5OWY0NjgwYjQyY2I5NDM0ZjBjMTVjNjM3ZDU0MWExMTVj
-    MDE4ODE4OTVkZDliNTg4YTAxMDRlYzBjODAwMTUwODFmMjU5YjE=
+    NTFjZTVhZjIyNDEwN2VlMGMxZDg1MTYxY2ExZDVmZGMzYzI5NjExODgxZjY0
+    NWQ5NTBhNzYxNWRlYTUzNTE1NzU5MWY5YTE0MTJkMGEyYzdkNjM2OWNlMmZh
+    MmVkOGRiNmQ2OGIyZWVmMTVkN2FkZGZlZDY5MzdhY2YxY2I2M2Q=

data/.gitignore

@@ -18,4 +18,4 @@ test/tmp
 test/version_tmp
 tmp
 
-test/resources/Vagrantfile
+test/resources/vagrantfiles/Vagrantfile

data/CHANGELOG.md

@@ -1,3 +1,27 @@
+# 1.2.0 (October 24th, 2013)
+
+* Now showing better error message to the end-user when a parameter
+  cannot be found. The message displays which key could not be found.
+  Moreover, we show the context within the Vagrantfile where we think
+  the error occurred:
+
+  ```
+  Nugrant: Parameter 'param' was not found, is it defined in
+  your .vagrantuser file? Here where we think the error
+  could be in your Vagrantfile:
+
+   1:     Vagrant.configure("2") do |config|
+   2:>>     puts config.user.param
+   3:     end
+  ```
+
+  See [GH-8] (https://github.com/maoueh/nugrant/issues/8).
+* Ensured that keys used within a `Bag` are always symbol. This make
+  sure that it is possible to retrieve a value with any access method.
+  See [GH-9](https://github.com/maoueh/nugrant/issues/9).
+* Now using [multi_json](https://rubygems.org/gems/multi_json)
+  for JSON handling.
+
 # 1.1.0 (May 17th, 2013)
 
 * Rewrite completely classes `Parameters` and `Bag`.

data/Gemfile

@@ -5,16 +5,16 @@ gemspec
 group :development do
   vagrant_dependencies = {
     'v1' => {
-        'home' => "~/.vagrant.d.v1",
-        'gem' => Proc.new do
-            gem "vagrant", "~> 1.0.7"
-        end,
+      'home' => "~/.vagrant.d.v1",
+      'gem' => Proc.new do
+        gem "vagrant", "~> 1.0.7"
+      end,
     },
     'v2' => {
-        'home' => "~/.vagrant.d",
-        'gem' => Proc.new do
-            gem "vagrant", :git => "git://github.com/mitchellh/vagrant.git"
-        end,
+      'home' => "~/.vagrant.d",
+      'gem' => Proc.new do
+        gem "vagrant", :git => "git://github.com/mitchellh/vagrant.git", :tag => "v1.3.5"
+      end,
     },
   }
 

data/README.md

@@ -1,4 +1,14 @@
-# Nugrant [![Build Status](https://travis-ci.org/maoueh/nugrant.png)](https://travis-ci.org/maoueh/nugrant)
+# Nugrant
+
+[![Gem Version](https://badge.fury.io/rb/nugrant.png)][gem]
+[![Build Status](https://secure.travis-ci.org/maoueh/nugrant.png?branch=master)][travis]
+[![Dependency Status](https://gemnasium.com/maoueh/nugrant.png?travis)][gemnasium]
+[![Code Climate](https://codeclimate.com/github/maoueh/nugrant.png)][codeclimate]
+
+[gem]: https://rubygems.org/gems/nugrant
+[travis]: http://travis-ci.org/maoueh/nugrant
+[gemnasium]: https://gemnasium.com/maoueh/nugrant
+[codeclimate]: https://codeclimate.com/github/maoueh/nugrant
 
 Nugrant is a library to easily handle parameters that need to be
 injected into an application via different sources (system, user,

data/lib/nugrant.rb

@@ -7,7 +7,26 @@ unless defined?(KeyError)
   end
 end
 
+module Nugrant
+  def self.create_parameters(options)
+    config = Nugrant::Config.new(options)
+
+    return Nugrant::Parameters.new(config)
+  end
+
+  def self.setup_i18n()
+    I18n.load_path << File.expand_path("locales/en.yml", Nugrant.source_root)
+    I18n.reload!
+  end
+
+  def self.source_root
+    @source_root ||= Pathname.new(File.expand_path("../../", __FILE__))
+  end
+end
+
 if defined?(Vagrant)
+  Nugrant.setup_i18n()
+
   case
   when defined?(Vagrant::Plugin::V2)
     require 'nugrant/vagrant/v2/plugin'
@@ -17,11 +36,3 @@ if defined?(Vagrant)
     abort("You are trying to use Nugrant with an unsupported Vagrant version [#{Vagrant::VERSION}]")
   end
 end
-
-module Nugrant
-  def self.create_parameters(options)
-    config = Nugrant::Config.new(options)
-
-    return Nugrant::Parameters.new(config)
-  end
-end

data/lib/nugrant/bag.rb

@@ -15,11 +15,11 @@ module Nugrant
     end
 
     def method_missing(method, *args, &block)
-      return __fetch(method.to_s)
+      return __fetch(method)
     end
 
     def has?(key)
-      return @__elements.has_key?(key)
+      return @__elements.has_key?(key.to_sym)
     end
 
     def empty?()
@@ -76,12 +76,12 @@ module Nugrant
 
       hash.each do |key, value|
         if not value.kind_of?(Hash)
-          @__elements[key] = value
+          @__elements[key.to_sym] = value
           next
         end
 
         # It is a hash, transform it into a bag
-        @__elements[key] = Bag.new(value)
+        @__elements[key.to_sym] = Bag.new(value)
       end
     end
 
@@ -90,7 +90,7 @@ module Nugrant
         raise KeyError, "Undefined parameter '#{key}'"
       end
 
-      return @__elements[key]
+      return @__elements[key.to_sym]
     end
   end
 end

data/lib/nugrant/helper/bag.rb

@@ -1,4 +1,4 @@
-require 'json'
+require 'multi_json'
 require 'yaml'
 
 require 'nugrant/bag'
@@ -29,7 +29,7 @@ module Nugrant
       end
 
       def self.parse_json(data_string)
-        JSON.parse(data_string)
+        MultiJson.load(data_string)
       end
 
       def self.parse_yml(data_string)

data/lib/nugrant/helper/stack.rb

@@ -0,0 +1,86 @@
+module Nugrant
+  module Helper
+    class Stack
+      @@DEFAULT_MATCHER = /^(.+):([0-9]+)/
+
+      def self.fetch_error_region(stack, options = {})
+        entry = find_entry(stack, options)
+        location = extract_error_location(entry, options)
+
+        return (options[:unknown] || "Unknown") if not location[:file] and not location[:line]
+        return location[:file] if not location[:line]
+
+        fetch_error_region_from_location(location, options)
+      end
+
+      def self.fetch_error_region_from_location(location, options = {})
+        prefix = options[:prefix] || "   "
+        width = options[:width] || 4
+        file = File.new(location[:file], "r")
+        line = location[:line]
+
+        index = 0
+
+        lines = []
+        while (line_string = file.gets())
+          index += 1
+          next if (line - index).abs > width
+
+          line_prefix = "#{prefix}#{index}:"
+          line_prefix += (line == index ? ">>   " : "     ")
+
+          lines << "#{line_prefix}#{line_string}"
+        end
+
+        lines.join().chomp()
+      rescue
+        return (options[:unknown] || "Unknown") if not location[:file] and not location[:line]
+        return location[:file] if not location[:line]
+
+        "#{location[:file]}:#{location[:line]}"
+      ensure
+        file.close() if file
+      end
+
+      ##
+      # Search a stack list (as simple string array) for the first
+      # entry that match the +:matcher+.
+      #
+      def self.find_entry(stack, options = {})
+        matcher = options[:matcher] || @@DEFAULT_MATCHER
+
+        stack.find do |entry|
+          entry =~ matcher
+        end
+      end
+
+      ##
+      # Extract error location information from a stack entry using the
+      # matcher received in arguments.
+      #
+      # The usual stack entry format is:
+      #   > /home/users/joe/work/lib/ruby.rb:4:Error message
+      #
+      # This function will extract the file and line information from
+      # the stack entry using the matcher. The matcher is expected to
+      # have two groups, the first for the file and the second for
+      # line.
+      #
+      # The results is returned in form of a hash with two keys, +:file+
+      # for the file information and +:line+ for the line information.
+      #
+      # If the matcher matched zero group, return +{:file => nil, :line => nil}+.
+      # If the matcher matched one group, return +{:file => file, :line => nil}+.
+      # If the matcher matched two groups, return +{:file => file, :line => line}+.
+      #
+      def self.extract_error_location(entry, options = {})
+        matcher = options[:matcher] || @@DEFAULT_MATCHER
+
+        result = matcher.match(entry)
+        captures = result ? result.captures : []
+
+        {:file => captures[0], :line => captures[1] ? captures[1].to_i() : nil}
+      end
+    end
+  end
+end

data/lib/nugrant/parameters.rb

@@ -21,7 +21,7 @@ module Nugrant
     end
 
     def method_missing(method, *args, &block)
-      return  @__all[method.to_s]
+      return @__all[method]
     end
 
     def empty?()

data/lib/nugrant/vagrant/errors.rb

@@ -0,0 +1,27 @@
+require 'vagrant/errors'
+
+require 'nugrant/helper/stack'
+
+module Nugrant
+  module Vagrant
+    module Errors
+      class NugrantVagrantError < ::Vagrant::Errors::VagrantError
+        error_namespace("nugrant.vagrant.errors")
+      end
+
+      class ParameterNotFoundError < NugrantVagrantError
+        error_key(:parameter_not_found)
+
+        def initialize(options = nil, *args)
+          super({:context => compute_context()}.merge(options || {}), *args)
+        end
+
+        def compute_context()
+          Helper::Stack.fetch_error_region(caller(), {
+            :matcher => /(.+Vagrantfile):([0-9]+)/
+          })
+        end
+      end
+    end
+  end
+end

data/lib/nugrant/vagrant/v1/config/user.rb

@@ -1,4 +1,5 @@
 require 'nugrant'
+require 'nugrant/vagrant/errors'
 
 module Nugrant
   module Vagrant
@@ -13,10 +14,14 @@ module Nugrant
 
           def [](param_name)
             return @parameters[param_name]
+          rescue KeyError
+            raise Errors::ParameterNotFoundError, :key => param_name
           end
 
           def method_missing(method, *args, &block)
             return @parameters.method_missing(method, *args, &block)
+          rescue KeyError
+            raise Errors::ParameterNotFoundError, :key => method
           end
 
           def each(&block)

data/lib/nugrant/vagrant/v2/config/user.rb

@@ -1,4 +1,5 @@
 require 'nugrant'
+require 'nugrant/vagrant/errors'
 
 module Nugrant
   module Vagrant
@@ -13,10 +14,14 @@ module Nugrant
 
           def [](param_name)
             return @parameters[param_name]
+          rescue KeyError
+            raise Errors::ParameterNotFoundError, :key => param_name
           end
 
           def method_missing(method, *args, &block)
             return @parameters.method_missing(method, *args, &block)
+          rescue KeyError
+            raise Errors::ParameterNotFoundError, :key => method
           end
 
           def each(&block)

data/lib/nugrant/version.rb

@@ -1,3 +1,3 @@
 module Nugrant
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

data/locales/en.yml

@@ -0,0 +1,13 @@
+en:
+  nugrant:
+    vagrant:
+      errors:
+        parameter_not_found: |-
+          Nugrant: Parameter '%{key}' was not found, is it defined in
+          your .vagrantuser file? Here where we think the error
+          could be in your Vagrantfile:
+
+          %{context}
+
+          If you think it should be found, don't hesitate to fill an
+          issue @ https://github.com/maoueh/nugrant/issues.

data/nugrant.gemspec

@@ -28,5 +28,7 @@ Gem::Specification.new do |gem|
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
+  gem.add_dependency 'multi_json', '~> 1.8'
+
   gem.add_development_dependency "rake"
 end

data/test/lib/nugrant/helper/test_stack.rb

@@ -0,0 +1,149 @@
+require 'test/unit'
+
+require 'nugrant/helper/stack'
+
+module Nugrant
+  module Helper
+    class TestStack < Test::Unit::TestCase
+      def assert_error_location(expected, entry, matcher = nil)
+        assert_equal(expected, Stack::extract_error_location(entry, :matcher => matcher))
+      end
+
+      def assert_error_region(expected, region)
+        expected_lines = expected.split()
+        region_lines = region.split()
+
+        assert_equal(expected_lines.length(), region_lines.length())
+
+        expected_lines.each_with_index do |expected_line, index|
+          assert_equal(expected_line.strip(), region_lines[index].strip())
+        end
+      end
+
+      def create_stack(options = {})
+        pattern = options[:pattern] || "Vagrantfile:%s"
+        count = options[:count] || 4
+
+        stack = []
+        (0..count).each do |index|
+          stack << pattern.gsub("%s", index.to_s())
+        end
+
+        stack
+      end
+
+      def create_location(name, line)
+        resource_path = File.expand_path("#{File.dirname(__FILE__)}/../../../resources/vagrantfiles")
+
+        {:file => "#{resource_path}/#{name}", :line => line}
+      end
+
+      def test_fetch_error_region_from_location()
+        location = create_location("v1.defaults_using_symbol", 4)
+        error_region = Stack::fetch_error_region_from_location(location)
+        expected_region = <<-EOT
+          1:     Vagrant::Config.run do |config|
+          2:       config.user.defaults = {
+          3:         :single => 1,
+          4:>>       :local => {
+          5:             :first => "value1",
+          6:             :second => "value2"
+          7:         }
+          8:       }
+        EOT
+
+        assert_error_region(expected_region, error_region)
+      end
+
+      def test_fetch_error_region_from_location_custom_prefix()
+        location = create_location("v1.defaults_using_symbol", 4)
+        error_region = Stack::fetch_error_region_from_location(location, :prefix => "**")
+        expected_region = <<-EOT
+          **1:     Vagrant::Config.run do |config|
+          **2:       config.user.defaults = {
+          **3:         :single => 1,
+          **4:>>       :local => {
+          **5:             :first => "value1",
+          **6:             :second => "value2"
+          **7:         }
+          **8:       }
+        EOT
+
+        assert_error_region(expected_region, error_region)
+      end
+
+      def test_fetch_error_region_from_location_custom_width()
+        location = create_location("v1.defaults_using_symbol", 4)
+        error_region = Stack::fetch_error_region_from_location(location, :width => 2)
+        expected_region = <<-EOT
+          2:       config.user.defaults = {
+          3:         :single => 1,
+          4:>>       :local => {
+          5:             :first => "value1",
+          6:             :second => "value2"
+        EOT
+
+        assert_error_region(expected_region, error_region)
+      end
+
+      def test_fetch_error_region_from_location_wrong_location()
+        location = {:file => nil, :line => nil}
+        assert_equal("Unknown", Stack::fetch_error_region_from_location(location))
+        assert_equal("Failed", Stack::fetch_error_region_from_location(location, :unknown => "Failed"))
+
+        location = {:file => "Vagrantfile", :line => nil}
+        assert_equal("Vagrantfile", Stack::fetch_error_region_from_location(location))
+
+        location = {:file => "NonExistingVagrantfile", :line => 4}
+        assert_equal("NonExistingVagrantfile:4", Stack::fetch_error_region_from_location(location))
+      end
+
+      def test_find_entry()
+        entries = ["First", "Second:", "Third:a", "Fourth:4"]
+
+        assert_equal("Fourth:4", Stack::find_entry(entries))
+        assert_equal("Third:a", Stack::find_entry(entries, :matcher => /^(.+):([a-z]+)/))
+      end
+
+      def test_extract_error_location_default_matcher()
+        # Matches
+        assert_error_location({:file => "/work/irb/workspace.rb", :line => 80}, "/work/irb/workspace.rb:80:in `eval'")
+        assert_error_location({:file => "workspace.rb", :line => 80}, "workspace.rb:80:in `eval'")
+        assert_error_location({:file => "/work/irb/workspace.rb", :line => 80}, "/work/irb/workspace.rb:80")
+
+        # No match
+        assert_error_location({:file => nil, :line => nil}, "/work/irb/workspace.rb?80")
+        assert_error_location({:file => nil, :line => nil}, "/work/irb/workspace.rb")
+        assert_error_location({:file =>nil, :line => nil}, "")
+      end
+
+      def test_extract_error_location_custom_matcher()
+        # Matches
+        assert_error_location(
+          {:file => "/work/Vagrantfile", :line => 80},
+          "/work/Vagrantfile:80:in `eval'",
+          /(.*Vagrantfile):([0-9]+)/
+        )
+
+        assert_error_location(
+          {:file => "Vagrantfile", :line => 80},
+          "Vagrantfile:80:in `eval'",
+          /(.*Vagrantfile):([0-9]+)/
+        )
+
+        assert_error_location(
+          {:file => "/work/irb/Vagrantfile", :line => 80},
+          "/work/irb/Vagrantfile:80",
+          /(.*Vagrantfile):([0-9]+)/
+        )
+
+        # Partial match
+        assert_error_location(
+          {:file => "/work/Vagrantfile", :line => nil},
+          "/work/Vagrantfile:80:in `eval'",
+          /(.*Vagrantfile)/
+        )
+      end
+    end
+  end
+end

data/test/lib/nugrant/test_bag.rb

@@ -8,18 +8,28 @@ module Nugrant
       return Bag.new(parameters)
     end
 
+    def assert_all_access_equal(value, bag, key)
+      assert_equal(value, bag.method_missing(key.to_sym), "bag.#{key.to_sym.inspect}")
+      assert_equal(value, bag[key.to_s], "bag[#{key.to_s.inspect}]")
+      assert_equal(value, bag[key.to_sym], "bag[#{key.to_sym.inspect}]")
+    end
+
+    def assert_all_access_bag(value, bag, key)
+      assert_bag(value, bag.method_missing(key.to_sym))
+      assert_bag(value, bag[key.to_s])
+      assert_bag(value, bag[key.to_sym])
+    end
+
     def assert_bag(parameters, bag)
       assert_kind_of(Bag, bag)
 
       parameters.each do |key, value|
         if not value.kind_of?(Hash)
-          assert_equal(value, bag.send(key))
-          assert_equal(value, bag[key])
+          assert_all_access_equal(value, bag, key)
           next
         end
 
-        assert_bag(value, bag.send(key))
-        assert_bag(value, bag[key])
+        assert_all_access_bag(value, bag, key)
       end
     end
 
@@ -81,6 +91,10 @@ module Nugrant
       assert_raise(KeyError) do
         bag["invalid_value"]
       end
+
+      assert_raise(KeyError) do
+        bag[:invalid_value]
+      end
     end
 
     def test_to_hash()
@@ -89,29 +103,29 @@ module Nugrant
       assert_kind_of(Hash, hash)
       assert_equal({}, hash)
 
-      hash = create_bag({"value" => {"one" => "value", "two" => "value"}}).__to_hash()
+      hash = create_bag({"value" => {:one => "value", "two" => "value"}}).__to_hash()
 
       assert_kind_of(Hash, hash)
-      assert_kind_of(Hash, hash['value'])
-      assert_kind_of(String, hash['value']['one'])
-      assert_kind_of(String, hash['value']['two'])
-      assert_equal({"value" => {"one" => "value", "two" => "value"}}, hash)
+      assert_kind_of(Hash, hash[:value])
+      assert_kind_of(String, hash[:value][:one])
+      assert_kind_of(String, hash[:value][:two])
+      assert_equal({:value => {:one => "value", :two => "value"}}, hash)
     end
 
     def test_merge_array()
       bag1 = create_bag({"first" => [1, 2]})
-      bag2 = create_bag({"first" => [2, 3]})
+      bag2 = create_bag({:first => [2, 3]})
 
       bag1.__merge!(bag2);
 
-      assert_equal({"first" => [1, 2, 3]}, bag1.__to_hash())
+      assert_equal({:first => [1, 2, 3]}, bag1.__to_hash())
 
       bag1 = create_bag({"first" => [1, 2]})
-      bag2 = create_bag({"first" => "string"})
+      bag2 = create_bag({:first => "string"})
 
       bag1.__merge!(bag2);
 
-      assert_equal({"first" => "string"}, bag1.__to_hash())
+      assert_equal({:first => "string"}, bag1.__to_hash())
     end
   end
 end

data/test/lib/nugrant/test_parameters.rb

@@ -22,10 +22,15 @@ class Nugrant::TestParameters < Test::Unit::TestCase
     })
   end
 
+  def assert_all_access_equal(value, parameters, key)
+    assert_equal(value, parameters.method_missing(key.to_sym), "parameters.#{key.to_sym.inspect}")
+    assert_equal(value, parameters[key.to_s], "parameters[#{key.to_s.inspect}]")
+    assert_equal(value, parameters[key.to_sym], "parameters[#{key.to_sym.inspect}]")
+  end
+
   def assert_level(parameters, results)
     results.each do |key, value|
-      assert_equal(value, parameters.send(key), "array[#{key}]")
-      assert_equal(value, parameters[key], "array[#{key}]")
+      assert_all_access_equal(value, parameters, key)
     end
 
     assert_key_error(parameters, "0.0.0")
@@ -146,13 +151,13 @@ class Nugrant::TestParameters < Test::Unit::TestCase
   def run_test_file_invalid(invalid_value)
     filetypes.each do |params_filetype|
       parameters = create_parameters(params_filetype, "params_simple", invalid_path, invalid_path)
-      assert_equal("value", parameters["test"])
+      assert_all_access_equal("value", parameters, "test")
 
       parameters = create_parameters(params_filetype, invalid_path, "params_simple", invalid_path)
-      assert_equal("value", parameters["test"])
+      assert_all_access_equal("value", parameters, "test")
 
       parameters = create_parameters(params_filetype, invalid_path, invalid_path, "params_simple")
-      assert_equal("value", parameters["test"])
+      assert_all_access_equal("value", parameters, "test")
 
       parameters = create_parameters(params_filetype, invalid_path, invalid_path, invalid_path)
       assert_not_nil(parameters)
@@ -171,8 +176,8 @@ class Nugrant::TestParameters < Test::Unit::TestCase
     filetypes.each do |params_filetype|
       parameters = create_parameters(params_filetype, file_path, invalid_path, invalid_path)
 
-      assert_equal("value1", parameters['level1'])
-      assert_equal("value2", parameters['level2']['first'])
+      assert_all_access_equal("value1", parameters, 'level1')
+      assert_all_access_equal("value2", parameters['level2'], 'first')
     end
   end
 
@@ -180,13 +185,13 @@ class Nugrant::TestParameters < Test::Unit::TestCase
     filetypes.each do |params_filetype|
       parameters = create_parameters(params_filetype, "params_defaults_at_root", invalid_path, invalid_path)
 
-      assert_equal("value", parameters.defaults)
+      assert_all_access_equal("value", parameters, :defaults)
     end
 
     filetypes.each do |params_filetype|
       parameters = create_parameters(params_filetype, "params_defaults_not_at_root", invalid_path, invalid_path)
 
-      assert_equal("value", parameters.level.defaults)
+      assert_all_access_equal("value", parameters.level, :defaults)
     end
   end
 
@@ -195,8 +200,8 @@ class Nugrant::TestParameters < Test::Unit::TestCase
       parameters = create_parameters(params_filetype, "params_simple", invalid_path, invalid_path)
       parameters.defaults = {"test" => "override1", "level" => "new1"}
 
-      assert_equal("value", parameters['test'])
-      assert_equal("new1", parameters['level'])
+      assert_all_access_equal("value", parameters, 'test')
+      assert_all_access_equal("new1", parameters, 'level')
     end
   end
 
@@ -205,7 +210,7 @@ class Nugrant::TestParameters < Test::Unit::TestCase
       parameters = create_parameters(params_filetype, "params_empty", invalid_path, invalid_path)
       parameters.defaults = {"test" => "value"}
 
-      assert_equal("value", parameters['test'])
+      assert_all_access_equal("value", parameters, 'test')
     end
   end
 
@@ -215,7 +220,7 @@ class Nugrant::TestParameters < Test::Unit::TestCase
         parameters = create_parameters(params_filetype, "params_#{wrong_type}", invalid_path, invalid_path)
         parameters.defaults = {"test" => "value"}
 
-        assert_equal("value", parameters['test'])
+        assert_all_access_equal("value", parameters, 'test')
       end
     end
   end

data/test/resources/vagrantfiles/v1.defaults_mixed_string_symbols

@@ -0,0 +1,18 @@
+Vagrant::Config.run do |config|
+  config.user.defaults = {
+    'single' => 1,
+    :local => {
+        'first' => "value1",
+        :second => "value2"
+    }
+  }
+
+  puts config.user.single
+  puts config.user[:single]
+
+  puts config.user.local.first
+  puts config.user[:local]["first"]
+
+  puts config.user.local.second
+  puts config.user["local"][:second]
+end

data/test/resources/vagrantfiles/v1.defaults_using_string

@@ -0,0 +1,18 @@
+Vagrant::Config.run do |config|
+  config.user.defaults = {
+    'single' => 1,
+    'local' => {
+        'first' => "value1",
+        'second' => "value2"
+    }
+  }
+
+  puts config.user.single
+  puts config.user[:single]
+
+  puts config.user.local.first
+  puts config.user[:local]["first"]
+
+  puts config.user.local.second
+  puts config.user["local"][:second]
+end

data/test/resources/vagrantfiles/v1.defaults_using_symbol

@@ -0,0 +1,18 @@
+Vagrant::Config.run do |config|
+  config.user.defaults = {
+    :single => 1,
+    :local => {
+        :first => "value1",
+        :second => "value2"
+    }
+  }
+
+  puts config.user.single
+  puts config.user[:single]
+
+  puts config.user.local.first
+  puts config.user[:local]["first"]
+
+  puts config.user.local.second
+  puts config.user["local"][:second]
+end

data/test/resources/vagrantfiles/v1.missing_parameter

@@ -0,0 +1,3 @@
+Vagrant::Config.run do |config|
+  puts config.user.unkown_for_sure
+end

data/test/resources/vagrantfiles/v2.defaults_mixed_string_symbols

@@ -0,0 +1,20 @@
+Vagrant.require_plugin('nugrant')
+
+Vagrant.configure("2") do |config|
+  config.user.defaults = {
+    'single' => 1,
+    :local => {
+        'first' => "value1",
+        :second => "value2"
+    }
+  }
+
+  puts config.user.single
+  puts config.user[:single]
+
+  puts config.user.local.first
+  puts config.user[:local]["first"]
+
+  puts config.user.local.second
+  puts config.user["local"][:second]
+end

data/test/resources/vagrantfiles/v2.defaults_using_string

@@ -0,0 +1,20 @@
+Vagrant.require_plugin('nugrant')
+
+Vagrant.configure("2") do |config|
+  config.user.defaults = {
+    'single' => 1,
+    'local' => {
+        'first' => "value1",
+        'second' => "value2"
+    }
+  }
+
+  puts config.user.single
+  puts config.user[:single]
+
+  puts config.user.local.first
+  puts config.user[:local]["first"]
+
+  puts config.user.local.second
+  puts config.user["local"][:second]
+end

data/test/resources/vagrantfiles/v2.defaults_using_symbol

@@ -0,0 +1,20 @@
+Vagrant.require_plugin('nugrant')
+
+Vagrant.configure("2") do |config|
+  config.user.defaults = {
+    :single => 1,
+    :local => {
+        :first => "value1",
+        :second => "value2"
+    }
+  }
+
+  puts config.user.single
+  puts config.user[:single]
+
+  puts config.user.local.first
+  puts config.user[:local]["first"]
+
+  puts config.user.local.second
+  puts config.user["local"][:second]
+end

data/test/resources/{Vagrantfile.v2.fake → vagrantfiles/v2.fake}

@@ -7,7 +7,7 @@ Vagrant.configure("2") do |config|
         'second' => "value2"
     }
   }
-  
+
   config.user.local.each do |name, value|
     puts "Parameters #{name}: #{value}"
   end

data/test/resources/vagrantfiles/v2.missing_parameter

@@ -0,0 +1,5 @@
+Vagrant.require_plugin('nugrant')
+
+Vagrant.configure("2") do |config|
+  puts config.user.unkown_for_sure
+end

metadata

@@ -1,15 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: nugrant
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Matthieu Vachon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-17 00:00:00.000000000 Z
+date: 2013-10-25 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: multi_json
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.8'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.8'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -52,8 +66,10 @@ files:
 - lib/nugrant/bag.rb
 - lib/nugrant/config.rb
 - lib/nugrant/helper/bag.rb
+- lib/nugrant/helper/stack.rb
 - lib/nugrant/helper/yaml.rb
 - lib/nugrant/parameters.rb
+- lib/nugrant/vagrant/errors.rb
 - lib/nugrant/vagrant/v1/command/parameters.rb
 - lib/nugrant/vagrant/v1/command/root.rb
 - lib/nugrant/vagrant/v1/config/user.rb
@@ -64,18 +80,14 @@ files:
 - lib/nugrant/vagrant/v2/plugin.rb
 - lib/nugrant/version.rb
 - lib/vagrant_init.rb
+- locales/en.yml
 - nugrant.gemspec
+- test/lib/nugrant/helper/test_stack.rb
 - test/lib/nugrant/test_bag.rb
 - test/lib/nugrant/test_config.rb
 - test/lib/nugrant/test_parameters.rb
 - test/resources/.vagrantuser
 - test/resources/README.md
-- test/resources/Vagrantfile.v1.empty
-- test/resources/Vagrantfile.v1.fake
-- test/resources/Vagrantfile.v1.real
-- test/resources/Vagrantfile.v2.empty
-- test/resources/Vagrantfile.v2.fake
-- test/resources/Vagrantfile.v2.real
 - test/resources/json/params_array.json
 - test/resources/json/params_boolean.json
 - test/resources/json/params_combinations.json
@@ -92,6 +104,20 @@ files:
 - test/resources/json/params_user_1.json
 - test/resources/json/params_user_2.json
 - test/resources/json/params_windows_eol.json
+- test/resources/vagrantfiles/v1.defaults_mixed_string_symbols
+- test/resources/vagrantfiles/v1.defaults_using_string
+- test/resources/vagrantfiles/v1.defaults_using_symbol
+- test/resources/vagrantfiles/v1.empty
+- test/resources/vagrantfiles/v1.fake
+- test/resources/vagrantfiles/v1.missing_parameter
+- test/resources/vagrantfiles/v1.real
+- test/resources/vagrantfiles/v2.defaults_mixed_string_symbols
+- test/resources/vagrantfiles/v2.defaults_using_string
+- test/resources/vagrantfiles/v2.defaults_using_symbol
+- test/resources/vagrantfiles/v2.empty
+- test/resources/vagrantfiles/v2.fake
+- test/resources/vagrantfiles/v2.missing_parameter
+- test/resources/vagrantfiles/v2.real
 - test/resources/yml/params_array.yml
 - test/resources/yml/params_boolean.yml
 - test/resources/yml/params_combinations.yml
@@ -132,17 +158,12 @@ signing_key:
 specification_version: 4
 summary: Library to handle user specific parameters from various location.
 test_files:
+- test/lib/nugrant/helper/test_stack.rb
 - test/lib/nugrant/test_bag.rb
 - test/lib/nugrant/test_config.rb
 - test/lib/nugrant/test_parameters.rb
 - test/resources/.vagrantuser
 - test/resources/README.md
-- test/resources/Vagrantfile.v1.empty
-- test/resources/Vagrantfile.v1.fake
-- test/resources/Vagrantfile.v1.real
-- test/resources/Vagrantfile.v2.empty
-- test/resources/Vagrantfile.v2.fake
-- test/resources/Vagrantfile.v2.real
 - test/resources/json/params_array.json
 - test/resources/json/params_boolean.json
 - test/resources/json/params_combinations.json
@@ -159,6 +180,20 @@ test_files:
 - test/resources/json/params_user_1.json
 - test/resources/json/params_user_2.json
 - test/resources/json/params_windows_eol.json
+- test/resources/vagrantfiles/v1.defaults_mixed_string_symbols
+- test/resources/vagrantfiles/v1.defaults_using_string
+- test/resources/vagrantfiles/v1.defaults_using_symbol
+- test/resources/vagrantfiles/v1.empty
+- test/resources/vagrantfiles/v1.fake
+- test/resources/vagrantfiles/v1.missing_parameter
+- test/resources/vagrantfiles/v1.real
+- test/resources/vagrantfiles/v2.defaults_mixed_string_symbols
+- test/resources/vagrantfiles/v2.defaults_using_string
+- test/resources/vagrantfiles/v2.defaults_using_symbol
+- test/resources/vagrantfiles/v2.empty
+- test/resources/vagrantfiles/v2.fake
+- test/resources/vagrantfiles/v2.missing_parameter
+- test/resources/vagrantfiles/v2.real
 - test/resources/yml/params_array.yml
 - test/resources/yml/params_boolean.yml
 - test/resources/yml/params_combinations.yml