configurate 0.3.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: ce3f9cd0e402209f0d1f6658072604e926a0553b
-  data.tar.gz: 685a8812aebdcad989056ffe75d6d9ecff65d7f6
+SHA256:
+  metadata.gz: c7ee31139c43b2f17023d605dee685f0618fcc0e56b068733cddb8e5e10cf006
+  data.tar.gz: 5ee1c764538c194795750718c6a530cbd94684d334782afc7fa47a35083547af
 SHA512:
-  metadata.gz: c7b64517e364ae891790d10737deb7feadcb326eb650d6f5b8dcf05628a5e8d99073d939e05fcf8fb75a5874043751c32b47c859adece475b16b56c050c08993
-  data.tar.gz: 7ddc889f275b59ca061289392deb07bda601c28520b5957168470b1bcf9efc33e012db50b648a642e3a4fe3552ccb633eabc0d336e30374a54e2a880744ff2b7
+  metadata.gz: c221bc2ed5cf3b7705a207776628621301cd73c3e891dac13ba9d2ada574fc265a5678edff29974da657daa373d712ccb6753ee2083ad6ab0d94b36cd1314225
+  data.tar.gz: 338aaf220e3ccc945a35079fb15c50c6af87132c78c97fb7fe0ff1d2d8bdd54e5a0406aa01d890eb910cfee55fb649667e67c042c54e005d0772ce654ba9be1a

data/Changelog.md

@@ -1,3 +1,9 @@
+# 0.4.0
+
+* `Configurate::Proxy` returns its own singleton class if the target does not support creating one.
+* Extract `Configurate::Provider::StringHash` as a new base class for hash based providers.
+* Add `Configurate::Provider::TOML`.
+
 # 0.3.1
 
 * Configurate::Provider::Dynamic returns true when passed the special

data/README.md

@@ -1,8 +1,7 @@
 # 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/jhass/configurate.png)](https://gemnasium.com/jhass/configurate)
-[![Code Climate](https://codeclimate.com/github/jhass/configurate.png)](https://codeclimate.com/github/jhass/configurate)
+[![Gem Version](https://badge.fury.io/rb/configurate.svg)](https://badge.fury.io/rb/configurate)
+[![Build Status](https://travis-ci.org/jhass/configurate.svg?branch=master)](https://travis-ci.org/jhass/configurate)
+[![Code Climate](https://codeclimate.com/github/jhass/configurate.svg)](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
@@ -107,14 +106,6 @@ will always output `unknown`. Again use `.get`
 
 ## Shipped providers
 
-### Configurate::Provider::Base
-
-A convenience base class changing the interface for implementers. It provides a basic `#lookup` method
-which just passes all parameters through to `#lookup_path`.
-The result of `#lookup_path` is returned, unless it's `nil`
-then `Configurate::SettingNotFoundError` is raised. Subclasses are expected to implement `#lookup_path`.
-Do not use this class directly as a provider!
-
 ### Configurate::Provider::Env
 
 This class transforms a query string into a name for a environment variable and looks up this variable then.
@@ -124,6 +115,40 @@ into arrays.
 
 This provider does not take any additional initialization parameters.
 
+### Configurate::Provider::TOML
+
+This provider reads settings from a given [TOML](https://github.com/toml-lang/toml) file. It converts the sections of
+query string to a nested value. For a given TOML file
+
+```toml
+[stuff]
+enable = true
+param = "foo"
+
+[stuff.nested]
+param = "bar"
+```
+
+the following queries would be valid:
+
+```ruby
+Config.stuff.enable?      # => true
+Config.stuff.param        # => "foo"
+Config.stuff.nested.param # => "bar"
+```
+
+This provider depends on the [tomlrb](https://github.com/fbernier/tomlrb) gem. This is why it is not loaded by default and
+needs an explicit `require 'configurate/provider/toml' to be available.
+
+The initializer takes a path to the configuration file a the mandatory first argument and
+the following optional parameters:
+
+* *namespace:* Specify a alternative root. This is useful if you for example add the same file multiple
+  times through multiple providers, with different namespaces, letting you override settings depending on
+  the rails environment, without duplicating common settings. Defaults to none.
+* *required:* Whether to raise an error if the the file isn't found or, if one is given, the namespace
+  doesn't exist in the file.
+
 ### Configurate::Provider::YAML
 
 This provider reads settings from a given [YAML](http://www.yaml.org) file. It converts the sections of
@@ -145,8 +170,8 @@ Config.stuff.param        # => "foo"
 Config.stuff.nested.param # => "bar"
 ```
 
-The initializer takes a path to the configuration file as mandatory first argument and
-the following optional parameters, as a hash:
+The initializer takes a path to the configuration file a the mandatory first argument and
+the following optional parameters:
 
 * *namespace:* Specify a alternative root. This is useful if you for example add the same file multiple
   times through multiple providers, with different namespaces, letting you override settings depending on
@@ -154,6 +179,46 @@ the following optional parameters, as a hash:
 * *required:* Whether to raise an error if the the file isn't found or, if one is given, the namespace
   doesn't exist in the file.
 
+
+### Configurate::Provider::StringHash
+
+A provider taking a (nested) `Hash` where all keys are strings. The query string is then looked up in this hash.
+
+So for a given `Hash`
+
+```ruby
+{
+  "stuff" => {
+    "enable" => true,
+    "param" => "foo",
+    "nested" => {
+      "param" => "bar"
+    }
+  }
+}
+```
+
+the following queries would be valid:
+
+```ruby
+Config.stuff.enable?      # => true
+Config.stuff.param        # => "foo"
+Config.stuff.nested.param # => "bar"
+```
+
+The initializer takes the hash as the mandatory first argument and
+the following optional parameters:
+
+* *namespace:* Specify a alternative root. This is useful if you for example add the same file multiple
+  times through multiple providers, with different namespaces, letting you override settings depending on
+  the rails environment, without duplicating common settings. Defaults to none.
+* *required:* Whether to raise an error if the namespace
+  doesn't exist in the hash.
+* *source:* A hint text about the origin of the configuration data to be used in error messages.
+
+As you may have noticed by now, `Configurate::Provider::YAML` and `Configurate::Provider::TOML` are merely convenience
+subclasses of this provider, loading the file for you.
+
 ### Configurate::Provider::Dynamic
 
 A provider which stores the first additional parameter if the query string ends with an equal sign and can
@@ -167,6 +232,14 @@ Config.reset_dynamic!
 Config.foo.bar         # => nil
 ```
 
+### Configurate::Provider::Base
+
+A convenience base class changing the interface for implementers. It provides a basic `#lookup` method
+which just passes all parameters through to `#lookup_path`.
+The result of `#lookup_path` is returned, unless it's `nil`
+then `Configurate::SettingNotFoundError` is raised. Subclasses are expected to implement `#lookup_path`.
+Do not use this class directly as a provider!
+
 ## Writing a provider
 
 ...should be pretty easy. For example here is the `Configurate::Provider::Env` provider:
@@ -184,6 +257,8 @@ class Configurate::Provider::Env < Configurate::Provider::Base
 end
 ```
 
+`Configurate::Provider::StringHash` should also serve as a useful baseclass for most providers.
+
 
 ## Documentation
 

data/lib/configurate.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "forwardable"
 
 require "configurate/setting_path"
@@ -57,10 +59,14 @@ module Configurate
 
     def_delegators :@lookup_chain, :lookup, :add_provider, :[]
 
+    # rubocop:disable Style/MethodMissingSuper we handle all calls
+    # rubocop:disable Style/MissingRespondToMissing we override respond_to? instead
+
     # See description and {#lookup}, {#[]} and {#add_provider}
     def method_missing(method, *args, &block)
       Proxy.new(@lookup_chain).public_send(method, *args, &block)
     end
+    # rubocop:enable all
 
     # Create a new configuration object
     # @yield the given block will be evaluated in the context of the new object

data/lib/configurate/lookup_chain.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Configurate
   # This object builds a chain of configuration providers to try to find
   # the value of a setting.

data/lib/configurate/provider.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Configurate
   module Provider
     # This provides a basic {#lookup} method for other providers to build
@@ -9,6 +11,7 @@ module Configurate
       def lookup(*args)
         result = lookup_path(*args)
         return result unless result.nil?
+
         raise Configurate::SettingNotFoundError, "The setting #{args.first} was not found"
       end
     end
@@ -24,11 +27,13 @@ module Configurate
         hash = hash[setting_path.shift]
       end
       return fallback.call unless setting_path.empty?
+
       hash
     end
   end
 end
 
+require "configurate/provider/string_hash"
 require "configurate/provider/yaml"
 require "configurate/provider/env"
 require "configurate/provider/dynamic"

data/lib/configurate/provider/dynamic.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Configurate
   module Provider
     # This provider knows nothing upon initialization, however if you access
@@ -13,7 +15,7 @@ module Configurate
           return true
         end
 
-        if setting_path.setter? && args.length > 0
+        if setting_path.setter? && !args.empty?
           *root, key = setting_path.to_a
           hash = root.inject(settings) {|hash, key| hash[key] }
           hash[key] = extract_value(args)

data/lib/configurate/provider/env.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Configurate
   module Provider
     # This provider looks for settings in the environment.

data/lib/configurate/provider/string_hash.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+module Configurate
+  module Provider
+    # This provider takes a nested string keyed hash and does nested lookups in it.
+    class StringHash < Base
+      # @param hash [::Hash] the string keyed hash to provide values from
+      # @param namespace [String] optionally set this as the root
+      # @param required [Boolean] whether or not to raise an error if
+      #   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+.
+      # @param source [String] optional hint of what's the source of this configuration. Used in error messages.
+      # @raise [ArgumentError] if the namespace isn't found in the hash or the given object is not a hash
+      def initialize hash, namespace: nil, required: true, raise_on_missing: false, source: nil
+        raise ArgumentError, "Please provide a hash" unless hash.is_a?(Hash)
+
+        @required = required
+        @raise_on_missing = raise_on_missing
+        @source = source
+        @settings = root_from hash, namespace
+      end
+
+      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
+
+      private
+
+      def root_from hash, namespace
+        return hash if namespace.nil?
+
+        Provider.lookup_in_hash(SettingPath.new(namespace), hash) do
+          raise ArgumentError, "Namespace #{namespace} not found #{"in #{@source}" if @source}" if @required
+
+          warn "WARNING: Namespace #{namespace} not found #{"in #{@source}" if @source}"
+          nil
+        end
+      end
+    end
+  end
+end

data/lib/configurate/provider/toml.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require "configurate"
+require "tomlrb"
+
+module Configurate
+  module Provider
+    # This provider tries to open a TOML file and does nested lookups
+    # in it.
+    class TOML < StringHash
+      # @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
+        super(Tomlrb.load_file(file),
+          namespace:        namespace,
+          required:         required,
+          raise_on_missing: raise_on_missing,
+          source:           file
+        )
+      rescue Errno::ENOENT => e
+        warn "WARNING: Configuration file #{file} not found, ensure it's present"
+        raise e if required
+      end
+    end
+  end
+end

data/lib/configurate/provider/yaml.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 require "yaml"
 
 module Configurate
   module Provider
     # This provider tries to open a YAML file and does nested lookups
     # in it.
-    class YAML < Base
+    class YAML < StringHash
       # @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
@@ -14,29 +16,16 @@ module Configurate
       # @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)
-
-        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
+        super(::YAML.load_file(file),
+          namespace:        namespace,
+          required:         required,
+          raise_on_missing: raise_on_missing,
+          source:           file
+        )
       rescue Errno::ENOENT => e
         warn "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) {
-          raise MissingSetting.new "#{setting_path} is not a valid setting." if @raise_on_missing
-          nil
-        }
-      end
     end
   end
 end

data/lib/configurate/proxy.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Configurate
   # Proxy object to support nested settings
   #
@@ -23,7 +25,7 @@ module Configurate
       !target
     end
 
-    %i(!= == 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
@@ -38,6 +40,7 @@ module Configurate
       define_method method do
         value = target
         return value.public_send converter if value.respond_to? converter
+
         value.public_send method
       end
     end
@@ -55,6 +58,17 @@ module Configurate
     end
     alias_method :public_send, :send
 
+    def singleton_class
+      target.singleton_class
+    rescue ::TypeError
+      class << self
+        self
+      end
+    end
+
+    # rubocop:disable Style/MethodMissingSuper we handle all calls
+    # rubocop:disable Style/MissingRespondToMissing we override respond_to? instead
+
     def method_missing setting, *args, &block
       return target.public_send(setting, *args, &block) if target_respond_to? setting
 
@@ -64,6 +78,7 @@ module Configurate
 
       self
     end
+    # rubocop:enable all
 
     # Get the setting at the current path, if found.
     # (see LookupChain#lookup)
@@ -76,7 +91,7 @@ module Configurate
 
     private
 
-    COMMON_KEY_NAMES = %i(key method)
+    COMMON_KEY_NAMES = %i[key method].freeze
 
     def target_respond_to? setting, include_private=false
       return false if COMMON_KEY_NAMES.include? setting

data/lib/configurate/setting_path.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "forwardable"
 
 module Configurate
@@ -40,18 +42,19 @@ module Configurate
 
     def each
       return to_enum(:each) unless block_given?
+
       @path.each do |component|
         yield clean_special_characters(component)
       end
     end
 
-    %i(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
 
-    %i(<< 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

data/spec/configurate/lookup_chain_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 class InvalidConfigurationProvider; end
@@ -71,34 +73,34 @@ describe Configurate::LookupChain do
 
     it "converts numbers to strings" do
       allow(@provider[0]).to receive(:lookup).and_return(5)
-      expect(subject.lookup "foo").to eq "5"
+      expect(subject.lookup("foo")).to eq "5"
     end
 
     it "does not convert false to a string" do
       allow(@provider[0]).to receive(:lookup).and_return(false)
-      expect(subject.lookup "enable").to be_falsey
+      expect(subject.lookup("enable")).to be_falsey
     end
 
     it "converts 'true' to true" do
       allow(@provider[0]).to receive(:lookup).and_return("true")
-      expect(subject.lookup "enable").to be_truthy
+      expect(subject.lookup("enable")).to be_truthy
     end
 
     it "converts 'false' to false" do
       allow(@provider[0]).to receive(:lookup).and_return("false")
-      expect(subject.lookup "enable").to be_falsey
+      expect(subject.lookup("enable")).to be_falsey
     end
 
     it "returns the value unchanged if it can't be converted" do
       value = double
       allow(value).to receive(:respond_to?).with(:to_s).and_return(false)
       allow(@provider[0]).to receive(:lookup).and_return(value)
-      expect(subject.lookup "enable").to eq value
+      expect(subject.lookup("enable")).to eq value
     end
 
     it "returns nil if no value is found" do
       @provider.each {|p| allow(p).to receive(:lookup).and_raise(Configurate::SettingNotFoundError) }
-      expect(subject.lookup "not.me").to be_nil
+      expect(subject.lookup("not.me")).to be_nil
     end
   end
 end

data/spec/configurate/provider/dynamic_spec.rb

@@ -1,16 +1,18 @@
+# frozen_string_literal: true
+
 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(%w(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 +22,14 @@ describe Configurate::Provider::Dynamic do
     end
 
     it "resolves nested calls after group assignment" do
-      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"
+      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"
-      expect(subject.lookup_path Configurate::SettingPath.new(["reset_dynamic!"])).to eq true
-      expect(subject.lookup_path Configurate::SettingPath.new(%w(find_me later))).to_not eq "there"
+      expect(subject.lookup_path(Configurate::SettingPath.new(["reset_dynamic!"]))).to eq true
+      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,9 +1,11 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Configurate::Provider::Env do
   subject { described_class.new }
-  let(:existing_path) { %w(existing setting) }
-  let(:not_existing_path) { %w(not existing path) }
+  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"
@@ -22,11 +24,11 @@ describe Configurate::Provider::Env do
     end
 
     it "returns nil if the setting isn't available" do
-      expect(subject.lookup_path not_existing_path).to be_nil
+      expect(subject.lookup_path(not_existing_path)).to be_nil
     end
 
     it "makes an array out of comma separated values" do
-      expect(subject.lookup_path array_path).to eq %w(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/string_hash_spec.rb

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe Configurate::Provider::StringHash do
+  let(:settings) {
+    {
+      "toplevel" => "bar",
+      "some"     => {
+        "nested" => {"some" => "lala", "setting" => "foo"}
+      }
+    }
+  }
+
+  describe "#initialize" do
+    it "raises if the argument is not hash" do
+      expect {
+        described_class.new "foo"
+      }.to raise_error ArgumentError
+    end
+
+    context "with a namespace" do
+      it "looks in the hash for that namespace" do
+        namespace = "some.nested"
+        provider = described_class.new settings, namespace: namespace
+        expect(provider.instance_variable_get(:@settings)).to eq settings["some"]["nested"]
+      end
+
+      it "raises if the namespace isn't found" do
+        expect {
+          described_class.new({}, namespace: "bar")
+        }.to raise_error ArgumentError
+      end
+
+      it "works with an empty namespace in the file" do
+        expect {
+          described_class.new({"foo" => {"bar" => nil}}, namespace: "foo.bar")
+        }.to_not raise_error
+      end
+    end
+
+    context "with required set to false" do
+      it "doesn't raise if a namespace isn't found" do
+        expect {
+          silence_stderr do
+            described_class.new({}, namespace: "foo", required: false)
+          end
+        }.not_to raise_error
+      end
+    end
+  end
+
+  describe "#lookup_path" do
+    before do
+      @provider = described_class.new settings
+    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 be_nil
+    end
+
+    context "with raise_on_missing set to true" do
+      before do
+        @provider = described_class.new settings, 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/toml_spec.rb

@@ -0,0 +1,110 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+require "configurate/provider/toml"
+
+describe Configurate::Provider::TOML do
+  let(:settings) {
+    {
+      "toplevel" => "bar",
+      "some"     => {
+        "nested" => {"some" => "lala", "setting" => "foo"}
+      }
+    }
+  }
+
+  describe "#initialize" do
+    it "loads the file" do
+      file = "foobar.toml"
+      expect(Tomlrb).to receive(:load_file).with(file).and_return({})
+      described_class.new file
+    end
+
+    it "raises if the file is not found" do
+      allow(Tomlrb).to receive(:load_file).and_raise(Errno::ENOENT)
+      expect {
+        silence_stderr do
+          described_class.new "foo"
+        end
+      }.to raise_error Errno::ENOENT
+    end
+
+    context "with a namespace" do
+      it "looks in the file for that namespace" do
+        namespace = "some.nested"
+        allow(Tomlrb).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"]
+      end
+
+      it "raises if the namespace isn't found" do
+        allow(Tomlrb).to receive(:load_file).and_return({})
+        expect {
+          silence_stderr do
+            described_class.new "bla", namespace: "bar"
+          end
+        }.to raise_error ArgumentError
+      end
+
+      it "works with an empty namespace in the file" do
+        allow(Tomlrb).to receive(:load_file).and_return("foo" => {"bar" => nil})
+        expect {
+          silence_stderr do
+            described_class.new "bla", namespace: "foo.bar"
+          end
+        }.to_not raise_error
+      end
+    end
+
+    context "with required set to false" do
+      it "doesn't raise if a file isn't found" do
+        allow(Tomlrb).to receive(:load_file).and_raise(Errno::ENOENT)
+        expect {
+          silence_stderr do
+            described_class.new "not_me", required: false
+          end
+        }.not_to raise_error
+      end
+
+      it "doesn't raise if a namespace isn't found" do
+        allow(Tomlrb).to receive(:load_file).and_return({})
+        expect {
+          silence_stderr do
+            described_class.new "bla", namespace: "foo", required: false
+          end
+        }.not_to raise_error
+      end
+    end
+  end
+
+  describe "#lookup_path" do
+    before do
+      allow(Tomlrb).to receive(:load_file).and_return(settings)
+      @provider = described_class.new "dummy"
+    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 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/yaml_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Configurate::Provider::YAML do
@@ -31,14 +33,14 @@ describe Configurate::Provider::YAML 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"]
+        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"
-        }.to raise_error
+        }.to raise_error ArgumentError
       end
 
       it "works with an empty namespace in the file" do
@@ -53,14 +55,18 @@ describe Configurate::Provider::YAML do
       it "doesn't raise if a file isn't found" do
         allow(::YAML).to receive(:load_file).and_raise(Errno::ENOENT)
         expect {
-          described_class.new "not_me", required: false
+          silence_stderr do
+            described_class.new "not_me", required: false
+          end
         }.not_to raise_error
       end
 
       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
+          silence_stderr do
+            described_class.new "bla", namespace: "foo", required: false
+          end
         }.not_to raise_error
       end
     end
@@ -73,11 +79,11 @@ describe Configurate::Provider::YAML do
     end
 
     it "looks up the whole nesting" do
-      expect(@provider.lookup_path %w(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
+      expect(@provider.lookup_path(["not_me"])).to be_nil
     end
 
     context "with raise_on_missing set to true" do
@@ -86,7 +92,7 @@ describe Configurate::Provider::YAML do
       end
 
       it "looks up the whole nesting" do
-        expect(@provider.lookup_path %w(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

data/spec/configurate/provider_spec.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Configurate::Provider::Base do
   describe "#lookup" do
     subject { described_class.new }
     it "calls #lookup_path" do
-      path = Configurate::SettingPath.new(%w(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,17 +17,16 @@ 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
+      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
+      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,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Configurate::Proxy do
@@ -54,8 +56,8 @@ describe Configurate::Proxy do
   end
 
   describe "#target" do
-    %i(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)
@@ -89,7 +91,7 @@ describe Configurate::Proxy do
 
     it "converts to an array" do
       allow(lookup_chain).to receive(:lookup).and_return([1, 2])
-      expect(%i(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

data/spec/configurate/setting_path_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Configurate::SettingPath do
@@ -10,7 +12,7 @@ describe Configurate::SettingPath do
   describe "#initialize" do
     context "with a string" do
       it "creates a path" do
-        expect(described_class.new long_path.to_s).to eq long_path
+        expect(described_class.new(long_path.to_s)).to eq long_path
       end
     end
   end
@@ -53,7 +55,7 @@ describe Configurate::SettingPath do
 
   describe "#initialize_copy" do
     it "modifying a copy leaves the original unchanged" do
-      original = described_class.new %w(foo bar)
+      original = described_class.new %w[foo bar]
       copy = original.clone
       copy << "baz"
       expect(copy).to include "baz"
@@ -89,14 +91,14 @@ describe Configurate::SettingPath do
     end
   end
 
-  %i(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
 
-  %i(<< unshift push).each do |method|
+  %i[<< unshift push].each do |method|
     describe "##{method}" do
       it "converts the argument to a string" do
         arg = double
@@ -119,7 +121,7 @@ describe Configurate::SettingPath do
 
   describe "#inspect" do
     it "includes the dotted path" do
-      path = described_class.new(%i(foo bar))
+      path = described_class.new(%i[foo bar])
       expect(path.inspect).to include "foo.bar"
     end
   end

data/spec/configurate_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Configurate::Settings do
@@ -13,7 +15,7 @@ describe Configurate::Settings do
   end
 
   %i(lookup add_provider []).each do |method|
-    describe "#{method}" do
+    describe method.to_s do
       subject { described_class.create }
 
       it "delegates the call to #lookup_chain" do

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # This file was generated by the `rspec --init` command. Conventionally, all
 # specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
 # Require this file using `require "spec_helper"` to ensure that it is only
@@ -8,13 +10,15 @@
 begin
   require "coveralls"
   Coveralls.wear!
-rescue LoadError; end
+rescue LoadError
+end
 
 require "configurate"
 
 def silence_stderr
   $stderr = StringIO.new
   yield
+ensure
   $stderr = STDERR
 end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: configurate
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.0
 platform: ruby
 authors:
 - Jonne Haß
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-06 00:00:00.000000000 Z
+date: 2020-04-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -38,6 +38,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: tomlrb
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.3.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.3.0
 description: Configurate is a flexible configuration system that can read settings
   from multiple sources at the same time.
 email: me@jhass.eu
@@ -53,12 +67,16 @@ files:
 - lib/configurate/provider.rb
 - lib/configurate/provider/dynamic.rb
 - lib/configurate/provider/env.rb
+- lib/configurate/provider/string_hash.rb
+- lib/configurate/provider/toml.rb
 - lib/configurate/provider/yaml.rb
 - lib/configurate/proxy.rb
 - lib/configurate/setting_path.rb
 - spec/configurate/lookup_chain_spec.rb
 - spec/configurate/provider/dynamic_spec.rb
 - spec/configurate/provider/env_spec.rb
+- spec/configurate/provider/string_hash_spec.rb
+- spec/configurate/provider/toml_spec.rb
 - spec/configurate/provider/yaml_spec.rb
 - spec/configurate/provider_spec.rb
 - spec/configurate/proxy_spec.rb
@@ -77,26 +95,26 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.2
+      version: 2.0.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Flexbile configuration system
 test_files:
+- spec/spec_helper.rb
+- spec/configurate/lookup_chain_spec.rb
+- spec/configurate/provider/toml_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/provider/string_hash_spec.rb
+- spec/configurate/provider/env_spec.rb
 - spec/configurate/proxy_spec.rb
 - spec/configurate/provider_spec.rb
-- spec/configurate/lookup_chain_spec.rb
-- spec/spec_helper.rb
+- spec/configurate/setting_path_spec.rb
 - spec/configurate_spec.rb
-has_rdoc: