feature_map 1.2.4 → 1.2.6

data/lib/feature_map/private/documentation_site.rb

@@ -1,3 +1,4 @@
+# @feature Extension System
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/extension_loader.rb

@@ -1,3 +1,4 @@
+# @feature Extension System
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/feature_assigner.rb

@@ -1,3 +1,4 @@
+# @feature Feature Assignment
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/feature_metrics_calculator.rb

@@ -1,3 +1,4 @@
+# @feature Metrics Calculation
 # frozen_string_literal: true
 
 require 'rubocop'

data/lib/feature_map/private/feature_plugins/assignment.rb

@@ -1,3 +1,4 @@
+# @feature Code Features
 module FeatureMap
   module Private
     module FeaturePlugins

data/lib/feature_map/private/glob_cache.rb

@@ -1,3 +1,4 @@
+# @feature Assignment Mapping
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/health_calculator.rb

@@ -1,3 +1,4 @@
+# @feature Metrics Calculation
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/lines_of_code_calculator.rb

@@ -1,3 +1,4 @@
+# @feature Metrics Calculation
 # frozen_string_literal: true
 
 require 'parser/current'

data/lib/feature_map/private/metrics_file.rb

@@ -1,3 +1,4 @@
+# @feature Metrics Storage
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/percentile_metrics_calculator.rb

@@ -1,3 +1,4 @@
+# @feature Metrics Calculation
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/release_notification_builder.rb

@@ -1,3 +1,4 @@
+# @feature Extension System
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/simple_cov_resultsets.rb

@@ -0,0 +1,30 @@
+# @feature Testing Tools
+class SimpleCovResultsets
+  def self.fetch_coverage_stats(simplecov_resultsets)
+    root_dir = "#{Dir.pwd}/"
+
+    simplecov_resultsets.reduce({}) do |combined_stats, simplecov_resultset|
+      coverage_data = simplecov_resultset['RSpec']['coverage']
+
+      file_stats = coverage_data.map do |absolute_path, file_data|
+        relative_path = absolute_path.sub(root_dir, '')
+        lines_data = file_data['lines']
+
+        executable_lines = lines_data.compact.size
+        covered_lines = lines_data.count { |line| line&.positive? }
+        missed_lines = executable_lines - covered_lines
+        coverage_percentage = executable_lines.positive? ? ((covered_lines.to_f / executable_lines) * 100).round(2) : 0
+
+        # Return a tuple of [path, stats]
+        [relative_path, {
+          'lines' => executable_lines,
+          'hits' => covered_lines,
+          'misses' => missed_lines,
+          'coverage_ratio' => coverage_percentage
+        }]
+      end
+
+      combined_stats.merge(file_stats.to_h)
+    end
+  end
+end

data/lib/feature_map/private/test_coverage_file.rb

@@ -1,3 +1,4 @@
+# @feature Testing Tools
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private/test_pyramid/jest_mapper.rb

@@ -0,0 +1,44 @@
+module FeatureMap
+  module Private
+    module TestPyramid
+      class JestMapper
+        class << self
+          def map_tests_by_assignment(test_suites, assignments)
+            # Transform test suites into a hash of filepath => assertion results
+            tests_by_path = test_suites.each_with_object({}) do |suite, result|
+              result[filepath(suite['name'])] = suite['assertionResults']
+            end
+
+            assignments.each_with_object({}) do |(feature_name, files), result|
+              counts = count_tests_for_feature(tests_by_path, files)
+              result[feature_name] = counts
+            end
+          end
+
+          private
+
+          def count_tests_for_feature(tests_by_path, files)
+            files.each_with_object({ count: 0, pending: 0 }) do |file, counts|
+              file_path = filepath(file)
+              assertions = tests_by_path[file_path]
+              next unless assertions
+
+              passed, pending = assertions.partition { |assertion| !%w[pending skipped todo].include?(assertion['status']) }
+              counts[:count] += passed.size
+              counts[:pending] += pending.size
+            end
+          end
+
+          def filepath(pathlike)
+            # Get the base directory of the script execution
+            project_root = Dir.pwd
+            path = File.join(File.dirname(pathlike), File.basename(pathlike, '.*'))
+
+            # Strip absolute path prefix, keeping path relative to the project root
+            path.gsub("#{project_root}/", '')
+          end
+        end
+      end
+    end
+  end
+end

data/lib/feature_map/private/test_pyramid/mapper.rb

@@ -0,0 +1,33 @@
+module FeatureMap
+  module Private
+    module TestPyramid
+      class Mapper
+        class << self
+          def examples_by_feature(examples_path, assignments)
+            return {} if examples_path =~ /.skip$/
+
+            examples_file = File.read(examples_path)
+            normalized_assignments = assignments.transform_values { |feature| feature['files'] || [] }
+
+            case examples_path
+            when /\.rspec$/
+              examples = JSON.parse(examples_file)&.fetch('examples', [])
+              TestPyramid::RspecMapper.map_tests_by_assignment(
+                examples,
+                normalized_assignments
+              )
+            when /\.jest$/
+              examples = JSON.parse(examples_file)&.fetch('testResults', [])
+              TestPyramid::JestMapper.map_tests_by_assignment(
+                examples,
+                normalized_assignments
+              )
+            else
+              raise "Unhandled filetype for unit path: #{examples_path}"
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/feature_map/private/test_pyramid/rspec_mapper.rb

@@ -0,0 +1,54 @@
+module FeatureMap
+  module Private
+    module TestPyramid
+      class RspecMapper
+        class << self
+          def map_tests_by_assignment(examples, assignments)
+            normalized_assignments = transform_assignments(assignments)
+            examples_by_file = examples.group_by { |ex| filepath(ex['id']) }
+
+            normalized_assignments.transform_values do |files|
+              count_example_files(examples_by_file, files)
+            end
+          end
+
+          private
+
+          def count_example_files(examples_by_file, files)
+            files.each_with_object({ count: 0, pending: 0 }) do |file, counts|
+              file_examples = examples_by_file[file]
+              next unless file_examples
+
+              passed, pending = file_examples.partition { |ex| ex['status'] == 'passed' }
+              counts[:count] += passed.size
+              counts[:pending] += pending.size
+            end
+          end
+
+          # NOTE:  We normalize paths to remove the app/ and spec/ prefix,
+          #        as well as the _spec suffix.  This allows for spec files to
+          #        be mapped to feature-assigned app-paths such that the specs
+          #        do not need to be directly assigned to features.  It still
+          #        works if they are directly assigned to features, too.
+          def filepath(pathlike)
+            File
+              .join(File.dirname(pathlike), File.basename(pathlike, '.*'))
+              .gsub(%r{^\./}, '')
+              .gsub(%r{^spec/}, '')
+              .gsub(%r{^app/}, '')
+              .gsub(/_spec$/, '')
+          end
+
+          def transform_assignments(assignments)
+            assignments.transform_values do |files|
+              [
+                *files,
+                files.map { |f| filepath(f) }
+              ].flatten
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/feature_map/private/test_pyramid_file.rb

@@ -1,3 +1,4 @@
+# @feature Testing Tools
 # frozen_string_literal: true
 
 module FeatureMap
@@ -12,19 +13,9 @@ module FeatureMap
 
       FEATURES_KEY = 'features'
 
-      def self.write!(unit_examples, integration_examples, regression_examples, regression_assignments)
+      def self.write!(unit_by_feature, integration_by_feature, regression_by_feature)
         FileUtils.mkdir_p(path.dirname) if !path.dirname.exist?
-
-        regression_file_assignments = regression_assignments['features']&.transform_values do |feature|
-          feature['files']&.map { |file| filepath(file) } || []
-        end || {}
-
-        content = generate_content(
-          unit_examples.group_by { |ex| filepath(ex['id']) },
-          integration_examples.group_by { |ex| filepath(ex['id']) },
-          regression_examples.group_by { |ex| filepath(ex['id']) },
-          regression_file_assignments
-        )
+        content = generate_content(unit_by_feature, integration_by_feature, regression_by_feature)
         path.write([header_comment, "\n", { FEATURES_KEY => content }.to_yaml].join)
       end
 
@@ -46,36 +37,15 @@ module FeatureMap
         HEADER
       end
 
-      def self.generate_content(unit_examples, integration_examples, regression_examples, regression_file_assignments)
-        Private.feature_file_assignments.reduce({}) do |content, (feature_name, files)|
-          regression_files = regression_file_assignments[feature_name] || []
-          regression_count, regression_pending = regression_files.reduce([0, 0]) do |accumulated_counts, file|
-            accumulated_count, accumulated_pending = accumulated_counts
-            count, pending = split(regression_examples[file])
-
-            [accumulated_count + count, accumulated_pending + pending]
-          end
-
-          pyramid = files.reduce({}) do |acc, file|
-            normalized_path = filepath(file)
-
-            unit_count, unit_pending = split(unit_examples["#{normalized_path}_spec"])
-            integration_count, integration_pending = split(integration_examples[normalized_path])
-
-            {
-              'unit_count' => (acc['unit_count'] || 0) + unit_count,
-              'unit_pending' => (acc['unit_pending'] || 0) + unit_pending,
-              'integration_count' => (acc['integration_count'] || 0) + integration_count,
-              'integration_pending' => (acc['integration_pending'] || 0) + integration_pending
-            }
-          end
-
-          {
-            feature_name => pyramid.merge(
-              'regression_count' => regression_count,
-              'regression_pending' => regression_pending
-            ),
-            **content
+      def self.generate_content(unit_by_feature, integration_by_feature, regression_by_feature)
+        CodeFeatures.all.map(&:name).each_with_object({}) do |feature_name, content|
+          content[feature_name] = {
+            'unit_count' => unit_by_feature.dig(feature_name, :count) || 0,
+            'unit_pending' => unit_by_feature.dig(feature_name, :pending) || 0,
+            'integration_count' => integration_by_feature.dig(feature_name, :count) || 0,
+            'integration_pending' => integration_by_feature.dig(feature_name, :pending) || 0,
+            'regression_count' => regression_by_feature.dig(feature_name, :count) || 0,
+            'regression_pending' => regression_by_feature.dig(feature_name, :pending) || 0
           }
         end
       end
@@ -91,20 +61,6 @@ module FeatureMap
       rescue Errno::ENOENT
         raise FileContentError, "No feature test coverage file found at #{path}. Use `bin/featuremap test_coverage` to generate it and try again."
       end
-
-      def self.split(examples)
-        return [0, 0] if examples.nil?
-
-        examples.partition { |ex| ex['status'] == 'passed' }.map(&:count)
-      end
-
-      def self.filepath(pathlike)
-        File
-          .join(File.dirname(pathlike), File.basename(pathlike, '.*'))
-          .gsub(%r{^\./}, '')
-          .gsub(%r{^spec/}, '')
-          .gsub(%r{^app/}, '')
-      end
     end
   end
 end

data/lib/feature_map/private/todo_inspector.rb

@@ -1,3 +1,4 @@
+# @feature Testing Tools
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/private.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 # frozen_string_literal: true
 
 require 'code_ownership'
@@ -21,7 +22,11 @@ require 'feature_map/private/code_cov'
 require 'feature_map/private/test_coverage_file'
 require 'feature_map/private/test_pyramid_file'
 require 'feature_map/private/additional_metrics_file'
+require 'feature_map/private/simple_cov_resultsets'
 require 'feature_map/private/feature_plugins/assignment'
+require 'feature_map/private/test_pyramid/mapper'
+require 'feature_map/private/test_pyramid/rspec_mapper'
+require 'feature_map/private/test_pyramid/jest_mapper'
 require 'feature_map/private/validations/files_have_features'
 require 'feature_map/private/validations/features_up_to_date'
 require 'feature_map/private/validations/files_have_unique_features'
@@ -103,11 +108,21 @@ module FeatureMap
     end
 
     def self.generate_test_pyramid!(unit_path, integration_path, regression_path, regression_assignments_path)
-      unit_examples = JSON.parse(File.read(unit_path))&.fetch('examples')
-      integration_examples = JSON.parse(File.read(integration_path))&.fetch('examples')
-      regression_examples = regression_path ? JSON.parse(File.read(regression_path))&.fetch('examples') : []
-      regression_assignments = regression_assignments_path ? YAML.load_file(regression_assignments_path) : {}
-      TestPyramidFile.write!(unit_examples, integration_examples, regression_examples, regression_assignments)
+      assignments = AssignmentsFile.load_features!
+      regression_assignments = regression_assignments_path ? YAML.load_file(regression_assignments_path)&.fetch('features') : assignments
+
+      TestPyramidFile.write!(
+        TestPyramid::Mapper.examples_by_feature(unit_path, assignments),
+        TestPyramid::Mapper.examples_by_feature(integration_path, assignments),
+        regression_path ? TestPyramid::Mapper.examples_by_feature(regression_path, regression_assignments) : {}
+      )
+    end
+
+    def self.gather_simplecov_test_coverage!(simplecov_paths)
+      simplecov_resultsets = simplecov_paths.map { |path| JSON.parse(File.read(path)) }
+      coverage_stats = SimpleCovResultsets.fetch_coverage_stats(simplecov_resultsets)
+
+      TestCoverageFile.write!(coverage_stats)
     end
 
     def self.gather_test_coverage!(commit_sha, code_cov_token)

data/lib/feature_map/validator.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 module FeatureMap
   module Validator
     def validation_errors(files:, autocorrect: true, stage_changes: true); end

data/lib/feature_map.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 # frozen_string_literal: true
 
 require 'set'
@@ -97,6 +98,10 @@ module FeatureMap
     Private.generate_test_pyramid!(unit_path, integration_path, regression_path, regression_assignments_path)
   end
 
+  def gather_simplecov_test_coverage!(simplecov_resultsets)
+    Private.gather_simplecov_test_coverage!(simplecov_resultsets)
+  end
+
   def gather_test_coverage!(commit_sha, code_cov_token)
     Private.gather_test_coverage!(commit_sha, code_cov_token)
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: feature_map
 version: !ruby/object:Gem::Version
-  version: 1.2.4
+  version: 1.2.6
 platform: ruby
 authors:
 - Beyond Finance
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-03-07 00:00:00.000000000 Z
+date: 2025-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: code_ownership
@@ -150,6 +150,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -175,9 +189,13 @@ extensions: []
 extra_rdoc_files: []
 files:
 - README.md
+- bin/bundle
 - bin/docs
 - bin/featuremap
+- bin/jekyll
 - bin/readme
+- bin/rspec
+- bin/rubocop
 - lib/feature_map.rb
 - lib/feature_map/cli.rb
 - lib/feature_map/code_features.rb
@@ -210,7 +228,11 @@ files:
 - lib/feature_map/private/metrics_file.rb
 - lib/feature_map/private/percentile_metrics_calculator.rb
 - lib/feature_map/private/release_notification_builder.rb
+- lib/feature_map/private/simple_cov_resultsets.rb
 - lib/feature_map/private/test_coverage_file.rb
+- lib/feature_map/private/test_pyramid/jest_mapper.rb
+- lib/feature_map/private/test_pyramid/mapper.rb
+- lib/feature_map/private/test_pyramid/rspec_mapper.rb
 - lib/feature_map/private/test_pyramid_file.rb
 - lib/feature_map/private/todo_inspector.rb
 - lib/feature_map/private/validations/features_up_to_date.rb