featureparity 0.0.10 → 0.0.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e4c131ba204a5f2e3b872551963af559ee1cc2ba941eb192228cd261a12c6f48
-  data.tar.gz: 4919cdca3946beec35921271be6998938150085879a3b6c9ed4d8cd4c84aa622
+  metadata.gz: f3bf1014e57b31bb56a73f768b635114ea15621f81986e0f8e94aded5c32c06b
+  data.tar.gz: edc45b069c7cee17cfd382096db52d1a835cbecb3a0af1d19652c4ee242aacef
 SHA512:
-  metadata.gz: 8977bbdb59e48dd3d19878858ebc0e5363ec9be1313b8352054f8227e9a84a99c8b3e7343ae74cb034bc96f2f45fc5700656a5218f46db61abcaa5a32f9fa146
-  data.tar.gz: e37abf6853447467b7b91937f26447d4e2c7b4de5d52bcc3b17ac6c348ce01fbd996e806e0a10add88f04b8c4823af292302a81b6cb81b7eae5b3cb306846caf
+  metadata.gz: 87f36cb76299fc104298a23ed41cb9e6ab82e54fa773f88a8808f32758e77f1c9e7373476e8d843bc7ed0fd1df6fbc7471491cec14217c5a41eb7a7d67613843
+  data.tar.gz: 73f0e7fc3e58db8acebd37ad645e8a2a0dca300b54db51aa31d84a8d5727726c1e4fc24f40dc69b7d4c57959922d2fbc14d97ad50bf2ca2bbb42eff86010e4ba

data/lib/fp/commands/check.rb

@@ -180,6 +180,7 @@ module Fp
 
           print_state_counts(summary)
           print_violations(violations)
+          print_orphans(report)
           print_pending_reviews(report, project_slug)
           print_unscoped_warning(summary, project_slug)
           print_verdict(passed, violations, opts)
@@ -227,6 +228,38 @@ module Fp
         puts
       end
 
+      # Markers the run carried that bind to no requirement in this project — the test
+      # claims to cover something that doesn't exist. FP identifies them; it does not
+      # propose the requirement, because a slug and a test name aren't enough to write a
+      # good one. Whether this fails the build is the server's call (orphans_violate,
+      # true at covered/strict), already folded into `passing` — here we only render it,
+      # as a violation when it counts and a note when it doesn't. Only the --junit path
+      # has a run to find these in; plain `fp check` reads recorded state and shows none.
+      def print_orphans(report)
+        orphans = report['orphans'] || []
+        return if orphans.empty?
+
+        violates = report['orphans_violate']
+        lead = violates ? "#{orphans.size} orphan marker(s) — a violation at this level:" \
+                        : "#{orphans.size} orphan marker(s) (not failing the gate at this level):"
+        puts lead
+        output.table(
+          %w[SLUG SURFACE WHERE],
+          orphans.map { |o| [o['slug'], o['surface'], marker_location(o)] }
+        )
+        puts 'These tests are marked fp:<slug> for a requirement that does not exist. ' \
+             'Either propose it (fp propose --slug <slug> ...) or fix the marker\'s slug — ' \
+             'FP will not invent a requirement from a marker.'
+        puts
+      end
+
+      def marker_location(orphan)
+        file = orphan['source_file']
+        return '—' if file.nil? || file.empty?
+
+        orphan['source_line'] ? "#{file}:#{orphan['source_line']}" : file
+      end
+
       # Informational, never a violation: an active requirement naming no surface this
       # project has can't be satisfied, so it would pass any gate silently. Worth
       # saying out loud without blocking a merge on someone else's data-entry gap.

data/lib/fp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Fp
-  VERSION = '0.0.10'
+  VERSION = '0.0.11'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: featureparity
 version: !ruby/object:Gem::Version
-  version: 0.0.10
+  version: 0.0.11
 platform: ruby
 authors:
 - Stowzilla