familia 2.0.0.pre19 → 2.0.0.pre21

data/try/features/encrypted_fields/context_isolation_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/context_isolation_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encryption_fields/context_isolation_try.rb
 
 require 'base64'

data/try/features/encrypted_fields/encrypted_fields_core_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/encrypted_fields_core_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encrypted_fields_core_try.rb
 
 require_relative '../../support/helpers/test_helpers'
@@ -122,4 +126,33 @@ end
 @user2.api_key.reveal { |decrypted| decrypted }
 #=> 'secret-key-123'
 
+## encrypted_data? returns false when no encrypted fields have values
+class SecureUser6 < Familia::Horreum
+  feature :encrypted_fields
+  identifier_field :user_id
+  field :user_id
+  encrypted_field :ssn
+  encrypted_field :api_key
+end
+
+@user3 = SecureUser6.new(user_id: 'test-user-006')
+@user3.encrypted_data?
+#=> false
+
+## encrypted_data? returns true when at least one encrypted field has a value
+@user3.ssn = '123-45-6789'
+@user3.encrypted_data?
+#=> true
+
+## encrypted_data? returns true with multiple encrypted fields set
+@user3.api_key = 'secret-key-456'
+@user3.encrypted_data?
+#=> true
+
+## encrypted_data? returns false after clearing encrypted fields
+@user3.ssn = nil
+@user3.api_key = nil
+@user3.encrypted_data?
+#=> false
+
 Fiber[:familia_key_cache]&.clear if Fiber[:familia_key_cache]

data/try/features/encrypted_fields/encrypted_fields_integration_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/encrypted_fields_integration_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encrypted_fields_integration_try.rb
 
 # Test constants will be redefined in each test since variables don't persist

data/try/features/encrypted_fields/encrypted_fields_no_cache_security_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/encrypted_fields_no_cache_security_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encrypted_fields_no_cache_security_try.rb
 #
 # Security tests for the no-cache encryption strategy

data/try/features/encrypted_fields/encrypted_fields_security_try.rb

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

data/try/features/encrypted_fields/error_conditions_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/error_conditions_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encryption_fields/error_conditions_try.rb
 
 require 'base64'

data/try/features/encrypted_fields/fresh_key_derivation_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/fresh_key_derivation_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encryption_fields/fresh_key_derivation_try.rb
 
 require 'base64'

data/try/features/encrypted_fields/fresh_key_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/fresh_key_try.rb
+#
+# frozen_string_literal: true
+
 require_relative '../../support/helpers/test_helpers'
 require 'base64'
 

data/try/features/encrypted_fields/key_rotation_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/key_rotation_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encryption_fields/key_rotation_try.rb
 
 require 'base64'

data/try/features/encrypted_fields/memory_security_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/memory_security_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encryption_fields/memory_security_try.rb
 
 require 'base64'

data/try/features/encrypted_fields/missing_current_key_version_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/missing_current_key_version_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encryption_fields/missing_current_key_version_try.rb
 
 require 'base64'

data/try/features/encrypted_fields/nonce_uniqueness_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/nonce_uniqueness_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encryption_fields/nonce_uniqueness_try.rb
 
 require 'base64'

data/try/features/encrypted_fields/secure_by_default_behavior_try.rb

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

data/try/features/encrypted_fields/thread_safety_try.rb

@@ -1,3 +1,7 @@
+# try/features/encrypted_fields/thread_safety_try.rb
+#
+# frozen_string_literal: true
+
 # try/features/encryption_fields/thread_safety_try.rb
 
 require 'concurrent'

data/try/features/encrypted_fields/universal_serialization_safety_try.rb

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

data/try/features/encryption/config_persistence_try.rb

@@ -1,3 +1,7 @@
+# try/features/encryption/config_persistence_try.rb
+#
+# frozen_string_literal: true
+
 # try/encryption/debug2_try.rb
 
 # - Tests configuration persistence between test sections

data/try/features/encryption/core_try.rb

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

data/try/features/encryption/instance_variable_scope_try.rb

@@ -1,3 +1,7 @@
+# try/features/encryption/instance_variable_scope_try.rb
+#
+# frozen_string_literal: true
+
 # try/encryption/debug3_try.rb
 
 # - Tests instance variable scoping in tryouts framework

data/try/features/encryption/module_loading_try.rb

@@ -1,3 +1,7 @@
+# try/features/encryption/module_loading_try.rb
+#
+# frozen_string_literal: true
+
 # try/encryption/debug_try.rb
 
 # - Tests that the encryption module loads correctly

data/try/features/encryption/providers/aes_gcm_provider_try.rb

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

data/try/features/encryption/providers/xchacha20_poly1305_provider_try.rb

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

data/try/features/encryption/roundtrip_validation_try.rb

@@ -1,3 +1,7 @@
+# try/features/encryption/roundtrip_validation_try.rb
+#
+# frozen_string_literal: true
+
 # try/encryption/debug4_try.rb
 
 # - Tests full encryption/decryption round trips

data/try/features/encryption/secure_memory_handling_try.rb

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

data/try/features/expiration/expiration_try.rb

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

data/try/features/external_identifier/external_identifier_try.rb

@@ -1,4 +1,6 @@
 # try/features/external_identifier/external_identifier_try.rb
+#
+# frozen_string_literal: true
 
 require_relative '../../support/helpers/test_helpers'
 
@@ -15,10 +17,28 @@ class ExternalIdTest < Familia::Horreum
   field :name
 end
 
-# Class with custom prefix
+# Class with custom format using different prefix
 class CustomPrefixTest < Familia::Horreum
   feature :object_identifier
-  feature :external_identifier, prefix: 'cust'
+  feature :external_identifier, format: 'cust_%{id}'
+  identifier_field :id
+  field :id
+  field :name
+end
+
+# Class with custom format template using hyphen separator
+class CustomFormatTest < Familia::Horreum
+  feature :object_identifier
+  feature :external_identifier, format: 'ext-%{id}'
+  identifier_field :id
+  field :id
+  field :name
+end
+
+# Class with format template without traditional prefix
+class NoPrefixFormatTest < Familia::Horreum
+  feature :object_identifier
+  feature :external_identifier, format: 'api/%{id}'
   identifier_field :id
   field :id
   field :name
@@ -77,6 +97,28 @@ custom_obj = CustomPrefixTest.new
 custom_obj.extid.start_with?('cust_')
 #==> true
 
+## Custom format with hyphen separator
+format_obj = CustomFormatTest.new
+format_obj.extid.start_with?('ext-')
+#==> true
+
+## Custom format uses hyphen not underscore
+format_obj = CustomFormatTest.new
+extid = format_obj.extid
+extid.include?('-') && !extid.include?('_')
+#==> true
+
+## Format without prefix placeholder uses literal format
+no_prefix_obj = NoPrefixFormatTest.new
+no_prefix_obj.extid.start_with?('api/')
+#==> true
+
+## Format without prefix does not include underscore
+no_prefix_obj = NoPrefixFormatTest.new
+extid = no_prefix_obj.extid
+!extid.include?('_') && !extid.include?('ext')
+#==> true
+
 ## External ID is URL-safe base-36 format
 obj = ExternalIdTest.new
 extid = obj.extid
@@ -154,13 +196,21 @@ obj1.extid != obj2.extid
 ExternalIdTest.field_types[:extid]
 #=:> Familia::Features::ExternalIdentifier::ExternalIdentifierFieldType
 
-## Feature options contain correct prefix
-ExternalIdTest.feature_options(:external_identifier)[:prefix]
-#=> "ext"
+## Feature options contain default format
+ExternalIdTest.feature_options(:external_identifier)[:format]
+#=> "ext_%{id}"
+
+## Custom prefix format options
+CustomPrefixTest.feature_options(:external_identifier)[:format]
+#=> "cust_%{id}"
+
+## Custom format with hyphen options
+CustomFormatTest.feature_options(:external_identifier)[:format]
+#=> "ext-%{id}"
 
-## Custom prefix feature options
-CustomPrefixTest.feature_options(:external_identifier)[:prefix]
-#=> "cust"
+## No traditional prefix format options
+NoPrefixFormatTest.feature_options(:external_identifier)[:format]
+#=> "api/%{id}"
 
 ## External ID is shorter than UUID objid
 obj = ExternalIdTest.new
@@ -224,3 +274,116 @@ destroy_test_obj.destroy!
 # Verify mapping was cleaned up
 ExternalIdTest.extid_lookup.key?(destroy_extid)
 #=> false
+
+# ========================================
+# Comprehensive Format Option Test Coverage
+# ========================================
+
+## Format option: default format produces ext_ prefix
+default_obj = ExternalIdTest.new
+default_obj.extid.start_with?('ext_')
+#==> true
+
+## Format option: custom format with different prefix works
+custom_prefix_obj = CustomPrefixTest.new
+custom_prefix_obj.extid.start_with?('cust_')
+#==> true
+
+## Format option: custom format with hyphen separator works
+hyphen_obj = CustomFormatTest.new
+hyphen_obj.extid.start_with?('ext-')
+#==> true
+
+## Format option: hyphen format does not contain underscore
+hyphen_obj = CustomFormatTest.new
+!hyphen_obj.extid.include?('_')
+#==> true
+
+## Format option: format without prefix works
+no_prefix_obj = NoPrefixFormatTest.new
+no_prefix_obj.extid.start_with?('api/')
+#==> true
+
+## Format option: no prefix format uses slash separator
+no_prefix_obj = NoPrefixFormatTest.new
+no_prefix_obj.extid.include?('/')
+#==> true
+
+## Format option: %{id} placeholder is properly interpolated
+obj = ExternalIdTest.new
+# The extid should not contain the literal string '%{id}'
+!obj.extid.include?('%{id}')
+#==> true
+
+## Format option: generated ID part is base36 alphanumeric
+obj = ExternalIdTest.new
+extid = obj.extid
+id_part = extid.gsub(/^ext_/, '')
+id_part.match?(/\A[0-9a-z]+\z/)
+#==> true
+
+## Format option: hyphen format ID part is base36 alphanumeric
+hyphen_obj = CustomFormatTest.new
+extid = hyphen_obj.extid
+id_part = extid.gsub(/^ext-/, '')
+id_part.match?(/\A[0-9a-z]+\z/)
+#==> true
+
+## Format option: no prefix format ID part is base36 alphanumeric
+no_prefix_obj = NoPrefixFormatTest.new
+extid = no_prefix_obj.extid
+id_part = extid.gsub(/^api\//, '')
+id_part.match?(/\A[0-9a-z]+\z/)
+#==> true
+
+## Format option: different formats produce different prefixes but same ID length
+default_obj = ExternalIdTest.new
+custom_obj = CustomPrefixTest.new
+hyphen_obj = CustomFormatTest.new
+# All should have similar length IDs (within a few characters due to prefix differences)
+default_id_len = default_obj.extid.length
+custom_id_len = custom_obj.extid.length
+hyphen_id_len = hyphen_obj.extid.length
+(default_id_len - custom_id_len).abs <= 5 && (default_id_len - hyphen_id_len).abs <= 5
+#==> true
+
+## Format option: custom formats maintain determinism
+obj = CustomPrefixTest.new
+first_extid = obj.extid
+second_extid = obj.extid
+first_extid == second_extid
+#==> true
+
+## Format option: formats work correctly with save/load cycle
+format_save_obj = CustomFormatTest.new(id: 'format_save_test', name: 'Format Save Test')
+original_extid = format_save_obj.extid
+format_save_obj.save
+loaded_obj = CustomFormatTest.new(id: 'format_save_test')
+loaded_obj.extid == original_extid
+#==> true
+
+## Format option: find_by_extid works with custom formats
+findable_custom = CustomPrefixTest.new(id: 'findable_custom', name: 'Findable Custom')
+findable_custom.save
+found_custom = CustomPrefixTest.find_by_extid(findable_custom.extid)
+found_custom&.id
+#=> "findable_custom"
+
+## Format option: find_by_extid works with hyphen format
+findable_hyphen = CustomFormatTest.new(id: 'findable_hyphen', name: 'Findable Hyphen')
+findable_hyphen.save
+found_hyphen = CustomFormatTest.find_by_extid(findable_hyphen.extid)
+found_hyphen&.id
+#=> "findable_hyphen"
+
+## Format option: find_by_extid works with no-prefix format
+findable_no_prefix = NoPrefixFormatTest.new(id: 'findable_no_prefix', name: 'Findable No Prefix')
+findable_no_prefix.save
+found_no_prefix = NoPrefixFormatTest.find_by_extid(findable_no_prefix.extid)
+found_no_prefix&.id
+#=> "findable_no_prefix"
+
+# Cleanup format test objects
+findable_custom.destroy! rescue nil
+findable_hyphen.destroy! rescue nil
+findable_no_prefix.destroy! rescue nil

data/try/features/feature_dependencies_try.rb

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

data/try/features/feature_improvements_try.rb

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

data/try/features/field_groups_try.rb

@@ -1,3 +1,5 @@
+# try/features/field_groups_try.rb
+#
 # frozen_string_literal: true
 
 # try/features/field_groups_try.rb

data/try/features/object_identifier/object_identifier_integration_try.rb

@@ -1,4 +1,6 @@
 # try/features/object_identifier/object_identifier_integration_try.rb
+#
+# frozen_string_literal: true
 
 require_relative '../../support/helpers/test_helpers'
 
@@ -19,7 +21,7 @@ end
 # Class with custom configurations for both features
 class ::CustomIntegrationTest < Familia::Horreum
   feature :object_identifier, generator: :hex
-  feature :external_identifier, prefix: 'custom'
+  feature :external_identifier, format: 'custom_%{id}'
   identifier_field :id
   field :id
   field :name
@@ -96,8 +98,9 @@ original_extid = persistence_obj.extid
 persistence_obj.save
 
 # Load from Valkey/Redis
-loaded_obj = PersistenceTest.load(id: 'persistence_test')
-#=> nil
+loaded_obj = PersistenceTest.load('persistence_test')
+loaded_obj.objid == original_objid && loaded_obj.extid == original_extid
+#==> true
 
 ## objid persists after save/load
 persistence_obj = PersistenceTest.new
@@ -169,17 +172,17 @@ opts.key?(:external_identifier)
 IntegrationTest.feature_options(:object_identifier)[:generator]
 #=> :uuid_v7
 
-## Prefix default configuration is applied correctly
-IntegrationTest.feature_options(:external_identifier)[:prefix]
-#=> "ext"
+## Format default configuration is applied correctly
+IntegrationTest.feature_options(:external_identifier)[:format]
+#=> "ext_%{id}"
 
 ## Custom generator configuration is applied correctly
 CustomIntegrationTest.feature_options(:object_identifier)[:generator]
 #=> :hex
 
-## Custom prefix configuration is applied correctly
-CustomIntegrationTest.feature_options(:external_identifier)[:prefix]
-#=> "custom"
+## Custom format configuration is applied correctly
+CustomIntegrationTest.feature_options(:external_identifier)[:format]
+#=> "custom_%{id}"
 
 ## objid is URL-safe (UUID format)
 obj = IntegrationTest.new

data/try/features/object_identifier/object_identifier_try.rb

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

data/try/features/quantization/quantization_try.rb

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

data/try/features/real_feature_integration_try.rb

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

data/try/features/relationships/indexing_commands_verification_try.rb

@@ -1,5 +1,7 @@
 # try/features/relationships/indexing_commands_verification_try.rb
 #
+# frozen_string_literal: true
+
 # Verification of proper Redis command generation for indexing operations
 # This test ensures the indexing system uses proper DataType methods instead of direct Redis calls
 #