tryouts 3.7.1 → 4.0.0.pre1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d3934c079df9042f8ef837ea6eb7dc14a04706a1fcba4853d26a5f1ce69672bf
-  data.tar.gz: adf6be71e206ad4747f83268370772ffdaeddfc6a9b49576bbd74cb6cf9b650c
+  metadata.gz: b43c7d018d4bb7bd546bb516a584903d7922d87a616dbec87d2e4b6d00e06652
+  data.tar.gz: d4068b53bbd8428f88bf1a4d4af62aa7db0ff4ecccfecca0c0b268f1d8fb0381
 SHA512:
-  metadata.gz: d88b6998f1bd4d35329f501f4cbf710d1f653ca3faece95bc88c89ef626e6fe1413bd0129890817ff5d4cc15341e8c7bebd1209056cfc4fbd8bb730dc3ef4574
-  data.tar.gz: 3d919417a2bc28cebb759a77a340bd2dd996ed290de6a7b96c2fe40f4936d5df6d952242b6991fb12d34646e325699f0837f817fc5aa2b6e03257e6d903a541c
+  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'
 

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
@@ -161,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