poise 2.2.3 → 2.3.0

data/test/spec/resource_spec.rb

@@ -73,7 +73,7 @@ describe Poise::Resource do
     end # /context with an optional parent
   end # /describe #poise_subresource
 
-  context 'with fused' do
+  describe '#poise_fused' do
     resource(:poise_test) do
       include described_class
       poise_fused
@@ -81,5 +81,34 @@ describe Poise::Resource do
 
     it { is_expected.to include Poise::Resource }
     it { is_expected.to include Poise::Helpers::Fused }
-  end # /context with fused
+  end # /describe #poise_fused
+
+  describe '#poise_inversion' do
+    context 'with an options resource' do
+      resource(:poise_test) do
+        include described_class
+        poise_inversion
+        provides(:poise_test)
+        Halite::SpecHelper::Patcher.post_create_cleanup(:poise_test_options, inversion_options_resource_class)
+        Halite::SpecHelper::Patcher.post_create_cleanup(:poise_test_options, inversion_options_provider_class)
+      end
+      it { is_expected.to include Poise::Resource }
+      it { is_expected.to include Poise::Helpers::Inversion }
+      its(:inversion_options_resource_class) { is_expected.to be_truthy }
+      its(:inversion_options_provider_class) { is_expected.to be_truthy }
+    end # /context with an options resource
+
+    context 'without an options resource' do
+      resource(:poise_test) do
+        include described_class
+        poise_inversion(false)
+        provides(:poise_test)
+      end
+      it { is_expected.to include Poise::Resource }
+      it { is_expected.to include Poise::Helpers::Inversion }
+      its(:inversion_options_resource_class) { is_expected.to be nil }
+      its(:inversion_options_provider_class) { is_expected.to be nil }
+      it { expect { Chef::Resource.resource_for_node(:poise_test_options, chef_run.node) }.to raise_error Chef::Exceptions::NoSuchResourceType }
+    end # /context without an options resource
+  end # /describe #poise_inversion
 end

data/test/spec/subcontext/resource_collection_spec.rb

@@ -0,0 +1,99 @@
+#
+# Copyright 2015, Noah Kantrowitz
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'spec_helper'
+
+describe Poise::Subcontext::ResourceCollection do
+  let(:top) { Chef::ResourceCollection.new }
+  let(:sub) { described_class.new(top) }
+  let(:inner) { described_class.new(sub) }
+  let(:sibling) { described_class.new(top) }
+  subject(:subject_context) { sub }
+
+  # Helper for use in #before.
+  def res(name)
+    Chef::Resource::RubyBlock.new(name, nil)
+  end
+
+  # Populate the various collections with test data.
+  before do
+    top << res('top1')
+    top << res('top2')
+    sub << res('sub1')
+    sub << res('sub2')
+    inner << res('inner1')
+    inner << res('inner2')
+    sibling << res('sibling1')
+    sibling << res('sibling2')
+  end
+
+  describe '#lookup' do
+    let(:name) { '' }
+    subject { subject_context.lookup("ruby_block[#{name}]") }
+
+    context 'with a resource in the subcontext' do
+      let(:name) { 'sub1' }
+      it { is_expected.to be_a Chef::Resource }
+    end # /context with a resource in the subcontext
+
+    context 'with a resource in a parent context' do
+      let(:name) { 'top1' }
+      it { is_expected.to be_a Chef::Resource }
+    end # /context with a resource in a parent context
+
+    context 'with a resource in a sibling context' do
+      let(:name) { 'sibling1' }
+      it { expect { subject }.to raise_error Chef::Exceptions::ResourceNotFound }
+    end # /context with a resource in a sibling context
+
+    context 'with a resource in a nested context' do
+      let(:name) { 'inner1' }
+      it { expect { subject }.to raise_error Chef::Exceptions::ResourceNotFound }
+    end # /context with a resource in a nested context
+  end # /describe #lookup
+
+  describe '#recursive_each' do
+    subject do
+      [].tap do |ary|
+        subject_context.recursive_each do |res|
+          ary << res.name
+        end
+      end
+    end
+    it { is_expected.to eq %w{top1 top2 sub1 sub2} }
+
+    context 'from a deeply nested context' do
+      let(:subject_context) { inner }
+      it { is_expected.to eq %w{top1 top2 sub1 sub2 inner1 inner2} }
+    end # /context from a deeply nested context
+  end # /describe #recursive_each
+
+  describe '#reverse_recursive_each' do
+    subject do
+      [].tap do |ary|
+        subject_context.reverse_recursive_each do |res|
+          ary << res.name
+        end
+      end
+    end
+    it { is_expected.to eq %w{sub2 sub1 top2 top1} }
+
+    context 'from a deeply nested context' do
+      let(:subject_context) { inner }
+      it { is_expected.to eq %w{inner2 inner1 sub2 sub1 top2 top1} }
+    end # /context from a deeply nested context
+  end # /describe #reverse_recursive_each
+end

data/test/spec/utils/resource_provider_mixin_spec.rb

@@ -59,6 +59,16 @@ module ResourceProviderMixinTest
     include Test
     include Test2
   end
+
+  module Mixins
+    module Resource
+      include Test
+    end
+
+    module Provider
+      include Test
+    end
+  end
 end
 
 describe Poise::Utils::ResourceProviderMixin do
@@ -85,4 +95,16 @@ describe Poise::Utils::ResourceProviderMixin do
       it { is_expected.to be < ResourceProviderMixinTest::Test2::Provider }
     end
   end # /context with nested usage
+
+  context 'with a mixin' do
+    context 'in a resource' do
+      subject { ResourceProviderMixinTest::Mixins::Resource }
+      it { is_expected.to include ResourceProviderMixinTest::Test::Resource }
+    end
+
+    context 'in a provider' do
+      subject { ResourceProviderMixinTest::Mixins::Provider }
+      it { is_expected.to include ResourceProviderMixinTest::Test::Provider }
+    end
+  end # /context with a mixin
 end

data/test/spec/utils_spec.rb

@@ -17,10 +17,52 @@
 require 'spec_helper'
 require 'chef/cookbook_version'
 
+module PoiseUtilsSpecTopLevel
+  module ClassMethods
+    def something(name=nil)
+      @something = name if name
+      @something
+    end
+
+    def included(klass)
+      super
+      klass.extend(ClassMethods)
+    end
+  end
+
+  extend ClassMethods
+end
+
+Poise::Utils.parameterized_module(PoiseUtilsSpecTopLevel) do |name|
+  something(name)
+end
+
+module PoiseUtilsSpecNested
+  module Inner
+    module ClassMethods
+      def something(name=nil)
+        @something = name if name
+        @something
+      end
+
+      def included(klass)
+        super
+        klass.extend(ClassMethods)
+      end
+    end
+
+    extend ClassMethods
+  end
+end
+
+Poise::Utils.parameterized_module(PoiseUtilsSpecNested::Inner) do |name|
+  something(name)
+end
+
 describe Poise::Utils do
   describe '.find_cookbook_name' do
     let(:cookbooks) { [] }
-    let(:run_context) { instance_double('Chef::RunContext', cookbook_collection: cookbooks.inject({}) {|memo, ver| memo[ver.name] = ver; memo })}
+    let(:run_context) { instance_double('Chef::RunContext', cookbook_collection: cookbooks.inject({}) {|memo, ver| memo[ver.name] = ver; memo }, node: {})}
     let(:filename) { '/test/my_cookbook/libraries/default.rb' }
     subject { described_class.find_cookbook_name(run_context, filename) }
 
@@ -194,5 +236,149 @@ describe Poise::Utils do
 
       its(:poise_test_val) { is_expected.to be_nil }
     end # /context with no value set
+
+    context 'with a branching ancestor tree' do
+      let(:mod_child1) do
+        parent = mod_parent
+        Module.new do
+          include parent
+          poise_test_val(:child1)
+        end
+      end
+      let(:mod_child2) do
+        parent = mod_parent
+        Module.new do
+          include parent
+        end
+      end
+      subject do
+        mod1 = mod_child1
+        mod2 = mod_child2
+        Class.new do
+          include mod1
+          include mod2
+        end
+      end
+
+      its(:poise_test_val) { is_expected.to eq :child1 }
+
+      context 'with ignorable values' do
+        let(:mod_parent) do
+          Module.new do
+            class_methods = Module.new do
+              def poise_test_val(val=nil)
+                if val
+                  @poise_test_val = val
+                end
+                @poise_test_val || Poise::Utils.ancestor_send(self, :poise_test_val, ignore: [true])
+              end
+
+              define_method(:included) do |klass|
+                super(klass)
+                klass.extend(class_methods)
+              end
+            end
+
+            extend class_methods
+          end
+        end
+        let(:mod_child2) do
+          parent = mod_parent
+          Module.new do
+            include parent
+            poise_test_val(true)
+          end
+        end
+
+        its(:poise_test_val) { is_expected.to eq :child1 }
+      end # /context with ignorable values
+    end # /context with a branching ancestor tree
   end # /describe .ancestor_send
+
+  describe '.parameterized_module' do
+    context 'with a top-level module' do
+      subject do
+        Module.new do
+          include PoiseUtilsSpecTopLevel('top')
+        end
+      end
+
+      its(:something) { is_expected.to eq 'top' }
+      it { expect(PoiseUtilsSpecTopLevel('top').name).to eq 'PoiseUtilsSpecTopLevel' }
+    end # /context with a top-level module
+
+    context 'with a nested module' do
+      subject do
+        Module.new do
+          include PoiseUtilsSpecNested::Inner('inner')
+        end
+      end
+
+      its(:something) { is_expected.to eq 'inner' }
+      it { expect(PoiseUtilsSpecNested::Inner('inner').name).to eq 'PoiseUtilsSpecNested::Inner' }
+    end # /context with a nested module
+
+    context 'with an anonymous module' do
+      subject { Poise::Utils.parameterized_module(Module.new) }
+      it { expect { subject }.to raise_error Poise::Error }
+    end # /context with an anonymous module
+
+    context 'with the wrong arugments' do
+      subject { PoiseUtilsSpecTopLevel('top', 'other') }
+      it { expect { subject }.to raise_error ArgumentError }
+    end # /context with the wrong arugments
+  end # /describe .parameterized_module
+
+  describe '.check_block_arity!' do
+    let(:block) { }
+    let(:args) { }
+    subject { Poise::Utils.send(:check_block_arity!, block, args) }
+
+    context 'with a positive arity' do
+      let(:block) do
+        proc {|a, b| nil }
+      end
+
+      context 'with 0 arguments' do
+        let(:args) { [] }
+        it { expect { subject }.to raise_error ArgumentError, /wrong number of arguments \(0 for 2\)/ }
+      end # /context with 0 arguments
+
+      context 'with 1 argument' do
+        let(:args) { [1] }
+        it { expect { subject }.to raise_error ArgumentError, /wrong number of arguments \(1 for 2\)/ }
+      end # /context with 1 argument
+
+      context 'with 2 arguments' do
+        let(:args) { [1, 2] }
+        it { expect { subject }.to_not raise_error }
+      end # /context with 2 arguments
+
+      context 'with 3 arguments' do
+        let(:args) { [1, 2, 3] }
+        it { expect { subject }.to raise_error ArgumentError, /wrong number of arguments \(3 for 2\)/ }
+      end # /context with 3 arguments
+    end # /context with a positive arity
+
+    context 'with a negative arity' do
+      let(:block) do
+        proc {|a, *b| nil }
+      end
+
+      context 'with 0 arguments' do
+        let(:args) { [] }
+        it { expect { subject }.to raise_error ArgumentError, /wrong number of arguments \(0 for 1\+\)/ }
+      end # /context with 0 arguments
+
+      context 'with 1 argument' do
+        let(:args) { [1] }
+        it { expect { subject }.to_not raise_error }
+      end # /context with 1 argument
+
+      context 'with 2 arguments' do
+        let(:args) { [1, 2] }
+        it { expect { subject }.to_not raise_error }
+      end # /context with 2 arguments
+    end # /context with a negative arity
+  end # /describe .check_block_arity!
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: poise
 version: !ruby/object:Gem::Version
-  version: 2.2.3
+  version: 2.3.0
 platform: ruby
 authors:
 - Noah Kantrowitz
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-21 00:00:00.000000000 Z
+date: 2015-08-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: halite
@@ -52,13 +52,14 @@ files:
 - ".travis.yml"
 - ".yardopts"
 - Berksfile
-- Berksfile.lock
 - CHANGELOG.md
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
 - lib/poise.rb
+- lib/poise/backports.rb
+- lib/poise/backports/not_passed.rb
 - lib/poise/error.rb
 - lib/poise/helpers.rb
 - lib/poise/helpers/chefspec_matchers.rb
@@ -74,6 +75,7 @@ files:
 - lib/poise/helpers/option_collector.rb
 - lib/poise/helpers/resource_cloning.rb
 - lib/poise/helpers/resource_name.rb
+- lib/poise/helpers/resource_subclass.rb
 - lib/poise/helpers/subcontext_block.rb
 - lib/poise/helpers/subresources.rb
 - lib/poise/helpers/subresources/child.rb
@@ -106,6 +108,7 @@ files:
 - test/gemfiles/master.gemfile
 - test/integration/default/serverspec/default_spec.rb
 - test/integration/default/serverspec/inversion_spec.rb
+- test/spec/backports/not_passed_spec.rb
 - test/spec/helpers/chefspec_matchers_spec.rb
 - test/spec/helpers/defined_in_spec.rb
 - test/spec/helpers/fused_spec.rb
@@ -118,6 +121,7 @@ files:
 - test/spec/helpers/option_collector_spec.rb
 - test/spec/helpers/resource_cloning_spec.rb
 - test/spec/helpers/resource_name_spec.rb
+- test/spec/helpers/resource_subclass_spec.rb
 - test/spec/helpers/subcontext_block_spec.rb
 - test/spec/helpers/subresources/child_spec.rb
 - test/spec/helpers/subresources/container_spec.rb
@@ -126,6 +130,7 @@ files:
 - test/spec/provider_spec.rb
 - test/spec/resource_spec.rb
 - test/spec/spec_helper.rb
+- test/spec/subcontext/resource_collection_spec.rb
 - test/spec/utils/resource_provider_mixin_spec.rb
 - test/spec/utils_spec.rb
 homepage: https://github.com/poise/poise
@@ -170,6 +175,7 @@ test_files:
 - test/gemfiles/master.gemfile
 - test/integration/default/serverspec/default_spec.rb
 - test/integration/default/serverspec/inversion_spec.rb
+- test/spec/backports/not_passed_spec.rb
 - test/spec/helpers/chefspec_matchers_spec.rb
 - test/spec/helpers/defined_in_spec.rb
 - test/spec/helpers/fused_spec.rb
@@ -182,6 +188,7 @@ test_files:
 - test/spec/helpers/option_collector_spec.rb
 - test/spec/helpers/resource_cloning_spec.rb
 - test/spec/helpers/resource_name_spec.rb
+- test/spec/helpers/resource_subclass_spec.rb
 - test/spec/helpers/subcontext_block_spec.rb
 - test/spec/helpers/subresources/child_spec.rb
 - test/spec/helpers/subresources/container_spec.rb
@@ -190,6 +197,7 @@ test_files:
 - test/spec/provider_spec.rb
 - test/spec/resource_spec.rb
 - test/spec/spec_helper.rb
+- test/spec/subcontext/resource_collection_spec.rb
 - test/spec/utils/resource_provider_mixin_spec.rb
 - test/spec/utils_spec.rb
 has_rdoc: