cuke_sniffer 0.0.7 → 0.0.8

data/lib/cuke_sniffer/rule_target.rb

@@ -1,62 +1,78 @@
-require 'roxml'
-module CukeSniffer
-
-  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
-  # License::   Distributes under the MIT License
-  # Parent class for all objects that have rules executed against it
-  # Mixins: CukeSniffer::Constants, CukeSniffer::RuleConfig, ROXML
-  class RuleTarget
-    include CukeSniffer::Constants
-    include CukeSniffer::RuleConfig
-    include ROXML
-
-    xml_accessor :score, :location
-    xml_accessor :rules_hash, :as => {:key => "phrase", :value => "score"}, :in => "rules", :from => "rule"
-
-    # int: Sum of the rules fired
-    attr_accessor :score
-
-    # string: Location in which the object was found
-    attr_accessor :location
-
-    # hash: Contains the phrase every rule fired against the object and times it fired
-    # * Key: string
-    # * Value: int
-    attr_accessor :rules_hash
-
-    # Location must be in the format of "file_path\file_name.rb:line_number"
-    def initialize(location)
-      @location = location
-      @score = 0
-      @rules_hash = {}
-      @class_type = self.class.to_s.gsub(/.*::/, "")
-    end
-
-    # Compares the score against the objects threshold
-    # If a score is below the threshold it is good and returns true
-    # Return: Boolean
-    def good?
-      score <= Constants::THRESHOLDS[@class_type]
-    end
-
-    # Calculates the score to threshold percentage of an object
-    # Return: Float
-    def problem_percentage
-      score.to_f / Constants::THRESHOLDS[@class_type].to_f
-    end
-
-    def == (comparison_object) # :nodoc:
-      comparison_object.location == location &&
-      comparison_object.score == score &&
-      comparison_object.rules_hash == rules_hash
-    end
-
-    private
-
-    #TODO Abstraction needed for this regex matcher (constants?)
-    def is_comment?(line)
-      true if line =~ /^\#.*$/
-    end
-  end
-end
+require 'cuke_sniffer/constants'
+require 'cuke_sniffer/rule_config'
+
+module CukeSniffer
+
+  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
+  # Copyright:: Copyright (C) 2014 Robert Cochran
+  # License::   Distributes under the MIT License
+  # Parent class for all objects that have rules executed against it
+  # Mixins: CukeSniffer::Constants, CukeSniffer::RuleConfig, ROXML
+  class RuleTarget
+    include CukeSniffer::Constants
+    include CukeSniffer::RuleConfig
+    include ROXML
+
+    xml_accessor :score, :location
+    xml_accessor :rules_hash, :as => {:key => "phrase", :value => "score"}, :in => "rules", :from => "rule"
+    xml_accessor :type
+
+    # int: Sum of the rules fired
+    attr_accessor :score
+
+    # string: Location in which the object was found
+    attr_accessor :location
+
+    # hash: Contains the phrase every rule fired against the object and times it fired
+    # * Key: string
+    # * Value: int
+    attr_accessor :rules_hash
+
+    # string: Type of the object being evaluated
+    attr_accessor :type
+
+    # Location must be in the format of "file_path\file_name.rb:line_number"
+    def initialize(location)
+      @location = location
+      @score = 0
+      @rules_hash = {}
+      @class_type = self.class.to_s.gsub(/.*::/, "")
+    end
+
+    # Compares the score against the objects threshold
+    # If a score is below the threshold it is good and returns true
+    # Return: Boolean
+    def good?
+      score <= Constants::THRESHOLDS[@class_type]
+    end
+
+    # Calculates the score to threshold percentage of an object
+    # Return: Float
+    def problem_percentage
+      score.to_f / Constants::THRESHOLDS[@class_type].to_f
+    end
+
+    def == (comparison_object) # :nodoc:
+      comparison_object.location == location &&
+          comparison_object.score == score &&
+          comparison_object.rules_hash == rules_hash
+    end
+
+    #TODO Abstraction needed for this regex matcher (constants?)
+    def is_comment?(line)
+      true if line =~ /^\#.*$/
+    end
+
+    def store_rule(rule, phrase = rule.phrase)
+      @score += rule.score
+      @rules_hash[phrase] ||= 0
+      @rules_hash[phrase] += 1
+    end
+
+    def store_rule_many_times(rule, count, phrase = rule.phrase)
+      count.times do
+        store_rule(rule, phrase)
+      end
+    end
+  end
+end

data/lib/cuke_sniffer/rules_evaluator.rb

@@ -1,65 +1,53 @@
-require 'roxml'
-
-module CukeSniffer
-
-  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
-  # License::   Distributes under the MIT License
-  # Evaluates all cucumber components found in CukeSniffer with the passed rules
-  class RulesEvaluator
-    include CukeSniffer::Constants
-    attr_accessor :rules
-
-    def initialize(cli, rules)
-      raise "A CLI must be provided for evaluation." if cli.nil?
-      raise "Rules must be provided for evaluation." if rules.nil? or rules.empty?
-      @rules = rules
-      judge_features(cli.features)
-      judge_objects(cli.step_definitions, "StepDefinition")
-      judge_objects(cli.hooks, "Hook")
-    end
-
-    private
-
-    def judge_features(features)
-      features.each do |feature|
-        judge_feature(feature)
-      end
-    end
-
-    def judge_feature(feature)
-      judge_object(feature, "Feature")
-      judge_object(feature.background, "Background") unless feature.background.nil?
-      judge_objects(feature.scenarios, "Scenario")
-      feature.total_score += feature.update_score
-
-    end
-
-    def judge_objects(objects, type)
-      objects.each do | object |
-        judge_object(object, type)
-      end
-    end
-
-    def judge_object(object, type)
-      @rules.each do |rule|
-        fail "No targets for rule: #{rule.phrase}" if rule.targets.nil? or rule.targets.empty?
-        next unless rule.targets.include? type and rule.enabled
-        if eval(rule.reason) == true
-          phrase = rule.phrase.gsub("{class}", type)
-          store_rule(object, rule, phrase)
-        end
-      end
-    end
-
-    def store_rule(object, rule, phrase = rule.phrase)
-      object.score += rule.score
-      object.rules_hash[phrase] ||= 0
-      object.rules_hash[phrase] += 1
-    end
-
-    def is_comment?(line)
-      true if line =~ /^\#.*$/
-    end
-  end
+module CukeSniffer
+
+  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
+  # Copyright:: Copyright (C) 2014 Robert Cochran
+  # License::   Distributes under the MIT License
+  # Evaluates all cucumber components found in CukeSniffer with the passed rules
+  class RulesEvaluator
+    include CukeSniffer::Constants
+    attr_accessor :rules
+
+    def initialize(cli, rules)
+      raise "A CLI must be provided for evaluation." if cli.nil?
+      raise "Rules must be provided for evaluation." if rules.nil? or rules.empty?
+      @rules = rules
+      judge_features(cli.features)
+      judge_objects(cli.step_definitions, "StepDefinition")
+      judge_objects(cli.hooks, "Hook")
+    end
+
+    private
+
+    def judge_features(features)
+      features.each do |feature|
+        judge_feature(feature)
+      end
+    end
+
+    def judge_feature(feature)
+      judge_object(feature, "Feature")
+      judge_object(feature.background, "Background") unless feature.background.nil?
+      judge_objects(feature.scenarios, "Scenario")
+      feature.total_score += feature.update_score
+
+    end
+
+    def judge_objects(objects, type)
+      objects.each do | object |
+        judge_object(object, type)
+      end
+    end
+
+    def judge_object(object, type)
+      @rules.each do |rule|
+        fail "No targets for rule: #{rule.phrase}" if rule.targets.nil? or rule.targets.empty?
+        next unless rule.targets.include? type and rule.enabled
+        if rule.reason.(object, rule) == true
+          phrase = rule.phrase.gsub("{class}", type)
+          object.store_rule(rule, phrase)
+        end
+      end
+    end
+  end
 end

data/lib/cuke_sniffer/scenario.rb

@@ -1,102 +1,123 @@
-module CukeSniffer
-
-  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
-  # License::   Distributes under the MIT License
-  # This class is a representation of the cucumber objects
-  # Background, Scenario, Scenario Outline
-  #
-  # Extends CukeSniffer::FeatureRulesEvaluator
-  class Scenario < FeatureRuleTarget
-
-    xml_accessor :start_line
-    xml_accessor :steps, :as => [], :in => "steps"
-    xml_accessor :examples_table, :as => [], :in => "examples"
-
-    # int: Line on which the scenario begins
-    attr_accessor :start_line
-
-    # string: The type of scenario
-    # Background, Scenario, Scenario Outline
-    attr_accessor :type
-
-    # string array: List of each step call in a scenario
-    attr_accessor :steps
-
-    # hash: Keeps each location and content of an inline table
-    # * Key: Step string the inline table is attached to
-    # * Value: Array of all of the lines in the table
-    attr_accessor :inline_tables
-
-    # string array: List of each example row in a scenario outline
-    attr_accessor :examples_table
-
-    # Location must be in the format of "file_path\file_name.rb:line_number"
-    # Scenario must be a string array containing everything from the first tag to the last example table
-    # where applicable.
-    def initialize(location, scenario)
-      super(location)
-      @start_line = location.match(/:(?<line>\d*)$/)[:line].to_i
-      @steps = []
-      @inline_tables = {}
-      @examples_table = []
-      split_scenario(scenario)
-    end
-
-    def ==(comparison_object) # :nodoc:
-      super(comparison_object) &&
-      comparison_object.steps == steps &&
-      comparison_object.examples_table == examples_table
-    end
-
-    def get_step_order
-      order = []
-      @steps.each do |line|
-        next if is_comment?(line)
-        match = line.match(STEP_REGEX)
-        order << match[:style] unless match.nil?
-      end
-      order
-    end
-
-    private
-
-    def split_scenario(scenario)
-      index = 0
-      until index >= scenario.length or scenario[index] =~ SCENARIO_TITLE_STYLES
-        update_tag_list(scenario[index])
-        index += 1
-      end
-
-      until index >= scenario.length or scenario[index].match STEP_REGEX or scenario[index].include?("Examples:")
-        match = scenario[index].match(SCENARIO_TITLE_STYLES)
-        @type = match[:type] unless match.nil?
-        create_name(scenario[index], SCENARIO_TITLE_STYLES)
-        index += 1
-      end
-
-      until index >= scenario.length or scenario[index].include?("Examples:")
-        if scenario[index] =~ /^\|.*\|/
-          step = scenario[index - 1]
-          @inline_tables[step] = []
-          until index >= scenario.length or scenario[index] =~ /(#{STEP_REGEX}|^\s*Examples:)/
-            @inline_tables[step] << scenario[index]
-            index += 1
-          end
-        else
-          @steps << scenario[index] if scenario[index] =~ STEP_REGEX
-          index += 1
-        end
-      end
-
-      if index < scenario.length and scenario[index].include?("Examples:")
-        index += 1
-        until index >= scenario.length
-          index += 2 if scenario[index].include?("Examples:")
-          @examples_table << scenario[index] if scenario[index] =~ /#{COMMENT_REGEX}\|.*\|/
-          index += 1
-        end
-      end
-    end
-  end
-end
+module CukeSniffer
+
+  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
+  # Copyright:: Copyright (C) 2014 Robert Cochran
+  # License::   Distributes under the MIT License
+  # This class is a representation of the cucumber objects
+  # Background, Scenario, Scenario Outline
+  #
+  # Extends CukeSniffer::FeatureRulesEvaluator
+  class Scenario < FeatureRuleTarget
+
+    xml_accessor :start_line
+    xml_accessor :steps, :as => [], :in => "steps"
+    xml_accessor :examples_table, :as => [], :in => "examples"
+
+    # int: Line on which the scenario begins
+    attr_accessor :start_line
+
+    # string: The type of scenario
+    # Background, Scenario, Scenario Outline
+    attr_accessor :type
+
+    # string array: List of each step call in a scenario
+    attr_accessor :steps
+
+    # hash: Keeps each location and content of an inline table
+    # * Key: Step string the inline table is attached to
+    # * Value: Array of all of the lines in the table
+    attr_accessor :inline_tables
+
+    # string array: List of each example row in a scenario outline
+    attr_accessor :examples_table
+
+    # Location must be in the format of "file_path\file_name.rb:line_number"
+    # Scenario must be a string array containing everything from the first tag to the last example table
+    # where applicable.
+    def initialize(location, scenario)
+      super(location)
+      @start_line = location.match(/:(?<line>\d*)$/)[:line].to_i
+      @steps = []
+      @inline_tables = {}
+      @examples_table = []
+      split_scenario(scenario)
+    end
+
+    def ==(comparison_object) # :nodoc:
+      super(comparison_object) &&
+          comparison_object.steps == steps &&
+          comparison_object.examples_table == examples_table
+    end
+
+    def get_step_order
+      order = []
+      @steps.each do |line|
+        next if is_comment?(line)
+        match = line.match(STEP_REGEX)
+        order << match[:style] unless match.nil?
+      end
+      order
+    end
+
+    def outline?
+      type === 'Scenario Outline'
+    end
+
+    def commented_examples
+      @examples_table.select do |example|
+        is_comment?(example)
+      end
+    end
+
+    def get_steps(step_start)
+      if step_start != "*"
+        regex = /^\s*#{step_start}/
+      else
+        regex = /^\s*[*]/
+      end
+      @steps.select do |step|
+        step =~ regex
+      end
+    end
+
+    private
+
+    def split_scenario(scenario)
+      index = 0
+      until index >= scenario.length or scenario[index] =~ SCENARIO_TITLE_STYLES
+        update_tag_list(scenario[index])
+        index += 1
+      end
+
+      until index >= scenario.length or scenario[index].match STEP_REGEX or scenario[index].include?("Examples:")
+        match = scenario[index].match(SCENARIO_TITLE_STYLES)
+        @type = match[:type] unless match.nil?
+        create_name(scenario[index], SCENARIO_TITLE_STYLES)
+        index += 1
+      end
+
+      until index >= scenario.length or scenario[index].include?("Examples:")
+        if scenario[index] =~ /^\|.*\|/
+          step = scenario[index - 1]
+          @inline_tables[step] = []
+          until index >= scenario.length or scenario[index] =~ /(#{STEP_REGEX}|^\s*Examples:)/
+            @inline_tables[step] << scenario[index]
+            index += 1
+          end
+        else
+          @steps << scenario[index] if scenario[index] =~ STEP_REGEX
+          index += 1
+        end
+      end
+
+      if index < scenario.length and scenario[index].include?("Examples:")
+        index += 1
+        until index >= scenario.length
+          index += 2 if scenario[index].include?("Examples:")
+          @examples_table << scenario[index] if scenario[index] =~ /#{COMMENT_REGEX}\|.*\|/
+          index += 1
+        end
+      end
+    end
+  end
+end