cucumber_lint 0.0.3 → 0.0.4

data/features/cucumber_lint/steps_formatter.feature

@@ -1,30 +0,0 @@
-Feature: table formatter
-
-  Scenario: unformatted steps
-    Given steps
-      """
-      Given A
-      Given B
-      When C
-      Then D
-      Then E
-      """
-    Then the formatted steps are
-      """
-      Given A
-      And B
-      When C
-      Then D
-      And E
-      """
-
-  Scenario: formatted steps
-    Given steps
-      """
-      Given A
-      And B
-      When C
-      Then D
-      And E
-      """
-    Then the steps are formatted

data/features/cucumber_lint/table_formatter.feature

@@ -1,45 +0,0 @@
-Feature: table formatter
-
-  Scenario: minimal table
-    Given table lines
-      """
-      |header_column1|header_column2|
-      |row1_column1|row1_column2|
-      |row2_column1|row2_column2|
-      |row3_column1|row3_column2|
-      """
-    Then the formatted table lines are
-      """
-      | header_column1 | header_column2 |
-      | row1_column1   | row1_column2   |
-      | row2_column1   | row2_column2   |
-      | row3_column1   | row3_column2   |
-      """
-
-
-  Scenario: misaligned spacing in table
-    Given table lines
-      """
-      | header_column1 | header_column2      |
-      | row1_column1 | row1_column2           |
-      | row2_column1      | row2_column2    |
-      | row3_column1 | row3_column2      |
-      """
-    Then the formatted table lines are
-      """
-      | header_column1 | header_column2 |
-      | row1_column1   | row1_column2   |
-      | row2_column1   | row2_column2   |
-      | row3_column1   | row3_column2   |
-      """
-
-
-  Scenario: formatted table
-    Given table lines
-      """
-      | header_column1 | header_column2 |
-      | row1_column1   | row1_column2   |
-      | row2_column1   | row2_column2   |
-      | row3_column1   | row3_column2   |
-      """
-    Then the table is formatted

data/features/step_definitions/fixtures/formatted.feature.example

@@ -1,9 +0,0 @@
-Feature: Test Feature
-
-  Scenario: Test Scenario
-    Given a table
-      | header_column1 | header_column2 |
-      | row1_column1   | row1_column2   |
-      | row2_column1   | row2_column2   |
-      | row3_column1   | row3_column2   |
-    Then my tests pass

data/features/step_definitions/fixtures/unformatted.feature.example

@@ -1,9 +0,0 @@
-Feature: Test Feature
-
-  Scenario: Test Scenario
-    Given a table
-      |header_column1|header_column2|
-      |row1_column1|row1_column2|
-      |row2_column1|row2_column2|
-      |row3_column1|row3_column2|
-    Then my tests pass

data/features/step_definitions/steps_formatter_steps.rb

@@ -1,15 +0,0 @@
-Given(/^steps$/) do |steps|
-  @steps = steps.split("\n", -1)
-end
-
-
-Then(/^the formatted steps are$/) do |steps|
-  expected = steps.split("\n", -1)
-  actual = CucumberLint::StepsFormatter.new(@steps).formatted_content
-  expect(actual).to eql expected
-end
-
-
-Then(/^the steps are formatted$/) do
-  expect(CucumberLint::StepsFormatter.new(@steps)).to be_formatted
-end

data/features/step_definitions/table_formatter_steps.rb

@@ -1,15 +0,0 @@
-Given(/^table lines$/) do |lines|
-  @lines = lines.split("\n", -1)
-end
-
-
-Then(/^the formatted table lines are$/) do |lines|
-  expected = lines.split("\n", -1)
-  actual = CucumberLint::TableFormatter.new(@lines).formatted_content
-  expect(actual).to eql expected
-end
-
-
-Then(/^the table is formatted$/) do
-  expect(CucumberLint::TableFormatter.new(@lines)).to be_formatted
-end

data/lib/core_ext/array.rb

@@ -1,22 +0,0 @@
-# Monkey-patching Array
-class Array
-
-  def group
-    groups = []
-    last_group = { passing: true, values: [] }
-
-    each do |element|
-      passing = yield element
-
-      if passing != last_group[:passing]
-        groups << last_group unless last_group[:values].empty?
-        last_group = { passing: passing, values: [] }
-      end
-
-      last_group[:values] << element
-    end
-
-    groups + [last_group]
-  end
-
-end

data/lib/cucumber_lint/feature_formatter.rb

@@ -1,48 +0,0 @@
-require 'core_ext/array'
-
-module CucumberLint
-  # A class that takes in the content that make up a cucumber feature
-  # and can determine if it is formatted and output the formatted content
-  class FeatureFormatter
-
-    attr_reader :formatted_content
-
-    def initialize content
-      @content = content
-      @formatted_content = format
-    end
-
-
-    def formatted?
-      @content == @formatted_content
-    end
-
-
-    private
-
-
-    def format
-      lines = format_steps @content.split("\n", -1)
-      groupings = lines.group { |line| line =~ /^\s*\|/ }
-
-      groupings.map do |group|
-        if group[:passing]
-          format_table group[:values]
-        else
-          group[:values]
-        end
-      end.flatten.join("\n")
-    end
-
-
-    def format_table lines
-      TableFormatter.new(lines).formatted_content
-    end
-
-
-    def format_steps steps
-      StepsFormatter.new(steps).formatted_content
-    end
-
-  end
-end

data/lib/cucumber_lint/steps_formatter.rb

@@ -1,50 +0,0 @@
-module CucumberLint
-  # A class that takes in the lines of a cucumber feature
-  # and can determine if the steps are formatted and output the formatted steps
-  class StepsFormatter
-
-    attr_reader :formatted_content
-
-    def initialize lines
-      @lines = lines
-      @formatted_content = format
-    end
-
-
-    def formatted?
-      @lines == @formatted_content
-    end
-
-    private
-
-    RESET_KEYWORDS = %w(Background Scenario)
-    STEP_TYPES = %w(Given When Then)
-
-    def format
-      @lines.map { |line| format_line line }
-    end
-
-
-    def format_line line
-      @previous_step_type = nil if should_reset_previous_step_type? line
-
-      step_type = line.split(' ', -1)[0]
-
-      if STEP_TYPES.include?(step_type)
-        if @previous_step_type == step_type
-          line.sub! step_type, 'And'
-        else
-          @previous_step_type = step_type
-        end
-      end
-
-      line
-    end
-
-
-    def should_reset_previous_step_type? line
-      RESET_KEYWORDS.any? { |keyword| line.start_with? keyword }
-    end
-
-  end
-end

data/lib/cucumber_lint/table_formatter.rb

@@ -1,65 +0,0 @@
-module CucumberLint
-  # A class that takes in the lines that make up a cucumber table
-  # and can determine if it is formatted and output the formatted table
-  class TableFormatter
-
-    attr_reader :formatted_content
-
-    def initialize lines
-      @lines = lines
-      @column_widths = column_widths
-      @formatted_content = format
-    end
-
-
-    def formatted?
-      @lines == @formatted_content
-    end
-
-
-    private
-
-
-    def column_widths
-      widths_by_line = @lines.map { |line| line_column_widths line }
-      grouped_widths = widths_by_line[0].zip(*widths_by_line[1..-1])
-      grouped_widths.each_with_index.map do |width, i|
-        i == 0 ? width.min : width.max
-      end
-    end
-
-
-    def format
-      @lines.each_with_index.map do |line|
-        line.split('|', -1).each_with_index.map do |piece, index|
-          format_piece piece, index
-        end.join('|')
-      end
-    end
-
-
-    def format_piece piece, index
-      if index == 0
-        ' ' * column_widths[index]
-      elsif index == column_widths.length - 1
-        piece
-      else
-        " #{piece.strip} ".ljust column_widths[index]
-      end
-    end
-
-
-    def line_column_widths line
-      pieces = line.split('|', -1)
-      pieces.each_with_index.map do |piece, index|
-        if index == 0 || index == pieces.length - 1
-          piece.length
-        else
-          piece.strip!
-          piece.length + 2 # one space padding on each side
-        end
-      end
-    end
-
-  end
-end

data/spec/core_ext/array_spec.rb

@@ -1,19 +0,0 @@
-require 'core_ext/array'
-
-describe Array do
-  describe '#group' do
-    it 'returns elements grouped by whether of not they pass the given block' do
-      array = [1, 1, 2, 3, 5, 8, 13, 21, 34]
-      expected = array.group(&:odd?)
-      actual = [
-        { passing: true, values: [1, 1] },
-        { passing: false, values: [2] },
-        { passing: true, values: [3, 5] },
-        { passing: false, values: [8] },
-        { passing: true, values: [13, 21] },
-        { passing: false, values: [34] }
-      ]
-      expect(expected).to eql actual
-    end
-  end
-end