cuke_sniffer 0.0.7 → 0.0.8

data/lib/cuke_sniffer.rb

@@ -1,23 +1,8 @@
-require 'nokogiri'
-require 'roxml'
-
-require 'cuke_sniffer/constants'
-require 'cuke_sniffer/rule_config'
-require 'cuke_sniffer/rule_target'
-require 'cuke_sniffer/feature_rules_evaluator'
-require 'cuke_sniffer/feature'
-require 'cuke_sniffer/scenario'
-require 'cuke_sniffer/step_definition'
-require 'cuke_sniffer/hook'
-require 'cuke_sniffer/rule'
-require 'cuke_sniffer/summary_node'
-require 'cuke_sniffer/rules_evaluator'
-require 'cuke_sniffer/cuke_sniffer_helper'
-require 'cuke_sniffer/summary_helper'
-require 'cuke_sniffer/dead_steps_helper'
-require 'cuke_sniffer/cli'
-require 'cuke_sniffer/formatter'
-
-module CukeSniffer
-
-end
+require 'nokogiri'
+require 'roxml'
+
+require 'cuke_sniffer/cli'
+
+module CukeSniffer
+
+end

data/lib/cuke_sniffer/cli.rb

@@ -1,10 +1,21 @@
-require 'erb'
-require 'roxml'
+require 'cuke_sniffer/constants'
+require 'cuke_sniffer/dead_steps_helper'
+require 'cuke_sniffer/feature'
+require 'cuke_sniffer/hook'
+require 'cuke_sniffer/rule'
+require 'cuke_sniffer/rule_config'
+require 'cuke_sniffer/step_definition'
+require 'cuke_sniffer/summary_node'
+
+require 'cuke_sniffer/cuke_sniffer_helper'
+require 'cuke_sniffer/rules_evaluator'
+require 'cuke_sniffer/summary_helper'
+require 'cuke_sniffer/formatter'
 
 module CukeSniffer
 
   # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
+  # Copyright:: Copyright (C) 2014 Robert Cochran
   # License::   Distributes under the MIT License
   # Mixins: CukeSniffer::Constants, ROXML
   class CLI
@@ -151,10 +162,11 @@ module CukeSniffer
     def catalog_step_calls
       puts "\nCataloging Step Calls: "
       steps = CukeSniffer::CukeSnifferHelper.get_all_steps(@features, @step_definitions)
+      steps_map = build_steps_map(steps)
       @step_definitions.each do |step_definition|
         print '.'
-        calls = steps.find_all { |location, step| step.gsub(STEP_STYLES, "") =~ step_definition.regex }
-        calls.each { |call| step_definition.add_call(call[0], call[1].gsub(STEP_STYLES, "")) }
+        calls = steps_map.find_all {|step, location| step =~ step_definition.regex }
+        step_definition.calls = build_stored_calls_map(calls)
       end
 
       steps_with_expressions = CukeSniffer::CukeSnifferHelper.get_steps_with_expressions(steps)
@@ -286,5 +298,26 @@ module CukeSniffer
       object_list
     end
 
+    def build_stored_calls_map(calls)
+      stored_calls = {}
+      calls.each do |step, locations|
+        locations.each { |location| stored_calls[location] = step}
+      end
+      stored_calls
+    end
+
+    def build_steps_map(steps)
+      calls_map = {}
+      steps.each do |location, step|
+        sanitized_step = step.gsub(STEP_STYLES, "")
+        if(calls_map.keys.include?(sanitized_step))
+          calls_map[sanitized_step] << location
+        else
+          calls_map[sanitized_step] = [location]
+        end
+      end
+      calls_map
+    end
+
   end
 end

data/lib/cuke_sniffer/constants.rb

@@ -1,6 +1,6 @@
 module CukeSniffer
   # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
+  # Copyright:: Copyright (C) 2014 Robert Cochran
   # License::   Distributes under the MIT License
   # A collection of constants that are used throughout the gem
   module Constants

data/lib/cuke_sniffer/cuke_sniffer_helper.rb

@@ -1,6 +1,6 @@
 module CukeSniffer
   # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
+  # Copyright:: Copyright (C) 2014 Robert Cochran
   # License::   Distributes under the MIT License
   # Static class used for aiding cuke_sniffer in various tasks
   class CukeSnifferHelper

data/lib/cuke_sniffer/dead_steps_helper.rb

@@ -1,6 +1,6 @@
 module CukeSniffer
   # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
+  # Copyright:: Copyright (C) 2014 Robert Cochran
   # License::   Distributes under the MIT License
   # Mixins: CukeSniffer::Constants
   # A static class to aid in the identification of dead steps.

data/lib/cuke_sniffer/feature.rb

@@ -1,104 +1,109 @@
-module CukeSniffer
-
-  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
-  # License::   Distributes under the MIT License
-  # Handles feature files and disassembles and evaluates
-  # its components.
-  # Extends CukeSniffer::FeatureRulesEvaluator
-  class Feature < FeatureRuleTarget
-
-    xml_accessor :scenarios, :as => [CukeSniffer::FeatureRuleTarget], :in => "scenarios"
-
-    SCENARIO_TITLE_REGEX = /#{COMMENT_REGEX}#{SCENARIO_TITLE_STYLES}(?<name>.*)/ # :nodoc:
-
-    # Scenario: The background of a Feature, created as a Scenario object
-    attr_accessor :background
-
-    # Scenario array: A list of all scenarios contained in a feature file
-    attr_accessor :scenarios
-
-    # int: Total score from all of the scenarios contained in the feature
-    attr_accessor :scenarios_score
-
-    # int: Total score of the feature and its scenarios
-    attr_accessor :total_score
-
-    # String array: A list of all the lines in a feature file
-    attr_accessor :feature_lines
-
-    # file_name must be in the format of "file_path\file_name.feature"
-    def initialize(file_name)
-      super(file_name)
-      @scenarios = []
-      @scenarios_score = 0
-      @total_score = 0
-      @feature_lines = IO.readlines(file_name)
-      split_feature(file_name, feature_lines) unless @feature_lines == []
-    end
-
-    def ==(comparison_object) # :nodoc:
-      super(comparison_object) &&
-          comparison_object.scenarios == scenarios
-    end
-
-    def update_score
-      @scenarios_score += @background.score unless @background.nil?
-      @scenarios.each { |scenario| @scenarios_score += scenario.score }
-      @total_score = @scenarios_score + @score
-    end
-
-    private
-
-    def split_feature(file_name, feature_lines)
-      index = 0
-      until feature_lines[index].match /Feature:\s*(?<name>.*)/
-        update_tag_list(feature_lines[index])
-        index += 1
-      end
-
-      until index >= feature_lines.length or feature_lines[index].match TAG_REGEX or feature_lines[index].match SCENARIO_TITLE_REGEX
-        create_name(feature_lines[index], "Feature:")
-        index += 1
-      end
-
-      scenario_title_found = false
-      index_of_title = nil
-      code_block = []
-      until index >= feature_lines.length
-        if scenario_title_found and feature_lines[index].match SCENARIO_TITLE_REGEX
-          not_our_code = []
-          code_block.reverse.each do |line|
-            break if line =~ /#{SCENARIO_TITLE_STYLES}|#{STEP_STYLES}|^\|.*\||Examples:/
-            not_our_code << line
-          end
-
-          if not_our_code.empty?
-            add_scenario_to_feature(code_block, index_of_title)
-          else
-            add_scenario_to_feature(code_block[0...(-1 * not_our_code.length)], index_of_title)
-          end
-          scenario_title_found = false
-          code_block = not_our_code.reverse
-        end
-        code_block << feature_lines[index].strip
-        if feature_lines[index].match SCENARIO_TITLE_REGEX
-          scenario_title_found = true
-          index_of_title = "#{file_name}:#{index + 1}"
-        end
-        index += 1
-      end
-      #TODO - Last scenario falling through above logic, needs a fix (code_block related)
-      add_scenario_to_feature(code_block, index_of_title) unless code_block==[]
-    end
-
-    def add_scenario_to_feature(code_block, index_of_title)
-      scenario = CukeSniffer::Scenario.new(index_of_title, code_block)
-      if scenario.type == "Background"
-        @background = scenario
-      else
-        @scenarios << scenario
-      end
-    end
-  end
-end
+require 'cuke_sniffer/feature_rules_evaluator'
+
+require 'cuke_sniffer/scenario'
+
+module CukeSniffer
+
+  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
+  # Copyright:: Copyright (C) 2014 Robert Cochran
+  # License::   Distributes under the MIT License
+  # Handles feature files and disassembles and evaluates
+  # its components.
+  # Extends CukeSniffer::FeatureRulesEvaluator
+  class Feature < FeatureRuleTarget
+
+    xml_accessor :scenarios, :as => [CukeSniffer::FeatureRuleTarget], :in => "scenarios"
+
+    SCENARIO_TITLE_REGEX = /#{COMMENT_REGEX}#{SCENARIO_TITLE_STYLES}(?<name>.*)/ # :nodoc:
+
+    # Scenario: The background of a Feature, created as a Scenario object
+    attr_accessor :background
+
+    # Scenario array: A list of all scenarios contained in a feature file
+    attr_accessor :scenarios
+
+    # int: Total score from all of the scenarios contained in the feature
+    attr_accessor :scenarios_score
+
+    # int: Total score of the feature and its scenarios
+    attr_accessor :total_score
+
+    # String array: A list of all the lines in a feature file
+    attr_accessor :feature_lines
+
+    # file_name must be in the format of "file_path\file_name.feature"
+    def initialize(file_name)
+      super(file_name)
+      @type = "Feature"
+      @scenarios = []
+      @scenarios_score = 0
+      @total_score = 0
+      @feature_lines = IO.readlines(file_name)
+      split_feature(file_name, feature_lines) unless @feature_lines == []
+    end
+
+    def ==(comparison_object) # :nodoc:
+      super(comparison_object) &&
+          comparison_object.scenarios == scenarios
+    end
+
+    def update_score
+      @scenarios_score += @background.score unless @background.nil?
+      @scenarios.each { |scenario| @scenarios_score += scenario.score }
+      @total_score = @scenarios_score + @score
+    end
+
+    private
+
+    def split_feature(file_name, feature_lines)
+      index = 0
+      until index >= feature_lines.length or feature_lines[index].match /Feature:\s*(?<name>.*)/
+        update_tag_list(feature_lines[index])
+        index += 1
+      end
+
+      until index >= feature_lines.length or feature_lines[index].match TAG_REGEX or feature_lines[index].match SCENARIO_TITLE_REGEX
+        create_name(feature_lines[index], "Feature:")
+        index += 1
+      end
+
+      scenario_title_found = false
+      index_of_title = nil
+      code_block = []
+      until index >= feature_lines.length
+        if scenario_title_found and feature_lines[index].match SCENARIO_TITLE_REGEX
+          not_our_code = []
+          code_block.reverse.each do |line|
+            break if line =~ /#{SCENARIO_TITLE_STYLES}|#{STEP_STYLES}|^\|.*\||Examples:/
+            not_our_code << line
+          end
+
+          if not_our_code.empty?
+            add_scenario_to_feature(code_block, index_of_title)
+          else
+            add_scenario_to_feature(code_block[0...(-1 * not_our_code.length)], index_of_title)
+          end
+          scenario_title_found = false
+          code_block = not_our_code.reverse
+        end
+        code_block << feature_lines[index].strip
+        if feature_lines[index].match SCENARIO_TITLE_REGEX
+          scenario_title_found = true
+          index_of_title = "#{file_name}:#{index + 1}"
+        end
+        index += 1
+      end
+      #TODO - Last scenario falling through above logic, needs a fix (code_block related)
+      add_scenario_to_feature(code_block, index_of_title) unless code_block==[]
+    end
+
+    def add_scenario_to_feature(code_block, index_of_title)
+      scenario = CukeSniffer::Scenario.new(index_of_title, code_block)
+      if scenario.type == "Background"
+        @background = scenario
+      else
+        @scenarios << scenario
+      end
+    end
+  end
+end

data/lib/cuke_sniffer/feature_rules_evaluator.rb

@@ -1,56 +1,55 @@
-require 'cuke_sniffer/constants'
-require 'cuke_sniffer/rule_config'
-require 'cuke_sniffer/rule_target'
-
-module CukeSniffer
-
-  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
-  # Copyright:: Copyright (C) 2013 Robert Cochran
-  # License::   Distributes under the MIT License
-  # Parent class for Feature and Scenario objects
-  # holds shared attributes and rules.
-  # Extends CukeSniffer::RuleTarget
-  class FeatureRuleTarget < RuleTarget
-
-    # string array: Contains all tags attached to a Feature or Scenario
-    attr_accessor :tags
-
-    # string: Name of the Feature or Scenario
-    attr_accessor :name
-
-    # Location must be in the format of "file_path\file_name.rb:line_number"
-    def initialize(location)
-      @name = ""
-      @tags = []
-      super(location)
-    end
-
-    def == (comparison_object) # :nodoc:
-      super(comparison_object) &&
-      comparison_object.name == name &&
-      comparison_object.tags == tags
-    end
-
-    private
-
-    def create_name(line, filter)
-      line.gsub!(/#{COMMENT_REGEX}#{filter}/, "")
-      line.strip!
-      @name += " " unless @name.empty? or line.empty?
-      @name += line
-    end
-
-    def update_tag_list(line)
-      comment_start = (line =~ /([^@\w]#)|(^#)/)
-
-      if comment_start
-        line[0...comment_start].split.each { |single_tag| @tags << single_tag }
-        @tags << line[comment_start..line.length].strip
-      else
-        line.split.each { |single_tag| @tags << single_tag }
-      end
-    end
-
-  end
-end
-
+require 'cuke_sniffer/constants'
+require 'cuke_sniffer/rule_target'
+
+module CukeSniffer
+
+  # Author::    Robert Cochran  (mailto:cochrarj@miamioh.edu)
+  # Copyright:: Copyright (C) 2014 Robert Cochran
+  # License::   Distributes under the MIT License
+  # Parent class for Feature and Scenario objects
+  # holds shared attributes and rules.
+  # Extends CukeSniffer::RuleTarget
+  class FeatureRuleTarget < RuleTarget
+
+    # string array: Contains all tags attached to a Feature or Scenario
+    attr_accessor :tags
+
+    # string: Name of the Feature or Scenario
+    attr_accessor :name
+
+    # Location must be in the format of "file_path\file_name.rb:line_number"
+    def initialize(location)
+      @name = ""
+      @tags = []
+      super(location)
+    end
+
+    def == (comparison_object) # :nodoc:
+      super(comparison_object) &&
+      comparison_object.name == name &&
+      comparison_object.tags == tags
+    end
+
+    private
+
+    def create_name(line, filter)
+      line.gsub!(/#{COMMENT_REGEX}#{filter}/, "")
+      line.strip!
+      @name += " " unless @name.empty? or line.empty?
+      @name += line
+    end
+
+    def update_tag_list(line)
+      comment_start = (line =~ /([^@\w]#)|(^#)/)
+
+      if comment_start
+        line[0...comment_start].split.each { |single_tag| @tags << single_tag }
+        @tags << line[comment_start..line.length].strip
+      else
+        line.split.each { |single_tag| @tags << single_tag }
+      end
+    end
+
+  end
+end
+