rspec_pretty_html_reporter 1.1.1 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fb778b6e59fc1d897dd91626bdc82f763f32010ec6e618d10536a24fa7137260
-  data.tar.gz: 28be25ab79d74a6f363700dde7a7af73953d79ce3289a4dc9a3795267b28c86b
+  metadata.gz: 19e2385411126f381ee6200d7355751a730acda12387aadf622c129be11a0c0d
+  data.tar.gz: 49c8601dbd91f42b5f25debe6c501d3770260416b9502190ad0c51e01c1e0b53
 SHA512:
-  metadata.gz: c65d6d8a1f5fbe9086aa2b028b108e69503c813d8f827fc5646ef668d66e700c28a38b03e1720b2d5c18ca37eed9afec7243ac485f38e4b68be900f091a5e8b6
-  data.tar.gz: bc1ffcc7489649cab6466b0403059e258c0bb43114faa9b5e3cb9a4dff7bcb41b363f92547db592579ed8b8ce19712867e4bc6ca148f8dc68f705cde7bb5c019
+  metadata.gz: c53a7027d3f170a25ac9cb3290944b9939197acf3207ae4929c28026f4bf998b6f0e8eaa8090b92fb7682e9876d630b5f838bf5801f5fc4a1a495d2abb0fa23f
+  data.tar.gz: 28aad8ae1532df88d43fae0ee76d6d68acbadaee57573c586c50a48bdc44f0306d0f526b6c7edfd5b584554def289544d8516090a00c4dc36d03def57d4a682a

data/lib/rspec_pretty_html_reporter/example.rb

@@ -1,5 +1,6 @@
 require 'rspec_pretty_html_reporter/oopsy'
 
+# Processes the array of example information for all of the example group that has been run
 class Example
   def self.load_spec_comments!(examples)
     examples.group_by(&:file_path).each do |file_path, file_examples|
@@ -7,8 +8,8 @@ class Example
 
       file_examples.zip(file_examples.rotate).each do |ex, next_ex|
         lexically_next = next_ex &&
-          next_ex.file_path == ex.file_path &&
-          next_ex.metadata[:line_number] > ex.metadata[:line_number]
+                         next_ex.file_path == ex.file_path &&
+                         next_ex.metadata[:line_number] > ex.metadata[:line_number]
         start_line_idx = ex.metadata[:line_number] - 1
         next_start_idx = (lexically_next ? next_ex.metadata[:line_number] : lines.size) - 1
         spec_lines = lines[start_line_idx...next_start_idx].select { |l| l.match(/#->/) }
@@ -21,7 +22,6 @@ class Example
               :metadata, :spec, :screenshots, :screenrecord, :failed_screenshot
 
   def initialize(example)
-
     @example_group = example.example_group.to_s
     @description = example.description
     @full_description = example.full_description

data/lib/rspec_pretty_html_reporter/helpers.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Helpers
+
+  ##
+  # Appends a number to duplicate filename to avoid them being overwritten
+  # by Specs with the same description.
+  #
+  # Renames duplicate filenames for specs that have the same description
+  # @param [String] filename  Duplicate filename to rename
+  # @return [String]          Renamed filename
+  #
+  def self.rename_duplicate_filename(filename)
+    if File.exist?(filename)
+      base, ext = /\A(.+?)(\.[^.]+)?\Z/.match(filename).to_a[1..]
+      number = 2
+      number += 1 while File.exist?(filename = "#{base}-#{number}#{ext}")
+    end
+    filename
+  end
+
+  ##
+  # Appends a number to a duplicate description to avoid them being overwritten
+  # by Specs with the same description.
+  #
+  # Renames duplicate descriptions for specs that have the same description
+  # @param [Hash] group_name    Group name to check for duplicate keys
+  # @param [String] description Duplicate description to rename
+  # @return [String]            Renamed description
+  #
+  def self.rename_duplicate_description(group_name, description)
+    if group_name.key?(description)
+      number = group_name.select { |k, _v| k.include?(description) }.length
+      description = "#{description}-#{number + 1}"
+    end
+    description
+  end
+end

data/lib/rspec_pretty_html_reporter.rb

@@ -6,10 +6,12 @@ require 'active_support/inflector'
 require 'fileutils'
 require 'erb'
 require 'rspec_pretty_html_reporter/example'
+require 'rspec_pretty_html_reporter/helpers'
 
 I18n.enforce_available_locales = false
 
 class RspecPrettyHtmlReporter < RSpec::Core::Formatters::BaseFormatter
+  include Helpers
   DEFAULT_REPORT_PATH = File.join(Bundler.root, 'reports', Time.now.strftime('%Y%m%d-%H%M%S'))
   REPORT_PATH = ENV['REPORT_PATH'] || DEFAULT_REPORT_PATH
 
@@ -48,8 +50,6 @@ class RspecPrettyHtmlReporter < RSpec::Core::Formatters::BaseFormatter
   end
 
   def example_passed(notification)
-    # require 'byebug'; byebug
-
     @group_example_success_count += 1
     @examples << Example.new(notification.example)
   end
@@ -68,8 +68,9 @@ class RspecPrettyHtmlReporter < RSpec::Core::Formatters::BaseFormatter
     @group_level -= 1
 
     if @group_level.zero?
+      @filename = Helpers.rename_duplicate_filename("#{REPORT_PATH}/#{notification.group.description.parameterize}.html")
 
-      File.open("#{REPORT_PATH}/#{notification.group.description.parameterize}.html", 'w') do |f|
+      File.open(@filename, 'w') do |f|
         @passed = @group_example_success_count
         @failed = @group_example_failure_count
         @pending = @group_example_pending_count
@@ -90,7 +91,6 @@ class RspecPrettyHtmlReporter < RSpec::Core::Formatters::BaseFormatter
 
         @total_group_examples = @passed + @failed + @pending
 
-
         class_map = { passed: 'success', failed: 'danger', pending: 'warning' }
         statuses = @examples.map(&:status)
         @status = if statuses.include?('failed')
@@ -98,7 +98,8 @@ class RspecPrettyHtmlReporter < RSpec::Core::Formatters::BaseFormatter
                   else
                     (statuses.include?('passed') ? 'passed' : 'pending')
                   end
-        @all_groups[notification.group.description.parameterize] = {
+        @spec_description = Helpers.rename_duplicate_description(@all_groups, notification.group.description.parameterize)
+        @all_groups[@spec_description] = {
           group: notification.group.description,
           examples: @examples.size,
           status: @status,
@@ -108,8 +109,7 @@ class RspecPrettyHtmlReporter < RSpec::Core::Formatters::BaseFormatter
           pending: statuses.select { |s| s == 'pending' },
           duration: @summary_duration
         }
-
-        @example_status = @all_groups[notification.group.description.parameterize][:klass]
+        @example_status = @all_groups[@spec_description][:klass]
 
         template_file = File.read("#{File.dirname(__FILE__)}/../templates/report.erb")
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspec_pretty_html_reporter
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.2
 platform: ruby
 authors:
 - Carlos Gutierrez
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-21 00:00:00.000000000 Z
+date: 2022-04-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -24,6 +24,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '7.0'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.3'
 - !ruby/object:Gem::Dependency
   name: rouge
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +58,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: 3.11.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.4'
+        version: 3.11.0
 - !ruby/object:Gem::Dependency
   name: byebug
   requirement: !ruby/object:Gem::Requirement
@@ -66,6 +80,34 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '11.1'
+- !ruby/object:Gem::Dependency
+  name: capybara
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.36'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.36'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -86,14 +128,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.4.0
+        version: 3.11.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.4.0
+        version: 3.11.0
 - !ruby/object:Gem::Dependency
   name: turnip
   requirement: !ruby/object:Gem::Requirement
@@ -114,6 +156,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 2.0.2
+- !ruby/object:Gem::Dependency
+  name: webdrivers
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '5.0'
 description: A custom reporter for RSpec which generates pretty HTML reports
 email:
 - testing@spartan-testsolutions.co.uk
@@ -125,6 +181,7 @@ files:
 - README.md
 - lib/rspec_pretty_html_reporter.rb
 - lib/rspec_pretty_html_reporter/example.rb
+- lib/rspec_pretty_html_reporter/helpers.rb
 - lib/rspec_pretty_html_reporter/oopsy.rb
 - resources/bootstrap-4.0.0-dist/css/bootstrap.min.css
 - resources/bootstrap-4.0.0-dist/js/bootstrap.js