condition_coverage 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b23a1bb5174201acad8b95876d179f5d7e818bea50eeca8799e189e41ae4014
-  data.tar.gz: 0527c568699e2fb994438308eeeafaf7b1183a3cbbb632d85efe60e7c482acbd
+  metadata.gz: e0b1aba7a743838c35d3b4d139b40efebafd48edef44b82ffc48f7c045eb6861
+  data.tar.gz: '074496c6beaf83b9c6ec438019eb20e7b4f8231230ba94cd5e7a87cc2dc01eb1'
 SHA512:
-  metadata.gz: 0d7593c42bfeb6e8a319e92740786829d7be675e5c2c66a9b96ced249cc811c5a8dc3ae0cae05d40afec05d650fc0d11e75db4171924a25435c88750967d1a41
-  data.tar.gz: 7831a347fae127eb253f7d806eb2d7ba3594718266d952da88e0030fd66027ab09d4936598efb04a23eef53c0ed070c082216aa4491964ae4ea96c3991652839
+  metadata.gz: debb313edef170467b8146405bc25f212b852798c19492b20699a111d255c05015ade7357a90cc9666e9a5642674fa31a0dcea3f930978bd2ef8bad2b37cab04
+  data.tar.gz: e90aa8e71930f57e5e50b5f70155d7166ace8741f300abb7092bafbb3d47bafaaa3ce409cfe7102fc1795dfe7cea5224961a8497e027a617351b371010d83432

data/README.md

@@ -3,7 +3,7 @@
 Condition coverage for Ruby, to run alongside SimpleCov's line and branch
 coverage.
 
-Status: 0.0.2 covers `&&`, `||`, `and` and `or`: each leaf operand of a
+Status: 0.0.3 covers `&&`, `||`, `and` and `or`: each leaf operand of a
 condition, however its operators are chained, nested or grouped. Decisions
 hidden inside calls come next (see Roadmap).
 
@@ -96,11 +96,15 @@ the file:
 ```ruby
 ok?(id) && new?(id)
 # compiles as
-ConditionCoverage.record("k.rb:2:2", 0, (ok?(id))) && ConditionCoverage.record("k.rb:2:2", 1, (new?(id)))
+ConditionCoverage.record("k.rb:2:2", 0, ok?(id)) && ConditionCoverage.record("k.rb:2:2", 1, new?(id))
 ```
 
-Each operand gets parentheses of its own, since one such as `ok? id` (after
-`and`) is not valid as a bare argument.
+A command call such as `ok? id` (after `and`) is not valid as a bare
+argument, so that operand alone gets parentheses of its own. No other operand
+does: parentheses make a statement, and a statement on a continuation line
+gives that line a line-coverage count of its own, which would change
+SimpleCov's line totals. With that one exception, an instrumented file has
+the same relevant lines as an uninstrumented one.
 
 `record` counts whether the operand was truthy or falsy and returns it
 unchanged, so short-circuiting and the value of the expression are

data/lib/condition_coverage/condition.rb

@@ -1,8 +1,14 @@
 module ConditionCoverage
   # One site's condition as the manifest found it: its operator
   # and its operands, each the byte range of a leaf or a nested
-  # Condition.
-  Condition = Data.define(:operator, :operands) do
+  # Condition. A site's commands are the byte ranges of its
+  # leaves that are command calls, eg `ok? id`.
+  Condition = Data.define(:operator, :operands, :commands) do
+    # A condition with no command-call leaves by default.
+    def initialize(operator:, operands:, commands: [])
+      super
+    end
+
     # The byte range of every leaf operand, left to right; a
     # leaf's index here is the index it is recorded under.
     def leaves

data/lib/condition_coverage/kernel.rb

@@ -1,3 +1,4 @@
+require 'fileutils'
 require_relative 'configuration'
 require_relative 'gaps'
 require_relative 'html_report'
@@ -73,10 +74,12 @@ module ConditionCoverage
   end
 
   # Writes the JSON report, then runs the chosen reporters, for
-  # every covered file whether or not the run loaded it.
+  # every covered file whether or not the run loaded it, into
+  # coverage_dir, created first if need be.
   def self.report(config)
     files = parsed_files(config)
     result = Result.new(merged_sites(files), @recorder.counts)
+    FileUtils.mkdir_p(config.output_dir)
     JsonWriter.write(result, config.json_path)
     run_reporters(result, files, config)
   end

data/lib/condition_coverage/manifest.rb

@@ -19,6 +19,7 @@ module ConditionCoverage
       def initialize(path)
         @path = path
         @sites = {}
+        @commands = []
         super()
       end
 
@@ -43,7 +44,11 @@ module ConditionCoverage
       def add_site(node)
         site = Site.of(@path, node)
         @sites[site] = nil
-        @sites[site] = condition_of(node)
+        outer = @commands
+        @commands = []
+        condition = condition_of(node)
+        @sites[site] = condition.with(commands: @commands)
+        @commands = outer
       end
 
       # node as a Condition, operands and all, named by its
@@ -72,11 +77,21 @@ module ConditionCoverage
       end
 
       # The byte range of a leaf operand, once any site inside
-      # it is noted too.
+      # it is noted too, and the range among its site's commands
+      # if it is a command call.
       def leaf(operand)
         operand.accept(self)
         loc = operand.location
-        loc.start_offset...loc.end_offset
+        range = loc.start_offset...loc.end_offset
+        @commands << range if command?(operand)
+        range
+      end
+
+      # True for a call to a named method with arguments and no
+      # parentheses, eg `id.include? 'N'`, but not `id.size == 6`.
+      def command?(node)
+        node.type == :call_node && node.arguments &&
+          node.opening_loc.nil? && node.name.match?(/\A[a-z_]/i)
       end
 
       # The one statement inside a parenthesised operand, or the

data/lib/condition_coverage/rewriter.rb

@@ -9,18 +9,21 @@ module ConditionCoverage
     def self.rewrite(source, sites)
       inserts = sites.flat_map do |site, condition|
         condition.leaves.each_with_index.flat_map do |range, index|
-          wrap(site, index, range)
+          command = condition.commands.include?(range)
+          wrap(site, index, range, command)
         end
       end
       apply(source.dup, inserts)
     end
 
-    # The two inserts that wrap one operand in a record call.
-    # The operand gets parentheses of its own, since one such
-    # as `ok? id` is not valid as a bare argument.
-    def self.wrap(site, index, range)
-      call = "ConditionCoverage.record(#{site.inspect}, #{index}, ("
-      [[range.begin, call], [range.end, '))']]
+    # The two inserts that wrap one operand in a record call. A
+    # command call is parenthesised, as it is no bare argument;
+    # nothing else is, since parentheses make a statement, which
+    # gives its line a line-coverage count of its own.
+    def self.wrap(site, index, range, command)
+      open, close = command ? ['(', '))'] : ['', ')']
+      call = "ConditionCoverage.record(#{site.inspect}, #{index}, "
+      [[range.begin, call + open], [range.end, close]]
     end
 
     # Splices each [offset, text] insert into source,

data/lib/condition_coverage/version.rb

@@ -1,3 +1,3 @@
 module ConditionCoverage
-  VERSION = '0.0.2'.freeze
+  VERSION = '0.0.3'.freeze
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: condition_coverage
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Jon Jagger
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-10-31 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: prism
@@ -26,7 +25,6 @@ dependencies:
         version: '1.0'
 description: Records whether each operand of every &&, ||, and and or condition was
   seen both true and false, and reports the gaps in the terminal and as HTML.
-email:
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -62,7 +60,6 @@ licenses:
 - BSD-2-Clause
 metadata:
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -77,8 +74,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
-signing_key:
+rubygems_version: 4.0.23
 specification_version: 4
 summary: Condition coverage for Ruby, alongside SimpleCov branch coverage
 test_files: []