refactor: Modularize strategies test files

- test_abstract_strategy.jl: Wrap in TestStrategiesAbstractStrategy module
- test_builders.jl: Wrap in TestStrategiesBuilders module
- test_configuration.jl: Wrap in TestStrategiesConfiguration module
- test_introspection.jl: Wrap in TestStrategiesIntrospection module
- test_metadata.jl: Wrap in TestStrategiesMetadata module
- test_registry.jl: Wrap in TestStrategiesRegistry module
- test_strategy_options.jl: Wrap in TestStrategiesStrategyOptions module
- test_utilities.jl: Wrap in TestStrategiesUtilities module
- test_validation.jl: Wrap in TestStrategiesValidation module

All files now follow the standard test module pattern with VERBOSE/SHOWTIMING support.
421/423 tests pass (2 intentional error tests for validation).

Author: ocots

test/suite/strategies/test_abstract_strategy.jl

@@ -1,7 +1,11 @@
-# Tests for abstract strategy contract
+module TestStrategiesAbstractStrategy
 
+using Test
+using CTModels
 using CTModels.Strategies
 using CTModels.Options
+using CTBase
+using Main.TestOptions: VERBOSE, SHOWTIMING
 
 # ============================================================================
 # Fake strategy types for testing (must be at module top-level)
@@ -49,6 +53,11 @@ struct UnimplementedStrategy <: CTModels.Strategies.AbstractStrategy end
 # Test function
 # ============================================================================
 
+"""
+    test_abstract_strategy()
+
+Tests for abstract strategy contract.
+"""
 function test_abstract_strategy()
     Test.@testset "Abstract Strategy" verbose=VERBOSE showtiming=SHOWTIMING begin
 
@@ -162,3 +171,7 @@ function test_abstract_strategy()
         end
     end
 end
+
+end # module
+
+test_abstract_strategy() = TestStrategiesAbstractStrategy.test_abstract_strategy()

test/suite/strategies/test_builders.jl

@@ -1,7 +1,11 @@
-# Tests for strategy builders
+module TestStrategiesBuilders
 
+using Test
+using CTModels
 using CTModels.Strategies
 using CTModels.Options
+using CTBase
+using Main.TestOptions: VERBOSE, SHOWTIMING
 
 # ============================================================================
 # Test strategy types (reuse from test_abstract_strategy.jl)
@@ -120,6 +124,11 @@ end
 # Test function
 # ============================================================================
 
+"""
+    test_builders()
+
+Tests for strategy builders.
+"""
 function test_builders()
     Test.@testset "Strategy Builders" verbose=VERBOSE showtiming=SHOWTIMING begin
 
@@ -286,3 +295,7 @@ function test_builders()
         end
     end
 end
+
+end # module
+
+test_builders() = TestStrategiesBuilders.test_builders()

test/suite/strategies/test_configuration.jl

@@ -1,6 +1,10 @@
-# Tests for strategy configuration
+module TestStrategiesConfiguration
 
+using Test
+using CTModels
+using CTModels.Strategies
 using CTModels.Options: OptionDefinition, OptionValue
+using Main.TestOptions: VERBOSE, SHOWTIMING
 
 # ============================================================================
 # Test strategies with metadata
@@ -64,6 +68,11 @@ CTModels.Strategies.options(s::Union{TestStrategyA, TestStrategyB}) = s.options
 # Test function
 # ============================================================================
 
+"""
+    test_configuration()
+
+Tests for strategy configuration.
+"""
 function test_configuration()
     Test.@testset "Strategy Configuration" verbose=VERBOSE showtiming=SHOWTIMING begin
 
@@ -226,3 +235,7 @@ function test_configuration()
         end
     end
 end
+
+end # module
+
+test_configuration() = TestStrategiesConfiguration.test_configuration()

test/suite/strategies/test_introspection.jl

@@ -1,7 +1,10 @@
-# Tests for strategy introspection utilities
+module TestStrategiesIntrospection
 
+using Test
+using CTModels
 using CTModels.Strategies
 using CTModels.Options
+using Main.TestOptions: VERBOSE, SHOWTIMING
 
 # ============================================================================
 # Fake strategy types for testing (must be at module top-level)
@@ -50,6 +53,11 @@ CTModels.Strategies.metadata(::Type{<:EmptyOptionsStrategy}) = CTModels.Strategi
 # Test function
 # ============================================================================
 
+"""
+    test_introspection()
+
+Tests for strategy introspection utilities.
+"""
 function test_introspection()
     Test.@testset "Strategy Introspection" verbose=VERBOSE showtiming=SHOWTIMING begin
 
@@ -308,3 +316,7 @@ function test_introspection()
         end
     end
 end
+
+end # module
+
+test_introspection() = TestStrategiesIntrospection.test_introspection()

test/suite/strategies/test_metadata.jl

@@ -1,5 +1,17 @@
-# Tests for strategy metadata functionality
+module TestStrategiesMetadata
 
+using Test
+using CTModels
+using CTModels.Strategies
+using CTModels.Options
+using CTBase
+using Main.TestOptions: VERBOSE, SHOWTIMING
+
+"""
+    test_metadata()
+
+Tests for strategy metadata functionality.
+"""
 function test_metadata()
     Test.@testset "StrategyMetadata" verbose=VERBOSE showtiming=SHOWTIMING begin
 
@@ -229,3 +241,7 @@ function test_metadata()
         end
     end
 end
+
+end # module
+
+test_metadata() = TestStrategiesMetadata.test_metadata()

test/suite/strategies/test_registry.jl

@@ -1,7 +1,11 @@
-# Tests for strategy registry API
+module TestStrategiesRegistry
 
+using Test
+using CTModels
 using CTModels.Strategies
 using CTModels.Options
+using CTBase
+using Main.TestOptions: VERBOSE, SHOWTIMING
 
 # ============================================================================
 # Fake strategy types for testing (must be at module top-level)
@@ -44,8 +48,13 @@ CTModels.Strategies.metadata(::Type{<:WrongTypeStrategy}) = CTModels.Strategies.
 # Test function
 # ============================================================================
 
+"""
+    test_registry()
+
+Tests for strategy registry API.
+"""
 function test_registry()
-    Test.@testset "Strategy Registry API" verbose=VERBOSE showtiming=SHOWTIMING begin
+    Test.@testset "Strategy Registry" verbose=VERBOSE showtiming=SHOWTIMING begin
 
         # ========================================================================
         # UNIT TESTS
@@ -250,3 +259,7 @@ function test_registry()
         end
     end
 end
+
+end # module
+
+test_registry() = TestStrategiesRegistry.test_registry()

test/suite/strategies/test_strategy_options.jl

@@ -1,12 +1,21 @@
-# Tests for strategy-specific options handling
+module TestStrategiesStrategyOptions
 
+using Test
+using CTModels
 using CTModels.Strategies
 using CTModels.Options
+using CTBase
+using Main.TestOptions: VERBOSE, SHOWTIMING
 
 # ============================================================================
 # Test function
 # ============================================================================
 
+"""
+    test_strategy_options()
+
+Tests for strategy-specific options handling.
+"""
 function test_strategy_options()
     Test.@testset "Strategy Options" verbose=VERBOSE showtiming=SHOWTIMING begin
 
@@ -239,3 +248,7 @@ function test_strategy_options()
         end
     end
 end
+
+end # module
+
+test_strategy_options() = TestStrategiesStrategyOptions.test_strategy_options()

test/suite/strategies/test_utilities.jl

@@ -1,6 +1,10 @@
-# Tests for strategy utilities
+module TestStrategiesUtilities
 
+using Test
+using CTModels
+using CTModels.Strategies
 using CTModels.Options: OptionDefinition
+using Main.TestOptions: VERBOSE, SHOWTIMING
 
 # ============================================================================
 # Test strategy for suggestions
@@ -43,6 +47,11 @@ CTModels.Strategies.options(s::TestUtilStrategy) = s.options
 # Test function
 # ============================================================================
 
+"""
+    test_utilities()
+
+Tests for strategy utilities.
+"""
 function test_utilities()
     Test.@testset "Strategy Utilities" verbose=VERBOSE showtiming=SHOWTIMING begin
 
@@ -202,3 +211,7 @@ function test_utilities()
         end
     end
 end
+
+end # module
+
+test_utilities() = TestStrategiesUtilities.test_utilities()

test/suite/strategies/test_validation.jl

@@ -1,6 +1,11 @@
-# Tests for strategy validation API
+module TestStrategiesValidation
 
+using Test
+using CTModels
+using CTModels.Strategies
 using CTModels.Options: OptionDefinition
+using CTBase
+using Main.TestOptions: VERBOSE, SHOWTIMING
 
 # ============================================================================
 # Valid test strategies
@@ -327,6 +332,11 @@ CTModels.Strategies.options(s::NoOptionsStrategy) = s.options
 # Test function
 # ============================================================================
 
+"""
+    test_validation()
+
+Tests for strategy validation API.
+"""
 function test_validation()
     Test.@testset "Strategy Validation" verbose=VERBOSE showtiming=SHOWTIMING begin
 
@@ -549,3 +559,7 @@ function test_validation()
         end
     end
 end
+
+end # module
+
+test_validation() = TestStrategiesValidation.test_validation()