tryouts 3.5.1 → 3.5.2

data/lib/tryouts/test_runner.rb

@@ -1,7 +1,7 @@
 # lib/tryouts/test_runner.rb
 
 require 'concurrent'
-require_relative 'parsers/prism_parser'
+require_relative 'parsers/legacy_parser'
 require_relative 'parsers/enhanced_parser'
 require_relative 'test_batch'
 require_relative 'translators/rspec_translator'
@@ -24,11 +24,11 @@ class Tryouts
     }.freeze
 
     def initialize(files:, options:, output_manager:)
-      @files          = files
-      @options        = apply_framework_defaults(options)
-      @output_manager = output_manager
-      @translator     = initialize_translator
-      @global_tally   = initialize_global_tally
+      @files           = files
+      @options         = apply_framework_defaults(options)
+      @output_manager  = output_manager
+      @translator      = initialize_translator
+      @global_tally    = initialize_global_tally
       @file_line_specs = options[:file_line_specs] || {}
     end
 
@@ -44,8 +44,11 @@ class Tryouts
       end
 
       # For agent critical mode, only count errors as failures
-      if @options[:agent] && (@options[:agent_focus] == :critical || @options[:agent_focus] == 'critical')
-        @global_tally[:aggregator].get_display_counts[:errors]
+      if @options[:agent] && ([:critical, 'critical'].include?(@options[:agent_focus]))
+        # Include infrastructure failures as errors for agent critical mode
+        display_errors        = @global_tally[:aggregator].get_display_counts[:errors]
+        infrastructure_errors = @global_tally[:aggregator].infrastructure_failure_count
+        display_errors + infrastructure_errors
       else
         result
       end
@@ -118,7 +121,7 @@ class Tryouts
         min_threads: 1,
         max_threads: pool_size,
         max_queue: @files.length, # Queue size must accommodate all files
-        fallback_policy: :abort # Raise exception if pool and queue are exhausted
+        fallback_policy: :abort, # Raise exception if pool and queue are exhausted
       )
 
       # Submit all file processing tasks to the thread pool
@@ -132,15 +135,15 @@ class Tryouts
       failure_count = 0
       futures.each_with_index do |future, idx|
         begin
-          result = future.value # This blocks until the future completes
+          result         = future.value # This blocks until the future completes
           failure_count += result unless result.zero?
 
           status = result.zero? ? Console.color(:green, 'PASS') : Console.color(:red, 'FAIL')
-          file = @files[idx]
+          file   = @files[idx]
           @output_manager.info "#{status} #{Console.pretty_path(file)} (#{result} failures)", 1
         rescue StandardError => ex
           failure_count += 1
-          file = @files[idx]
+          file           = @files[idx]
           @output_manager.info "#{Console.color(:red, 'ERROR')} #{Console.pretty_path(file)} (#{ex.message})", 1
         end
       end
@@ -184,10 +187,10 @@ class Tryouts
     end
 
     def show_grand_total
-      elapsed_time = Time.now - @global_tally[:start_time]
-      aggregator = @global_tally[:aggregator]
+      elapsed_time   = Time.now - @global_tally[:start_time]
+      aggregator     = @global_tally[:aggregator]
       display_counts = aggregator.get_display_counts
-      file_counts = aggregator.get_file_counts
+      file_counts    = aggregator.get_file_counts
 
       @output_manager.grand_total(
         display_counts[:total_tests],

data/lib/tryouts/version.rb

@@ -1,5 +1,5 @@
 # lib/tryouts/version.rb
 
 class Tryouts
-  VERSION = '3.5.1'
+  VERSION = '3.5.2'
 end

data/lib/tryouts.rb

@@ -8,7 +8,7 @@ TRYOUTS_LIB_HOME = __dir__ unless defined?(TRYOUTS_LIB_HOME)
 require_relative 'tryouts/console'
 require_relative 'tryouts/test_batch'
 require_relative 'tryouts/version'
-require_relative 'tryouts/parsers/prism_parser'
+require_relative 'tryouts/parsers/legacy_parser'
 require_relative 'tryouts/parsers/enhanced_parser'
 require_relative 'tryouts/cli'
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tryouts
 version: !ruby/object:Gem::Version
-  version: 3.5.1
+  version: 3.5.2
 platform: ruby
 authors:
 - Delano Mandelbaum
@@ -171,9 +171,10 @@ files:
 - lib/tryouts/failure_collector.rb
 - lib/tryouts/file_processor.rb
 - lib/tryouts/parser_warning.rb
+- lib/tryouts/parsers/CLAUDE.md
 - lib/tryouts/parsers/base_parser.rb
 - lib/tryouts/parsers/enhanced_parser.rb
-- lib/tryouts/parsers/prism_parser.rb
+- lib/tryouts/parsers/legacy_parser.rb
 - lib/tryouts/parsers/shared_methods.rb
 - lib/tryouts/test_batch.rb
 - lib/tryouts/test_case.rb

data/lib/tryouts/parsers/prism_parser.rb

@@ -1,122 +0,0 @@
-# lib/tryouts/parsers/prism_parser.rb
-
-require_relative '../test_case'
-require_relative 'base_parser'
-
-class Tryouts
-  # Fixed PrismParser with pattern matching for robust token filtering
-  class PrismParser < Tryouts::Parsers::BaseParser
-
-    def parse
-      return handle_syntax_errors if @prism_result.failure?
-
-      tokens           = tokenize_content
-      test_boundaries  = find_test_case_boundaries(tokens)
-      tokens           = classify_potential_descriptions_with_boundaries(tokens, test_boundaries)
-      test_blocks      = group_into_test_blocks(tokens)
-      process_test_blocks(test_blocks)
-    end
-
-    private
-
-    # Tokenize content using pattern matching for clean line classification
-    def tokenize_content
-      tokens = []
-
-      @lines.each_with_index do |line, index|
-        token = case line
-                in /^##\s*(.*)$/ # Test description format: ## description
-                  { type: :description, content: $1.strip, line: index }
-                in /^#\s*TEST\s*\d*:\s*(.*)$/  # rubocop:disable Lint/DuplicateBranch
-                  { type: :description, content: $1.strip, line: index }
-                in /^#\s*=!>\s*(.*)$/ # Exception expectation (updated for consistency)
-                  { type: :exception_expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in /^#\s*=<>\s*(.*)$/ # Intentional failure expectation
-                  { type: :intentional_failure_expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in /^#\s*==>\s*(.*)$/ # Boolean true expectation
-                  { type: :true_expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in %r{^#\s*=/=>\s*(.*)$} # Boolean false expectation
-                  { type: :false_expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in /^#\s*=\|>\s*(.*)$/ # Boolean (true or false) expectation
-                  { type: :boolean_expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in /^#\s*=\*>\s*(.*)$/ # Non-nil expectation
-                  { type: :non_nil_expectation, content: $1.strip, line: index }
-                in /^#\s*=:>\s*(.*)$/ # Result type expectation
-                  { type: :result_type_expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in /^#\s*=~>\s*(.*)$/ # Regex match expectation
-                  { type: :regex_match_expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in /^#\s*=%>\s*(.*)$/ # Performance time expectation
-                  { type: :performance_time_expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in /^#\s*=(\d+)>\s*(.*)$/ # Output expectation (stdout/stderr with pipe number)
-                  { type: :output_expectation, content: $2.strip, pipe: $1.to_i, line: index, ast: parse_expectation($2.strip) }
-                in /^#\s*=>\s*(.*)$/ # Regular expectation
-                  { type: :expectation, content: $1.strip, line: index, ast: parse_expectation($1.strip) }
-                in /^##\s*=>\s*(.*)$/ # Commented out expectation (should be ignored)
-                  { type: :comment, content: '=>' + $1.strip, line: index }
-                in /^#\s*(.*)$/ # Single hash comment - potential description
-                  { type: :potential_description, content: $1.strip, line: index }
-                in /^\s*$/ # Blank line
-                  { type: :blank, line: index }
-                else # Ruby code
-                  { type: :code, content: line, line: index, ast: parse_ruby_line(line) }
-                end
-
-        tokens << token
-      end
-
-      # Return tokens with potential_descriptions - they'll be classified later with test boundaries
-      tokens
-    end
-
-
-    # Convert potential_descriptions to descriptions or comments based on context
-    def classify_potential_descriptions(tokens)
-      tokens.map.with_index do |token, index|
-        if token[:type] == :potential_description
-          # Check if this looks like a test description based on content and context
-          content = token[:content].strip
-
-          # Skip if it's clearly just a regular comment (short, lowercase, etc.)
-          # Test descriptions are typically longer and more descriptive
-          looks_like_regular_comment = content.length < 20 &&
-                                       content.downcase == content &&
-                                       !content.match?(/test|example|demonstrate|show/i)
-
-          # Check if there's code immediately before this (suggesting it's mid-test)
-          prev_token      = index > 0 ? tokens[index - 1] : nil
-          has_code_before = prev_token && prev_token[:type] == :code
-
-          if looks_like_regular_comment || has_code_before
-            # Treat as regular comment
-            token.merge(type: :comment)
-          else
-            # Look ahead for test pattern: code + at least one expectation within reasonable distance
-            following_tokens = tokens[(index + 1)..]
-
-            # Skip blanks and comments to find meaningful content
-            meaningful_following = following_tokens.reject { |t| [:blank, :comment].include?(t[:type]) }
-
-            # Look for test pattern: at least one code token followed by at least one expectation
-            # within the next 10 meaningful tokens (to avoid matching setup/teardown)
-            test_window     = meaningful_following.first(10)
-            has_code        = test_window.any? { |t| t[:type] == :code }
-            has_expectation = test_window.any? { |t| is_expectation_type?(t[:type]) }
-
-            if has_code && has_expectation
-              token.merge(type: :description)
-            else
-              token.merge(type: :comment)
-            end
-          end
-        else
-          token
-        end
-      end
-    end
-
-    # Parser type identification for metadata
-    def parser_type
-      :prism_v2_fixed
-    end
-  end
-end