tryouts 3.4.0 → 3.5.1

data/lib/tryouts/parsers/shared_methods.rb

@@ -1,5 +1,7 @@
 # lib/tryouts/parsers/shared_methods.rb
 
+require_relative '../parser_warning'
+
 class Tryouts
   module Parsers
     module SharedMethods
@@ -205,13 +207,19 @@ class Tryouts
         test_blocks     = classified_blocks.filter { |block| block[:type] == :test }
         teardown_blocks = classified_blocks.filter { |block| block[:type] == :teardown }
 
-        Testrun.new(
+        testrun = Testrun.new(
           setup: build_setup(setup_blocks),
           test_cases: test_blocks.map { |block| build_test_case(block) },
           teardown: build_teardown(teardown_blocks),
           source_file: @source_path,
           metadata: { parsed_at: @parsed_at, parser: parser_type },
+          warnings: warnings
         )
+
+        # Validate strict mode after collecting all warnings
+        validate_strict_mode(testrun)
+
+        testrun
       end
 
       def build_setup(setup_blocks)
@@ -354,6 +362,9 @@ class Tryouts
           start_line: Integer => start_line,
           end_line: Integer => end_line
         }
+          # Collect warning for unnamed test
+          collect_unnamed_test_warning(block)
+
           source_lines           = @lines[start_line..end_line]
           first_expectation_line = exp_tokens.empty? ? start_line : exp_tokens.first[:line]
 
@@ -411,6 +422,44 @@ class Tryouts
         :shared
       end
 
+      # Warning collection methods
+      def add_warning(warning)
+        @warnings ||= []
+        @warnings << warning
+      end
+
+      def warnings
+        @warnings ||= []
+      end
+
+      def collect_unnamed_test_warning(block)
+        return unless block[:type] == :test && block[:description].empty?
+
+        line_number = block[:start_line] + 1
+        context = @lines[block[:start_line]] || ''
+
+        add_warning(ParserWarning.unnamed_test(
+          line_number: line_number,
+          context: context.strip
+        ))
+      end
+
+      def validate_strict_mode(testrun)
+        return unless @options[:strict]
+
+        unnamed_test_warnings = warnings.select { |w| w.type == :unnamed_test }
+        return if unnamed_test_warnings.empty?
+
+        # In strict mode, fail with first unnamed test error
+        first_warning = unnamed_test_warnings.first
+        raise TryoutSyntaxError.new(
+          "Strict mode: #{first_warning.message} at line #{first_warning.line_number}. #{first_warning.suggestion}",
+          line_number: first_warning.line_number,
+          context: first_warning.context,
+          source_file: @source_path
+        )
+      end
+
     end
   end
 end

data/lib/tryouts/test_case.rb

@@ -67,7 +67,7 @@ class Tryouts
     end
   end
 
-  Testrun = Data.define(:setup, :test_cases, :teardown, :source_file, :metadata) do
+  Testrun = Data.define(:setup, :test_cases, :teardown, :source_file, :metadata, :warnings) do
     def total_tests
       test_cases.size
     end

data/lib/tryouts/test_executor.rb

@@ -11,6 +11,7 @@ class Tryouts
       @output_manager = output_manager
       @translator     = translator
       @global_tally   = global_tally
+
     end
 
     def execute
@@ -28,6 +29,7 @@ class Tryouts
 
     private
 
+
     def execute_direct_mode
       batch = TestBatch.new(
         @testrun,

data/lib/tryouts/test_runner.rb

@@ -29,6 +29,7 @@ class Tryouts
       @output_manager = output_manager
       @translator     = initialize_translator
       @global_tally   = initialize_global_tally
+      @file_line_specs = options[:file_line_specs] || {}
     end
 
     def run
@@ -37,7 +38,10 @@ class Tryouts
 
       result = process_files
       show_failure_summary
-      show_grand_total if @global_tally[:aggregator].get_file_counts[:total] > 1
+      # Always show grand total for agent mode to ensure output, otherwise only for multiple files
+      if @options[:agent] || @global_tally[:aggregator].get_file_counts[:total] > 1
+        show_grand_total
+      end
 
       # For agent critical mode, only count errors as failures
       if @options[:agent] && (@options[:agent_focus] == :critical || @options[:agent_focus] == 'critical')
@@ -148,10 +152,16 @@ class Tryouts
       failure_count
     end
 
-    def process_file(file_path)
+    def process_file(file)
+      # Pass line spec for this file if available
+      file_options = @options.dup
+      if @file_line_specs && @file_line_specs[file]
+        file_options[:line_spec] = @file_line_specs[file]
+      end
+
       processor = FileProcessor.new(
-        file: file_path,
-        options: @options,
+        file: file,
+        options: file_options,
         output_manager: @output_manager,
         translator: @translator,
         global_tally: @global_tally,
@@ -160,7 +170,7 @@ class Tryouts
     rescue StandardError => ex
       handle_file_error(ex)
       @global_tally[:aggregator].add_infrastructure_failure(
-        :file_processing, file_path, ex.message, ex
+        :file_processing, file, ex.message, ex
       )
       1
     end

data/lib/tryouts/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tryouts
 version: !ruby/object:Gem::Version
-  version: 3.4.0
+  version: 3.5.1
 platform: ruby
 authors:
 - Delano Mandelbaum
@@ -148,6 +148,7 @@ files:
 - lib/tryouts/cli/formatters/token_budget.rb
 - lib/tryouts/cli/formatters/tty_status_display.rb
 - lib/tryouts/cli/formatters/verbose.rb
+- lib/tryouts/cli/line_spec_parser.rb
 - lib/tryouts/cli/modes/generate.rb
 - lib/tryouts/cli/modes/inspect.rb
 - lib/tryouts/cli/opts.rb
@@ -169,6 +170,7 @@ files:
 - lib/tryouts/expectation_evaluators/true.rb
 - lib/tryouts/failure_collector.rb
 - lib/tryouts/file_processor.rb
+- lib/tryouts/parser_warning.rb
 - lib/tryouts/parsers/base_parser.rb
 - lib/tryouts/parsers/enhanced_parser.rb
 - lib/tryouts/parsers/prism_parser.rb