drsi 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2e2a89f6f00a413fd3ae85e494e75e0f4bcc4aac
-  data.tar.gz: b44a051e6678c13077ae65a55e8c11ba36530128
+  metadata.gz: 6b99bdc02a78b72391449fe80e587f114dc2fca0
+  data.tar.gz: 707d75c55602ea782f5fdda5441c475f263de7bd
 SHA512:
-  metadata.gz: abd2d518e9b8952cfff3d150e323e0c440529aadf51c5dc2d2df388c8e93353d86bdd542faeaaeb276635ca503ce1cfa48b7bc8a43f1a8c94fa3facf18a7fb40
-  data.tar.gz: eae8418a3b0e7949b569cc73a510998d6a7a358ce4836d1548eb5c713b31ee0b79fca8c69cf51b7641c93bfa72c41e547b9edf2bc5393108e7faf01b487ec7aa
+  metadata.gz: 6a994734ded3c5039e348dc4fcd2daa40f2927b8a48606daf6bc8fa01d3c8dc2113030fb05faa82279a2391ec8f23df287d061afeb76e2bfa90f48e2fc6f43b5
+  data.tar.gz: 3831df641d260d23c583a1d79d93870cc81c8ef82c336b94703187bfe7693717da83376506e277ca4b889c943ac5eb26cf8c15cfb87b3d4cd812700ae2468bc1

data/README.md

@@ -69,9 +69,9 @@ Inside a context instance, every player object incorporates the behaviour (metho
 The Account instances above are players. They are accesible inside #run through #source_account and #target_account private methods.
 Also, every role player has private access to the rest of role players in the context.
 
-Unlike the Presenter approach in dci-ruby (where the object to play a role and the one inside the context playing it are associated but different), this extending/unextending approach preserves unique identity of objects playing roles.
+Unlike the Presenter approach in dci-ruby (where the object to play a role and the one inside the context playing it are associated but different), this injection approach preserves unique identity of objects playing roles.
 
-When instanciating a Context, the extra no-role pairs given as arguments are read-only attributes accessible via #settings:
+When instantiating a Context, the extra no-role pairs given as arguments are read-only attributes accessible via #settings:
 
     MoneyTransfer[:source_account => Account.new(1),
                   :target_account => Account.new(2),

data/lib/drsi/rolable.rb

@@ -8,11 +8,18 @@ require 'drsi/module'
 # reusing the empty ones or adding and extending new ones when it is needed.
 module Rolable
 
-  # Make an object play the role defined as a module in 'mod'
-  def __play_role!(role_klass, context)
+  # Make an object play (in the given 'context') the role defined as a module in 'role_mod':
+  #  - Get or extend a new empty role module,
+  #  - Copy role_mod instance_methods into it,
+  #  - Inject context and settings methods.
+  def __play_role!(role_mod, a_context)
     new_role = __next_empty_role
-    new_role.__copy_instance_methods_from(role_klass)
-    new_role.send(:define_method, :context) {context}
+    new_role.__copy_instance_methods_from(role_mod)
+    new_role.class_exec(a_context) do |the_context|
+      private
+      define_method(:context) {the_context}
+      define_method(:settings) {|*keys| context.send(:settings, *keys)}
+    end
   end
 
   # Make an object stop playing the last role it plays, if any.
@@ -55,12 +62,7 @@ module Rolable
     end
 
     # The context a role is played within. This method must be overidden in every __role definition module.
-    def context
-      nil
-    end
-
-    # The role definition code also have private access to the extra args given in the context instantiation.
-    def settings(*keys)
-      context.send(:settings, *keys) if context
-    end
+    # def context
+    #   nil
+    # end
 end

data/lib/drsi/version.rb

@@ -1,3 +1,3 @@
 module Drsi
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/spec/roleplayers_spec.rb

@@ -38,7 +38,7 @@ describe 'RolePlayers' do
         end
 
         def check_role1_context_access
-          role1.context == self
+          [!role1.respond_to?(:context), role1.private_methods.map(&:to_s).include?('context'), role1.send(:context) == self].uniq == [true]
         end
 
         def check_role1_settings_access
@@ -95,9 +95,8 @@ describe 'RolePlayers' do
       @player1.name.should eq('player1')
       @player2.should_not respond_to(:role1)
       @player2.should_not respond_to(:context)
-      @player1.should_not respond_to(:settings)
-      @player1.private_methods.map(&:to_s).should include('settings')
-      @player1.send(:settings).should be_nil
+      @player2.should_not respond_to(:settings)
+      (@player2.private_methods.map(&:to_s) & ['role1', 'context', 'settings']).should be_empty
     end
 
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: drsi
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Lorenzo Tello
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-12 00:00:00.000000000 Z
+date: 2014-01-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec