wizardwerdna-pluggable 0.0.0 → 0.1.0

data/VERSION

@@ -1 +1 @@
-0.0.0
+0.1.0

data/lib/pluggable.rb

@@ -4,23 +4,34 @@ $:.unshift(File.dirname(__FILE__)) unless
 require 'singleton'
 require 'forwardable'
 module Pluggable
-  VERSION = '0.0.1'
   
-  def plugins
-    Plugins.instance
+  class Plugins < Array
+    extend Forwardable
+    def self.from_array_of_instance_and_name_pairs array
+      result = new
+      array.each do |each_pair| 
+        result << each_pair[:instance]
+        result.instance_variable_set each_pair[:name], each_pair[:instance]
+      end
+      result
+    end
   end
   
-  class Plugins < Array
+  class PluginFactory < Array
     include Singleton
-    extend Forwardable
-
-    def delegate_public_methods_to_plugins_except *excluded_methods
+    def build_plugins
+      array_of_instance_and_name_pairs = map do |each| 
+        instance = each.new
+        {:name => variable_name_for_plugin_instance(instance), :instance => instance}
+      end
+      Plugins.from_array_of_instance_and_name_pairs(array_of_instance_and_name_pairs)
+    end
+    def delegate_plugin_public_methods_to_plugins_class_except *excluded_methods
       excluded_methods = excluded_methods.map{|each| each.to_s}
-      each do |instance|
+      build_plugins.each do |instance|
         delegated_methods = instance.public_methods-Plugin.public_instance_methods-excluded_methods
         variable_name = variable_name_for_plugin_instance instance
-        instance_variable_set variable_name, instance
-        self.class.def_delegators variable_name, *delegated_methods
+        Plugins.def_delegators variable_name, *delegated_methods
       end
     end
     private
@@ -31,7 +42,28 @@ module Pluggable
   
   class Plugin
     def self.inherited(klass)
-      Plugins.instance << klass.new
+      PluginFactory.instance << klass
   	end
   end
+  
+  def install_plugins
+    instance_variable_set :@pluggable_module_plugins, PluginFactory.instance.build_plugins
+  end
+
+  def plugins
+    instance_variable_get :@pluggable_module_plugins
+  end
+
+  def self.included(klass)
+    klass.extend ClassMethods
+  end
+  
+  module ClassMethods
+    def plugin_factory
+      PluginFactory.instance
+    end
+    def delegate_plugin_public_methods_except *excluded_methods
+      PluginFactory.instance.delegate_plugin_public_methods_to_plugins_class_except *excluded_methods
+    end
+  end
 end

data/pluggable.gemspec

@@ -0,0 +1,53 @@
+# Generated by jeweler
+# DO NOT EDIT THIS FILE
+# Instead, edit Jeweler::Tasks in Rakefile, and run `rake gemspec`
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = %q{pluggable}
+  s.version = "0.1.0"
+
+  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.authors = ["Andrew C. Greenberg"]
+  s.date = %q{2009-08-20}
+  s.description = %q{TODO: longer description of your gem}
+  s.email = %q{wizardwerdna@gmail.com}
+  s.extra_rdoc_files = [
+    "LICENSE",
+     "README.rdoc"
+  ]
+  s.files = [
+    ".document",
+     ".gitignore",
+     "LICENSE",
+     "README.rdoc",
+     "Rakefile",
+     "VERSION",
+     "lib/pluggable.rb",
+     "pluggable.gemspec",
+     "spec/pluggable_spec.rb",
+     "spec/spec_helper.rb"
+  ]
+  s.homepage = %q{http://github.com/wizardwerdna/pluggable}
+  s.rdoc_options = ["--charset=UTF-8"]
+  s.require_paths = ["lib"]
+  s.rubygems_version = %q{1.3.5}
+  s.summary = %q{TODO: one-line summary of your gem}
+  s.test_files = [
+    "spec/pluggable_spec.rb",
+     "spec/spec_helper.rb"
+  ]
+
+  if s.respond_to? :specification_version then
+    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
+      s.add_development_dependency(%q<rspec>, [">= 0"])
+    else
+      s.add_dependency(%q<rspec>, [">= 0"])
+    end
+  else
+    s.add_dependency(%q<rspec>, [">= 0"])
+  end
+end

data/spec/pluggable_spec.rb

@@ -2,47 +2,41 @@ require 'forwardable'
 require File.dirname(__FILE__) + '/spec_helper.rb'
 
 class Test
-  extend Forwardable
   include Pluggable
+  def process
+    plugins.map{|each| each.process}
+  end
+  private
   def method_missing symbol, *args
     plugins.send(symbol, *args)
   end
 end
 
 class Plugin1 < Test::Plugin
-  def foo
-    "foo"
-  end
-  def exception_public
-  end
-  private
-  def exception_private
-  end
+  def foo; "foo"; end
+  def process; foo; end
+  private 
+  def exception_private; end
 end
 
 class Plugin2 < Test::Plugin
-  def bar
-    "bar"
-  end
-  def exception_public
-  end
+  def bar; "bar"; end
+  def process; bar; end
   private
-  def exception_private
-  end
+  def exception_private; end
 end
 
 class Plugin3 < Test::Plugin
-  def baz
-    "baz"
-  end
-  def exception_public
-  end
+  def baz; "baz"; end
+  def process; baz; end
   private
-  def exception_private
-  end
+  def exception_private; end
+end
+
+if Test.respond_to? :delegate_plugin_public_methods_except
+  Test.delegate_plugin_public_methods_except :exception_public
 end
 
-Test.new.plugins.delegate_public_methods_to_plugins_except :exception_public
 
 describe Pluggable, "when included in a class" do
   
@@ -50,16 +44,55 @@ describe Pluggable, "when included in a class" do
     @test_instance = Test.new
   end
   
-  it "adds a plugins method to the class instance returning an instance of Plugins" do
+  it "should install a class method #plugin_factory, an array of all the plugin classes" do
+    Test.should respond_to(:plugin_factory)
+    Test.plugin_factory.should have(3).items
+    Test.plugin_factory.should include(Plugin1, Plugin2, Plugin3)
+  end
+  
+  it "should add a method #plugins that should be nil upon creation" do
     @test_instance.should respond_to(:plugins)
-    @test_instance.plugins.should be_a_kind_of(Test::Plugins)
+    @test_instance.plugins.should be_nil
+  end
+  
+  it "should add a method #install_plugins" do
+    Test.new.should respond_to(:install_plugins)
   end
   
-  it "should include instances of all the plugins" do
+  it "should install a class method #delegate_plugin_public_methods_except" do
+    Test.should respond_to(:delegate_plugin_public_methods_except)
+  end
+end
+
+describe Pluggable, "after installing plugins" do
+  before(:each) do
+    @test_instance = Test.new
+    @test_instance.install_plugins
+  end
+  
+  it "should have #plugins answer an object containing a collection of new plugin instances after #install_plugins" do
+    @test_instance.should_not be_nil
+    @test_instance.plugins.should_not be_nil
+    @test_instance.plugins.should be_an_instance_of(Test::Plugins)
     @test_instance.plugins.should have(3).items
     @test_instance.plugins.map{|each| each.class}.should include(Plugin1, Plugin2, Plugin3)
   end
   
+  it "should have #plugins answer an object containing a collection of new plugin instances after #install_plugins a second time" do
+    @test_instance.should_not be_nil
+    @test_instance.plugins.should_not be_nil
+    @test_instance.plugins.should be_an_instance_of(Test::Plugins)
+    @test_instance.plugins.should have(3).items
+    @test_instance.plugins.map{|each| each.class}.should include(Plugin1, Plugin2, Plugin3)
+  end
+end
+
+describe Pluggable, "after Test has delegated all but excepted plugin public methods" do
+  before(:each) do
+    @test_instance = Test.new
+    @test_instance.install_plugins
+  end
+
   it "should delegate public methods of plugins" do
     @test_instance.plugins.should respond_to(:foo)
     @test_instance.plugins.should respond_to(:bar)
@@ -80,10 +113,33 @@ describe Pluggable, "when included in a class" do
   end
 end
 
+describe Test::PluginFactory, "instances" do
+  before(:each) do
+    @instance = Test::PluginFactory.instance
+  end
+  
+  it "should be an array of all the plugins" do
+    @instance.should have(3).items
+    @instance.should include(Plugin1, Plugin2, Plugin3)
+  end
+  
+  it "should build a Plugins object with fresh instances of each plugin" do
+    @installed_plugins = @instance.build_plugins
+    @installed_plugins.should be_a_kind_of(Test::Plugins)
+    @installed_plugins.should have(3).items
+    @installed_plugins.map{|each| each.class}.should include(Plugin1, Plugin2, Plugin3)
+  end
+end
+
 describe Test, "when using message_missing to simulate delegation from the parent" do
   
   before(:each) do
     @test_instance = Test.new
+    @test_instance.install_plugins
+  end
+
+  it "should properly process using all the plugins" do
+    @test_instance.process.should == ["foo", "bar", "baz"]
   end
 
   it "should delegate public methods of plugins" do
@@ -91,4 +147,4 @@ describe Test, "when using message_missing to simulate delegation from the paren
     @test_instance.bar.should == "bar"
     @test_instance.baz.should == "baz"
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: wizardwerdna-pluggable
 version: !ruby/object:Gem::Version 
-  version: 0.0.0
+  version: 0.1.0
 platform: ruby
 authors: 
 - Andrew C. Greenberg
@@ -39,6 +39,7 @@ files:
 - Rakefile
 - VERSION
 - lib/pluggable.rb
+- pluggable.gemspec
 - spec/pluggable_spec.rb
 - spec/spec_helper.rb
 has_rdoc: false