collapsium 0.4.1 → 0.5.0

data/spec/support_path_components.rb

@@ -0,0 +1,85 @@
+require 'spec_helper'
+require_relative '../lib/collapsium/support/path_components'
+
+describe ::Collapsium::Support::PathComponents do
+  let(:tester) { Class.new { extend ::Collapsium::Support::PathComponents } }
+
+  context "#separator" do
+    it "defaults to DEFAULT_SEPARATOR" do
+      expect(tester.separator).to eql \
+        ::Collapsium::Support::PathComponents::DEFAULT_SEPARATOR
+    end
+
+    it "can be set" do
+      expect { tester.separator = ':' }.not_to raise_error
+      expect(tester.separator).to eql ':'
+
+      tester.separator = ::Collapsium::Support::PathComponents::DEFAULT_SEPARATOR
+    end
+  end
+
+  context "#path_prefix" do
+    it "defaults to an empty string" do
+      expect(tester.path_prefix).to eql '.' # == DEFAULT_SEPARATOR
+    end
+
+    it "can be set, but normalizes its value" do
+      expect { tester.path_prefix = 'foo' }.not_to raise_error
+      expect(tester.path_prefix).to eql '.foo' # == DEFAULT_SEPARATOR + 'foo'
+
+      tester.path_prefix = ''
+    end
+  end
+
+  context "#path_components" do
+    it "splits a path into components" do
+      expect(tester.path_components("foo.bar")).to eql %w(foo bar)
+    end
+
+    it "strips empty components at the beginning" do
+      expect(tester.path_components("..foo.bar")).to eql %w(foo bar)
+    end
+
+    it "strips empty components at the end" do
+      expect(tester.path_components("foo.bar..")).to eql %w(foo bar)
+    end
+
+    it "strips empty components in the middle" do
+      expect(tester.path_components("foo...bar")).to eql %w(foo bar)
+    end
+  end
+
+  context "#join_path" do
+    it "joins path components" do
+      expect(tester.join_path(%w(foo bar))).to eql "foo.bar"
+    end
+
+    it "joins empty components to an empty string" do
+      expect(tester.join_path([])).to eql ""
+    end
+  end
+
+  context "#normalize_path" do
+    it "normalizes a path" do
+      expect(tester.normalize_path("foo..bar..baz.")).to eql ".foo.bar.baz"
+    end
+
+    it "normalizes an array path" do
+      expect(tester.normalize_path(['foo', '', 'bar'])).to eql ".foo.bar"
+    end
+  end
+
+  context "#parent_path" do
+    it "returns the parent of a path" do
+      expect(tester.parent_path(".foo.bar")).to eql ".foo"
+    end
+
+    it "can deal with children of the root" do
+      expect(tester.parent_path(".foo")).to eql "."
+    end
+
+    it "can deal with empty paths" do
+      expect(tester.parent_path("")).to eql "."
+    end
+  end
+end

data/spec/viral_capabilities_spec.rb

@@ -20,6 +20,10 @@ class ExtendedHash < Hash
   extend TestModule
 end
 
+class IncludedArray < Array
+  include TestModule
+end
+
 class DirectPrependedHash < Hash
   prepend ::Collapsium::ViralCapabilities
 
@@ -67,7 +71,7 @@ module ViralityModule
   extend ::Collapsium::ViralCapabilities
   include ::Collapsium::Support::Methods
 
-  def virality(value)
+  def virality(value, *_)
     # Wrap :delete to become a no-op
     wrap_method(value, :delete) do
       next true
@@ -80,7 +84,7 @@ describe ::Collapsium::ViralCapabilities do
   context PrependedHash do
     let(:tester) do
       x = PrependedHash.new
-      x.merge!(foo: { bar: { baz: true } })
+      x.merge!(foo: { bar: [{ baz: true }] })
     end
 
     before do
@@ -89,14 +93,15 @@ describe ::Collapsium::ViralCapabilities do
 
     it "replicates itself" do
       expect(tester[:foo].respond_to?(:find_me)).to be_truthy
-      expect(tester[:foo][:bar].respond_to?(:find_me)).to be_truthy
+      expect(tester[:foo][:bar][0].respond_to?(:find_me)).to be_truthy
+      expect(tester[:foo][:bar][0].class).to eql PrependedHash
     end
   end
 
   context IncludedHash do
     let(:tester) do
       x = IncludedHash.new
-      x.merge!(foo: { bar: { baz: true } })
+      x.merge!(foo: { bar: [{ baz: true }] })
     end
 
     before do
@@ -105,14 +110,32 @@ describe ::Collapsium::ViralCapabilities do
 
     it "replicates itself" do
       expect(tester[:foo].respond_to?(:find_me)).to be_truthy
-      expect(tester[:foo][:bar].respond_to?(:find_me)).to be_truthy
+      expect(tester[:foo][:bar][0].respond_to?(:find_me)).to be_truthy
+      expect(tester[:foo][:bar][0].class).to eql IncludedHash
+    end
+  end
+
+  context IncludedArray do
+    let(:tester) do
+      x = IncludedArray.new
+      x << { foo: { bar: [{ baz: true }, 42] } }
+    end
+
+    before do
+      expect(tester.respond_to?(:find_me)).to be_truthy
+    end
+
+    it "replicates itself" do
+      expect(tester[0][:foo].respond_to?(:find_me)).to be_truthy
+      expect(tester[0][:foo][:bar][0].respond_to?(:find_me)).to be_truthy
+      expect(tester[0][:foo][:bar].class).to eql IncludedArray
     end
   end
 
   context ExtendedHash do
     let(:tester) do
       x = ExtendedHash.new
-      x.merge!(foo: { bar: { baz: true } })
+      x.merge!(foo: { bar: [{ baz: true }] })
     end
 
     it "does not receive viral capabilities" do
@@ -123,7 +146,7 @@ describe ::Collapsium::ViralCapabilities do
   context DirectPrependedHash do
     let(:tester) do
       x = DirectPrependedHash.new
-      x.merge!(foo: { bar: { baz: true } })
+      x.merge!(foo: { bar: [{ baz: true }] })
     end
 
     before do
@@ -138,7 +161,7 @@ describe ::Collapsium::ViralCapabilities do
   context DirectIncludedHash do
     let(:tester) do
       x = DirectIncludedHash.new
-      x.merge!(foo: { bar: { baz: true } })
+      x.merge!(foo: { bar: [{ baz: true }] })
     end
 
     before do
@@ -147,14 +170,14 @@ describe ::Collapsium::ViralCapabilities do
 
     it "replicates itself" do
       expect(tester[:foo].respond_to?(:find_me)).to be_truthy
-      expect(tester[:foo][:bar].respond_to?(:find_me)).to be_truthy
+      expect(tester[:foo][:bar][0].respond_to?(:find_me)).to be_truthy
     end
   end
 
   context DirectExtendedHash do
     let(:tester) do
       x = DirectExtendedHash.new
-      x.merge!(foo: { bar: { baz: true } })
+      x.merge!(foo: { bar: [{ baz: true }] })
     end
 
     before do
@@ -163,7 +186,7 @@ describe ::Collapsium::ViralCapabilities do
 
     it "replicates itself" do
       expect(tester[:foo].respond_to?(:find_me)).to be_truthy
-      expect(tester[:foo][:bar].respond_to?(:find_me)).to be_truthy
+      expect(tester[:foo][:bar][0].respond_to?(:find_me)).to be_truthy
     end
   end
 
@@ -186,7 +209,7 @@ describe ::Collapsium::ViralCapabilities do
     let(:tester) do
       x = {}
       x.extend(TestModule)
-      x.merge!(foo: { bar: { baz: true } })
+      x.merge!(foo: { bar: [{ baz: { quux: true } }] })
     end
 
     before do
@@ -195,7 +218,7 @@ describe ::Collapsium::ViralCapabilities do
 
     it "replicates itself" do
       expect(tester[:foo].respond_to?(:find_me)).to be_truthy
-      expect(tester[:foo][:bar].respond_to?(:find_me)).to be_truthy
+      expect(tester[:foo][:bar][0][:baz].respond_to?(:find_me)).to be_truthy
     end
   end
 
@@ -203,7 +226,7 @@ describe ::Collapsium::ViralCapabilities do
     let(:tester) do
       x = {}
       x.extend(ViralityModule)
-      x.merge!(foo: { bar: { baz: true } })
+      x.merge!(foo: { bar: [{ baz: true }] })
     end
 
     before do
@@ -214,7 +237,7 @@ describe ::Collapsium::ViralCapabilities do
       # Just check that the module went viral; it doesn't check the viral
       # enhancements work.
       expect(tester[:foo].respond_to?(:virality)).to be_truthy
-      expect(tester[:foo][:bar].respond_to?(:virality)).to be_truthy
+      expect(tester[:foo][:bar][0].respond_to?(:virality)).to be_truthy
 
       # Now also check the viral enhancements. In this case, :delete becomes a
       # no-op.
@@ -226,19 +249,23 @@ describe ::Collapsium::ViralCapabilities do
   end
 
   context "values already of the same class" do
+    # rubocop:disable Performance/RedundantMerge
     let(:innermost) do
       innermost = PrependedHash.new
       innermost.merge!(quux: true)
+      innermost
     end
 
     let(:inner) do
       inner = {}
       inner.merge!(foo: true, bar: innermost)
+      inner
     end
 
     let(:tester) do
       PrependedHash.new
     end
+    # rubocop:enable Performance/RedundantMerge
 
     it "peforms a no-op when adding the right class" do
       tester[:bar] = inner

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: collapsium
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.5.0
 platform: ruby
 authors:
 - Jens Finkhaeuser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-18 00:00:00.000000000 Z
+date: 2016-10-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -30,28 +30,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.42'
+        version: '0.44'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.42'
+        version: '0.44'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.2'
+        version: '11.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '11.2'
+        version: '11.3'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -121,8 +121,10 @@ files:
 - lib/collapsium/recursive_dup.rb
 - lib/collapsium/recursive_merge.rb
 - lib/collapsium/recursive_sort.rb
+- lib/collapsium/support/array_methods.rb
 - lib/collapsium/support/hash_methods.rb
 - lib/collapsium/support/methods.rb
+- lib/collapsium/support/path_components.rb
 - lib/collapsium/uber_hash.rb
 - lib/collapsium/version.rb
 - lib/collapsium/viral_capabilities.rb
@@ -135,6 +137,7 @@ files:
 - spec/recursive_sort_spec.rb
 - spec/spec_helper.rb
 - spec/support_methods_spec.rb
+- spec/support_path_components.rb
 - spec/uber_hash_spec.rb
 - spec/viral_capabilities_spec.rb
 homepage: https://github.com/jfinkhaeuser/collapsium
@@ -171,5 +174,6 @@ test_files:
 - spec/recursive_sort_spec.rb
 - spec/spec_helper.rb
 - spec/support_methods_spec.rb
+- spec/support_path_components.rb
 - spec/uber_hash_spec.rb
 - spec/viral_capabilities_spec.rb