feature_map 1.2.4 → 1.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '097b75ea1f00eafc127a2e43452ae185b1e3682d209c39ec96ab07c3ef75fe20'
-  data.tar.gz: eefa91bf65f4c9abcbddc8924bc80597596522f9d9a489ff60d02ed21a7ab77e
+  metadata.gz: 735b90dee1fa2dbc01d10b57bb2b53a8350edb3465c04e11760f19f6613893ce
+  data.tar.gz: f1bdfbdd9cca2a4f384b60c1598ef3fc0add164607cb51cd7aadc9e06fda1407
 SHA512:
-  metadata.gz: 3ae23faf8405ea7436cdd10cc20c08673bff4447997eef2849ba3e2261c9edf50be6715ccbe0829adfe099fd91c7f18e829d29dd11aefe8238ee5bc126fa55a2
-  data.tar.gz: 43c0a7372afa58dbb1bc301f2331ef9f6fe51d466d90e9223953ee7050548251ad6512dd7e62b5d41fe266280e6d1d02735651a468bb10db76bab208eaaeaab3
+  metadata.gz: 3d763d8fa726924f6d90653b748ae50d8d82849de44f20cc2de72affa2ddddd20d537d2dab2079767a72cdc33bd7fd6442c3d3cb93275b8ddfd34392001182b7
+  data.tar.gz: 7c21f6d7e0d9e1da75fd0ec638041e20c0be5997af75ed0fb8ac49be213c7066b2c2bf488f84939c6b0eb741435325946859c66bc5fd14d3b91a13c90c7832e4

data/README.md

@@ -42,13 +42,13 @@ The documentation site is a React application which is built on the Vite framewo
 
 Compilation of the build asset is done via `npm run build` from within the [docs](./docs) folder.  This compiles the React app into a single static file which is placed in [./lib/feature_map/private/docs/index.html](./lib/feature_map/private/docs/index.html]).
 
-The documentation site may be run locally to aid in development via `bin/docs`.  It uses sample data found in [docs/src/data/sample_config.js](./docs/src/data/sample_config.js).
+The documentation site may be run locally to aid in development via `bin/docs`.  It will generate test coverage and metrics data, and make it available to the docs site running in development mode.
 
 More information on the development of the documentation site may be found in the [Docs Readme](./docs/README.md).
 
 ### README Site
 
-The README site is built with Jekyll and TailwindCSS and is hosted via GitHub Pages at:  https://beyond-finance.github.io/feature_map.  It can be run locally to aid in development via `bin/readme`.
+The README site is built with Jekyll and TailwindCSS and is hosted via GitHub Pages at: [https://beyond-finance.github.io/feature_map](https://beyond-finance.github.io/feature_map/).  It can be run locally to aid in development via `bin/readme`.
 
 ### Publication
 

data/bin/docs

@@ -5,5 +5,23 @@ if ! gem list foreman -i --silent; then
   gem install foreman
 fi
 
+COVERAGE_FILE="coverage/.resultset.json"
+
+if [ ! -f $COVERAGE_FILE ] || [ $(find $COVERAGE_FILE -mtime +7 -print 2>/dev/null) ]; then
+  echo "###############################################"
+  echo "# No test coverage file found.  Regenerating. #"
+  echo "###############################################"
+
+  bundle exec rspec
+  bundle exec featuremap test_coverage --use-simplecov --simplecov-path $COVERAGE_FILE
+else
+  echo "#############################################################"
+  echo "# Using recent coverage file.  If you'd like to regenerate: #"
+  echo "# > rm $COVERAGE_FILE                             #"
+  echo "#############################################################"
+fi
+
+bundle exec featuremap docs
+cat .feature_map/docs/feature-map-config.js | sed 's/window\.FEATURE_MAP_CONFIG = /export const sampleConfig = /' > docs/sample_config.js
 cd docs && npm install && cd ..
 exec foreman start -f Procfile.docs "$@"

data/lib/feature_map/cli.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 require 'optparse'
 require 'pathname'
 require 'fileutils'
@@ -152,30 +153,67 @@ module FeatureMap
     end
 
     def self.test_coverage!(argv)
+      options = {
+        source: :codecov,
+        simplecov_paths: []
+      }
+
       parser = OptionParser.new do |opts|
         opts.banner = <<~MSG
           Usage: bin/featuremap test_coverage [options] [code_cov_commit_sha].
-          Note:  Requires environment variable `CODECOV_API_TOKEN`.
+
+          Options:
+            --use-simplecov             Use SimpleCov instead of CodeCov
+            --simplecov-path PATH       Path to a SimpleCov resultset.json file (can be specified multiple times)
+
+          Note:#{'  '}
+            - CodeCov mode requires environment variable `CODECOV_API_TOKEN`
+            - CodeCov mode uses the provided commit SHA or defaults to the latest commit on main
+            - SimpleCov mode requires at least one path to be specified with --simplecov-path
+            - SimpleCov and CodeCov modes cannot be used together
         MSG
 
+        opts.on('--use-simplecov', 'Use SimpleCov instead of CodeCov') do
+          options[:source] = :simplecov
+        end
+
+        opts.on('--simplecov-path PATH', 'Use SimpleCov JSON resultset file instead of CodeCov.  May be specified multiple times.') do |path|
+          options[:simplecov_paths] << path
+        end
+
         opts.on('--help', 'Shows this prompt') do
           puts opts
           exit
         end
       end
+
       args = parser.order!(argv)
       parser.parse!(args)
       non_flag_args = argv.reject { |arg| arg.start_with?('--') }
       custom_commit_sha = non_flag_args[0]
 
-      code_cov_token = ENV.fetch('CODECOV_API_TOKEN', '')
-      raise 'Please specify a CodeCov API token in your environment as `CODECOV_API_TOKEN`' if code_cov_token.empty?
+      case options[:source]
+      when :codecov
+        code_cov_token = ENV.fetch('CODECOV_API_TOKEN', '')
+        raise 'Please specify a CodeCov API token in your environment as `CODECOV_API_TOKEN`' if code_cov_token.empty?
+
+        # If no commit SHA was provided in the CLI command args, use the most recent commit of the main branch in the upstream remote.
+        commit_sha = custom_commit_sha || `git log -1 --format=%H origin/main`.chomp
+        puts "Pulling test coverage statistics for commit #{commit_sha}"
 
-      # If no commit SHA was providid in the CLI command args, use the most recent commit of the main branch in the upstream remote.
-      commit_sha = custom_commit_sha || `git log -1 --format=%H origin/main`.chomp
-      puts "Pulling test coverage statistics for commit #{commit_sha}"
+        FeatureMap.gather_test_coverage!(commit_sha, code_cov_token)
+      when :simplecov
+        missing_paths = options[:simplecov_paths].reject { |path| File.exist?(path) }
+        raise 'Error: When using --use-simplecov, you must specify at least one path with --simplecov-path.' if options[:simplecov_paths].empty?
+        raise "SimpleCov results file not found: #{missing_paths.join(', ')}" if missing_paths.any?
+        raise 'Error: Cannot specify a commit SHA when using --simplecov. These options are incompatible.' if custom_commit_sha
 
-      FeatureMap.gather_test_coverage!(commit_sha, code_cov_token)
+        puts "Gathering test coverage statistics from SimpleCov files: #{options[:simplecov_paths].join(', ')}"
+
+        FeatureMap.gather_simplecov_test_coverage!(options[:simplecov_paths])
+      else
+        raise 'Invalid source'
+      end
 
       puts OutputColor.green('FeatureMap test coverage statistics collected.')
       puts 'View the resulting test coverage for each feature in .feature_map/test-coverage.yml'

data/lib/feature_map/code_features.rb

@@ -1,3 +1,4 @@
+# @feature Code Features
 # frozen_string_literal: true
 
 require 'yaml'

data/lib/feature_map/commit.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 module FeatureMap
   class Commit
     attr_reader :sha

data/lib/feature_map/configuration.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 module FeatureMap
   class Configuration
     attr_reader :assigned_globs

data/lib/feature_map/constants.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/mapper.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 # frozen_string_literal: true
 
 module FeatureMap

data/lib/feature_map/output_color.rb

@@ -1,3 +1,4 @@
+# @feature Core Library
 # frozen_string_literal: true
 
 module OutputColor

data/lib/feature_map/private/additional_metrics_file.rb

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

data/lib/feature_map/private/assignment_applicator.rb

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

data/lib/feature_map/private/assignment_mappers/feature_definition_assignment.rb

@@ -13,6 +13,13 @@ module FeatureMap
           return @@map_files_to_features if @@map_files_to_features&.keys && @@map_files_to_features.keys.count.positive?
 
           @@map_files_to_features = CodeFeatures.all.each_with_object({}) do |feature, map| # rubocop:disable Style/ClassVars
+            # NOTE:  The FeatureDefinitionAssignment naively assumes that all
+            #        features will have a definition yaml file.  This comes from
+            #        the CodeOwnership implementation which does require these
+            #        files to exist.  This is not true in repositories using the
+            #        feature_definitions.csv style of feature definition.
+            next if feature.config_yml.nil?
+
             map[feature.config_yml] = feature
           end
         end
@@ -27,6 +34,13 @@ module FeatureMap
           return {} if Private.configuration.ignore_feature_definitions
 
           CodeFeatures.all.each_with_object({}) do |feature, map|
+            # NOTE:  The FeatureDefinitionAssignment naively assumes that all
+            #        features will have a definition yaml file.  This comes from
+            #        the CodeOwnership implementation which does require these
+            #        files to exist.  This is not true in repositories using the
+            #        feature_definitions.csv style of feature definition.
+            next if feature.config_yml.nil?
+
             map[feature.config_yml] = feature
           end
         end

data/lib/feature_map/private/assignments_file.rb

@@ -1,3 +1,4 @@
+# @feature Feature Assignment
 # frozen_string_literal: true
 
 require 'code_ownership'

data/lib/feature_map/private/code_cov.rb

@@ -1,3 +1,4 @@
+# @feature Testing Tools
 # frozen_string_literal: true
 
 require 'faraday'

data/lib/feature_map/private/cyclomatic_complexity_calculator.rb

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