familia 2.0.0.pre17 → 2.0.0.pre19

data/try/features/encrypted_fields/encrypted_fields_no_cache_security_try.rb

@@ -3,7 +3,7 @@
 # Security tests for the no-cache encryption strategy
 # These tests verify that we maintain security properties by NOT caching derived keys
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 test_keys = {
   v1: Base64.strict_encode64('a' * 32),

data/try/features/encrypted_fields/encrypted_fields_security_try.rb

@@ -1,6 +1,6 @@
 # try/features/encrypted_fields_security_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'base64'
 
 # Define all test classes up front to avoid tryouts retry conflicts

data/try/features/encrypted_fields/error_conditions_try.rb

@@ -2,7 +2,7 @@
 
 require 'base64'
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Setup encryption keys for error testing
 @test_keys = {

data/try/features/encrypted_fields/fresh_key_derivation_try.rb

@@ -2,7 +2,7 @@
 
 require 'base64'
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 test_keys = {
   v1: Base64.strict_encode64('a' * 32),

data/try/features/encrypted_fields/fresh_key_try.rb

@@ -1,4 +1,4 @@
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'base64'
 
 # Setup encryption configuration

data/try/features/encrypted_fields/key_rotation_try.rb

@@ -2,7 +2,7 @@
 
 require 'base64'
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Setup multiple key versions for rotation testing
 @test_keys = {

data/try/features/encrypted_fields/memory_security_try.rb

@@ -1,7 +1,7 @@
 # try/features/encryption_fields/memory_security_try.rb
 
 require 'base64'
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 test_keys = {
   v1: Base64.strict_encode64('a' * 32),

data/try/features/encrypted_fields/missing_current_key_version_try.rb

@@ -2,7 +2,7 @@
 
 require 'base64'
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # This tryouts file is based on the premise that there is no current key
 # version set. This is a global setting so if other tryouts rely on

data/try/features/encrypted_fields/nonce_uniqueness_try.rb

@@ -3,7 +3,7 @@
 require 'base64'
 require 'set'
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 test_keys = {
   v1: Base64.strict_encode64('a' * 32),

data/try/features/encrypted_fields/secure_by_default_behavior_try.rb

@@ -1,6 +1,6 @@
 # try/features/encryption_fields/secure_by_default_behavior_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'base64'
 
 Familia.debug = false

data/try/features/encrypted_fields/thread_safety_try.rb

@@ -3,7 +3,7 @@
 require 'concurrent'
 require 'base64'
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Setup encryption keys for testing
 test_keys = {

data/try/features/encrypted_fields/universal_serialization_safety_try.rb

@@ -1,6 +1,6 @@
 # try/features/encryption_fields/universal_serialization_safety_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'base64'
 
 Familia.debug = false

data/try/{encryption → features/encryption}/config_persistence_try.rb

@@ -5,7 +5,7 @@
 
 require 'base64'
 
-require_relative '../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'familia/encryption/providers/xchacha20_poly1305_provider'
 
 # SETUP

data/try/{encryption/encryption_core_try.rb → features/encryption/core_try.rb}

@@ -1,7 +1,7 @@
 # try/encryption/encryption_core_try.rb
 
-require_relative '../helpers/test_helpers'
-require_relative '../../lib/familia/encryption'
+require_relative '../../support/helpers/test_helpers'
+require_relative '../../../lib/familia/encryption'
 require 'base64'
 
 # Test constants will be redefined in each test since variables don't persist

data/try/{encryption → features/encryption}/instance_variable_scope_try.rb

@@ -6,7 +6,7 @@
 
 require 'base64'
 
-require_relative '../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 @test_keys = {
   v1: Base64.strict_encode64('a' * 32)

data/try/{encryption → features/encryption}/module_loading_try.rb

@@ -5,7 +5,7 @@
 
 require 'base64'
 
-require_relative '../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Test basic functionality
 

data/try/{encryption → features/encryption}/providers/aes_gcm_provider_try.rb

@@ -1,6 +1,6 @@
 # try/encryption/providers/aes_gcm_provider_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../../support/helpers/test_helpers'
 require 'base64'
 
 ## AES-GCM provider availability check (always available with OpenSSL)

data/try/{encryption → features/encryption}/providers/xchacha20_poly1305_provider_try.rb

@@ -1,6 +1,6 @@
 # try/encryption/providers/xchacha20_poly1305_provider_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../../support/helpers/test_helpers'
 require 'base64'
 
 ## XChaCha20Poly1305 provider availability check

data/try/{encryption → features/encryption}/roundtrip_validation_try.rb

@@ -5,7 +5,7 @@
 
 require 'base64'
 
-require_relative '../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 ## Test successful encryption
 test_keys = { v1: Base64.strict_encode64('a' * 32) }

data/try/{encryption → features/encryption}/secure_memory_handling_try.rb

@@ -1,7 +1,7 @@
 # try/encryption/secure_memory_handling_try.rb
 
-require_relative '../helpers/test_helpers'
-require_relative '../../lib/familia/encryption/providers/secure_xchacha20_poly1305_provider'
+require_relative '../../support/helpers/test_helpers'
+require_relative '../../../lib/familia/encryption/providers/secure_xchacha20_poly1305_provider'
 require 'base64'
 
 # SETUP

data/try/features/expiration/expiration_try.rb

@@ -1,6 +1,6 @@
 # try/features/expiration_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 Familia.debug = false
 
@@ -58,7 +58,7 @@ ExpiringTest.default_expiration
 #=> true
 
 ## Can call update_expiration method
-result = @test_obj.update_expiration(default_expiration: 180)
+result = @test_obj.update_expiration(expiration: 180)
 [result.class, result]
 #=> [FalseClass, false]
 

data/try/features/external_identifier/external_identifier_try.rb

@@ -1,6 +1,6 @@
 # try/features/external_identifier/external_identifier_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 Familia.debug = false
 

data/try/features/feature_dependencies_try.rb

@@ -1,6 +1,6 @@
 # try/features/feature_dependencies_try.rb
 
-require_relative '../helpers/test_helpers'
+require_relative '../support/helpers/test_helpers'
 
 Familia.debug = false
 

data/try/features/feature_improvements_try.rb

@@ -1,6 +1,6 @@
 # try/features/feature_improvements_try.rb
 
-require_relative '../helpers/test_helpers'
+require_relative '../support/helpers/test_helpers'
 
 # Test hierarchical feature registration
 class ::TestClass

data/try/features/object_identifier/object_identifier_integration_try.rb

@@ -1,6 +1,6 @@
 # try/features/object_identifier/object_identifier_integration_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 Familia.debug = false
 

data/try/features/object_identifier/object_identifier_try.rb

@@ -1,6 +1,6 @@
 # try/features/object_identifier/object_identifier_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 Familia.debug = false
 

data/try/features/quantization/quantization_try.rb

@@ -1,6 +1,6 @@
 # try/features/quantization_try.rb
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 Familia.debug = false
 

data/try/features/real_feature_integration_try.rb

@@ -1,6 +1,6 @@
 # try/features/real_feature_integration_try.rb
 
-require_relative '../helpers/test_helpers'
+require_relative '../support/helpers/test_helpers'
 
 Familia.debug = false
 
@@ -12,15 +12,17 @@ class ExpirationIntegrationTest < Familia::Horreum
   feature :expiration
 end
 
-# Safe dump feature integration with field categories
+# Safe dump feature integration with field types
 class SafeDumpCategoryTest < Familia::Horreum
+  feature :encrypted_fields
+  feature :transient_fields
+  feature :safe_dump
+
   identifier_field :id
   field :id
-  field :public_name, category: :persistent
-  field :email, category: :encrypted
-  field :tryouts_cache_data, category: :transient
-
-  feature :safe_dump
+  field :public_name
+  encrypted_field :email
+  transient_field :tryouts_cache_data
 
   # Use new SafeDump DSL
   safe_dump_field :id
@@ -30,14 +32,15 @@ end
 
 # Combined features work together
 class CombinedFeaturesTest < Familia::Horreum
-  identifier_field :id
-  field :id
-  field :name, category: :persistent
-  field :temp_data, category: :transient
-
+  feature :transient_fields
   feature :expiration
   feature :safe_dump
 
+  identifier_field :id
+  field :id
+  field :name
+  transient_field :temp_data
+
   # Use new SafeDump DSL
   safe_dump_field :id
   safe_dump_field :name
@@ -123,11 +126,11 @@ CombinedFeaturesTest.features_enabled.include?(:safe_dump)
 
 ## Test that feature() method returns current features when called with no args
 CombinedFeaturesTest.feature
-#=> [:expiration, :safe_dump]
+#=> [:transient_fields, :expiration, :safe_dump]
 
 ## Test that features_enabled() method returns the same results as feature() method
 CombinedFeaturesTest.feature
-#=> [:expiration, :safe_dump]
+#=> [:transient_fields, :expiration, :safe_dump]
 
 ## Features list is accessible
 QueryFeaturesTest.feature

data/try/features/relationships/indexing_commands_verification_try.rb

@@ -4,7 +4,7 @@
 # This test ensures the indexing system uses proper DataType methods instead of direct Redis calls
 #
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Enable database command logging for command verification tests
 Familia.enable_database_logging = true
@@ -44,8 +44,11 @@ end
 
 # Test data
 @user = TestIndexedUser.new(user_id: 'test_user_123', email: 'test@example.com', department: 'engineering')
+@user.save
 @company = TestIndexedCompany.new(company_id: 'test_company_456', name: 'Test Corp')
+@company.save
 @employee = TestIndexedEmployee.new(emp_id: 'test_emp_789', email: 'emp@example.com', department: 'sales')
+@employee.save
 
 ## Class-level indexing creates proper DataType field
 TestIndexedUser.respond_to?(:email_index)
@@ -91,6 +94,7 @@ end
 #=> false
 
 ## Instance-level indexing works with parent context
+@employee.save
 @employee.add_to_test_indexed_company_dept_index(@company)
 sample = @company.sample_from_department('sales')
 sample.first&.emp_id == @employee.emp_id
@@ -103,15 +107,16 @@ dept_index.class.name
 
 ## Multiple employees in same department
 @employee2 = TestIndexedEmployee.new(emp_id: 'test_emp_999', email: 'emp2@example.com', department: 'sales')
+@employee2.save
 @employee2.add_to_test_indexed_company_dept_index(@company)
 employees_in_sales = @company.find_all_by_department('sales')
-employees_in_sales.map(&:emp_id).sort
+employees_in_sales&.map(&:emp_id).sort
 #=> ["test_emp_789", "test_emp_999"]
 
 ## Removing from instance-level index works
 @employee.remove_from_test_indexed_company_dept_index(@company)
 remaining_employees = @company.find_all_by_department('sales')
-remaining_employees.map(&:emp_id)
+remaining_employees&.map(&:emp_id)
 #=> ["test_emp_999"]
 
 ## Index update methods work correctly

data/try/features/relationships/indexing_try.rb

@@ -4,7 +4,7 @@
 # Tests both multi_index (parent-context) and unique_index (class-level) indexing
 #
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Test classes for indexing functionality
 class ::TestUser < Familia::Horreum
@@ -55,13 +55,18 @@ end
 
 # Setup
 @user1 = TestUser.new(user_id: 'user_001', email: 'alice@example.com', username: 'alice', department: 'engineering', role: 'developer')
+@user1.save
 @user2 = TestUser.new(user_id: 'user_002', email: 'bob@example.com', username: 'bob', department: 'marketing', role: 'manager')
+@user2.save
 @user3 = TestUser.new(user_id: 'user_003', email: 'charlie@example.com', username: 'charlie', department: 'engineering', role: 'lead')
+@user3.save
 
 @company_id = "comp_#{rand(10000000)}"
-@company = TestCompany.create(company_id: @company_id, name: 'Acme Corp')
+@company = TestCompany.create!(company_id: @company_id, name: 'Acme Corp')
 @emp1 = TestEmployee.new(emp_id: 'emp_001', email: 'alice@acme.com', department: 'engineering', manager_id: 'mgr_001', badge_number: 'BADGE001')
+@emp1.save
 @emp2 = TestEmployee.new(emp_id: 'emp_002', email: 'bob@acme.com', department: 'sales', manager_id: 'mgr_002', badge_number: 'BADGE002')
+@emp2.save
 
 
 ## Context-scoped methods require context parameter
@@ -81,7 +86,7 @@ sample = @company.sample_from_department(@emp2.department)
 
 ## First indexing relationship has correct configuration
 config = @user1.class.indexing_relationships.first
-[config.field, config.index_name, config.target_class == TestUser, config.query]
+[config.field, config.index_name, config.scope_class == TestUser, config.query]
 #=> [:email, :email_lookup, true, true]
 
 ## Second indexing relationship has query disabled
@@ -184,7 +189,7 @@ TestUser.respond_to?(:find_by_username)
 
 ## Instance-scoped unique index has correct configuration
 config = @emp1.class.indexing_relationships.find { |r| r.field == :badge_number }
-[config.index_name, config.target_class, config.cardinality]
+[config.index_name, config.scope_class, config.cardinality]
 #=> [:badge_index, TestCompany, :unique]
 
 ## Target class gets finder method for unique index
@@ -246,6 +251,16 @@ found_emps = @company.find_all_by_badge_number('BADGE002')
 found_emps.map(&:emp_id)
 #=> ["emp_002"]
 
+## Instance-scoped bulk query filters nil inputs
+badges_with_nil = [nil, 'BADGE001', nil]
+found_emps = @company.find_all_by_badge_number(badges_with_nil)
+found_emps.map(&:emp_id)
+#=> ["emp_001"]
+
+## Instance-scoped bulk query with only nil returns empty
+@company.find_all_by_badge_number([nil, nil]).length
+#=> 0
+
 ## Update badge index entry
 old_badge = @emp1.badge_number
 @emp1.badge_number = 'BADGE001_NEW'
@@ -272,7 +287,7 @@ old_badge = @emp1.badge_number
 
 ## Context-scoped multi_index relationship has correct configuration
 config = @emp1.class.indexing_relationships.find { |r| r.field == :department }
-[config.field, config.index_name, config.target_class]
+[config.field, config.index_name, config.scope_class]
 #=> [:department, :dept_index, TestCompany]
 
 ## Context-scoped methods are generated with collision-free naming
@@ -415,6 +430,20 @@ found = TestUser.find_all_by_email(emails)
 found.map(&:user_id)
 #=> ["user_002"]
 
+## Bulk query filters nil inputs before querying
+emails_with_nil = [nil, 'bob@example.com', nil]
+found = TestUser.find_all_by_email(emails_with_nil)
+found.map(&:user_id)
+#=> ["user_002"]
+
+## Bulk query with only nil inputs returns empty array
+TestUser.find_all_by_email([nil, nil]).length
+#=> 0
+
+## Bulk query with nil as single value returns empty array
+TestUser.find_all_by_email(nil).length
+#=> 0
+
 ## Adding to index with nil field value does nothing
 @user_nil = TestUser.new(user_id: 'user_nil', email: nil)
 @user_nil.add_to_class_email_lookup

data/try/features/relationships/participation_commands_verification_spec.rb

@@ -7,7 +7,7 @@ RSpec.describe 'participation_commands_verification_try' do
     Timecop.freeze(Time.parse("2024-01-15 10:30:00"))
     puts Time.now  # Always returns 2024-01-15 10:30:00
     puts Date.today  # Always returns 2024-01-15
-    require_relative '../../../lib/middleware/database_middleware'
+    require_relative '../../../lib/middleware/database_logger'
     require_relative '../../../lib/familia'
     Familia.enable_database_logging = true
     Familia.enable_database_counter = true

data/try/features/relationships/participation_commands_verification_try.rb

@@ -17,7 +17,7 @@ puts Date.today  # Always returns 2024-01-15
 
 
 # Load middleware first
-require_relative '../../../lib/middleware/database_middleware'
+require_relative '../../../lib/middleware/database_logger'
 
 # Load Familia
 require_relative '../../../lib/familia'
@@ -83,7 +83,7 @@ instantiation_commands.empty?
 database_commands = DatabaseLogger.capture_commands do
   @customer.save
 end
-database_commands.map { |cmd| cmd[:command] } if database_commands
+database_commands.map { |cmd| cmd.command } if database_commands
 ##=> [["hmset", "reverse_index_customer:ri_cust_123:object", "customer_id", "ri_cust_123", "name", "Reverse Index Test Customer"], ["zadd", "reverse_index_customer:instances", "1705343400.0", "ri_cust_123"]]
 
 
@@ -91,14 +91,14 @@ database_commands.map { |cmd| cmd[:command] } if database_commands
 database_commands = DatabaseLogger.capture_commands do
   @domain1.save
 end
-database_commands[0][:command] if database_commands && database_commands[0]
+database_commands[0].command if database_commands && database_commands[0]
 ##=>  ["hmset", "reverse_index_domain:ri_dom_1:object", "domain_id", "ri_dom_1", "display_domain", "example1.com", "created_at", "1705343400.0"]
 
 ## Domain2 save functionality
 database_commands = DatabaseLogger.capture_commands do
   @domain2.save
 end
-database_commands[0][:command]if database_commands && database_commands[0]
+database_commands[0].command if database_commands && database_commands[0]
 ##=> ["hmset", "reverse_index_domain:ri_dom_2:object", "domain_id", "ri_dom_2", "display_domain", "example2.com", "created_at", "1705343401.0"]
 
 

data/try/features/relationships/participation_performance_improvements_try.rb

@@ -3,7 +3,7 @@
 # Tests for performance improvements in participation functionality
 # Verifies reverse index functionality and robust type comparison
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Test classes for performance improvements
 class PerfTestCustomer < Familia::Horreum

data/try/features/relationships/participation_reverse_index_try.rb

@@ -3,7 +3,7 @@
 # Tests for participation reverse index functionality
 # Verifies performance improvements and correct behavior
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Test classes for reverse index functionality
 class ReverseIndexCustomer < Familia::Horreum

data/try/features/relationships/relationships_api_changes_try.rb

@@ -4,7 +4,7 @@
 # Testing new class_participates_in and unique_index methods
 # Testing breaking changes and argument validation
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Test classes for new API
 class ::ApiTestUser < Familia::Horreum
@@ -264,17 +264,17 @@ participation_meta.target_class.familia_name
 
 ## unique_index stores correct target_class
 indexing_meta = ApiTestUser.indexing_relationships.find { |r| r.index_name == :email_lookup }
-indexing_meta.target_class
+indexing_meta.scope_class
 #=> ApiTestUser
 
-## unique_index stores correct target_class via familia_name
+## unique_index stores correct scope_class via familia_name
 indexing_meta = ApiTestUser.indexing_relationships.find { |r| r.index_name == :email_lookup }
-indexing_meta.target_class.familia_name
+indexing_meta.scope_class.familia_name
 #=> 'ApiTestUser'
 
 ## multi_index with within: stores correct metadata
 membership_meta = ApiTestMembership.indexing_relationships.find { |r| r.index_name == :user_memberships }
-membership_meta.target_class
+membership_meta.scope_class
 #=> ApiTestUser
 
 # =============================================

data/try/features/relationships/relationships_edge_cases_try.rb

@@ -1,6 +1,6 @@
 # Simplified edge case testing for Relationships v2 - focusing on core functionality
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Test classes for edge case testing
 class EdgeTestCustomer < Familia::Horreum
@@ -41,14 +41,14 @@ end
 @domain1 = EdgeTestDomain.new(
   domain_id: 'edge_dom_1',
   display_domain: 'edge1.example.com',
-  created_at: Time.new(2025, 6, 15, 12, 0, 0),
+  created_at: Time.new(2025, 6, 15, 12, 0, 0).to_i,  # Convert Time to Integer for JSON serialization
   score_value: 10
 )
 
 @domain2 = EdgeTestDomain.new(
   domain_id: 'edge_dom_2',
   display_domain: 'edge2.example.com',
-  created_at: Time.new(2025, 7, 20, 15, 30, 0),
+  created_at: Time.new(2025, 7, 20, 15, 30, 0).to_i,  # Convert Time to Integer for JSON serialization
   score_value: 25
 )
 

data/try/features/relationships/relationships_performance_minimal_try.rb

@@ -1,6 +1,6 @@
 # Minimal performance testing focusing on core Familia functionality
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'benchmark'
 
 # Simple test classes without relationships feature

data/try/features/relationships/relationships_performance_simple_try.rb

@@ -2,7 +2,7 @@
 #
 # Simplified performance testing for the Relationships feature
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'benchmark'
 
 # Test classes for performance testing

data/try/features/relationships/relationships_performance_try.rb

@@ -2,7 +2,7 @@
 #
 # Performance and integration testing for the Relationships feature
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'benchmark'
 
 # Test classes for performance testing

data/try/features/relationships/relationships_performance_working_try.rb

@@ -2,7 +2,7 @@
 #
 # Working performance test focusing on basic functionality
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 require 'benchmark'
 
 # Simple test class using only basic Familia features

data/try/features/relationships/relationships_try.rb

@@ -3,7 +3,7 @@
 # Simplified Familia v2 relationship functionality tests - focusing on core working features
 #
 
-require_relative '../../helpers/test_helpers'
+require_relative '../../support/helpers/test_helpers'
 
 # Test classes for Familia v2 relationship functionality
 class TestCustomer < Familia::Horreum