smartdown 0.12.0 → 0.12.1

data/lib/smartdown/api/previous_question_page.rb

@@ -29,9 +29,13 @@ module Smartdown
       def questions
         @questions ||= elements.slice_before do |element|
           element.is_a? Smartdown::Model::Element::MarkdownHeading
-        end.each_with_index.map do |question_element_group, index|
+        end.select { |question_element_group| 
+          question_element_group.any? { 
+            |element| element.class.to_s.include?("Smartdown::Model::Element::Question")
+          }
+        }.each_with_index.map do |question_element_group, index|
           Smartdown::Api::PreviousQuestion.new(question_element_group, responses[index])
-        end
+        end.compact
       end
 
       private

data/lib/smartdown/api/question_page.rb

@@ -25,7 +25,7 @@ module Smartdown
           elsif question_element_group.find{|element| element.is_a? Smartdown::Model::Element::Question::Postcode}
             Smartdown::Api::PostcodeQuestion.new(question_element_group)
           end
-        end
+        end.compact
       end
     end
   end

data/lib/smartdown/version.rb

@@ -1,3 +1,3 @@
 module Smartdown
-  VERSION = "0.12.0"
+  VERSION = "0.12.1"
 end

data/spec/acceptance/flow_spec.rb

@@ -56,6 +56,7 @@ describe Smartdown::Api::Flow do
 
   context "flow with two questions per page" do
     let(:input) { Smartdown::Parser::DirectoryInput.new(fixture("animal-example-multiple")) }
+
     it "has two scenario_sets" do
       expect(flow.scenario_sets.size).to eq(2)
     end
@@ -88,6 +89,8 @@ describe Smartdown::Api::Flow do
 
       context "with a valid answer given to the first question, only first answer to second page" do
         specify { expect(flow.state("y",["lion", "yes", nil]).current_node.name).to eq "question_2" }
+        specify { expect(flow.state("y",["lion", "yes", nil]).previous_question_pages.first.questions.count).to eq 1 }
+        specify { expect(flow.state("y",["lion", "yes", nil]).current_node.questions.count).to eq 2 }
         specify { expect(flow.state("y",["lion", "yes", nil]).accepted_responses).to eq ["lion"] }
         specify { expect(flow.state("y",["lion", "yes", nil]).current_answers.count).to eq 2 }
         specify { expect(flow.state("y",["lion", "yes", nil]).current_answers[0].valid?).to be true }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smartdown
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.12.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-08 00:00:00.000000000 Z
+date: 2015-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parslet
-  requirement: &8259380 !ruby/object:Gem::Requirement
+  requirement: &20811800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 1.6.2
   type: :runtime
   prerelease: false
-  version_requirements: *8259380
+  version_requirements: *20811800
 - !ruby/object:Gem::Dependency
   name: uk_postcode
-  requirement: &8258580 !ruby/object:Gem::Requirement
+  requirement: &20810920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 1.0.1
   type: :runtime
   prerelease: false
-  version_requirements: *8258580
+  version_requirements: *20810920
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &8257860 !ruby/object:Gem::Requirement
+  requirement: &20810340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 3.0.0
   type: :development
   prerelease: false
-  version_requirements: *8257860
+  version_requirements: *20810340
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &8257340 !ruby/object:Gem::Requirement
+  requirement: &20809660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *8257340
+  version_requirements: *20809660
 - !ruby/object:Gem::Dependency
   name: pry
-  requirement: &8256560 !ruby/object:Gem::Requirement
+  requirement: &20808900 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *8256560
+  version_requirements: *20808900
 - !ruby/object:Gem::Dependency
   name: gem_publisher
-  requirement: &8255920 !ruby/object:Gem::Requirement
+  requirement: &20808260 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *8255920
+  version_requirements: *20808260
 - !ruby/object:Gem::Dependency
   name: timecop
-  requirement: &8206260 !ruby/object:Gem::Requirement
+  requirement: &20937900 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,7 +87,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *8206260
+  version_requirements: *20937900
 description: 
 email: david.heath@digital.cabinet-office.gov.uk
 executables:
@@ -303,7 +303,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3641220914455318318
+      hash: 3149710437722645719
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -312,7 +312,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3641220914455318318
+      hash: 3149710437722645719
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.11