familia 2.0.0.pre19 → 2.0.0.pre22

data/try/features/relationships/participation_unresolved_target_try.rb

@@ -0,0 +1,109 @@
+# try/features/relationships/participation_unresolved_target_try.rb
+#
+# frozen_string_literal: true
+
+# Test for proper error handling when target class cannot be resolved
+#
+# This test verifies that participates_in raises a helpful error when
+# the target class hasn't been loaded yet or doesn't exist.
+
+require_relative '../../support/helpers/test_helpers'
+
+## Test error message when target class doesn't exist (Symbol)
+begin
+  class UnresolvedTargetTest1 < Familia::Horreum
+    feature :relationships
+    identifier_field :id
+    field :id
+
+    participates_in :NonExistentTargetClass, :items
+  end
+  @error_raised = false
+rescue ArgumentError => e
+  @error_raised = true
+  @error_message = e.message
+end
+@error_raised
+#=> true
+
+## Test error message includes the unresolved class name
+@error_message.include?('NonExistentTargetClass')
+#=> true
+
+## Test error message mentions load order issue
+@error_message.include?('load order')
+#=> true
+
+## Test error message mentions Familia.members
+@error_message.include?('Familia.members')
+#=> true
+
+## Test error message includes list of registered classes
+@error_message.include?('Current registered classes')
+#=> true
+
+## Test error when target class doesn't exist (String)
+begin
+  class UnresolvedTargetTest2 < Familia::Horreum
+    feature :relationships
+    identifier_field :id
+    field :id
+
+    participates_in 'AnotherNonExistentClass', :items
+  end
+  @string_error_raised = false
+rescue ArgumentError => e
+  @string_error_raised = true
+  @string_error_message = e.message
+end
+@string_error_raised
+#=> true
+
+## Test error message for String target includes class name
+@string_error_message.include?('AnotherNonExistentClass')
+#=> true
+
+## Test error provides solution hint
+@string_error_message.include?('Solution')
+#=> true
+
+## Test that Class objects don't trigger the error (they're already resolved)
+# This should work fine - no error expected
+class ExistingTargetClass < Familia::Horreum
+  feature :relationships
+  identifier_field :id
+  field :id
+end
+
+class WorkingParticipant < Familia::Horreum
+  feature :relationships
+  identifier_field :id
+  field :id
+
+  # This should work - ExistingTargetClass is already defined
+  participates_in ExistingTargetClass, :items
+end
+
+# Verify the class was created successfully
+WorkingParticipant.ancestors.include?(Familia::Horreum)
+#=> true
+
+## Test that resolved Symbol target works (no error)
+class PreDefinedTarget < Familia::Horreum
+  feature :relationships
+  identifier_field :id
+  field :id
+end
+
+class SymbolParticipant < Familia::Horreum
+  feature :relationships
+  identifier_field :id
+  field :id
+
+  # This should work - PreDefinedTarget is defined above
+  participates_in :PreDefinedTarget, :items
+end
+
+# Verify it worked
+SymbolParticipant.ancestors.include?(Familia::Horreum)
+#=> true

data/try/features/relationships/relationships_api_changes_try.rb

@@ -1,5 +1,7 @@
 # try/features/relationships/relationships_api_changes_try.rb
 #
+# frozen_string_literal: true
+
 # Test coverage for Familia v2 relationships API changes
 # Testing new class_participates_in and unique_index methods
 # Testing breaking changes and argument validation

data/try/features/relationships/relationships_edge_cases_try.rb

@@ -1,3 +1,7 @@
+# try/features/relationships/relationships_edge_cases_try.rb
+#
+# frozen_string_literal: true
+
 # Simplified edge case testing for Relationships v2 - focusing on core functionality
 
 require_relative '../../support/helpers/test_helpers'

data/try/features/relationships/relationships_performance_minimal_try.rb

@@ -1,3 +1,7 @@
+# try/features/relationships/relationships_performance_minimal_try.rb
+#
+# frozen_string_literal: true
+
 # Minimal performance testing focusing on core Familia functionality
 
 require_relative '../../support/helpers/test_helpers'

data/try/features/relationships/relationships_performance_simple_try.rb

@@ -1,3 +1,7 @@
+# try/features/relationships/relationships_performance_simple_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/relationships_performance_simple_try.rb
 #
 # Simplified performance testing for the Relationships feature

data/try/features/relationships/relationships_performance_try.rb

@@ -1,3 +1,7 @@
+# try/features/relationships/relationships_performance_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/relationships_performance_try.rb
 #
 # Performance and integration testing for the Relationships feature

data/try/features/relationships/relationships_performance_working_try.rb

@@ -1,3 +1,7 @@
+# try/features/relationships/relationships_performance_working_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/relationships_performance_working_try.rb
 #
 # Working performance test focusing on basic functionality

data/try/features/relationships/relationships_try.rb

@@ -1,5 +1,7 @@
 # try/features/relationships/relationships_try.rb
 #
+# frozen_string_literal: true
+
 # Simplified Familia v2 relationship functionality tests - focusing on core working features
 #
 
@@ -113,12 +115,12 @@ decoded[:permission_list].include?(:write)
 @customer.domains.class.name
 #=> "Familia::SortedSet"
 
-## Customer can add domains (generated method)
-@customer.respond_to?(:add_domain)
+## Customer can add domains (generated method with _instance suffix)
+@customer.respond_to?(:add_domains_instance)
 #=> true
 
-## Customer can remove domains (generated method)
-@customer.respond_to?(:remove_domain)
+## Customer can remove domains (generated method with _instance suffix)
+@customer.respond_to?(:remove_domains_instance)
 #=> true
 
 ## Domain can check membership in customer domains (collision-free naming)

data/try/features/safe_dump/safe_dump_advanced_try.rb

@@ -1,3 +1,7 @@
+# try/features/safe_dump/safe_dump_advanced_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/safe_dump_extended_try.rb
 
 # These tryouts test the safe dumping functionality.

data/try/features/safe_dump/safe_dump_try.rb

@@ -1,3 +1,7 @@
+# try/features/safe_dump/safe_dump_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/safe_dump_try.rb
 
 require_relative '../../support/helpers/test_helpers'

data/try/features/transient_fields/redacted_string_try.rb

@@ -1,4 +1,6 @@
 # try/features/transient_fields/redacted_string_try.rb
+#
+# frozen_string_literal: true
 
 require_relative '../../support/helpers/test_helpers'
 

data/try/features/transient_fields/refresh_reset_try.rb

@@ -1,4 +1,7 @@
 # try/features/transient_fields/refresh_reset_try.rb
+#
+# frozen_string_literal: true
+
 # Test that refresh! properly resets transient fields to nil
 
 require_relative '../../support/helpers/test_helpers'

data/try/features/transient_fields/simple_refresh_test.rb

@@ -1,3 +1,6 @@
+# try/features/transient_fields/simple_refresh_test.rb
+#
+# frozen_string_literal: true
 
 require_relative '../../helpers/test_helpers'
 

data/try/features/transient_fields/single_use_redacted_string_try.rb

@@ -1,4 +1,6 @@
 # try/features/transient_fields/single_use_redacted_string_try.rb
+#
+# frozen_string_literal: true
 
 require_relative '../../support/helpers/test_helpers'
 

data/try/features/transient_fields/transient_fields_core_try.rb

@@ -1,3 +1,7 @@
+# try/features/transient_fields/transient_fields_core_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/transient_fields_core_try.rb
 
 require_relative '../../support/helpers/test_helpers'

data/try/features/transient_fields/transient_fields_integration_try.rb

@@ -1,3 +1,7 @@
+# try/features/transient_fields/transient_fields_integration_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/transient_fields_integration_try.rb
 
 require_relative '../../support/helpers/test_helpers'

data/try/integration/connection/fiber_context_preservation_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/fiber_context_preservation_try.rb
+#
+# frozen_string_literal: true
+
 # Fiber Context Preservation Tryouts
 #
 # Tests that verify the removal of previous_conn preservation logic

data/try/integration/connection/handler_constraints_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/handler_constraints_try.rb
+#
+# frozen_string_literal: true
+
 # Handler Constraint Methods Tryouts
 #
 # Tests that each connection handler class correctly defines its operation

data/try/integration/connection/isolated_dbclient_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/isolated_dbclient_try.rb
+#
+# frozen_string_literal: true
+
 # try/core/isolated_dbclient_try.rb
 
 # Tryouts: Isolated connection functionality

data/try/integration/connection/middleware_reconnect_try.rb

@@ -1,5 +1,7 @@
 # try/integration/connection/middleware_reconnect_try.rb
 #
+# frozen_string_literal: true
+
 # Tests for Familia.reconnect! method that refreshes connection pools
 # with current middleware configuration
 

data/try/integration/connection/operation_mode_guards_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/operation_mode_guards_try.rb
+#
+# frozen_string_literal: true
+
 # Operation Mode Guards Tryouts
 #
 # Tests the connection handler operation mode enforcement that prevents Redis

data/try/integration/connection/pipeline_fallback_integration_try.rb

@@ -1,4 +1,7 @@
+# try/integration/connection/pipeline_fallback_integration_try.rb
 #
+# frozen_string_literal: true
+
 # Tests pipeline fallback modes when connection handlers don't support pipelines.
 # Validates that pipelined_mode configuration works correctly with cached connections
 # and that the fallback behavior matches transaction fallback patterns.

data/try/integration/connection/pools_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/pools_try.rb
+#
+# frozen_string_literal: true
+
 # try/pooling/connection_pool_test_try.rb
 
 # USAGE: FAMILIA_TRACE=1 FAMILIA_DEBUG=1 bundle exec tryouts try/pooling/connection_pool_test_try.rb

data/try/integration/connection/responsibility_chain_tracking_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/responsibility_chain_tracking_try.rb
+#
+# frozen_string_literal: true
+
 # ResponsibilityChain Handler Tracking Tryouts
 #
 # Tests that the ResponsibilityChain correctly tracks which handler provided

data/try/integration/connection/transaction_fallback_integration_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/transaction_fallback_integration_try.rb
+#
+# frozen_string_literal: true
+
 # Transaction Fallback Integration Tryouts
 #
 # Tests real-world scenarios where transaction fallback is needed, particularly

data/try/integration/connection/transaction_mode_permissive_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/transaction_mode_permissive_try.rb
+#
+# frozen_string_literal: true
+
 # Transaction Mode: Permissive Tryouts
 #
 # Tests permissive transaction mode behavior where operations silently

data/try/integration/connection/transaction_mode_strict_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/transaction_mode_strict_try.rb
+#
+# frozen_string_literal: true
+
 # Transaction Mode: Strict Tryouts
 #
 # Tests strict transaction mode behavior where operations fail fast

data/try/integration/connection/transaction_mode_warn_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/transaction_mode_warn_try.rb
+#
+# frozen_string_literal: true
+
 # Transaction Mode: Warn Tryouts
 #
 # Tests warn transaction mode behavior where operations log a warning

data/try/integration/connection/transaction_modes_try.rb

@@ -1,3 +1,7 @@
+# try/integration/connection/transaction_modes_try.rb
+#
+# frozen_string_literal: true
+
 # Transaction Modes Tryouts
 #
 # Tests the configurable transaction mode system that provides graceful fallback

data/try/integration/conventional_inheritance_try.rb

@@ -1,3 +1,7 @@
+# try/integration/conventional_inheritance_try.rb
+#
+# frozen_string_literal: true
+
 require_relative '../support/helpers/test_helpers'
 
 # Define test classes in global namespace

data/try/integration/create_method_try.rb

@@ -1,3 +1,7 @@
+# try/integration/create_method_try.rb
+#
+# frozen_string_literal: true
+
 # try/core/create_method_try.rb
 #
 # Comprehensive test coverage for the create method

data/try/integration/cross_component_try.rb

@@ -1,3 +1,7 @@
+# try/integration/cross_component_try.rb
+#
+# frozen_string_literal: true
+
 # Test cross-component integration scenarios
 
 require_relative '../support/helpers/test_helpers'

data/try/integration/data_types/datatype_pipelines_try.rb

@@ -1,3 +1,7 @@
+# try/integration/data_types/datatype_pipelines_try.rb
+#
+# frozen_string_literal: true
+
 # DataType Pipeline Support Tryouts
 #
 # Tests pipeline support for DataType objects. Pipelines provide performance
@@ -77,10 +81,12 @@ result = @user.scores.pipelined { |pipe| }
 #=> [true, true]
 
 ## Multiple DataType operations in single pipeline
+# Note: Raw Redis commands bypass Familia's JSON serialization.
+# Use serialize_value for values that will be looked up via Familia methods.
 result = @user.scores.pipelined do |pipe|
-  pipe.zadd(@user.scores.dbkey, 500, 'multi')
-  pipe.hset(@user.profile.dbkey, 'multi', 'pipeline')
-  pipe.sadd(@user.tags.dbkey, 'multi_tag')
+  pipe.zadd(@user.scores.dbkey, 500, @user.scores.serialize_value('multi'))
+  pipe.hset(@user.profile.dbkey, 'multi', @user.profile.serialize_value('pipeline'))
+  pipe.sadd(@user.tags.dbkey, @user.tags.serialize_value('multi_tag'))
 end
 [
   result.is_a?(MultiResult),

data/try/integration/data_types/datatype_transactions_try.rb

@@ -1,3 +1,7 @@
+# try/integration/data_types/datatype_transactions_try.rb
+#
+# frozen_string_literal: true
+
 # DataType Transaction Support Tryouts
 #
 # Tests transaction support for DataType objects, covering both parent-owned
@@ -148,6 +152,8 @@ conn_class
 #=> "Redis::MultiConnection"
 
 ## Transaction with direct_access works correctly
+# Note: direct_access bypasses serialize_value, so raw 'true' string
+# gets parsed as JSON boolean true on retrieval (Issue #190 behavior)
 result = @user.profile.transaction do |trans_conn|
   trans_conn.hset(@user.profile.dbkey, 'status', 'active')
 
@@ -158,7 +164,7 @@ result = @user.profile.transaction do |trans_conn|
   end
 end
 [@user.profile['status'], @user.profile['verified']]
-#=> ["active", "true"]
+#=> ["active", true]
 
 ## Transaction atomicity - all commands succeed or none
 test_zset = Familia::SortedSet.new('atomic:test')
@@ -178,11 +184,13 @@ end
 #=> ["initial"]
 
 ## Nested transactions with parent-owned DataTypes work
+# Note: Raw Redis commands bypass Familia's JSON serialization.
+# Use serialize_value or check raw members for consistency.
 outer_result = @user.scores.transaction do |outer_conn|
-  outer_conn.zadd(@user.scores.dbkey, 999, 'outer_member')
+  outer_conn.zadd(@user.scores.dbkey, 999, @user.scores.serialize_value('outer_member'))
 
   inner_result = @user.tags.transaction do |inner_conn|
-    inner_conn.sadd(@user.tags.dbkey, 'nested_tag')
+    inner_conn.sadd(@user.tags.dbkey, @user.tags.serialize_value('nested_tag'))
   end
 
   inner_result.is_a?(MultiResult)
@@ -227,12 +235,14 @@ TransactionTestUser.logical_database
 #=> 2
 
 ## Multiple DataType types in single transaction
+# Note: Raw Redis commands bypass Familia's JSON serialization.
+# Use serialize_value for values that will be looked up via Familia methods.
 result = @user.scores.transaction do |conn|
   # Can operate on different DataTypes using same connection
-  conn.zadd(@user.scores.dbkey, 777, 'multi_test')
-  conn.hset(@user.profile.dbkey, 'multi', 'yes')
-  conn.sadd(@user.tags.dbkey, 'multi_tag')
-  conn.rpush(@user.activity.dbkey, 'multi_action')
+  conn.zadd(@user.scores.dbkey, 777, @user.scores.serialize_value('multi_test'))
+  conn.hset(@user.profile.dbkey, 'multi', @user.profile.serialize_value('yes'))
+  conn.sadd(@user.tags.dbkey, @user.tags.serialize_value('multi_tag'))
+  conn.rpush(@user.activity.dbkey, @user.activity.serialize_value('multi_action'))
 end
 [
   result.is_a?(MultiResult),

data/try/integration/database_consistency_try.rb

@@ -1,3 +1,7 @@
+# try/integration/database_consistency_try.rb
+#
+# frozen_string_literal: true
+
 # try/core/database_consistency_try.rb
 #
 # Database consistency verification and edge case testing

data/try/integration/familia_extended_try.rb

@@ -1,3 +1,7 @@
+# try/integration/familia_extended_try.rb
+#
+# frozen_string_literal: true
+
 # try/core/familia_extended_try.rb
 
 require 'time'

data/try/integration/familia_members_methods_try.rb

@@ -1,3 +1,7 @@
+# try/integration/familia_members_methods_try.rb
+#
+# frozen_string_literal: true
+
 # try/core/familia_members_methods_try.rb
 
 require_relative '../support/helpers/test_helpers'

data/try/integration/models/customer_safe_dump_try.rb

@@ -1,3 +1,7 @@
+# try/integration/models/customer_safe_dump_try.rb
+#
+# frozen_string_literal: true
+
 # try/models/customer_safedump_try.rb
 
 require_relative '../../support/helpers/test_helpers'

data/try/integration/models/customer_try.rb

@@ -1,3 +1,7 @@
+# try/integration/models/customer_try.rb
+#
+# frozen_string_literal: true
+
 # try/models/customer_try.rb
 
 # Customer Tryouts
@@ -69,7 +73,8 @@ Customer.find_by_id(ident).planid
 #=> true
 
 ## Customer can be added to class-level sorted set
-Customer.instances.add(@customer.identifier)
+# Note: Add the object directly so identifier extraction is consistent
+Customer.instances.add(@customer)
 Customer.instances.member?(@customer)
 #=> true
 
@@ -93,11 +98,10 @@ multi_result = @customer.destroy!
 cust = Customer.find_by_id('test@example.com')
 exists = Customer.exists?('test@example.com')
 [multi_result.results, cust.nil?, exists]
-#=> [[1, 0, 1, 1, 1, 1, 1], true, false]
+#=> [[1, 0, 1, 1, 1, 1, 1, true], true, false]
 
 ## Customer.destroy! can be called on an already destroyed object
 @customer.destroy!
-#=:> MultiResult
 #==> result.successful?
 #=*> result.results
 

data/try/integration/models/datatype_base_try.rb

@@ -1,3 +1,7 @@
+# try/integration/models/datatype_base_try.rb
+#
+# frozen_string_literal: true
+
 # try/data_types/data_type_base_try.rb
 
 # Test DataType base functionality

data/try/integration/models/familia_object_try.rb

@@ -1,3 +1,7 @@
+# try/integration/models/familia_object_try.rb
+#
+# frozen_string_literal: true
+
 # try/models/familia_object_try.rb
 
 require_relative '../../support/helpers/test_helpers'

data/try/integration/persistence_operations_try.rb

@@ -1,3 +1,7 @@
+# try/integration/persistence_operations_try.rb
+#
+# frozen_string_literal: true
+
 # try/core/persistence_operations_try.rb
 #
 # Comprehensive test coverage for core persistence methods: exists?, save, save_if_not_exists, create