tryouts 3.7.0 → 4.0.0.pre1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7178f4f14ad2015fb3e5a795bbdd7813b4a184b56413535d7ddfff569badca9d
-  data.tar.gz: ff451b947073ebfbaece457e3b94cb090410ff86aa8ca09e8fdc8c998e04118c
+  metadata.gz: b43c7d018d4bb7bd546bb516a584903d7922d87a616dbec87d2e4b6d00e06652
+  data.tar.gz: d4068b53bbd8428f88bf1a4d4af62aa7db0ff4ecccfecca0c0b268f1d8fb0381
 SHA512:
-  metadata.gz: d15d3afd82d4cef85ffe9eb481b5bfd68971c2cbc4247f8f6b46fbf53e7292ae6d22d396b1b2070481f0034c12079f8183ff66903a05f6a36dc754a7b33aef55
-  data.tar.gz: 936ecb57906f16befe499de544032e9f1cd25ed50f757652556f357ac80b6ff7d14dbb3787c8ff2eb2d93460f3c59a224eecbc2b491a936038394013976a3b42
+  metadata.gz: c11a368db40e8335a3d9eae238581ad6e0b4ea8b56862bb82f10adbb9746626762db1a89247be6e8eb2473aff90506f151a5b18452ab625d2a6e2f2753968f2d
+  data.tar.gz: 7ebec1c8923c423cd3ec2117d78e3a4a95fec48139bb1af2ecee48ebdfea7d02d1ceccfa8608fb5914cb21c762dfafc04a1f81835fc18723c3fbe79d495540db

data/README.md

@@ -42,7 +42,7 @@ try try/core/basic_syntax_try.rb
 
 ## Writing Tests
 
-Tryouts use a comment-based syntax for expecations:
+Tryouts use a comment-based syntax for expectations:
 
 ```ruby
 # Setup code runs before all tests
@@ -98,9 +98,10 @@ Each test file is made up of three sections:
   | `#=:>` | Must match result type     | `#=:> String`                         | result, _     |
   | `#=~>` | Must match regex pattern   | `#=~> /^[^@]+@[^@]+\.[^@]+$/`         | result, _     |
   | `#=%>` | Must complete within time  | `#=%> 2000 # in milliseconds`         | result, _     |
+  | `#=*>` | Must be non-nil            | `#=*> result`                         | result, _     |
   | `#=1>` | Match content in STDOUT    | `#=1> "You have great success"`       | result, _     |
   | `#=2>` | Match content in STDERR    | `#=2> /[a-zA-Z0-9]+-?[0-9]{1,5}`      | result, _     |
-  | `#=<>` | Fails on purpose           | `#==<> result.include?(4)`            | result, _     |
+  | `#=<>` | Fails on purpose           | `#=<> result.include?(4)`            | result, _     |
 
 
 ### Using other test runners

data/lib/tryouts/cli/formatters/agent.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/agent.rb
+#
+# frozen_string_literal: true
 
 require_relative 'token_budget'
 
@@ -89,10 +91,11 @@ class Tryouts
       end
 
       def file_parsed(_file_path, test_count:, setup_present: false, teardown_present: false)
+        # Store per-file parsed test count for reference
+        # Total executed tests come from TestResultAggregator via grand_total()
         if @current_file_data
           @current_file_data[:tests] = test_count
         end
-        @total_stats[:tests] += test_count
       end
 
       def parser_warnings(file_path, warnings:)
@@ -117,9 +120,7 @@ class Tryouts
       end
 
       def file_result(file_path, total_tests:, failed_count:, error_count:, elapsed_time: nil)
-        # Always update global totals
-        @total_stats[:failures] += failed_count
-        @total_stats[:errors] += error_count
+        # Update elapsed time (other stats come from TestResultAggregator via grand_total)
         @total_stats[:elapsed_time] += elapsed_time if elapsed_time
 
         # Update per-file data - file_result is called AFTER file_end, so data is in @collected_files
@@ -168,24 +169,19 @@ class Tryouts
 
       # Summary operations - reliable trigger for rendering
       def batch_summary(failure_collector)
-        # This becomes the single, reliable trigger for rendering
-        grand_total(
-          total_tests: @total_stats[:tests],
-          failed_count: @collected_files.sum { |f| f[:failures].size },
-          error_count: @collected_files.sum { |f| f[:errors].size },
-          successful_files: @collected_files.size - @collected_files.count { |f| f[:failures].any? || f[:errors].any? },
-          total_files: @collected_files.size,
-          elapsed_time: @total_stats[:elapsed_time]
-        ) unless @output_rendered
+        # NOTE: We don't render here because test_runner.rb calls grand_total directly
+        # with accurate counts from the TestResultAggregator
       end
 
       def grand_total(total_tests:, failed_count:, error_count:, successful_files:, total_files:, elapsed_time:)
         return if @output_rendered  # Prevent double rendering
 
+        # Use the accurate counts from TestResultAggregator (passed as parameters)
+        # NOT the parsed counts we accumulated in file_parsed()
         @total_stats.merge!(
-          tests: total_tests,
-          failures: failed_count,
-          errors: error_count,
+          tests: total_tests,           # Actual executed test count from aggregator
+          failures: failed_count,       # Actual failures from aggregator
+          errors: error_count,          # Actual errors from aggregator
           successful_files: successful_files,
           total_files: total_files,
           elapsed_time: elapsed_time,

data/lib/tryouts/cli/formatters/base.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/base.rb
+#
+# frozen_string_literal: true
 
 require_relative 'output_manager'
 

data/lib/tryouts/cli/formatters/compact.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/compact.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI

data/lib/tryouts/cli/formatters/factory.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/factory.rb
+#
+# frozen_string_literal: true
 
 require_relative '../tty_detector'
 

data/lib/tryouts/cli/formatters/live_status_manager.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/live_status_manager.rb
+#
+# frozen_string_literal: true
 
 require_relative 'test_run_state'
 require_relative 'tty_status_display'

data/lib/tryouts/cli/formatters/output_manager.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/output_manager.rb
+#
+# frozen_string_literal: true
 
 require_relative 'live_status_manager'
 

data/lib/tryouts/cli/formatters/quiet.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/quiet.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI

data/lib/tryouts/cli/formatters/test_run_state.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/test_run_state.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI

data/lib/tryouts/cli/formatters/token_budget.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/token_budget.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI

data/lib/tryouts/cli/formatters/tty_status_display.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/tty_status_display.rb
+#
+# frozen_string_literal: true
 
 require 'tty-cursor'
 require 'tty-screen'

data/lib/tryouts/cli/formatters/verbose.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters/verbose.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI

data/lib/tryouts/cli/formatters.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/formatters.rb
+#
+# frozen_string_literal: true
 
 require_relative 'formatters/base'
 require_relative 'formatters/compact'

data/lib/tryouts/cli/line_spec_parser.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/line_spec_parser.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI

data/lib/tryouts/cli/modes/generate.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/modes/generate.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI

data/lib/tryouts/cli/modes/inspect.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/modes/inspect.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI
@@ -13,14 +15,24 @@ class Tryouts
 
       def handle
         @output_manager.raw("Inspecting: #{@file}")
-        @output_manager.separator(:heavy)
+        @output_manager.raw('=' * 60)
         @output_manager.raw("Found #{@testrun.total_tests} test cases")
         @output_manager.raw("Setup code: #{@testrun.setup.empty? ? 'None' : 'Present'}")
         @output_manager.raw("Teardown code: #{@testrun.teardown.empty? ? 'None' : 'Present'}")
         @output_manager.raw('')
 
-        @testrun.test_cases.each_with_index do |tc, i|
-          @output_manager.raw("Test #{i + 1}: #{tc.description}")
+        test_number = 0
+        @testrun.test_cases.each do |tc|
+          if tc.is_a?(Tryouts::OrphanBlock)
+            @output_manager.raw("Orphan code block (no description, no expectations)")
+            @output_manager.raw("  Code lines: #{tc.code.lines.count}")
+            @output_manager.raw("  Range: #{tc.line_range}")
+            @output_manager.raw('')
+            next
+          end
+
+          test_number += 1
+          @output_manager.raw("Test #{test_number}: #{tc.description}")
           @output_manager.raw("  Code lines: #{tc.code.lines.count}")
           @output_manager.raw("  Expectations: #{tc.expectations.size}")
           @output_manager.raw("  Range: #{tc.line_range}")

data/lib/tryouts/cli/opts.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/opts.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   class CLI
@@ -73,17 +75,18 @@ class Tryouts
         #=1>  STDOUT content        #=2>  STDERR content      #=<>  Intentional failure
 
       Exception Testing:
-        # Method 1: Rescue and test exception
+        ## Method 1: Rescue and test exception
         begin
           risky_operation
-        rescue StandardError => e
+        rescue MySpecificError => e
           e.class
         end
-        #=> StandardError
+        #=> MySpecificError
 
-        # Method 2: Let it raise and test with #=!>
+        ## Method 2: Let it raise and test with #=!> and #=~>
         risky_operation
-        #=!> error.is_a?(StandardError)
+        #=!> MySpecificError
+        #=~> /Could not complete action/
     HELP
 
     class << self
@@ -160,7 +163,6 @@ class Tryouts
 
           opts.separator "\nParser Options:"
           opts.on('--enhanced-parser', 'Use enhanced parser with inhouse comment extraction (default)') { options[:parser] = :enhanced }
-          opts.on('--legacy-parser', 'Use legacy prism parser') { options[:parser] = :prism }
 
           opts.separator "\nInspection Options:"
           opts.on('-i', '--inspect', 'Inspect file structure without running tests') { options[:inspect] = true }

data/lib/tryouts/cli/tty_detector.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli/tty_detector.rb
+#
+# frozen_string_literal: true
 
 require 'tty-screen'
 

data/lib/tryouts/cli.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/cli.rb
+#
+# frozen_string_literal: true
 
 require 'optparse'
 

data/lib/tryouts/console.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/console.rb
+#
+# frozen_string_literal: true
 
 require 'pathname'
 

data/lib/tryouts/expectation_evaluators/base.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/base.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   module ExpectationEvaluators

data/lib/tryouts/expectation_evaluators/boolean.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/boolean.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/exception.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/exception.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/expectation_result.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/expectation_result.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   module ExpectationEvaluators

data/lib/tryouts/expectation_evaluators/false.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/false.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/intentional_failure.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/intentional_failure.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 require_relative 'regular'

data/lib/tryouts/expectation_evaluators/non_nil.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/non_nil.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/output.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/output.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/performance_time.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/performance_time.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/regex_match.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/regex_match.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/registry.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/registry.rb
+#
+# frozen_string_literal: true
 
 require_relative 'expectation_result'
 require_relative 'base'

data/lib/tryouts/expectation_evaluators/regular.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/regular.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/result_type.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/result_type.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/expectation_evaluators/true.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/expectation_evaluators/true.rb
+#
+# frozen_string_literal: true
 
 require_relative 'base'
 

data/lib/tryouts/failure_collector.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/failure_collector.rb
+#
+# frozen_string_literal: true
 
 require_relative 'console'
 

data/lib/tryouts/file_processor.rb

@@ -1,6 +1,7 @@
 # lib/tryouts/file_processor.rb
+#
+# frozen_string_literal: true
 
-require_relative 'parsers/legacy_parser'
 require_relative 'parsers/enhanced_parser'
 require_relative 'test_executor'
 require_relative 'cli/modes/inspect'
@@ -9,7 +10,7 @@ require_relative 'cli/modes/generate'
 class Tryouts
   class FileProcessor
     # Supported parser types for validation and documentation
-    PARSER_TYPES = [:enhanced, :prism].freeze
+    PARSER_TYPES = [:enhanced].freeze
     def initialize(file:, options:, output_manager:, translator:, global_tally:)
       @file           = file
       @options        = options
@@ -52,8 +53,6 @@ class Tryouts
       case parser_type
       when :enhanced
         EnhancedParser.new(file, options)
-      when :prism
-        LegacyParser.new(file, options)
       end
     end
 

data/lib/tryouts/parser_warning.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/parser_warning.rb
+#
+# frozen_string_literal: true
 
 class Tryouts
   # Data structure for parser warnings

data/lib/tryouts/parsers/base_parser.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/parsers/base_parser.rb
+#
+# frozen_string_literal: true
 
 require 'prism'
 
@@ -11,8 +13,7 @@ class Tryouts
     #
     # BaseParser establishes the foundation for parsing tryout files by handling
     # file loading, Prism integration, and providing shared parsing infrastructure.
-    # All concrete parser implementations (EnhancedParser, LegacyParser) inherit
-    # from this class.
+    # All concrete parser implementations (EnhancedParser) inherit from this class.
     #
     # @abstract Subclass and implement {#parse} to create a concrete parser
     # @example Implementing a custom parser
@@ -75,7 +76,6 @@ class Tryouts
     # 4. Provide a unique `parser_type` identifier
     #
     # @see EnhancedParser For Prism-based comment extraction
-    # @see LegacyParser For line-by-line parsing approach
     # @see SharedMethods For common parsing utilities
     # @since 3.0.0
     class BaseParser

data/lib/tryouts/parsers/enhanced_parser.rb

@@ -1,4 +1,6 @@
 # lib/tryouts/parsers/enhanced_parser.rb
+#
+# frozen_string_literal: true
 
 require_relative '../test_case'
 require_relative 'base_parser'
@@ -31,7 +33,7 @@ class Tryouts
   # @!attribute [r] parser_type
   #   @return [Symbol] Returns :enhanced to identify parser type
   #
-  # ## Key Benefits over LegacyParser
+  # ## Key Benefits
   #
   # ### 1. HEREDOC Safety
   # - Uses Prism's `parse_comments()` to extract only actual Ruby comments
@@ -56,7 +58,6 @@ class Tryouts
   # Uses Ruby 3.4+ pattern matching (`case/in`) for token classification,
   # providing clean, modern syntax for expectation type detection.
   #
-  # @see LegacyParser For simpler regex-based parsing (legacy compatibility)
   # @see BaseParser For shared parsing functionality
   # @since 3.2.0
   class EnhancedParser < Tryouts::Parsers::BaseParser
@@ -72,6 +73,12 @@ class Tryouts
     def parse
       return handle_syntax_errors if @prism_result.failure?
 
+      # Line ranges of multi-line statements, computed once from the whole-file
+      # parse. A column-0 comment strictly inside one of these spans (e.g. a
+      # #=>-shaped line inside an unclosed array literal) cannot be a real
+      # block boundary or expectation, no matter what it looks like lexically.
+      @multiline_statement_ranges = multiline_statement_ranges
+
       # Use inhouse comment extraction instead of line-by-line regex parsing
       # This automatically excludes HEREDOC content!
       tokens           = tokenize_content_with_inhouse_extraction
@@ -104,22 +111,41 @@ class Tryouts
       comments        = Prism.parse_comments(@source)
       comment_by_line = comments.group_by { |comment| comment.location.start_line }
 
+      # A =begin/=end block comment is one Prism comment node spanning multiple
+      # lines, but comment_by_line only keys its start line. Its interior lines
+      # must be skipped entirely: re-emitting them as :code duplicates them, and
+      # running the marker regex cascade against the raw multi-line slice lets an
+      # interior #=>-shaped line match via ^/$ line anchors.
+      multiline_comment_spans = comments.filter_map do |comment|
+        loc = comment.location
+        ((loc.start_line + 1)..loc.end_line) if loc.end_line > loc.start_line
+      end
+
       # Process each line, handling multiple comments per line
       @lines.each_with_index do |line, index|
         line_number = index + 1
 
+        next if multiline_comment_spans.any? { |span| span.cover?(line_number) }
+
         if (comments_for_line = comment_by_line[line_number]) && !comments_for_line.empty?
           emitted_code = false
           comments_for_line.sort_by! { |c| c.location.start_column }
           comments_for_line.each do |comment|
             comment_content = comment.slice.strip
-            if comment.location.start_column > 0
+            if comment.location.end_line > comment.location.start_line
+              # Whole =begin/=end block: one plain comment token for the span
+              tokens << { type: :comment, content: comment_content, line: index }
+            elsif comment.location.start_column > 0
               unless emitted_code
                 tokens << { type: :code, content: line, line: index, ast: parse_ruby_line(line) }
                 emitted_code = true
               end
               # Inline comment (after code) - treat as regular comment, not expectation
               tokens << { type: :comment, content: comment_content.sub(/^#\s*/, ''), line: line_number - 1 }
+            elsif strictly_inside_multiline_statement?(index)
+              # Column-0 comment inside a multi-line statement (unclosed
+              # array/hash/call/...) - force plain :comment regardless of shape
+              tokens << { type: :comment, content: comment_content.sub(/^#\s*/, ''), line: line_number - 1 }
             else
               tokens << classify_comment_inhousely(comment_content, line_number)
             end
@@ -140,6 +166,34 @@ class Tryouts
       tokens
     end
 
+    # Collect 0-based, inclusive line ranges for every AST node that spans more
+    # than one physical line, walking the existing whole-file @prism_result.
+    # Built once per parse; no second Prism.parse call.
+    def multiline_statement_ranges
+      ranges = []
+      root   = @prism_result.value
+      return ranges if root.nil?
+
+      visit = lambda do |node|
+        next if node.nil?
+
+        loc = node.location
+        ranges << ((loc.start_line - 1)..(loc.end_line - 1)) if loc.start_line != loc.end_line
+        node.compact_child_nodes.each { |child| visit.call(child) }
+      end
+
+      root.statements.body.each { |stmt| visit.call(stmt) }
+      ranges
+    end
+
+    # True if the 0-based line index falls STRICTLY inside a multi-line
+    # statement span. A comment on the same line as the opening or closing
+    # delimiter is still eligible to be a real boundary; one on a line fully
+    # swallowed by the statement is not.
+    def strictly_inside_multiline_statement?(line_idx)
+      @multiline_statement_ranges.any? { |range| line_idx > range.first && line_idx < range.last }
+    end
+
     # Classify comment content into specific token types using pattern matching
     #
     # Takes a raw comment string and determines what type of tryout token it