macros4cuke 0.3.14 → 0.3.15

data/.rubocop.yml

@@ -1,3 +1,4 @@
+# This is disabled because some demos use UTF-8
 AsciiComments:
   Enabled: false
 
@@ -9,7 +10,13 @@ DefWithParentheses:
   
 EmptyLines:
   Enabled: false
+  
+# Avoid methods longer than 40 lines of code
+MethodLength:
+  Max: 40
 
+SpaceInsideBrackets:
+  Enabled: false
   
 StringLiterals:
   Enabled: false

data/CHANGELOG.md

@@ -1,27 +1,31 @@
-## 0.3.14 / 2013-05-14
-* [FIX] lib/ folder in secondary local repository was messed up. Gem was OK.
+## 0.3.15 / 2013-05-20
+* [FIX] File `.CHANGELOG.md` Date entries weren't incremented.
+* [CHANGE] File `.rubocop.yml`: Added more configuration entries.
+* [CHANGE] Spec files updated to better please Rubocop.
 
+## 0.3.14 / 2013-05-18
+* [FIX] lib/ folder in secondary local repository was messed up. Gem was OK.
 
-## 0.3.13 / 2013-05-14
+## 0.3.13 / 2013-05-18
 * [NEW] File `.rubocop.yml` Added.
 * [CHANGE] Many source files changed to please Rubocop.
 
-## 0.3.12 / 2013-05-14
+## 0.3.12 / 2013-05-17
 * [NEW] File `.ruby-gemset` Added (for RVM users).
 * [NEW] File `.ruby-version` Added (for RVM users).
 
-## 0.3.11 / 2013-05-14
+## 0.3.11 / 2013-05-16
 * [CHANGE] File `README.md`: Minor reformating.
 * [CHANGE] File `basic.feature` (in examples/): Added one more macro-step example.
 
-## 0.3.10 / 2013-05-14
+## 0.3.10 / 2013-05-15
 * [CHANGE] File `README.md`: Expanded section on conditional section.
 * [CHANGE] Method `Templating::Engine::parse` slightly refactored in order to decrease its complexity.
 
-## 0.3.09 / 2013-05-13
+## 0.3.09 / 2013-05-14
 * [CHANGE] File `.travis.yml`: Added jruby as a target Ruby
 
-## 0.3.08 / 2013-05-12
+## 0.3.08 / 2013-05-13
 * [CHANGE] File `README.md`: Added Gem Version badge
 
 ## 0.3.07 / 2013-05-12

data/lib/macros4cuke/constants.rb

@@ -4,7 +4,7 @@
 
 module Macros4Cuke # Module used as a namespace
   # The version number of the gem.
-  Version = '0.3.14'
+  Version = '0.3.15'
 
   # Brief description of the gem.
   Description = "Macros for Cucumber"

data/lib/macros4cuke/templating/engine.rb

@@ -155,7 +155,7 @@ public
   # @return [String] The text value assigned to the placeholder. 
   #   Returns an empty string when no value is assigned to the placeholder.
   def render(aContextObject, theLocals)
-    raise NotImplementedError, "Method Section::#{__method__} must be implemented in subclass(es)."
+    raise NotImplementedError, "Method Section.#{__method__} must be implemented in subclass."
   end    
 
 end # class

data/spec/macros4cuke/macro-collection_spec.rb

@@ -2,9 +2,11 @@
 # File: macro-collection_spec.rb
 
 require_relative '../spec_helper'
-require_relative '../../lib/macros4cuke/macro-collection'	# Load the class under test
 
-module Macros4Cuke # Open this namespace to get rid of module qualifier prefixes
+# Load the class under test
+require_relative '../../lib/macros4cuke/macro-collection'
+
+module Macros4Cuke # Open this namespace to avoid module qualifier prefixes
 
 describe MacroCollection do
 
@@ -32,12 +34,14 @@ SNIPPET
 
     it "should accept the addition of a new macro-step" do
       phrase = "[enter my credentials]"
-      lambda { singleton.add_macro(phrase, sample_substeps, true)}.should_not raise_error
+      args = [phrase, sample_substeps, true]
+      ->() { singleton.add_macro(*args)}.should_not raise_error
       singleton.should have(1).macro_steps
 
       # Error case: inserting another macro with same phrase.
-      error_message = "A macro-step with phrase '[enter my credentials]' already exist."
-      lambda { singleton.add_macro(phrase, sample_substeps, true) }.should raise_error(Macros4Cuke::DuplicateMacroError, error_message)
+      msg = "A macro-step with phrase '[enter my credentials]' already exist."
+      ->(){ singleton.add_macro(*args) }.should 
+        raise_error(Macros4Cuke::DuplicateMacroError, msg)
     end
   end
 

data/spec/macros4cuke/macro-step-support_spec.rb

@@ -1,7 +1,10 @@
+# encoding: utf-8
 # File: macro-step_spec.rb
 
 require_relative '../spec_helper'
-require_relative '../../lib/macros4cuke/macro-step-support'	# The class under test
+
+# The class under test
+require_relative '../../lib/macros4cuke/macro-step-support'	
 
 
 module Macros4Cuke	# Open the module to avoid lengthy qualified names
@@ -41,29 +44,33 @@ SNIPPET
   
   context "Defining macro(s):" do
     it "should add valid new macro" do
-      lambda { world.add_macro(m1_phrase, m1_substeps, true) }.should_not raise_error
+      ->(){ world.add_macro(m1_phrase, m1_substeps, true) }.should_not raise_error
     end
     
     it "should complain when entering the same macro again" do
       # Error case: trying to register another macro with same key/phrase.
-      error_message = "A macro-step with phrase 'enter the credentials' already exist."
-      lambda { world.add_macro(m1_phrase, m1_substeps, true) }.should raise_error(Macros4Cuke::DuplicateMacroError, error_message)
+      msg = "A macro-step with phrase 'enter the credentials' already exist."
+      -> { world.add_macro(m1_phrase, m1_substeps, true) }.should 
+        raise_error(Macros4Cuke::DuplicateMacroError, msg)
     end
     
     it "should complain macro uses no table and phrase is parameterless" do
-      # Error case: substeps have arguments, but the macro has no mechanism to pass the needed data.
-      error_message = "The sub-step argument 'userid' does not appear in the phrase."
-      lambda { world.add_macro("fill in the credentials", m1_substeps, false) }.should raise_error(Macros4Cuke::UnreachableSubstepArgument, error_message)    
+      # Error case: substeps have arguments, 
+      # but the macro has no mechanism to pass the needed data.
+      phrase = "fill in the credentials"
+      msg = "The sub-step argument 'userid' does not appear in the phrase."
+      ->(){ world.add_macro(phrase, m1_substeps, false) }.should 
+        raise_error(Macros4Cuke::UnreachableSubstepArgument, msg)    
     end
   end # context
   
   context "Invoking macro(s):" do
   
     it "should complain when invoking an unknown macro-step" do
-      phrase_unknown = "dream of a perfect world"
-      
-      error_message = "Unknown macro-step with phrase: 'dream of a perfect world'."
-      lambda { world.invoke_macro(phrase_unknown) }.should raise_error( Macros4Cuke::UnknownMacroError, error_message)
+      phrase_unknown = "dream of a perfect world" 
+      msg = "Unknown macro-step with phrase: 'dream of a perfect world'."
+      ->(){ world.invoke_macro(phrase_unknown) }.should 
+        raise_error(Macros4Cuke::UnknownMacroError, msg)
     end
 
   end # context  
@@ -71,10 +78,10 @@ SNIPPET
   context "Clearing macro(s):" do
   
     it "should clear all macros" do
-      lambda { world.clear_macros() }.should_not raise_error
+      ->(){ world.clear_macros() }.should_not raise_error
       
       # Control the post-condition
-      MacroCollection::instance.macro_steps.should be_empty
+      MacroCollection.instance.macro_steps.should be_empty
     end  
 
   end # context

data/spec/macros4cuke/macro-step_spec.rb

@@ -1,3 +1,4 @@
+# encoding: utf-8
 # File: macro-step_spec.rb
 
 
@@ -29,20 +30,22 @@ end
 
   context "Creation & initialization" do
     it "should be created with a phrase, substeps and a table use indicator" do
-      lambda { MacroStep.new(sample_phrase, sample_template, true) }.should_not raise_error
+      ->(){ MacroStep.new(sample_phrase, sample_template, true) }.should_not raise_error
     end
 
 
     it "should complain when a sub-step argument can never be assigned a value via the phrase" do
-      error_message = "The sub-step argument 'password' does not appear in the phrase."
-      lambda { MacroStep.new(sample_phrase, sample_template, false) }.should raise_error(Macros4Cuke::UnreachableSubstepArgument, error_message)
+      msg = "The sub-step argument 'password' does not appear in the phrase."
+      ->(){ MacroStep.new(sample_phrase, sample_template, false) }.should 
+        raise_error(Macros4Cuke::UnreachableSubstepArgument, msg)
     end
 
 
-    it "should complain when an argument from the phrase never occurs in a substep" do
-      a_phrase = "enter my credentials as <foobar>"
-      error_message = "The phrase argument 'foobar' does not appear in a sub-step."
-      lambda { MacroStep.new(a_phrase, sample_template, true) }.should raise_error(Macros4Cuke::UselessPhraseArgument, error_message)
+    it "should complain when an argument in phrase never occurs in substeps" do
+      phrase = "enter my credentials as <foobar>"
+      msg = "The phrase argument 'foobar' does not appear in a sub-step."
+      ->(){ MacroStep.new(phrase, sample_template, true) }.should 
+       raise_error(Macros4Cuke::UselessPhraseArgument, msg)
     end
 
 
@@ -63,10 +66,10 @@ end
 
   context "Provided services" do
 
-    let(:phrase_instance) {%Q|enter my credentials as "nobody"|}
+    let(:phrase_instance) { %Q|enter my credentials as "nobody"| }
 
     it "should render the substeps" do
-      text = subject.expand(phrase_instance, [ ['password', 'no-secret'] ])
+      text = subject.expand(phrase_instance, [ %w(password no-secret) ])
       expectation = <<-SNIPPET
   Given I landed in the homepage
   When I click "Sign in"
@@ -94,7 +97,7 @@ SNIPPET
     
     it "should un-escape the double-quotes for phrase arguments" do
       specific_phrase = %q|enter my credentials as "quotable\""|
-      text = subject.expand(specific_phrase, [ ['password', 'no-secret'] ])
+      text = subject.expand(specific_phrase, [ %w(password no-secret) ])
       expectation = <<-SNIPPET
   Given I landed in the homepage
   When I click "Sign in"
@@ -110,12 +113,13 @@ SNIPPET
     it "should complain when an unknown variable is used" do
       # Error case: there is no macro argument called <unknown>
       error_message = "Unknown macro-step argument 'unknown'."
-      lambda { subject.expand(phrase_instance, [ ['unknown', 'anything'] ]) }.should raise_error(UnknownArgumentError, error_message)
+      args = [ %w(unknown anything) ]
+      ->(){ subject.expand(phrase_instance, args) }.should 
+        raise_error(UnknownArgumentError, error_message)
     end
 
   end # context
 
-
 end # describe
 
 end # module

data/spec/macros4cuke/templating/engine_spec.rb

@@ -2,7 +2,9 @@
 # File: engine_spec.rb
 
 require_relative '../../spec_helper'
-require_relative '../../../lib/macros4cuke/templating/engine'	# Load the class under test
+
+# Load the class under test
+require_relative '../../../lib/macros4cuke/templating/engine'
 
 
 module Macros4Cuke
@@ -20,6 +22,8 @@ describe Engine do
   And I fill in "Password" with "<password>"
   And I click "Sign in"
 SNIPPET
+
+      source
     end
     
   # Template containing two conditional sections
@@ -34,6 +38,8 @@ SNIPPET
 <?dummy></dummy>
   And I click "Register"
 SNIPPET
+
+    source
     end    
   
   
@@ -51,32 +57,35 @@ SNIPPET
 
     it "should parse an empty text line" do
       # Expectation: result should be an empty array.
-      Engine::parse('').should be_empty
+      Engine.parse('').should be_empty
     end
 
     it "should parse a text line without tag" do
       sample_text = 'Mary has a little lamb'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
-      # Expectation: an array with one couple: [:static, the source text]
+      # Expectation: an array with one couple: 
+      # [:static, the source text]
       result.should have(1).items
       result[0].should == [:static, sample_text]
     end
 
     it "should parse a text line that consists of just a tag" do
       sample_text = '<some_tag>'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
-      # Expectation: an array with one couple: [:static, the source text]
+      # Expectation: an array with one couple: 
+      # [:static, the source text]
       result.should have(1).items
       result[0].should == [:dynamic, strip_chevrons(sample_text)]
     end
 
     it "should parse a text line with a tag at the start" do
       sample_text = '<some_tag>some text'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
-      # Expectation: an array with two couples: [dynamic, 'some_tag'][:static, some text]
+      # Expectation: an array with two couples: 
+      # [dynamic, 'some_tag'][:static, some text]
       result.should have(2).items
       result[0].should == [:dynamic, 'some_tag']
       result[1].should == [:static,  'some text']
@@ -84,9 +93,10 @@ SNIPPET
 
     it "should parse a text line with a tag at the end" do
       sample_text = 'some text<some_tag>'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
-      # Expectation: an array with two couples: [:static, some text] [dynamic, 'some_tag']
+      # Expectation: an array with two couples: 
+      # [:static, some text] [dynamic, 'some_tag']
       result.should have(2).items
       result[0].should == [:static,  'some text']
       result[1].should == [:dynamic, 'some_tag']
@@ -94,7 +104,7 @@ SNIPPET
 
     it "should parse a text line with a tag in the middle" do
       sample_text = 'begin <some_tag> end'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
       # Expectation: an array with three couples:
       result.should have(3).items
@@ -105,7 +115,7 @@ SNIPPET
 
     it "should parse a text line with two tags in the middle" do
       sample_text = 'begin <some_tag>middle<another_tag> end'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
       # Expectation: an array with items couples:
       result.should have(5).items
@@ -117,7 +127,7 @@ SNIPPET
 
       # Case: two consecutive tags
       sample_text = 'begin <some_tag><another_tag> end'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
       # Expectation: an array with four couples:
       result.should have(4).items
@@ -129,7 +139,7 @@ SNIPPET
 
     it "should parse a text line with escaped chevrons" do
       sample_text = 'Mary has a \<little\> lamb'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
       # Expectation: an array with one couple: [:static, the source text]
       result.should have(1).items
@@ -138,7 +148,7 @@ SNIPPET
 
     it "should parse a text line with escaped chevrons in a tag" do
       sample_text = 'begin <some_\<\\>weird\>_tag> end'
-      result = Engine::parse(sample_text)
+      result = Engine.parse(sample_text)
 
       # Expectation: an array with three couples:
       result.should have(3).items
@@ -150,19 +160,22 @@ SNIPPET
     it "should complain if a tag misses an closing chevron" do
       sample_text = 'begin <some_tag\> end'
       error_message = "Missing closing chevron '>'."
-      lambda { Engine::parse(sample_text) }.should raise_error(StandardError, error_message)
+      ->(){ Engine.parse(sample_text) }.should raise_error(
+        StandardError, error_message)
     end
 
     it  "should complain if a text misses an opening chevron" do
       sample_text = 'begin <some_tag> > end'
       error_message = "Missing opening chevron '<'."
-      lambda { Engine::parse(sample_text) }.should raise_error(StandardError, error_message)
+      ->(){ Engine.parse(sample_text) }.should raise_error(
+        StandardError, error_message)
     end
 
     it  "should complain if a text has nested opening chevrons" do
       sample_text = 'begin <<some_tag> > end'
       error_message = "Nested opening chevron '<'."
-      lambda { Engine::parse(sample_text) }.should raise_error(StandardError, error_message)
+      ->(){ Engine.parse(sample_text) }.should raise_error(
+        StandardError, error_message)
     end
 
   end # context
@@ -170,11 +183,11 @@ SNIPPET
   context "Creation and initialization" do
 
     it "should accept an empty template text" do
-      lambda { Engine.new '' }.should_not raise_error
+      ->(){ Engine.new '' }.should_not raise_error
     end
 
     it "should be created with a template text" do
-      lambda { Engine.new sample_template }.should_not raise_error
+      ->(){ Engine.new sample_template }.should_not raise_error
     end
 
     it "should know the source text" do
@@ -186,20 +199,26 @@ SNIPPET
     end
     
     it "should accept conditional section" do
-      lambda { Engine.new sophisticated_template }.should_not raise_error
+      ->(){ Engine.new sophisticated_template }.should_not raise_error
       instance = Engine.new sophisticated_template
+      elements = instance.instance_variable_get(:@representation)
+      sections = elements.select { |e| e.is_a?(Section) }
+      names = sections.map { |e| e.to_s }
+      names.should == %w(<?address> <?birthdate> <?dummy>)
     end
 
     it "should complain when a placeholder is empty or blank" do
       text_w_empty_arg = sample_template.sub(/userid/, '')
-      error_message = %Q|An empty or blank argument occurred in 'And I fill in "Username" with "<>"'.|
-      lambda { Engine.new text_w_empty_arg }.should raise_error(Macros4Cuke::EmptyArgumentError, error_message)
+      msg = %q(An empty or blank argument occurred in 'And I fill in "Username" with "<>"'.)
+      ->(){ Engine.new text_w_empty_arg }.should 
+        raise_error(Macros4Cuke::EmptyArgumentError, msg)
     end
 
     it "should complain when a placeholder contains an invalid character" do
       text_w_empty_arg = sample_template.sub(/userid/, 'user%id')
-      error_message = "The invalid sign '%' occurs in the argument/tag 'user%id'."
-      lambda { Engine.new text_w_empty_arg }.should raise_error(Macros4Cuke::InvalidCharError, error_message)
+      msg = "The invalid sign '%' occurs in the argument/tag 'user%id'."
+      ->(){ Engine.new text_w_empty_arg }.should 
+        raise_error(Macros4Cuke::InvalidCharError, msg)
     end
     
     it "should complain when a section has no closing tag" do
@@ -207,22 +226,25 @@ SNIPPET
       text_w_open_section = sophisticated_template.sub(/<\/address>/, '')
 
       error_message = "Unterminated section <?address>."
-      lambda { Engine.new text_w_open_section}.should raise_error(StandardError, error_message)
+      ->(){ Engine.new text_w_open_section}.should 
+        raise_error(StandardError, error_message)
     end
     
-    it "should complain when a closing tag does not correspond to currently open section" do
+    it "should complain when a closing tag has no corresponding opening tag" do
       # Replacing an end of section tag by another...
       text_w_wrong_end = sophisticated_template.sub(/<\/address>/, '</foobar>')
 
-      error_message = "End of section</foobar> doesn't match current section 'address'."
-      lambda { Engine.new text_w_wrong_end}.should raise_error(StandardError, error_message)
+      msg = "End of section</foobar> doesn't match current section 'address'."
+      ->(){ Engine.new text_w_wrong_end }.should 
+        raise_error(StandardError, msg)
     end
     
-    it "should complain when a closing tag is found while no section is open" do
+    it "should complain when a closing tag is found without opening tag" do
       # Replacing an end of section tag by another...
-      text_w_wrong_end = sophisticated_template.sub(/<\?birthdate>/, '</foobar>')
-      error_message = "End of section</foobar> found while no corresponding section is open."
-      lambda { Engine.new text_w_wrong_end}.should raise_error(StandardError, error_message)
+      wrong_end = sophisticated_template.sub(/<\?birthdate>/, '</foobar>')
+      msg = "End of section</foobar> found while no corresponding section is open."
+      ->(){ Engine.new wrong_end }.should 
+        raise_error(StandardError, msg)
     end    
     
   end # context
@@ -243,12 +265,12 @@ SNIPPET
       substeps = "  # Comment 1 <miscellaneous>\n" + sample_template
       substeps += "  #\n Comment 2 <haphazard>"
       instance = Engine.new substeps
-      subject.variables == [:userid, :password]
+      instance.variables == [:userid, :password]
     end
 
 
     it "should render the text given the actuals" do
-      locals = {'userid' => "johndoe"}
+      locals = { 'userid' => "johndoe" }
 
       rendered_text = subject.render(Object.new, locals)
       expected = <<-SNIPPET
@@ -262,13 +284,13 @@ SNIPPET
       rendered_text.should == expected
 
       # Case of an actual that's not a String
-      locals = {'userid' => "johndoe", "password" => 12345678 }
+      locals = { 'userid' => "johndoe", "password" => 12345 }
       rendered_text = subject.render(Object.new, locals)
       expected = <<-SNIPPET
   Given I landed in the homepage
   # The credentials are entered here
   And I fill in "Username" with "johndoe"
-  And I fill in "Password" with "12345678"
+  And I fill in "Password" with "12345"
   And I click "Sign in"
 SNIPPET
 
@@ -278,7 +300,7 @@ SNIPPET
       # Place actual value in context object
       Context = Struct.new(:userid, :password)
       context = Context.new("sherlock", "holmes")
-      rendered_text = subject.render(context, {'userid' => 'susan'})
+      rendered_text = subject.render(context, { 'userid' => 'susan' })
       expected = <<-SNIPPET
   Given I landed in the homepage
   # The credentials are entered here
@@ -299,7 +321,7 @@ SNIPPET
     it "should render conditional sections" do
       instance = Engine.new(sophisticated_template)
       
-      locals = {'firstname' => "Anon", 
+      locals = { 'firstname' => "Anon", 
         "lastname" => "Eemoos" ,
         "birthdate" => "1976-04-21"
         }
@@ -330,7 +352,7 @@ SNIPPET
     
 
     it "should render multivalued actuals" do
-      locals = {'userid' => ["johndoe", "yeti"] } # Silly case
+      locals = { 'userid' => %w(johndoe yeti) } # Silly case
 
       rendered_text = subject.render(Object.new, locals)
       expected = <<-SNIPPET

data/spec/macros4cuke/templating/placeholder_spec.rb

@@ -2,7 +2,9 @@
 # File: section_spec.rb
 
 require_relative '../../spec_helper'
-require_relative '../../../lib/macros4cuke/templating/engine'	# Load the classes under test
+
+# Load the classes under test
+require_relative '../../../lib/macros4cuke/templating/engine'
 
 module Macros4Cuke
 
@@ -16,44 +18,48 @@ describe Placeholder do
   context "Creation and initialization:" do
 
     it "should be created with a variable name" do
-      lambda { Placeholder.new('foobar') }.should_not raise_error
+      ->() { Placeholder.new('foobar') }.should_not raise_error
     end
-    
-    it "should know the name of its variable" do 
+
+    it "should know the name of its variable" do
       subject.name.should == 'foobar'
-    end    
+    end
 
-  end # context 
+  end # context
 
-  context "Provided services:" do 
-    it "should render an empty string when no actual value is bound to the placeholder" do
+  context "Provided services:" do
+    it "should render an empty string when no actual value is absent" do
       # Case: context has no value associated to 'foobar'
       rendered_text = subject.render(Object.new, {})
       rendered_text.should be_empty
-      
+
       # Case: locals Hash has a nil value associated to 'foobar'
-      rendered_text = subject.render(Object.new, {'foobar' => nil})
+      rendered_text = subject.render(Object.new, { 'foobar' => nil })
       rendered_text.should be_empty
 
       # Case: context object has a nil value associated to 'foobar'
       context = Object.new
-      def context.foobar; nil; end  # Add singleton method foobar that returns nil
+      def context.foobar  # Add singleton method foobar that returns nil
+        nil
+      end 
       rendered_text = subject.render(context, {})
-      rendered_text.should be_empty        
+      rendered_text.should be_empty
     end
-    
+
     it "should render the actual value bound to the placeholder" do
       # Case: locals Hash has a value associated to 'foobar'
-      rendered_text = subject.render(Object.new, {'foobar' => 'hello'})
+      rendered_text = subject.render(Object.new, { 'foobar' => 'hello' })
       rendered_text.should == 'hello'
 
       # Case: context object has a value associated to 'foobar'
       context = Object.new
-      def context.foobar; 'world'; end  # Add singleton method foobar that returns 'world'
+      def context.foobar # Add singleton method foobar that returns 'world'
+        'world'
+      end
       rendered_text = subject.render(context, {})
-      rendered_text.should == 'world'       
+      rendered_text.should == 'world'
     end
-  
+
   end # context
 
 end # describe

data/spec/macros4cuke/templating/section_spec.rb

@@ -2,7 +2,9 @@
 # File: section_spec.rb
 
 require_relative '../../spec_helper'
-require_relative '../../../lib/macros4cuke/templating/engine'	# Load the classes under test
+
+# Load the classes under test
+require_relative '../../../lib/macros4cuke/templating/engine'
 
 module Macros4Cuke
 
@@ -11,33 +13,33 @@ module Templating # Open this namespace to get rid of module qualifier prefixes
 # Spec for abstract class Section.
 describe Section do
   # Default instantiation rule
-  subject { Section.new('foobar') } 
-  
+  subject { Section.new('foobar') }
+
   # Return a list of possible child elements
   let(:sample_children) do
     [ StaticText.new("Hello "),
       Placeholder.new("user"),
       EOLine.new
     ]
-  end 
+  end
 
   context "Creation and initialization" do
- 
+
     it "should be created with a variable name" do
-      lambda { Section.new('foobar') }.should_not raise_error 
+      ->() { Section.new('foobar') }.should_not raise_error
     end
-    
-    it "should know the name of its variable" do 
+
+    it "should know the name of its variable" do
       subject.name.should == 'foobar'
     end
 
     it "should have no child at start" do
       subject.should have(0).children
-    end    
-  
+    end
+
   end # context
-  
-  context "Provided services:" do  
+
+  context "Provided services:" do
     it "should add child element(s)" do
       sample_children.each do |a_child|
         subject.add_child(a_child)
@@ -46,31 +48,30 @@ describe Section do
       # Control that the addition work as expected
       subject.children.should == sample_children
     end
-    
+
     it "should know the name all child placeholders" do
       # Case: simple flat list of children
       sample_children.each { |a_child| subject.add_child(a_child) }
       subject.variables.should == [ 'user' ]
-      
-      # Case: at least one child is a group    
+
+      # Case: at least one child is a group
       parent = Section.new('son')
-      [ 
-        subject,
+      [ subject,
         StaticText.new("Bye "),
         Placeholder.new("firstname"),
         EOLine.new
-      ].each { |a_child| parent.add_child(a_child) } 
+      ].each { |a_child| parent.add_child(a_child) }
 
-      parent.variables.should == [ 'user', 'firstname']
-      
+      parent.variables.should == %w(user firstname)
     end
-    
-    
+
+
     it "should expect that its subclasses render the children" do
-      error_message = "Method Section::render must be implemented in subclass(es)."
-      lambda {subject.send(:render, Object.new, {}) }.should raise_error(NotImplementedError, error_message)
+      error_message = "Method Section.render must be implemented in subclass."
+      ->(){ subject.send(:render, Object.new, {}) }.should raise_error(
+        NotImplementedError, error_message)
     end
-    
+
   end # context
 
 end # describe
@@ -83,11 +84,11 @@ describe ConditionalSection do
   context "Creation and initialization:" do
 
     it "should be created with a variable name and a boolean" do
-      lambda { ConditionalSection.new('foobar', false) }.should_not raise_error
-      lambda { ConditionalSection.new('foobar', true) }.should_not raise_error
+      ->(){ ConditionalSection.new('foobar', false) }.should_not raise_error
+      ->(){ ConditionalSection.new('foobar', true) }.should_not raise_error
     end
 
-    it "should know whether the rendition on existence of actual value or not" do
+    it "should know whether the rendition on existence of actual value" do
       [false, true].each do |existence|
         instance = ConditionalSection.new('foobar', existence)
         instance.existence.should == existence
@@ -107,7 +108,7 @@ describe ConditionalSection do
 
     # Default instantiation rule
     subject { ConditionalSection.new('foobar', true) }
-    
+
     it "should know its original source text" do
       subject.to_s.should == '<?foobar>'
     end
@@ -117,17 +118,18 @@ describe ConditionalSection do
       sample_children.each { |a_child| subject.add_child(a_child) }
 
       # Case of an existing actual
-      locals = {'user' => "joe", "foobar" => 'exists' }
+      locals = { 'user' => "joe", "foobar" => 'exists' }
       rendered_text = subject.render(Object.new, locals)
       expected_text = "Hello joe\n"
       rendered_text.should == expected_text
 
-      # Case of a conditional section that should be rendering when value is non-existing.
+      # Case of a conditional section that should be
+      # rendering when value is non-existing.
       instance = ConditionalSection.new('foobar', false)
       sample_children.each { |a_child| instance.add_child(a_child) }
 
       # Case of a non-existing actual
-      locals = {'user' => "joe" }
+      locals = { 'user' => "joe" }
       rendered_text = instance.render(Object.new, locals)
       rendered_text.should == expected_text
     end
@@ -137,16 +139,17 @@ describe ConditionalSection do
       sample_children.each { |a_child| subject.add_child(a_child) }
 
       # Case of a non-existing actual
-      locals = {'user' => "joe"}
+      locals = { 'user' => "joe" }
       rendered_text = subject.render(Object.new, locals)
       rendered_text.should == ""
 
-      # Case of a conditional section that should be rendering when value is non-existing.
+      # Case of a conditional section that should be
+      # rendering when value is non-existing.
       instance = ConditionalSection.new('foobar', false)
       sample_children.each { |a_child| instance.add_child(a_child) }
 
       # Case of a non-existing actual
-      locals = {'user' => "joe", "foobar" => 'exists' }
+      locals = { 'user' => "joe", "foobar" => 'exists' }
       rendered_text = instance.render(Object.new, locals)
       rendered_text.should == ""
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: macros4cuke
 version: !ruby/object:Gem::Version
-  version: 0.3.14
+  version: 0.3.15
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-18 00:00:00.000000000 Z
+date: 2013-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber