configurate 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 996d3e9be532044af0c5ab4597eb547661683c3b
-  data.tar.gz: 2e9b90ac7a188be0c8624fcd5e52619b7f267b53
+  metadata.gz: cee20b7c12f6d7b03d65511ea8c501955dc437cc
+  data.tar.gz: 6b23db2069cf5b8128c9b2d1c6b1f3e1ddac26b7
 SHA512:
-  metadata.gz: 0c51c9bc229cff96e092dee805cb4f4517cdc315831b79f65b46253687b692a2db562278bb7831178433ff2768d403a4eb7c7151c9859b8884dd8e1413b08157
-  data.tar.gz: e3af9cf48b009e97fe81cd7fcbee2d5e55586d6aff33a45be88b562d9115c69a4b1157af12d52d525dc6e06d866b49586ba04576b72613e37b1a9b39649c69c5
+  metadata.gz: 53094d245312357e844542e7342fc9c1e07c864320109c35a883bcd250bfcd81a81fffef41792cf092bb81f4ec96f61d5ba6faa577e79f0678dca2543a68c9ec
+  data.tar.gz: ddf023b472ebcd5388e6e454939970d7744add4784cf4d1d434fd1b6dfe9f5f8d9c0d4f7e20bd295dc97fd587a9eb5cd0a2afc13aaf18d897bdff4fec227fd11

data/Changelog.md

@@ -1,3 +1,10 @@
+# 0.3.0
+
+* Add new exception: Configurate::MissingSetting to be raised and bubble up to the user
+  if a setting wasn't found and the user requested to be informed.
+* Configurate::Provider::YAML got the new option raise_on_missing to raise
+  Configurate::MissingSetting if the requested key is not in the YAML document.
+
 # 0.2.0
 
 * Dynamic provider listens to reset_dynamic! message and forgets all settings on it.

data/README.md

@@ -1,9 +1,9 @@
 # Configurate - A flexible configuration system
 [![Gem Version](https://badge.fury.io/rb/configurate.png)](https://rubygems.org/gems/configurate)
 [![Build Status](https://secure.travis-ci.org/jhass/configurate.png?branch=master)](https://travis-ci.org/jhass/configurate)
-[![Gemnasium](https://gemnasium.com/MrZYX/configurate.png)](https://gemnasium.com/MrZYX/configurate)
-[![Code Climate](https://codeclimate.com/github/jhass/configurate.png)](https://codeclimate.com/github/MrZYX/configurate)
-[![Coverage Status](https://coveralls.io/repos/jhass/configurate/badge.png?branch=master)](https://coveralls.io/r/MrZYX/configurate)
+[![Gemnasium](https://gemnasium.com/jhass/configurate.png)](https://gemnasium.com/jhass/configurate)
+[![Code Climate](https://codeclimate.com/github/jhass/configurate.png)](https://codeclimate.com/github/jhass/configurate)
+[![Coverage Status](https://coveralls.io/repos/jhass/configurate/badge.svg?branch=master)](https://coveralls.io/r/jhass/configurate?branch=master)
 
 Configurate allows you to specify a chain of configuration providers which are
 queried in order until one returns a value. This allows scenarios like overriding
@@ -11,7 +11,7 @@ your default settings with a user configuration file and let those be overridden
 by environment variables. The query interface allows to group and nest your configuration options
 to a practically unlimited level.
 
-Configurate works with Ruby 1.9.2 or later.
+Configurate supports Ruby 2.0 or later.
 
 ## Installation
 

data/lib/configurate.rb

@@ -1,10 +1,9 @@
-require 'forwardable'
-
-require 'configurate/setting_path'
-require 'configurate/lookup_chain'
-require 'configurate/provider'
-require 'configurate/proxy'
+require "forwardable"
 
+require "configurate/setting_path"
+require "configurate/lookup_chain"
+require "configurate/provider"
+require "configurate/proxy"
 
 # A flexible and extendable configuration system.
 # The calling logic is isolated from the lookup logic
@@ -20,15 +19,15 @@ require 'configurate/proxy'
 module Configurate
   # This is your main entry point. Instead of lengthy explanations
   # let an example demonstrate its usage:
-  # 
+  #
   #     require 'configuration_methods'
-  #     
+  #
   #     AppSettings = Configurate::Settings.create do
   #       add_provider Configurate::Provider::Env
   #       add_provider Configurate::Provider::YAML, '/etc/app_settings.yml',
   #                    namespace: Rails.env, required: false
   #       add_provider Configurate::Provider::YAML, 'config/default_settings.yml'
-  #       
+  #
   #       extend YourConfigurationMethods
   #     end
   #
@@ -36,18 +35,17 @@ module Configurate
   #
   # Please also read the note at {Proxy}!
   class Settings
-  
     attr_reader :lookup_chain
-    
+
     undef_method :method # Remove possible conflicts with common setting names
 
     extend Forwardable
 
     def initialize
       @lookup_chain = LookupChain.new
-      $stderr.puts "Warning you called Configurate::Settings.new with a block, you really meant to call #create" if block_given?
+      warn "Warning you called Configurate::Settings.new with a block, you really meant to call #create" if block_given?
     end
-    
+
     # @!method lookup(setting)
     # (see {LookupChain#lookup})
 
@@ -63,15 +61,22 @@ module Configurate
     def method_missing(method, *args, &block)
       Proxy.new(@lookup_chain).public_send(method, *args, &block)
     end
-    
+
     # Create a new configuration object
     # @yield the given block will be evaluated in the context of the new object
     def self.create(&block)
-      config = self.new
+      config = new
       config.instance_eval(&block) if block_given?
       config
     end
   end
-  
+
+  # This is supposed to be raised by providers if the requested setting
+  # does not exist, (remember, nil is a valid value and thus rarely a sufficient check)
+  # and this should be communicated to the end user.
+  class MissingSetting < RuntimeError; end
+
+  # This is supposed to be raised by providers if the requested setting
+  # cannot be found and the next provider in the chain should be tried.
   class SettingNotFoundError < RuntimeError; end
 end

data/lib/configurate/lookup_chain.rb

@@ -21,7 +21,6 @@ module Configurate
       @provider << provider.new(*args)
     end
 
-
     # Tries all providers in the order they were added to provide a response
     # for setting.
     #
@@ -45,19 +44,20 @@ module Configurate
     private
 
     def special_value_or_string(value)
-      if [TrueClass, FalseClass, NilClass, Array, Hash].include?(value.class)
-        return value
-      elsif value.is_a?(String)
-        return case value.strip
+      case value
+      when TrueClass, FalseClass, NilClass, Array, Hash
+        value
+      else
+        if value.respond_to?(:to_s)
+          case value.to_s.strip
           when "true" then true
           when "false" then false
           when "", "nil" then nil
-          else value
+          else value.to_s
+          end
+        else
+          value
         end
-      elsif value.respond_to?(:to_s)
-        return value.to_s
-      else
-        return value
       end
     end
   end

data/lib/configurate/provider.rb

@@ -1,32 +1,34 @@
-module Configurate; module Provider
-  # This provides a basic {#lookup} method for other providers to build
-  # upon. Childs are expected to define +lookup_path(path, *args)+.
-  # The method should return nil if the setting
-  # wasn't found and {#lookup} will raise an {SettingNotFoundError} in that
-  # case.
-  class Base
-    def lookup(*args)
-      result = lookup_path(*args)
-      return result unless result.nil?
-      raise Configurate::SettingNotFoundError, "The setting #{args.first} was not found"
+module Configurate
+  module Provider
+    # This provides a basic {#lookup} method for other providers to build
+    # upon. Childs are expected to define +lookup_path(path, *args)+.
+    # The method should return nil if the setting
+    # wasn't found and {#lookup} will raise an {SettingNotFoundError} in that
+    # case.
+    class Base
+      def lookup(*args)
+        result = lookup_path(*args)
+        return result unless result.nil?
+        raise Configurate::SettingNotFoundError, "The setting #{args.first} was not found"
+      end
     end
-  end
 
-  # Utility function to lookup a settings path in a hash
-  # @param setting_path [SettingPath]
-  # @param hash [Hash]
-  # @yield fallback value if not found
-  # @return [Object]
-  def self.lookup_in_hash setting_path, hash, &fallback
-    fallback ||= proc { nil }
-    while hash.is_a?(Hash) && hash.has_key?(setting_path.first) && !setting_path.empty?
-      hash = hash[setting_path.shift]
+    # Utility function to lookup a settings path in a hash
+    # @param setting_path [SettingPath]
+    # @param hash [Hash]
+    # @yield fallback value if not found
+    # @return [Object]
+    def self.lookup_in_hash setting_path, hash, &fallback
+      fallback ||= proc { nil }
+      while hash.is_a?(Hash) && hash.has_key?(setting_path.first) && !setting_path.empty?
+        hash = hash[setting_path.shift]
+      end
+      return fallback.call unless setting_path.empty?
+      hash
     end
-    return fallback.call unless setting_path.empty?
-    hash
   end
-end; end
+end
 
-require 'configurate/provider/yaml'
-require 'configurate/provider/env'
-require 'configurate/provider/dynamic'
+require "configurate/provider/yaml"
+require "configurate/provider/env"
+require "configurate/provider/dynamic"

data/lib/configurate/provider/dynamic.rb

@@ -1,30 +1,38 @@
-module Configurate; module Provider
-  # This provider knows nothing upon initialization, however if you access
-  # a setting ending with +=+ and give one argument to that call it remembers
-  # that setting, stripping the +=+ and will return it on the next call
-  # without +=+. Sending +reset_dynamic!+ to it will make it forget all
-  # settings. Also assigning nil will have the effect of it forgetting
-  # a setting.
-  class Dynamic < Base
-    def initialize
-      @settings = {}
-    end
+module Configurate
+  module Provider
+    # This provider knows nothing upon initialization, however if you access
+    # a setting ending with +=+ and give one argument to that call it remembers
+    # that setting, stripping the +=+ and will return it on the next call
+    # without +=+. Sending +reset_dynamic!+ to it will make it forget all
+    # settings. Also assigning nil will have the effect of it forgetting
+    # a setting.
+    class Dynamic < Base
+      def lookup_path(setting_path, *args)
+        if setting_path.to_s == "reset_dynamic!"
+          @settings = nil
+          return
+        end
+
+        if setting_path.setter? && args.length > 0
+          *root, key = setting_path.to_a
+          hash = root.inject(settings) {|hash, key| hash[key] }
+          hash[key] = extract_value(args)
+        end
+
+        Provider.lookup_in_hash setting_path, settings
+      end
+
+      private
 
-    def lookup_path(setting_path, *args)
-      if setting_path.to_s == "reset_dynamic!"
-        @settings = {}
-        return
+      def settings
+        @settings ||= Hash.new {|hash, key| hash[key] = Hash.new(&hash.default_proc) }
       end
 
-      if setting_path.setter? && args.length > 0
+      def extract_value args
         value = args.first
         value = value.get if value.respond_to?(:_proxy?) && value._proxy?
-        *root, key = setting_path.to_a
-        hash = root.inject(@settings) {|hash, key| hash[key] ||= {} }
-        hash[key] = value
+        value
       end
-
-      Provider.lookup_in_hash setting_path, @settings
     end
   end
-end; end
+end

data/lib/configurate/provider/env.rb

@@ -1,17 +1,19 @@
-module Configurate; module Provider
-  # This provider looks for settings in the environment.
-  # For the setting +foo.bar_baz+ this provider will look for an
-  # environment variable +FOO_BAR_BAZ+, joining all components of the 
-  # setting with underscores and upcasing the result.
-  # If an value contains any commas (,) it's split at them and returned as array.
-  class Env < Base
-    def lookup_path(setting_path, *args)
-      value = ENV[setting_path.join("_").upcase]
-      unless value.nil?
-        value = value.dup
-        value = value.split(",") if value.include?(",")
+module Configurate
+  module Provider
+    # This provider looks for settings in the environment.
+    # For the setting +foo.bar_baz+ this provider will look for an
+    # environment variable +FOO_BAR_BAZ+, joining all components of the
+    # setting with underscores and upcasing the result.
+    # If an value contains any commas (,) it's split at them and returned as array.
+    class Env < Base
+      def lookup_path(setting_path, *_args)
+        value = ENV[setting_path.join("_").upcase]
+        unless value.nil?
+          value = value.dup
+          value = value.split(",") if value.include?(",")
+        end
+        value
       end
-      value
     end
   end
-end; end
+end

data/lib/configurate/provider/yaml.rb

@@ -1,37 +1,42 @@
-require 'yaml'
+require "yaml"
 
-module Configurate; module Provider
-  # This provider tries to open a YAML file and does nested lookups
-  # in it.
-  class YAML < Base
-    # @param file [String] the path to the file
-    # @param opts [Hash]
-    # @option opts [String] :namespace optionally set this as the root
-    # @option opts [Boolean] :required wheter or not to raise an error if
-    #   the file or the namespace, if given, is not found. Defaults to +true+.
-    # @raise [ArgumentError] if the namespace isn't found in the file
-    # @raise [Errno:ENOENT] if the file isn't found
-    def initialize file, opts = {}
-      @settings = {}
-      required = opts.delete(:required) { true }
+module Configurate
+  module Provider
+    # This provider tries to open a YAML file and does nested lookups
+    # in it.
+    class YAML < Base
+      # @param file [String] the path to the file
+      # @param namespace [String] optionally set this as the root
+      # @param required [Boolean] whether or not to raise an error if
+      #   the file or the namespace, if given, is not found. Defaults to +true+.
+      # @param raise_on_missing [Boolean]  whether to raise {Configurate::MissingSetting}
+      #   if a setting can't be provided. Defaults to +false+.
+      # @raise [ArgumentError] if the namespace isn't found in the file
+      # @raise [Errno:ENOENT] if the file isn't found
+      def initialize file, namespace: nil, required: true, raise_on_missing: false
+        @raise_on_missing = raise_on_missing
+        @settings = {}
 
-      @settings = ::YAML.load_file(file)
+        @settings = ::YAML.load_file(file)
 
-      namespace = opts.delete(:namespace)
-      unless namespace.nil?
-        @settings = Provider.lookup_in_hash(SettingPath.new(namespace), @settings) do
-          raise ArgumentError, "Namespace #{namespace} not found in #{file}" if required
-          $stderr.puts "WARNING: Namespace #{namespace} not found in #{file}"
-          nil
+        unless namespace.nil?
+          @settings = Provider.lookup_in_hash(SettingPath.new(namespace), @settings) do
+            raise ArgumentError, "Namespace #{namespace} not found in #{file}" if required
+            $stderr.puts "WARNING: Namespace #{namespace} not found in #{file}"
+            nil
+          end
         end
+      rescue Errno::ENOENT => e
+        warn "WARNING: Configuration file #{file} not found, ensure it's present"
+        raise e if required
       end
-    rescue Errno::ENOENT => e
-      $stderr.puts "WARNING: Configuration file #{file} not found, ensure it's present"
-      raise e if required
-    end
 
-    def lookup_path setting_path, *_
-      Provider.lookup_in_hash(setting_path, @settings)
+      def lookup_path setting_path, *_
+        Provider.lookup_in_hash(setting_path, @settings) {
+          raise MissingSetting.new "#{setting_path} is not a valid setting." if @raise_on_missing
+          nil
+        }
+      end
     end
   end
-end; end
+end

data/lib/configurate/proxy.rb

@@ -23,17 +23,17 @@ module Configurate
       !target
     end
 
-    [:!=, :==, :eql?, :coerce].each do |method|
+    %i(!= == eql? coerce).each do |method|
       define_method method do |other|
         target.public_send method, target_or_object(other)
       end
     end
 
     {
-      :to_int => :to_i,
-      :to_hash => :to_h,
-      :to_str => :to_s,
-      :to_ary => :to_a
+      to_int:  :to_i,
+      to_hash: :to_h,
+      to_str:  :to_s,
+      to_ary:  :to_a
     }.each do |method, converter|
       define_method method do
         value = target
@@ -75,7 +75,8 @@ module Configurate
     alias_method :get, :target
 
     private
-    COMMON_KEY_NAMES = [:key, :method]
+
+    COMMON_KEY_NAMES = %i(key method)
 
     def target_respond_to? setting, include_private=false
       return false if COMMON_KEY_NAMES.include? setting

data/lib/configurate/setting_path.rb

@@ -1,4 +1,4 @@
-require 'forwardable'
+require "forwardable"
 
 module Configurate
   # Class encapsulating the concept of a path to a setting
@@ -45,13 +45,13 @@ module Configurate
       end
     end
 
-    [:join, :first, :last, :shift, :pop].each do |method|
+    %i(join first last shift pop).each do |method|
       define_method method do |*args|
         clean_special_characters @path.public_send(method, *args)
       end
     end
 
-    [:<<, :unshift, :push].each do |method|
+    %i(<< unshift push).each do |method|
       define_method method do |*args|
         @path.public_send method, *args.map(&:to_s)
       end
@@ -66,7 +66,11 @@ module Configurate
     end
 
     def inspect
-      "<SettingPath:#{object_id.to_s(16)} path=#{to_s}:#{@path.object_id.to_s(16)} question=#{question?} action=#{action?} setter=#{setter?}>"
+      "<SettingPath:#{object_id.to_s(16)} "\
+      "path=#{self}:#{@path.object_id.to_s(16)} "\
+      "question=#{question?} "\
+      "action=#{action?} "\
+      "setter=#{setter?}>"
     end
 
     private

data/spec/configurate/lookup_chain_spec.rb

@@ -1,8 +1,8 @@
-require 'spec_helper'
+require "spec_helper"
 
 class InvalidConfigurationProvider; end
 class ValidConfigurationProvider
-  def lookup(setting, *args); end
+  def lookup(_setting, *_args); end
 end
 
 describe Configurate::LookupChain do
@@ -97,7 +97,7 @@ describe Configurate::LookupChain do
     end
 
     it "returns nil if no value is found" do
-      @provider.each { |p| allow(p).to receive(:lookup).and_raise(Configurate::SettingNotFoundError) }
+      @provider.each {|p| allow(p).to receive(:lookup).and_raise(Configurate::SettingNotFoundError) }
       expect(subject.lookup "not.me").to be_nil
     end
   end

data/spec/configurate/provider/dynamic_spec.rb

@@ -1,16 +1,16 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Configurate::Provider::Dynamic do
   subject { described_class.new }
   describe "#lookup_path" do
     it "returns nil if the setting was never set" do
-      expect(subject.lookup_path Configurate::SettingPath.new(["not_me"]) ).to be_nil
+      expect(subject.lookup_path Configurate::SettingPath.new(["not_me"])).to be_nil
     end
 
     it "remembers the setting if it ends with =" do
       subject.lookup_path Configurate::SettingPath.new(["find_me", "later="]), "there"
 
-      expect(subject.lookup_path Configurate::SettingPath.new(["find_me", "later"]) ).to eq "there"
+      expect(subject.lookup_path Configurate::SettingPath.new(%w(find_me later))).to eq "there"
     end
 
     it "calls .get on the argument if a proxy object is given" do
@@ -20,14 +20,14 @@ describe Configurate::Provider::Dynamic do
     end
 
     it "resolves nested calls after group assignment" do
-      subject.lookup_path Configurate::SettingPath.new(["find_me", "later="]), {"a" => "b"}
-      expect(subject.lookup_path Configurate::SettingPath.new(["find_me", "later", "a"])).to eq "b"
+      subject.lookup_path Configurate::SettingPath.new(%w(find_me later=)), "a" => "b"
+      expect(subject.lookup_path Configurate::SettingPath.new(%w(find_me later a))).to eq "b"
     end
 
     it "clears out all overrides on reset_dynamic!" do
       subject.lookup_path Configurate::SettingPath.new(["find_me", "later="]), "there"
       subject.lookup_path Configurate::SettingPath.new(["reset_dynamic!"])
-      expect(subject.lookup_path Configurate::SettingPath.new(["find_me", "later"]) ).to_not eq "there"
+      expect(subject.lookup_path Configurate::SettingPath.new(%w(find_me later))).to_not eq "there"
     end
   end
 end

data/spec/configurate/provider/env_spec.rb

@@ -1,21 +1,21 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Configurate::Provider::Env do
   subject { described_class.new }
-  let(:existing_path) { ['existing', 'setting']}
-  let(:not_existing_path) { ['not', 'existing', 'path']}
-  let(:array_path) { ['array'] }
+  let(:existing_path) { %w(existing setting) }
+  let(:not_existing_path) { %w(not existing path) }
+  let(:array_path) { ["array"] }
   before(:all) do
-    ENV['EXISTING_SETTING'] = "there"
-    ENV['ARRAY'] = "foo,bar,baz"
+    ENV["EXISTING_SETTING"] = "there"
+    ENV["ARRAY"] = "foo,bar,baz"
   end
 
   after(:all) do
-    ENV['EXISTING_SETTING'] = nil
-    ENV['ARRAY'] = nil
+    ENV["EXISTING_SETTING"] = nil
+    ENV["ARRAY"] = nil
   end
 
-  describe '#lookup_path' do
+  describe "#lookup_path" do
     it "joins and upcases the path" do
       expect(ENV).to receive(:[]).with("EXISTING_SETTING")
       subject.lookup_path existing_path
@@ -26,7 +26,7 @@ describe Configurate::Provider::Env do
     end
 
     it "makes an array out of comma separated values" do
-      expect(subject.lookup_path array_path).to eq ["foo", "bar", "baz"]
+      expect(subject.lookup_path array_path).to eq %w(foo bar baz)
     end
 
     it "returns a unfrozen string" do

data/spec/configurate/provider/yaml_spec.rb

@@ -1,11 +1,14 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Configurate::Provider::YAML do
-  let(:settings) { {"toplevel" => "bar",
-                    "some" => {
-                      "nested" => { "some" => "lala", "setting" => "foo"}
-                    }
-                   } }
+  let(:settings) {
+    {
+      "toplevel" => "bar",
+      "some"     => {
+        "nested" => {"some" => "lala", "setting" => "foo"}
+      }
+    }
+  }
 
   describe "#initialize" do
     it "loads the file" do
@@ -23,26 +26,25 @@ describe Configurate::Provider::YAML do
       }.to raise_error Errno::ENOENT
     end
 
-
     context "with a namespace" do
       it "looks in the file for that namespace" do
         namespace = "some.nested"
         allow(::YAML).to receive(:load_file).and_return(settings)
-        provider = described_class.new 'bla', namespace: namespace
-        expect(provider.instance_variable_get :@settings).to eq settings['some']['nested']
+        provider = described_class.new "bla", namespace: namespace
+        expect(provider.instance_variable_get :@settings).to eq settings["some"]["nested"]
       end
 
       it "raises if the namespace isn't found" do
         allow(::YAML).to receive(:load_file).and_return({})
         expect {
-          described_class.new 'bla', namespace: "bar"
+          described_class.new "bla", namespace: "bar"
         }.to raise_error
       end
 
       it "works with an empty namespace in the file" do
-        allow(::YAML).to receive(:load_file).and_return({'foo' => {'bar' => nil}})
+        allow(::YAML).to receive(:load_file).and_return("foo" => {"bar" => nil})
         expect {
-          described_class.new 'bla', namespace: "foo.bar"
+          described_class.new "bla", namespace: "foo.bar"
         }.to_not raise_error
       end
     end
@@ -58,7 +60,7 @@ describe Configurate::Provider::YAML do
       it "doesn't raise if a namespace isn't found" do
         allow(::YAML).to receive(:load_file).and_return({})
         expect {
-          described_class.new 'bla', namespace: "foo", required: false
+          described_class.new "bla", namespace: "foo", required: false
         }.not_to raise_error
       end
     end
@@ -67,15 +69,31 @@ describe Configurate::Provider::YAML do
   describe "#lookup_path" do
     before do
       allow(::YAML).to receive(:load_file).and_return(settings)
-      @provider = described_class.new 'dummy'
+      @provider = described_class.new "dummy"
     end
 
     it "looks up the whole nesting" do
-      expect(@provider.lookup_path ["some", "nested", "some"]).to eq settings["some"]["nested"]["some"]
+      expect(@provider.lookup_path %w(some nested some)).to eq settings["some"]["nested"]["some"]
     end
 
     it "returns nil if no setting is found" do
       expect(@provider.lookup_path ["not_me"]).to be_nil
     end
+
+    context "with raise_on_missing set to true" do
+      before do
+        @provider = described_class.new "dummy", raise_on_missing: true
+      end
+
+      it "looks up the whole nesting" do
+        expect(@provider.lookup_path %w(some nested some)).to eq settings["some"]["nested"]["some"]
+      end
+
+      it "returns nil if no setting is found" do
+        expect {
+          @provider.lookup_path ["not_me"]
+        }.to raise_error Configurate::MissingSetting
+      end
+    end
   end
 end

data/spec/configurate/provider_spec.rb

@@ -1,10 +1,10 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Configurate::Provider::Base do
   describe "#lookup" do
     subject { described_class.new }
     it "calls #lookup_path" do
-      path = Configurate::SettingPath.new(["foo", "bar"])
+      path = Configurate::SettingPath.new(%w(foo bar))
       expect(subject).to receive(:lookup_path).with(path).and_return("something")
       expect(subject.lookup(path)).to eq "something"
     end
@@ -15,5 +15,17 @@ describe Configurate::Provider::Base do
         subject.lookup("bla")
       }.to raise_error Configurate::SettingNotFoundError
     end
+
+  end
+
+  describe "::lookup_in_hash" do
+    let(:hash) { {foo: {bar: nil}} }
+    it "returns nil if key is nil" do
+      expect(Configurate::Provider.lookup_in_hash(%i(foo bar), hash) { :fallback }).to be_nil
+    end
+
+    it "returns fallback for a non-existent key" do
+      expect(Configurate::Provider.lookup_in_hash(%i(foo bar baz), hash) { :fallback }).to eq :fallback
+    end
   end
 end

data/spec/configurate/proxy_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Configurate::Proxy do
   let(:lookup_chain) { double(lookup: "something") }
@@ -9,9 +9,9 @@ describe Configurate::Proxy do
       pending "If anyone knows a sane way to overwrite Module#===, please tell me :P"
       result = case proxy
                when String
-                "string"
+                 "string"
                else
-                "wrong"
+                 "wrong"
                end
       expect(result).to eq "string"
     end
@@ -54,9 +54,8 @@ describe Configurate::Proxy do
   end
 
   describe "#target" do
-    [:to_s, :to_xml, :respond_to?, :present?, :!=, :eql?,
-     :each, :try, :size, :length, :count, :==, :=~, :gsub, :blank?, :chop,
-     :start_with?, :end_with?].each do |method|
+    %i(to_s to_xml respond_to? present? != eql? each try size length
+       count == =~ gsub blank? chop start_with? end_with?).each do |method|
       it "is called for accessing #{method} on the proxy" do
         target = double(respond_to?: true, _proxy?: false)
         allow(lookup_chain).to receive(:lookup).and_return(target)
@@ -80,22 +79,22 @@ describe Configurate::Proxy do
 
     it "converts to a string" do
       allow(lookup_chain).to receive(:lookup).and_return("bar")
-      expect("foo"+proxy.something).to eq "foobar"
+      expect("foo#{proxy.something}").to eq "foobar"
     end
 
     it "converts to a number" do
       allow(lookup_chain).to receive(:lookup).and_return(1)
-      expect(2+proxy.something).to eq 3
+      expect(2 + proxy.something).to eq 3
     end
 
     it "converts to an array" do
       allow(lookup_chain).to receive(:lookup).and_return([1, 2])
-      expect([:a, :b].zip(proxy.something)).to eq [[:a, 1], [:b, 2]]
+      expect(%i(a b).zip(proxy.something)).to eq [[:a, 1], [:b, 2]]
     end
 
     it "converts to a hash" do
-      allow(lookup_chain).to receive(:lookup).and_return({:a => :b})
-      expect({c: :d}.merge(proxy.something)).to eq({:a => :b, :c => :d})
+      allow(lookup_chain).to receive(:lookup).and_return(a: :b)
+      expect({c: :d}.merge(proxy.something)).to eq a: :b, c: :d
     end
   end
 end

data/spec/configurate/setting_path_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Configurate::SettingPath do
   let(:normal_path) { described_class.new([:foo]) }
@@ -53,7 +53,7 @@ describe Configurate::SettingPath do
 
   describe "#initialize_copy" do
     it "modifying a copy leaves the original unchanged" do
-      original = described_class.new ["foo", "bar"]
+      original = described_class.new %w(foo bar)
       copy = original.clone
       copy << "baz"
       expect(copy).to include "baz"
@@ -61,7 +61,6 @@ describe Configurate::SettingPath do
     end
   end
 
-
   describe "#question_action_or_setter?" do
     context "with a question signature as setting" do
       subject { question_path.question_action_or_setter? }
@@ -86,22 +85,22 @@ describe Configurate::SettingPath do
 
   describe "#each" do
     it "should strip special characters" do
-      expect(long_path.all? { |c| c.include? "?" }).to be_falsey
+      expect(long_path.all? {|c| c.include? "?" }).to be_falsey
     end
   end
 
-  [:join, :first, :last, :shift, :pop].each do |method|
+  %i(join first last shift pop).each do |method|
     describe "##{method}" do
       subject { question_path.public_send method }
       it { should_not include "?" }
     end
   end
 
-  [:<<, :unshift, :push].each do |method|
+  %i(<< unshift push).each do |method|
     describe "##{method}" do
-      it 'converts the argument to a string' do
+      it "converts the argument to a string" do
         arg = double
-        expect(arg).to receive(:to_s).and_return('bar')
+        expect(arg).to receive(:to_s).and_return("bar")
         described_class.new.public_send method, arg
       end
     end
@@ -120,7 +119,7 @@ describe Configurate::SettingPath do
 
   describe "#inspect" do
     it "includes the dotted path" do
-      path = described_class.new([:foo, :bar])
+      path = described_class.new(%i(foo bar))
       expect(path.inspect).to include "foo.bar"
     end
   end

data/spec/configurate_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Configurate::Settings do
   describe "#method_missing" do
@@ -12,7 +12,7 @@ describe Configurate::Settings do
     end
   end
 
-  [:lookup, :add_provider, :[]].each do |method|
+  %i(lookup add_provider []).each do |method|
     describe "#{method}" do
       subject { described_class.create }
 

data/spec/spec_helper.rb

@@ -5,13 +5,12 @@
 #
 # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 
-
 begin
-  require 'coveralls'
+  require "coveralls"
   Coveralls.wear!
 rescue LoadError; end
 
-require 'configurate'
+require "configurate"
 
 def silence_stderr
   $stderr = StringIO.new
@@ -27,7 +26,7 @@ RSpec.configure do |config|
   # order dependency and want to debug it, you can fix the order by providing
   # the seed, which is printed after each run.
   #     --seed 1234
-  config.order = 'random'
+  config.order = "random"
 
   config.expect_with :rspec do |expect_config|
     expect_config.syntax = :expect

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: configurate
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Jonne Haß
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-10-03 00:00:00.000000000 Z
+date: 2015-05-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -85,18 +85,18 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.1
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Flexbile configuration system
 test_files:
-- spec/configurate/provider_spec.rb
-- spec/configurate/lookup_chain_spec.rb
-- spec/configurate/provider/yaml_spec.rb
-- spec/configurate/provider/env_spec.rb
 - spec/configurate/provider/dynamic_spec.rb
+- spec/configurate/provider/env_spec.rb
+- spec/configurate/provider/yaml_spec.rb
 - spec/configurate/setting_path_spec.rb
 - spec/configurate/proxy_spec.rb
+- spec/configurate/provider_spec.rb
+- spec/configurate/lookup_chain_spec.rb
 - spec/spec_helper.rb
 - spec/configurate_spec.rb
 has_rdoc: 

checksums.yaml.gz.asc

@@ -1,11 +0,0 @@
------BEGIN PGP SIGNATURE-----
-Version: GnuPG v2
-
-iQEcBAABAgAGBQJULyg7AAoJEPNH4OtHrHDWLzkIAIwwqTjkZlNOPS2D7tUOcNJY
-ZF/zxymbrSL9yjzXwvYExgKy2i1xV5LrWpWP/mCiX9RX2eLxv8LcQGlWKupCwMc9
-2ca/yy1wziXLhLY6zJhWF5j6WNOfRlJTGaAUPSHtbCeT9WOjbpnDWfWTDZKNuonb
-i1HPUCZPx+YAXb6Gzgl39FcCH1UgEZy3D3ijc3FCSHigfnwSEDcjCwQtQuMedwNu
-l1/V5UdTPDiWPBxeGMucyZLBKdN8r70wfgNKvDZ1XHolO+9HUOsmcS6St1P5OpUp
-dJ25Tm3ACv+uydyQLr/e2bPJs+Lm901rKUMb+gIozoQCYMTRnpAoNMVNqK+KRBI=
-=+ZIR
------END PGP SIGNATURE-----

data.tar.gz.asc

@@ -1,11 +0,0 @@
------BEGIN PGP SIGNATURE-----
-Version: GnuPG v2
-
-iQEcBAABAgAGBQJULyg7AAoJEPNH4OtHrHDWgO0H/jLKCQPi2kguHzfJU/k4hII8
-VITtYbNZb4ZBbHecgiam4mI+m5EvDCfQu3KEzSN3w06czUoVFW0uCABJkAqb43He
-EvITE2z36GKFb8GqfA5EXnRPjmsGTvH8EnUG8DetTRNjFkERvNSi6t3RxgfNQ8yw
-UFwtascrsmM5vjzCvgRlJ0cm1HA3LZFqoEyw/MovOLDviLTz1ECKqxNmaEULdhp9
-Fb10e5Y9XalRbbF6wSOKWp06owiEcTVY/lqNOZBJ+ZIG1RQUD73yhAGW2b7bM6rg
-tiBEAal1Tt6c5v5SvucCcMrKxd3R8CXCtwETQVFl4rxB4eYhSgx0PkFiQ3OPicY=
-=hJkd
------END PGP SIGNATURE-----

metadata.gz.asc

@@ -1,11 +0,0 @@
------BEGIN PGP SIGNATURE-----
-Version: GnuPG v2
-
-iQEcBAABAgAGBQJULyg7AAoJEPNH4OtHrHDWD0IH/0e90hWnDOQqWhUnKY3JM+gf
-wOdHXiyB4p7cPIoO6QFsZq0Mm4t+bYLADI3adDq9kQAtIjo6s3jUJzK84QNjnsNS
-8Sc1OWNFYLrMVi/7rkJfyjiV6rMHYyB9O76vxH3w7/CZAl7fscadF4BW1PTqcti8
-z4IoXtBs5KVxk1U3vtpPLaYitYIx/mc2voL0Z5wEuMeBhTYHjx4XraLrNzuSh6yN
-cRnItaD70Z2uIvRF6bSyazDLWEO2cUfH/kTjsjHja4xwa4yYm3/ZkFZ2YZAXJNvU
-P5mtnv5cCcJKZy1uNKV5DRj9zATL+yrzgSaSKV1O9BPW7zL3eHeu156TOxIRW48=
-=sUo3
------END PGP SIGNATURE-----