smartdown 0.5.0 → 0.5.1

data/lib/smartdown/engine/transition.rb

@@ -26,24 +26,15 @@ module Smartdown
 
     private
       def next_node_from_next_node_rules
-        next_node_rules && first_matching_rule(next_node_rules.rules).outcome
+        node.next_node_rules && first_matching_rule(node.next_node_rules.rules).outcome
       end
 
       def next_node_from_start_button
-        start_button && start_button.start_node
+        node.start_button && node.start_button.start_node
       end
 
       def input_variable_names_from_question
-        questions = node.elements.select { |e| e.class.to_s.include?("Smartdown::Model::Element::Question") }
-        questions.map(&:name)
-      end
-
-      def next_node_rules
-        node.elements.find { |e| e.is_a?(Smartdown::Model::NextNodeRules) }
-      end
-
-      def start_button
-        node.elements.find { |e| e.is_a?(Smartdown::Model::Element::StartButton) }
+        node.questions.map(&:name)
       end
 
       def first_matching_rule(rules)

data/lib/smartdown/engine.rb

@@ -24,22 +24,22 @@ module Smartdown
       {}.merge(@initial_state)
     end
 
-    def process(responses, test_start_state = nil)
+    def process(raw_responses, test_start_state = nil)
       state = test_start_state || build_start_state
-      unprocessed_responses = responses
+      unprocessed_responses = raw_responses
       while !unprocessed_responses.empty? do
-        nb_questions = 0
         current_node = flow.node(state.get(:current_node))
-        nb_questions += current_node.elements.select{|element|
-          element.class.to_s.include?("Smartdown::Model::Element::Question")
-        }.count
-        #There is at least one relevant input per transition for now:
-        #Transition from start to first question relies on an input, regardless of its value
-        nb_relevant_inputs = [nb_questions, 1].max
-        input_array = unprocessed_responses.take(nb_relevant_inputs)
-        unprocessed_responses = unprocessed_responses.drop(nb_relevant_inputs)
-
-        transition = Transition.new(state, current_node, input_array)
+
+        if current_node.is_start_page_node?
+          # If it's a start page node, we've got to do something different because of the preceeding 'y' answer
+          transition = Transition.new(state, current_node, unprocessed_responses.shift(1))
+        else
+          answers = current_node.questions.map do |question|
+            question.answer_type.new(question, unprocessed_responses.shift)
+          end
+
+          transition = Transition.new(state, current_node, answers)
+        end
         state = transition.next_state
       end
       state

data/lib/smartdown/model/answer/base.rb

@@ -0,0 +1,42 @@
+module Smartdown
+  module Model
+    module Answer
+      class Base
+        extend Forwardable
+        include Comparable
+
+        def_delegators :value, :to_s, :to_i, :to_f, :+, :-, :*, :/
+
+        def value_type
+          ::String
+        end
+
+        def <=>(other)
+          value <=> parse_comparison_object(other)
+        end
+
+        attr_reader :question, :value
+
+        def initialize(question, value)
+          @question = question
+          @value = parse_value(value)
+        end
+
+      private
+        def parse_comparison_object(comparison_object)
+          if comparison_object.is_a? Base
+            comparison_object.value
+          elsif comparison_object.is_a? value_type
+            comparison_object
+          else
+            parse_value(comparison_object)
+          end
+        end
+
+        def parse_value(value)
+          value
+        end
+      end
+    end
+  end
+end

data/lib/smartdown/model/answer/date.rb

@@ -0,0 +1,22 @@
+require_relative "base"
+
+module Smartdown
+  module Model
+    module Answer
+      class Date < Base
+        def value_type
+          ::Date
+        end
+
+        def to_s
+          value.strftime("%Y-%-m-%-d")
+        end
+
+      private
+        def parse_value(value)
+          ::Date.parse(value)
+        end
+      end
+    end
+  end
+end

data/lib/smartdown/model/answer/multiple_choice.rb

@@ -0,0 +1,13 @@
+require_relative "base"
+
+module Smartdown
+  module Model
+    module Answer
+      class MultipleChoice < Base
+        def value_type
+          ::String
+        end
+      end
+    end
+  end
+end

data/lib/smartdown/model/answer/salary.rb

@@ -0,0 +1,37 @@
+require_relative "base"
+
+module Smartdown
+  module Model
+    module Answer
+      class Salary < Base
+        attr_reader :period, :amount_per_period
+
+        def value_type
+          ::Float
+        end
+
+        def to_s
+          "#{'%.2f' % amount_per_period} per #{period}"
+        end
+
+      private
+        def parse_value(value)
+          @amount_per_period, @period = value.split(/-|\W*per\W*/)
+          @amount_per_period = @amount_per_period.to_f
+          yearly_total
+        end
+
+        def yearly_total
+          case period
+          when "week"
+            amount_per_period * 52
+          when "month"
+            amount_per_period * 12
+          when "year"
+            amount_per_period
+          end
+        end
+      end
+    end
+  end
+end

data/lib/smartdown/model/element/question/date.rb

@@ -1,8 +1,14 @@
+require 'smartdown/model/answer/date'
+
 module Smartdown
   module Model
     module Element
       module Question
-        Date = Struct.new(:name)
+        class Date < Struct.new(:name)
+          def answer_type
+            Smartdown::Model::Answer::Date
+          end
+        end
       end
     end
   end

data/lib/smartdown/model/element/question/multiple_choice.rb

@@ -1,8 +1,14 @@
+require 'smartdown/model/answer/multiple_choice'
+
 module Smartdown
   module Model
     module Element
       module Question
-        MultipleChoice = Struct.new(:name, :choices)
+        class MultipleChoice < Struct.new(:name, :choices)
+          def answer_type
+            Smartdown::Model::Answer::MultipleChoice
+          end
+        end
       end
     end
   end

data/lib/smartdown/model/element/question/salary.rb

@@ -1,8 +1,14 @@
+require 'smartdown/model/answer/salary'
+
 module Smartdown
   module Model
     module Element
       module Question
-        Salary = Struct.new(:name)
+        class Salary < Struct.new(:name)
+          def answer_type
+            Smartdown::Model::Answer::Salary
+          end
+        end
       end
     end
   end

data/lib/smartdown/model/node.rb

@@ -7,18 +7,30 @@ module Smartdown
         super(name, elements, front_matter || Smartdown::Model::FrontMatter.new)
       end
 
+      def title
+        h1s.first ? h1s.first.content : ""
+      end
+
+      def body
+        markdown_blocks[1..-1].map { |block| as_markdown(block) }.compact.join("\n")
+      end
+
       def questions
-        elements.select do |element|
+        @questions ||= elements.select do |element|
           element.class.to_s.include?("Smartdown::Model::Element::Question")
         end
       end
 
-      def title
-        h1s.first ? h1s.first.content : ""
+      def next_node_rules
+        @next_node_rules ||= elements.find { |e| e.is_a?(Smartdown::Model::NextNodeRules) }
       end
 
-      def body
-        markdown_blocks[1..-1].map { |block| as_markdown(block) }.compact.join("\n")
+      def start_button
+        @start_button ||= elements.find { |e| e.is_a?(Smartdown::Model::Element::StartButton) }
+      end
+
+      def is_start_page_node?
+        @is_start_page_node ||= !!start_button
       end
 
       private

data/lib/smartdown/model/predicate/comparison/greater.rb

@@ -8,11 +8,7 @@ module Smartdown
         class Greater < Base
           def evaluate(state)
             variable = state.get(varname)
-            if /(\d{4})-(\d{1,2})-(\d{1,2})/.match(value)
-              Date.parse(variable) > Date.parse(value)
-            else
-              variable > value
-            end
+            variable > value
           end
 
           def humanize

data/lib/smartdown/model/predicate/comparison/greater_or_equal.rb

@@ -8,11 +8,7 @@ module Smartdown
         class GreaterOrEqual < Base
           def evaluate(state)
             variable = state.get(varname)
-            if /(\d{4})-(\d{1,2})-(\d{1,2})/.match(value)
-              Date.parse(variable) >= Date.parse(value)
-            else
-              variable >= value
-            end
+            variable >= value
           end
 
           def humanize

data/lib/smartdown/model/predicate/comparison/less.rb

@@ -8,11 +8,7 @@ module Smartdown
         class Less < Base
           def evaluate(state)
             variable = state.get(varname)
-            if /(\d{4})-(\d{1,2})-(\d{1,2})/.match(value)
-              Date.parse(variable) < Date.parse(value)
-            else
-              variable < value
-            end
+            variable < value
           end
 
           def humanize

data/lib/smartdown/model/predicate/comparison/less_or_equal.rb

@@ -8,11 +8,7 @@ module Smartdown
         class LessOrEqual < Base
           def evaluate(state)
             variable = state.get(varname)
-            if /(\d{4})-(\d{1,2})-(\d{1,2})/.match(value)
-              Date.parse(variable) <= Date.parse(value)
-            else
-              variable <= value
-            end
+            variable <= value
           end
 
           def humanize

data/lib/smartdown/model/predicate/otherwise.rb

@@ -9,6 +9,10 @@ module Smartdown
         def ==(o)
           o.class == self.class
         end
+
+        def humanize
+          "otherwise"
+        end
       end
     end
   end

data/lib/smartdown/model/predicate/set_membership.rb

@@ -3,7 +3,7 @@ module Smartdown
     module Predicate
       SetMembership = Struct.new(:varname, :values) do
         def evaluate(state)
-          values.include?(state.get(varname))
+          values.any? {|value| state.get(varname) == value }
         end
 
         def humanize

data/lib/smartdown/version.rb

@@ -1,3 +1,3 @@
 module Smartdown
-  VERSION = "0.5.0"
+  VERSION = "0.5.1"
 end

data/spec/model/answer/base_spec.rb

@@ -0,0 +1,23 @@
+require 'smartdown/model/answer/base'
+
+describe Smartdown::Model::Answer::Base do
+  let(:question) { :a_question }
+  let(:value) { 'a value' }
+
+  subject(:instance) {Smartdown::Model::Answer::Base.new(question, value)}
+
+  specify { expect(instance.question).to eql question }
+
+  describe "value parsing" do
+    it "should assign the return value of parse_value (to be implemented by sub-classes) to value" do
+      expect_any_instance_of(Smartdown::Model::Answer::Base).to receive(:parse_value).with(value).and_return("parsed value")
+      expect(instance.value).to eql "parsed value"
+    end
+  end
+
+  describe "simple type behaviour" do
+    [:==, :<, :>, :<=, :>=, :to_s, :to_i, :to_f, :+, :-, :*, :/].each do |method|
+      it { should respond_to(method) }
+    end
+  end
+end

data/spec/model/answer/date_spec.rb

@@ -0,0 +1,77 @@
+require 'smartdown/model/answer/date'
+require 'smartdown/model/element/question/date'
+
+describe Smartdown::Model::Answer::Date do
+  let(:date_string) { "2014-9-4" }
+  let(:question) { Smartdown::Model::Element::Question::Date.new("a_date") }
+  subject(:instance) { described_class.new(question, date_string) }
+
+  specify { expect(instance.value).to eql Date.new(2014, 9, 4) }
+  specify { expect(instance.to_s).to eql "2014-9-4" }
+
+
+  describe "comparisons" do
+    let(:date_string) { "2000-1-10" }
+
+    context "comparing against ::Dates" do
+
+      specify { expect(instance == Date.new(2000, 1, 10)).to eql true }
+
+      specify { expect(instance < Date.new(2000, 1, 11)).to eql true }
+      specify { expect(instance < Date.new(2000, 1, 10)).to eql false }
+      specify { expect(instance < Date.new(2000, 1, 9)).to eql false }
+
+      specify { expect(instance > Date.new(2000, 1, 11)).to eql false }
+      specify { expect(instance > Date.new(2000, 1, 10)).to eql false }
+      specify { expect(instance > Date.new(2000, 1, 9)).to eql true }
+
+      specify { expect(instance <= Date.new(2000, 1, 11)).to eql true }
+      specify { expect(instance <= Date.new(2000, 1, 10)).to eql true }
+      specify { expect(instance <= Date.new(2000, 1, 9)).to eql false }
+
+      specify { expect(instance >= Date.new(2000, 1, 11)).to eql false }
+      specify { expect(instance >= Date.new(2000, 1, 10)).to eql true }
+      specify { expect(instance >= Date.new(2000, 1, 9)).to eql true }
+    end
+
+    context "comparing against strings" do
+      specify { expect(instance == "2000-1-10").to eql true }
+
+      specify { expect(instance < "2000-1-11").to eql true }
+      specify { expect(instance < "2000-1-10").to eql false }
+      specify { expect(instance < "2000-1-9").to eql false }
+
+      specify { expect(instance > "2000-1-11").to eql false }
+      specify { expect(instance > "2000-1-10").to eql false }
+      specify { expect(instance > "2000-1-9").to eql true }
+
+      specify { expect(instance <= "2000-1-11").to eql true }
+      specify { expect(instance <= "2000-1-10").to eql true }
+      specify { expect(instance <= "2000-1-9").to eql false }
+
+      specify { expect(instance >= "2000-1-11").to eql false }
+      specify { expect(instance >= "2000-1-10").to eql true }
+      specify { expect(instance >= "2000-1-9").to eql true }
+    end
+
+    context "comparing against Answer::Dates" do
+      specify { expect(instance == described_class.new(nil, "2000-1-10")).to eql true }
+
+      specify { expect(instance < described_class.new(nil, "2000-1-11")).to eql true }
+      specify { expect(instance < described_class.new(nil, "2000-1-10")).to eql false }
+      specify { expect(instance < described_class.new(nil, "2000-1-9")).to eql false }
+
+      specify { expect(instance > described_class.new(nil, "2000-1-11")).to eql false }
+      specify { expect(instance > described_class.new(nil, "2000-1-10")).to eql false }
+      specify { expect(instance > described_class.new(nil, "2000-1-9")).to eql true }
+
+      specify { expect(instance <= described_class.new(nil, "2000-1-11")).to eql true }
+      specify { expect(instance <= described_class.new(nil, "2000-1-10")).to eql true }
+      specify { expect(instance <= described_class.new(nil, "2000-1-9")).to eql false }
+
+      specify { expect(instance >= described_class.new(nil, "2000-1-11")).to eql false }
+      specify { expect(instance >= described_class.new(nil, "2000-1-10")).to eql true }
+      specify { expect(instance >= described_class.new(nil, "2000-1-9")).to eql true }
+    end
+  end
+end

data/spec/model/answer/salary_spec.rb

@@ -0,0 +1,98 @@
+require 'smartdown/model/answer/date'
+require 'smartdown/model/element/question/date'
+
+describe Smartdown::Model::Answer::Salary do
+  let(:salary_string) { "500-week" }
+  let(:question) { Smartdown::Model::Element::Question::Date.new("a_date") }
+  subject(:instance) { described_class.new(question, salary_string) }
+
+  specify { expect(instance.period).to eql('week') }
+  specify { expect(instance.amount_per_period).to eql(500.00) }
+
+  it "as a string, it should declare itself in the initial format provided" do
+    expect(instance.to_s).to eql("500.00 per week")
+  end
+
+  context "declared by week" do
+    let(:salary_string) { "500-week" }
+    specify { expect(instance.value).to eql 500.0 * 52 }
+  end
+
+  context "declared by month" do
+    let(:salary_string) { "2000-month" }
+    specify { expect(instance.value).to eql 2000.0 * 12 }
+  end
+
+  context "declared by year" do
+    let(:salary_string) { "20000-year" }
+    specify { expect(instance.value).to eql 20000.0 }
+  end
+
+
+  describe "comparisons" do
+    let(:salary_string) { "1200-week" } # equivalent to 62,400 yearly or 5200 monthly
+
+    context "comparing against ::Floats" do
+      specify { expect(instance == 62400.0).to eql true }
+
+      specify { expect(instance < 62400.1).to eql true }
+      specify { expect(instance < 62400.0).to eql false }
+      specify { expect(instance < 62399.9).to eql false }
+
+      specify { expect(instance > 62400.1).to eql false }
+      specify { expect(instance > 62400.0).to eql false }
+      specify { expect(instance > 62399.9).to eql true }
+
+      specify { expect(instance <= 62400.1).to eql true }
+      specify { expect(instance <= 62400.0).to eql true }
+      specify { expect(instance <= 62399.9).to eql false }
+
+      specify { expect(instance >= 62400.1).to eql false }
+      specify { expect(instance >= 62400.0).to eql true }
+      specify { expect(instance >= 62399.9).to eql true }
+    end
+
+    context "comparing against strings" do
+      specify { expect(instance == "1200-week").to eql true }
+      specify { expect(instance == "5200-month").to eql true }
+      specify { expect(instance == "62400-year").to eql true }
+      specify { expect(instance == "10-month").to eql false }
+
+      specify { expect(instance < "62400.1-year").to eql true }
+      specify { expect(instance < "5200.0-month").to eql false }
+      specify { expect(instance < "1199.9-week").to eql false }
+
+      specify { expect(instance > "1200.1-week").to eql false }
+      specify { expect(instance > "62400-year").to eql false }
+      specify { expect(instance > "5199.9-month").to eql true }
+
+      specify { expect(instance <= "62400.1-year").to eql true }
+      specify { expect(instance <= "5200.0-month").to eql true }
+      specify { expect(instance <= "1199.9-week").to eql false }
+
+      specify { expect(instance >= "1200.1-week").to eql false }
+      specify { expect(instance >= "62400-year").to eql true }
+      specify { expect(instance >= "5199.9-month").to eql true }
+    end
+
+    context "comparing against Answer::Salaries" do
+      specify { expect(instance == described_class.new(nil, "1200-week")).to eql true }
+
+      specify { expect(instance < described_class.new(nil, "1200.1-week")).to eql true }
+      specify { expect(instance < described_class.new(nil, "1200.0-week")).to eql false }
+      specify { expect(instance < described_class.new(nil, "1199.9-week")).to eql false }
+
+      specify { expect(instance > described_class.new(nil, "1200.1-week")).to eql false }
+      specify { expect(instance > described_class.new(nil, "1200.0-week")).to eql false }
+      specify { expect(instance > described_class.new(nil, "1199.9-week")).to eql true }
+
+      specify { expect(instance <= described_class.new(nil, "1200.1-week")).to eql true }
+      specify { expect(instance <= described_class.new(nil, "1200.0-week")).to eql true }
+      specify { expect(instance <= described_class.new(nil, "1199.9-week")).to eql false }
+
+      specify { expect(instance >= described_class.new(nil, "1200.1-week")).to eql false }
+      specify { expect(instance >= described_class.new(nil, "1200.0-week")).to eql true }
+      specify { expect(instance >= described_class.new(nil, "1199.9-week")).to eql true }
+    end
+  end
+end

data/spec/model/node_spec.rb

@@ -3,9 +3,9 @@ require 'smartdown/model/node'
 describe Smartdown::Model::Node do
   let(:name) { "my node" }
   let(:elements) { [] }
+  subject(:node) { described_class.new(name, elements) }
 
   describe "#new" do
-    subject(:node) { described_class.new(name, elements) }
 
     it "accepts name and list of body blocks" do
       expect(node.name).to eq(name)
@@ -29,4 +29,48 @@ describe Smartdown::Model::Node do
       end
     end
   end
+
+  describe "#questions" do
+    context "with seval elements, some of which are questions" do
+      let(:elements) { [
+        Smartdown::Model::Element::MarkdownHeading.new("A Heading"),
+        Smartdown::Model::Element::Question::Date.new("a_date_question"),
+        Smartdown::Model::Element::Question::MultipleChoice.new("a_multiple_choice_question"),
+        Smartdown::Model::Element::MarkdownParagraph.new("Some text"),
+      ] }
+      specify { expect(node.questions).to eq elements[1..2] }
+    end
+  end
+
+  describe "#next_node_rules" do
+    context "with elements, one of which is a NextNodeRules questions" do
+      let(:elements) { [
+        Smartdown::Model::Element::MarkdownHeading.new("A Heading"),
+        Smartdown::Model::NextNodeRules.new(:rules)
+      ] }
+      specify { expect(node.next_node_rules).to eq elements[1] }
+    end
+  end
+
+  describe "#start_button" do
+    context "with elements, one of which is a StartButton questions" do
+      let(:elements) { [
+        Smartdown::Model::Element::StartButton.new(:a_start_node),
+        Smartdown::Model::NextNodeRules.new(:rules)
+      ] }
+      specify { expect(node.start_button).to eq elements[0] }
+    end
+  end
+
+  describe "#is_start_page_node?" do
+    context "with a StartButton element" do
+      let (:elements) { [Smartdown::Model::Element::StartButton.new(:some_start_node)] }
+      specify { expect(node.is_start_page_node?).to eql true }
+    end
+
+    context "without a StartButton element" do
+      let (:elements) { [] }
+      specify { expect(node.is_start_page_node?).to eql false }
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smartdown
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2014-07-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: parslet
-  requirement: &7733220 !ruby/object:Gem::Requirement
+  requirement: &6529700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 1.6.1
   type: :runtime
   prerelease: false
-  version_requirements: *7733220
+  version_requirements: *6529700
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &7731020 !ruby/object:Gem::Requirement
+  requirement: &6526340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -32,10 +32,10 @@ dependencies:
         version: 3.0.0
   type: :development
   prerelease: false
-  version_requirements: *7731020
+  version_requirements: *6526340
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &7744660 !ruby/object:Gem::Requirement
+  requirement: &6541220 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *7744660
+  version_requirements: *6541220
 - !ruby/object:Gem::Dependency
   name: gem_publisher
-  requirement: &7742640 !ruby/object:Gem::Requirement
+  requirement: &6539420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *7742640
+  version_requirements: *6539420
 description: 
 email: david.heath@digital.cabinet-office.gov.uk
 executables:
@@ -117,6 +117,10 @@ files:
 - lib/smartdown/model/predicate/otherwise.rb
 - lib/smartdown/model/node.rb
 - lib/smartdown/model/front_matter.rb
+- lib/smartdown/model/answer/date.rb
+- lib/smartdown/model/answer/base.rb
+- lib/smartdown/model/answer/salary.rb
+- lib/smartdown/model/answer/multiple_choice.rb
 - lib/smartdown/model/element/conditional.rb
 - lib/smartdown/model/element/markdown_paragraph.rb
 - lib/smartdown/model/element/start_button.rb
@@ -182,6 +186,9 @@ files:
 - spec/engine_spec.rb
 - spec/model/node_spec.rb
 - spec/model/flow_spec.rb
+- spec/model/answer/base_spec.rb
+- spec/model/answer/salary_spec.rb
+- spec/model/answer/date_spec.rb
 - spec/model/predicates/set_membership_spec.rb
 - spec/model/predicates/combined_spec.rb
 - spec/model/predicates/comparison_spec.rb
@@ -203,7 +210,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2695022002413728177
+      hash: -2794088758218871255
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -212,7 +219,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2695022002413728177
+      hash: -2794088758218871255
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.11
@@ -271,6 +278,9 @@ test_files:
 - spec/engine_spec.rb
 - spec/model/node_spec.rb
 - spec/model/flow_spec.rb
+- spec/model/answer/base_spec.rb
+- spec/model/answer/salary_spec.rb
+- spec/model/answer/date_spec.rb
 - spec/model/predicates/set_membership_spec.rb
 - spec/model/predicates/combined_spec.rb
 - spec/model/predicates/comparison_spec.rb