figleaf 0.3.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 37549d550c5b5f23f930cbbe09f7c94f554be0d61175ffd3c100da8d4da57132
-  data.tar.gz: 13de48551a3e5ba8b0ef24a8b206ad2e4ecbbe509bfacabc733d65c1e2d7fa57
+  metadata.gz: d1b89f2110836d4dd5f0ad63b08f4a4ce2b8c6b17edbb1103a73e85ffaa898c9
+  data.tar.gz: b798a685d89f189e37e89624f08d8c2a6f9bce2abad278f983e83dee5cf292fe
 SHA512:
-  metadata.gz: 99e014709696659c0c0cdded8e7e1ee79ac485fe82cea11d5f4d4dd63acddcaf8605a30e748d19beb477eb7a7eb9ec4bf13fb41eda39dca00e61ee5aee36d13a
-  data.tar.gz: 539305c3f50e86711a6d921a8a49962b131905b3939c619805d062ef00897af43295526a0ad3a6fd3cdee176124bfe838e945e300a1514906028086680ae850b
+  metadata.gz: b00325b3a45103b5b3d9859befeff699bea806e981757b86371ada1cf933ebdb9a4e1cbb139ae96127b221543d1f6092441dfb3e591624f289ae2e0bac91cccd
+  data.tar.gz: 51f854f9551e34b2b5492113f027cad42ad8519c1f86be2ed8c05bf8dc6ae498360edc0bd2578971e428bf39325ebe62f67a4aec9d43d812ba70972f04e2cab4

data/.rspec

@@ -1,2 +1,2 @@
 --color
---format progress
+--format documentation

data/.solargraph.yml

@@ -0,0 +1,16 @@
+---
+include:
+  - "**/*.rb"
+exclude:
+  - spec/**/*
+  - test/**/*
+  - vendor/**/*
+  - ".bundle/**/*"
+require: []
+domains: []
+reporters:
+  - update_errors
+formatter: null
+require_paths: []
+plugins: []
+max_files: 15000

data/lib/figleaf/config.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Figleaf
   # Convert a ruby block to nested hash
   class Config
@@ -33,17 +35,4 @@ module Figleaf
 
     attr_reader :property
   end
-
-  class LazyBlockHash < Hash
-    def [](attr)
-      val = super(attr)
-      if val.is_a?(Proc)
-        val.call
-      else
-        val
-      end
-    rescue => e
-      raise Settings::InvalidRb, "Configuration has invalid Ruby\n" + e.message
-    end
-  end
 end

data/lib/figleaf/fighash.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Figleaf
   class Fighash < Hashie::Mash
     def method_missing(method_name, *args, &blk)

data/lib/figleaf/lazy_block_hash.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module Figleaf
+  class LazyBlockHash < Hash
+    def [](attr)
+      val = super
+      if val.is_a?(Proc)
+        val.call
+      else
+        val
+      end
+    rescue => e
+      raise Settings::InvalidRb, "Configuration has invalid Ruby\n#{e.message}"
+    end
+  end
+end

data/lib/figleaf/settings.rb

@@ -1,7 +1,10 @@
+# frozen_string_literal: true
+
 module Figleaf
   class Settings
     InvalidYAML = Class.new(RuntimeError)
     InvalidRb = Class.new(RuntimeError)
+    MismatchedTypes = Class.new(RuntimeError)
 
     class_attribute :auto_define
     self.auto_define = false
@@ -70,7 +73,23 @@ module Figleaf
 
         default = config["default"]
         property = config[env_to_load]
-        property = default.merge(property || {}) if !default.nil?
+
+        property =
+          if property.nil? && default.nil?
+            return nil
+          elsif property.nil?
+            default
+          elsif default.nil?
+            property
+          elsif default.class.name != property.class.name
+            raise MismatchedTypes, "Error loading file #{file}: mismatch between default values (type #{default.class.name}) and #{env_to_load} (type #{property.class.name})"
+          else
+            case default
+            when Hash then default.deep_merge(property)
+            when Array then default + property
+            else property
+            end
+          end
 
         [property_name, use_hashie_if_hash(property)]
       end

data/lib/figleaf/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Figleaf
-  VERSION = "0.3.1"
+  VERSION = "0.4.0"
 end

data/lib/figleaf.rb

@@ -1,5 +1,8 @@
+# frozen_string_literal: true
+
 require "active_support/concern"
 require "active_support/core_ext/class/attribute"
+require "active_support/core_ext/hash"
 require "active_support/core_ext/module/attribute_accessors"
 require "active_support/core_ext/object/blank"
 require "erb"
@@ -10,6 +13,7 @@ require "yaml"
 module Figleaf
   autoload :Config, "figleaf/config"
   autoload :Fighash, "figleaf/fighash"
+  autoload :LazyBlockHash, "figleaf/lazy_block_hash"
   autoload :Settings, "figleaf/settings"
   autoload :VERSION, "figleaf/version"
 end

data/spec/figleaf/config_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 module Figleaf

data/spec/figleaf/configuration_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Figleaf::Settings do

data/spec/figleaf/fighash_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Figleaf::Fighash do

data/spec/figleaf/settings_spec.rb

@@ -1,12 +1,21 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Figleaf::Settings do
+  # Force a freshly loaded Figleaf::Settings class on every test run
+  before do
+    Figleaf.send(:remove_const, :Settings) if Figleaf.const_defined?(:Settings)
+    load "lib/figleaf/settings.rb"
+  end
+
+  # Force a new reference to the class
+  subject(:described_class) { Figleaf::Settings }
+
   describe "self.load_settings" do
-    before do
-      @fixtures_path = File.expand_path("../../fixtures/*.yml", __FILE__)
+    let(:fixtures_path) { File.expand_path("../../fixtures/*.yml", __FILE__) }
 
-      described_class.load_settings(@fixtures_path, "test")
-    end
+    before { described_class.load_settings(fixtures_path, "test") }
 
     it "load some service" do
       expect(described_class.service["foo"]).to eq("bar")
@@ -42,7 +51,7 @@ describe Figleaf::Settings do
     end
 
     it "and for boolean (false)" do
-      described_class.load_settings(@fixtures_path, "alt")
+      described_class.load_settings(fixtures_path, "alt")
       expect(described_class.boolean).to eq(false)
     end
 
@@ -81,6 +90,29 @@ describe Figleaf::Settings do
       end
     end
 
+    context "incompatible types" do
+      context "bad overrides" do
+        subject(:settings) { described_class.bad_overrides }
+
+        let(:fixtures_path) { File.expand_path("../../fixtures/type_errors/bad_overrides.yml", __FILE__) }
+
+        it "raises an error when overloading a hash with an array" do
+          expect { described_class.load_settings(fixtures_path, "array") }
+            .to raise_error(described_class::MismatchedTypes)
+        end
+
+        it "raises an error when overloading a hash with a string" do
+          expect { described_class.load_settings(fixtures_path, "string") }
+            .to raise_error(described_class::MismatchedTypes)
+        end
+
+        it "raises an error when overloading a hash with a bool" do
+          expect { described_class.load_settings(fixtures_path, "bool") }
+            .to raise_error(described_class::MismatchedTypes)
+        end
+      end
+    end
+
     context "overloading settings" do
       before do
         overload = File.expand_path("../../fixtures/extra/*.yml", __FILE__)
@@ -134,22 +166,68 @@ describe Figleaf::Settings do
     end
   end
 
-  context "load yaml files" do
-    before do
-      fixtures_path = File.expand_path("../../fixtures/*.yaml", __FILE__)
-      described_class.load_settings(fixtures_path, "test")
+  shared_examples_for "merged hashes" do
+    context "when using the default" do
+      before { described_class.load_settings(fixtures_path, "some_env") }
+
+      it "test a non overridden environment" do
+        expect(merged).to match(
+          "leaf1" => "default1",
+          "root1" => {"leaf11" => "default2"},
+          "root2" => {
+            "leaf21" => "default3",
+            "root21" => {
+              "leaf211" => "default4",
+              "root211" => {"leaf2111" => "default5"}
+            }
+          }
+        )
+      end
+    end
+
+    context "with an overridden environment" do
+      before { described_class.load_settings(fixtures_path, "overridden") }
+
+      it "merges the values" do
+        expect(merged).to match(
+          "leaf1" => "default1",
+          "leaf2" => "overridden1",
+          "root1" => {"leaf11" => "overridden2"},
+          "root2" => {
+            "leaf21" => "default3",
+            "leaf22" => "overridden3",
+            "root21" => {
+              "leaf211" => "overridden4",
+              "root211" => {
+                "leaf2111" => "default5",
+                "leaf2112" => "overridden5"
+              }
+            }
+          }
+        )
+      end
     end
+  end
+
+  context "load yaml files" do
+    let(:fixtures_path) { File.expand_path("../../fixtures/**/*.yaml", __FILE__) }
+
+    before { described_class.load_settings(fixtures_path, "test") }
 
     it "reads them just fine" do
       expect(described_class.yaml.works).to eq("here")
     end
+
+    context "hashes are merged" do
+      subject(:merged) { described_class.merged_yaml }
+      it_behaves_like "merged hashes"
+    end
   end
 
   context "load ruby files" do
-    before do
-      fixtures_path = File.expand_path("../../fixtures/extra/*.rb", __FILE__)
-      described_class.load_settings(fixtures_path, "test")
-    end
+    let(:fixtures_path) { File.expand_path("../../fixtures/extra/*.rb", __FILE__) }
+
+    before { described_class.load_settings(fixtures_path, "test") }
 
     it "load indifferently the key names" do
       expect(described_class.code["foo"]).to eq("bar")
@@ -188,5 +266,10 @@ describe Figleaf::Settings do
     it "and for regexp values" do
       expect(described_class.regexp.value).to eq(/\Amatcher\z/)
     end
+
+    context "hashes are merged" do
+      subject(:merged) { described_class.merged_rb }
+      it_behaves_like "merged hashes"
+    end
   end
 end

data/spec/fixtures/extra/merged_rb.rb

@@ -0,0 +1,33 @@
+default do
+  leaf1 "default1"
+  root1(leaf11: "default2")
+  root2(
+    leaf21: "default3",
+    root21: {
+      leaf211: "default4",
+      root211: {
+        leaf2111: "default5"
+      }
+    }
+  )
+end
+
+unused do
+  root1(leaf11: "unused11")
+  root2(root21: {root21: {leaf211: "unused2"}})
+end
+
+overridden do
+  leaf2 "overridden1"
+  root1(leaf11: "overridden2")
+
+  root2(
+    leaf22: "overridden3",
+    root21: {
+      leaf211: "overridden4",
+      root211: {
+        leaf2112: "overridden5"
+      }
+    }
+  )
+end

data/spec/fixtures/extra/merged_yaml.yaml

@@ -0,0 +1,29 @@
+---
+default:
+  leaf1: default1
+  root1:
+    leaf11: default2
+  root2:
+    leaf21: default3
+    root21:
+      leaf211: default4
+      root211:
+        leaf2111: default5
+
+unused:
+  root1:
+    leaf11: unused1
+  root2:
+    root21:
+      leaf211: unused2
+
+overridden:
+  leaf2: overridden1
+  root1:
+    leaf11: overridden2
+  root2:
+    leaf22: overridden3
+    root21:
+      leaf211: overridden4
+      root211:
+        leaf2112: overridden5

data/spec/fixtures/type_errors/bad_overrides.yml

@@ -0,0 +1,11 @@
+---
+default:
+  foo: bar
+
+array:
+  - foo
+
+string: "HELLO"
+
+bool:
+  true

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: figleaf
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.4.0
 platform: ruby
 authors:
 - Juan C. Müller
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-04-24 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -47,6 +46,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".solargraph.yml"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
@@ -59,6 +59,7 @@ files:
 - lib/figleaf.rb
 - lib/figleaf/config.rb
 - lib/figleaf/fighash.rb
+- lib/figleaf/lazy_block_hash.rb
 - lib/figleaf/settings.rb
 - lib/figleaf/version.rb
 - spec/figleaf/config_spec.rb
@@ -75,17 +76,19 @@ files:
 - spec/fixtures/extra/code.rb
 - spec/fixtures/extra/default.yml
 - spec/fixtures/extra/default_anchor.yml
+- spec/fixtures/extra/merged_rb.rb
+- spec/fixtures/extra/merged_yaml.yaml
 - spec/fixtures/extra/regexp.rb
 - spec/fixtures/extra/service.yml
 - spec/fixtures/regexp.yml
 - spec/fixtures/service.yml
 - spec/fixtures/string.yml
+- spec/fixtures/type_errors/bad_overrides.yml
 - spec/fixtures/yaml.yaml
 - spec/spec_helper.rb
 homepage: http://github.com/jcmuller/figleaf
 licenses: []
 metadata: {}
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -100,8 +103,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.19
-signing_key:
+rubygems_version: 3.7.1
 specification_version: 4
 summary: YAML based DRY settings manager.
 test_files: []