familia 2.0.0.pre18 → 2.0.0.pre21

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
@@ -37,7 +41,7 @@ end
 # =============================================
 
 ## create method successfully creates new object
-@created_obj = CreateTestModel.create(id: @first_test_id, name: 'Created Object', value: 'test_value')
+@created_obj = CreateTestModel.create!(id: @first_test_id, name: 'Created Object', value: 'test_value')
 [@created_obj.class, @created_obj.exists?, @created_obj.name]
 #=> [CreateTestModel, true, 'Created Object']
 
@@ -55,17 +59,17 @@ end
 # =============================================
 
 ## create method raises RecordExistsError for duplicate
-CreateTestModel.create(id: @first_test_id, name: 'Duplicate Attempt')
+CreateTestModel.create!(id: @first_test_id, name: 'Duplicate Attempt')
 #=!> Familia::RecordExistsError
 
 ## RecordExistsError includes the dbkey in the message
-CreateTestModel.create(id: @first_test_id, name: 'Another Duplicate')
+CreateTestModel.create!(id: @first_test_id, name: 'Another Duplicate')
 #=!> Familia::RecordExistsError
 #==> !!error.message.match(/create_test_model:#{@first_test_id}:object/)
 
 ## RecordExistsError message follows consistent format
 begin
-  CreateTestModel.create(id: @first_test_id, name: 'Yet Another Duplicate')
+  CreateTestModel.create!(id: @first_test_id, name: 'Yet Another Duplicate')
   false  # Should not reach here
 rescue Familia::RecordExistsError => e
   e.message.start_with?('Key already exists:')
@@ -74,10 +78,10 @@ end
 
 ## RecordExistsError exposes key property for programmatic access
 @final_test_id = next_test_id
-CreateTestModel.create(id: @final_test_id, name: 'Setup for Key Test')
+CreateTestModel.create!(id: @final_test_id, name: 'Setup for Key Test')
 
 begin
-  CreateTestModel.create(id: @final_test_id, name: 'Key Test Duplicate')
+  CreateTestModel.create!(id: @final_test_id, name: 'Key Test Duplicate')
   false  # Should not reach here
 rescue Familia::RecordExistsError => e
   # Key should be accessible and contain the identifier
@@ -90,22 +94,22 @@ end
 # =============================================
 
 ## create with empty identifier raises NoIdentifier error
-CreateTestModel.create(id: '')
+CreateTestModel.create!(id: '')
 #=!> Familia::NoIdentifier
 
 ## create with nil identifier raises NoIdentifier error
-CreateTestModel.create(id: nil)
+CreateTestModel.create!(id: nil)
 #=!> Familia::NoIdentifier
 
 ## create with only some fields set
 @partial_id = next_test_id
-@partial_obj = CreateTestModel.create(id: @partial_id, name: 'Partial Object')
+@partial_obj = CreateTestModel.create!(id: @partial_id, name: 'Partial Object')
 [@partial_obj.exists?, @partial_obj.name, @partial_obj.value]
 #=> [true, 'Partial Object', nil]
 
 ## create with no additional fields (only identifier)
 @minimal_id = next_test_id
-@minimal_obj = CreateTestModel.create(id: @minimal_id)
+@minimal_obj = CreateTestModel.create!(id: @minimal_id)
 [@minimal_obj.exists?, @minimal_obj.id]
 #=> [true, @minimal_id]
 
@@ -115,14 +119,14 @@ CreateTestModel.create(id: nil)
 
 ## create is atomic - no partial state on failure
 @concurrent_id = next_test_id
-@first_obj = CreateTestModel.create(id: @concurrent_id, name: 'First')
+@first_obj = CreateTestModel.create!(id: @concurrent_id, name: 'First')
 
 # Verify first object exists
 first_exists = @first_obj.exists?
 
 # Attempt to create duplicate should not affect existing object
 begin
-  CreateTestModel.create(id: @concurrent_id, name: 'Concurrent Attempt')
+  CreateTestModel.create!(id: @concurrent_id, name: 'Concurrent Attempt')
   false  # Should not reach here
 rescue Familia::RecordExistsError
   # Original object should be unchanged
@@ -134,7 +138,7 @@ end
 ## create failure doesn't leave partial data
 before_failed_create = Familia.dbclient.keys("create_test_model:#{@concurrent_id}:*").length
 begin
-  CreateTestModel.create(id: @concurrent_id, name: 'Should Fail')
+  CreateTestModel.create!(id: @concurrent_id, name: 'Should Fail')
 rescue Familia::RecordExistsError
   # Should not create any additional keys
   after_failed_create = Familia.dbclient.keys("create_test_model:#{@concurrent_id}:*").length
@@ -148,20 +152,20 @@ end
 
 ## Both create and save_if_not_exists raise same error type for duplicates
 @consistency_id = next_test_id
-@consistency_obj = CreateTestModel.create(id: @consistency_id, name: 'Consistency Test')
+@consistency_obj = CreateTestModel.create!(id: @consistency_id, name: 'Consistency Test')
 
 # Test create raises RecordExistsError
 create_error_class = begin
-  CreateTestModel.create(id: @consistency_id, name: 'Create Duplicate')
+  CreateTestModel.create!(id: @consistency_id, name: 'Create Duplicate')
   nil
 rescue => e
   e.class
 end
 
-# Test save_if_not_exists raises RecordExistsError
+# Test save_if_not_exists! raises RecordExistsError
 sine_error_class = begin
   duplicate_obj = CreateTestModel.new(id: @consistency_id, name: 'SINE Duplicate')
-  duplicate_obj.save_if_not_exists
+  duplicate_obj.save_if_not_exists!
   nil
 rescue => e
   e.class
@@ -172,17 +176,17 @@ end
 
 ## Both methods have similar error message patterns
 @error_comparison_id = next_test_id
-CreateTestModel.create(id: @error_comparison_id, name: 'Error Comparison')
+CreateTestModel.create!(id: @error_comparison_id, name: 'Error Comparison')
 
 create_error_msg = begin
-  CreateTestModel.create(id: @error_comparison_id, name: 'Create Error')
+  CreateTestModel.create!(id: @error_comparison_id, name: 'Create Error')
   nil
 rescue => e
   e.message
 end
 
 sine_error_msg = begin
-  CreateTestModel.new(id: @error_comparison_id, name: 'SINE Error').save_if_not_exists
+  CreateTestModel.new(id: @error_comparison_id, name: 'SINE Error').save_if_not_exists!
   nil
 rescue => e
   e.message
@@ -198,7 +202,7 @@ end
 
 ## create works with complex field values
 @complex_id = next_test_id
-@complex_obj = CreateTestModel.create(
+@complex_obj = CreateTestModel.create!(
   id: @complex_id,
   name: 'Complex Object',
   value: { nested: 'data', array: [1, 2, 3] }
@@ -214,7 +218,7 @@ end
 @consistency_check_id = next_test_id
 
 # Create via class method
-@class_created = CreateTestModel.create(id: @consistency_check_id, name: 'Class Created')
+@class_created = CreateTestModel.create!(id: @consistency_check_id, name: 'Class Created')
 
 # Both class and instance methods should see the object as existing
 class_sees_exists = CreateTestModel.exists?(@consistency_check_id)

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

@@ -0,0 +1,108 @@
+# 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
+# optimization by batching commands without the atomicity guarantee of transactions.
+
+require_relative '../../support/helpers/test_helpers'
+
+# Setup
+class PipelineTestUser < Familia::Horreum
+  logical_database 4
+  identifier_field :userid
+  field :userid
+  field :name
+
+  sorted_set :scores
+  hashkey :profile
+  set :tags
+  counter :visits
+end
+
+@user = PipelineTestUser.new(userid: 'pipe_user_001')
+@user.name = 'Pipeline Tester'
+@user.save
+
+## Parent-owned SortedSet can execute pipeline
+result = @user.scores.pipelined do |pipe|
+  pipe.zadd(@user.scores.dbkey, 100, 'p1')
+  pipe.zadd(@user.scores.dbkey, 200, 'p2')
+  pipe.zcard(@user.scores.dbkey)
+end
+[result.is_a?(MultiResult), @user.scores.members.size]
+#=> [true, 2]
+
+## Parent-owned HashKey can execute pipeline
+result = @user.profile.pipelined do |pipe|
+  pipe.hset(@user.profile.dbkey, 'city', 'NYC')
+  pipe.hset(@user.profile.dbkey, 'state', 'NY')
+  pipe.hgetall(@user.profile.dbkey)
+end
+[result.is_a?(MultiResult), @user.profile.keys.sort]
+#=> [true, ["city", "state"]]
+
+## Standalone SortedSet can execute pipeline
+leaderboard = Familia::SortedSet.new('pipeline:leaderboard')
+leaderboard.delete!
+result = leaderboard.pipelined do |pipe|
+  pipe.zadd(leaderboard.dbkey, 100, 'player1')
+  pipe.zadd(leaderboard.dbkey, 200, 'player2')
+  pipe.zcard(leaderboard.dbkey)
+end
+[result.is_a?(MultiResult), leaderboard.members.size]
+#=> [true, 2]
+
+## Pipeline with direct_access works correctly
+result = @user.profile.pipelined do |pipe_conn|
+  pipe_conn.hset(@user.profile.dbkey, 'pipeline_test', 'yes')
+
+  @user.profile.direct_access do |conn, key|
+    conn.object_id == pipe_conn.object_id &&
+    conn.hset(key, 'direct_test', 'yes')
+  end
+end
+[@user.profile['pipeline_test'], @user.profile['direct_test']]
+#=> ["yes", "yes"]
+
+## Pipeline returns MultiResult with correct structure
+result = @user.scores.pipelined do |pipe|
+  pipe.zadd(@user.scores.dbkey, 300, 'p3')
+  pipe.zadd(@user.scores.dbkey, 400, 'p4')
+end
+[result.is_a?(MultiResult), result.results.is_a?(Array)]
+#=> [true, true]
+
+## Empty pipeline returns empty MultiResult
+result = @user.scores.pipelined { |pipe| }
+[result.is_a?(MultiResult), result.results.empty?]
+#=> [true, true]
+
+## Multiple DataType operations in single pipeline
+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')
+end
+[
+  result.is_a?(MultiResult),
+  @user.scores.member?('multi'),
+  @user.profile['multi'],
+  @user.tags.member?('multi_tag')
+]
+#=> [true, true, "pipeline", true]
+
+## Standalone HashKey with logical_database option
+custom = Familia::HashKey.new('pipeline:custom', logical_database: 5)
+custom.delete!
+result = custom.pipelined do |pipe|
+  pipe.hset(custom.dbkey, 'key1', 'value1')
+  pipe.hget(custom.dbkey, 'key1')
+end
+result.is_a?(MultiResult)
+#=> true
+
+# Cleanup
+@user.destroy!

data/try/integration/data_types/datatype_transactions_try.rb

@@ -0,0 +1,251 @@
+# 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
+# DataTypes (delegating to parent) and standalone DataTypes (managing their
+# own connections). Validates atomic operations, connection context handling,
+# and integration with the transaction mode system.
+
+require_relative '../../support/helpers/test_helpers'
+
+# Setup - Create test model with various DataType fields
+class TransactionTestUser < Familia::Horreum
+  logical_database 2
+  identifier_field :userid
+  field :userid
+  field :name
+  field :email
+
+  # Instance-level DataTypes
+  sorted_set :scores
+  hashkey :profile
+  set :tags
+  list :activity
+  counter :visits
+  string :bio
+end
+
+@user = TransactionTestUser.new(userid: 'txn_user_001')
+@user.name = 'Transaction Tester'
+@user.save
+
+## Parent-owned SortedSet can execute transaction
+result = @user.scores.transaction do |conn|
+  conn.zadd(@user.scores.dbkey, 100, 'level1')
+  conn.zadd(@user.scores.dbkey, 200, 'level2')
+  conn.zadd(@user.scores.dbkey, 300, 'level3')
+end
+[result.is_a?(MultiResult), @user.scores.members.sort]
+#=> [true, ["level1", "level2", "level3"]]
+
+## Parent-owned HashKey can execute transaction
+result = @user.profile.transaction do |conn|
+  conn.hset(@user.profile.dbkey, 'city', 'San Francisco')
+  conn.hset(@user.profile.dbkey, 'country', 'USA')
+  conn.hget(@user.profile.dbkey, 'city')
+end
+[result.is_a?(MultiResult), result.results.last, @user.profile['country']]
+#=> [true, "San Francisco", "USA"]
+
+## Parent-owned UnsortedSet can execute transaction
+result = @user.tags.transaction do |conn|
+  conn.sadd(@user.tags.dbkey, 'ruby')
+  conn.sadd(@user.tags.dbkey, 'redis')
+  conn.scard(@user.tags.dbkey)
+end
+[result.is_a?(MultiResult), result.results.last, @user.tags.members.sort]
+#=> [true, 2, ["redis", "ruby"]]
+
+## Parent-owned List can execute transaction
+result = @user.activity.transaction do |conn|
+  conn.rpush(@user.activity.dbkey, 'login')
+  conn.rpush(@user.activity.dbkey, 'view_profile')
+  conn.rpush(@user.activity.dbkey, 'logout')
+  conn.llen(@user.activity.dbkey)
+end
+[result.is_a?(MultiResult), result.results.last, @user.activity.members]
+#=> [true, 3, ["login", "view_profile", "logout"]]
+
+## Parent-owned Counter can execute transaction
+result = @user.visits.transaction do |conn|
+  conn.set(@user.visits.dbkey, 0)
+  conn.incr(@user.visits.dbkey)
+  conn.incr(@user.visits.dbkey)
+  conn.get(@user.visits.dbkey)
+end
+[result.is_a?(MultiResult), result.results.last.to_i, @user.visits.value]
+#=> [true, 2, 2]
+
+## Parent-owned StringKey can execute transaction
+result = @user.bio.transaction do |conn|
+  conn.set(@user.bio.dbkey, 'Ruby developer')
+  conn.append(@user.bio.dbkey, ' and Redis enthusiast')
+  conn.get(@user.bio.dbkey)
+end
+[result.is_a?(MultiResult), result.results.last, @user.bio.value]
+#=> [true, "Ruby developer and Redis enthusiast", "Ruby developer and Redis enthusiast"]
+
+## Standalone SortedSet can execute transaction
+leaderboard = Familia::SortedSet.new('game:leaderboard')
+leaderboard.delete!
+result = leaderboard.transaction do |conn|
+  conn.zadd(leaderboard.dbkey, 500, 'player1')
+  conn.zadd(leaderboard.dbkey, 600, 'player2')
+  conn.zadd(leaderboard.dbkey, 450, 'player3')
+  conn.zcard(leaderboard.dbkey)
+end
+[result.is_a?(MultiResult), result.results.last, leaderboard.members.size]
+#=> [true, 3, 3]
+
+## Standalone HashKey can execute transaction
+cache = Familia::HashKey.new('app:cache')
+cache.delete!
+result = cache.transaction do |conn|
+  conn.hset(cache.dbkey, 'key1', 'value1')
+  conn.hset(cache.dbkey, 'key2', 'value2')
+  conn.hkeys(cache.dbkey)
+end
+[result.is_a?(MultiResult), result.results.last.sort, cache.keys.sort]
+#=> [true, ["key1", "key2"], ["key1", "key2"]]
+
+## Standalone UnsortedSet can execute transaction
+global_tags = Familia::UnsortedSet.new('app:tags')
+global_tags.delete!
+result = global_tags.transaction do |conn|
+  conn.sadd(global_tags.dbkey, 'tag1')
+  conn.sadd(global_tags.dbkey, 'tag2')
+  conn.smembers(global_tags.dbkey)
+end
+[result.is_a?(MultiResult), result.results.last.sort, global_tags.members.sort]
+#=> [true, ["tag1", "tag2"], ["tag1", "tag2"]]
+
+## Standalone StringKey can execute transaction
+session_data = Familia::StringKey.new('session:abc123')
+session_data.delete!
+result = session_data.transaction do |conn|
+  conn.set(session_data.dbkey, '{"user_id": 123}')
+  conn.expire(session_data.dbkey, 3600)
+  conn.get(session_data.dbkey)
+end
+[result.is_a?(MultiResult), result.results.last, session_data.value]
+#=> [true, "{\"user_id\": 123}", "{\"user_id\": 123}"]
+
+## Transaction with logical_database option works
+custom_cache = Familia::HashKey.new('custom:cache', logical_database: 3)
+custom_cache.delete!
+result = custom_cache.transaction do |conn|
+  conn.hset(custom_cache.dbkey, 'setting', 'enabled')
+  conn.hget(custom_cache.dbkey, 'setting')
+end
+[result.is_a?(MultiResult), result.results.last]
+#=> [true, "enabled"]
+
+## Transaction provides correct connection object type
+conn_class = nil
+@user.scores.transaction do |conn|
+  conn_class = conn.class.name
+end
+conn_class
+#=> "Redis::MultiConnection"
+
+## Transaction with direct_access works correctly
+result = @user.profile.transaction do |trans_conn|
+  trans_conn.hset(@user.profile.dbkey, 'status', 'active')
+
+  # direct_access should use the same transaction connection
+  @user.profile.direct_access do |conn, key|
+    conn.object_id == trans_conn.object_id &&
+    conn.hset(key, 'verified', 'true')
+  end
+end
+[@user.profile['status'], @user.profile['verified']]
+#=> ["active", "true"]
+
+## Transaction atomicity - all commands succeed or none
+test_zset = Familia::SortedSet.new('atomic:test')
+test_zset.delete!
+test_zset.add('initial', 1)
+
+begin
+  test_zset.transaction do |conn|
+    conn.zadd(test_zset.dbkey, 100, 'member1')
+    conn.zadd(test_zset.dbkey, 200, 'member2')
+    raise 'Intentional error to test rollback'
+  end
+rescue => e
+  # Transaction should have rolled back
+  test_zset.members
+end
+#=> ["initial"]
+
+## Nested transactions with parent-owned DataTypes work
+outer_result = @user.scores.transaction do |outer_conn|
+  outer_conn.zadd(@user.scores.dbkey, 999, 'outer_member')
+
+  inner_result = @user.tags.transaction do |inner_conn|
+    inner_conn.sadd(@user.tags.dbkey, 'nested_tag')
+  end
+
+  inner_result.is_a?(MultiResult)
+end
+[outer_result.is_a?(MultiResult), @user.tags.member?('nested_tag')]
+#=> [true, true]
+
+## Transaction respects transaction modes (permissive)
+begin
+  original_mode = Familia.transaction_mode
+  Familia.configure { |config| config.transaction_mode = :permissive }
+
+  # Force a cached connection to trigger fallback
+  @user.class.instance_variable_set(:@dbclient, Familia.create_dbclient)
+
+  result = @user.scores.transaction do |conn|
+    # Should be IndividualCommandProxy in fallback mode
+    conn.class == Familia::Connection::IndividualCommandProxy &&
+    conn.zadd(@user.scores.dbkey, 888, 'fallback_test')
+  end
+
+  result.is_a?(MultiResult)
+ensure
+  @user.class.remove_instance_variable(:@dbclient)
+  Familia.configure { |config| config.transaction_mode = original_mode }
+end
+#=> true
+
+## Transaction with empty block returns empty MultiResult
+result = @user.scores.transaction { |conn| }
+[result.is_a?(MultiResult), result.results.empty?]
+#=> [true, true]
+
+## Transaction connection uses parent's logical_database
+# TransactionTestUser has logical_database 2
+# Parent-owned DataType delegates to parent, verify via class setting
+@user.scores.delete!
+@user.scores.transaction do |conn|
+  conn.zadd(@user.scores.dbkey, 1, 'test_member')
+end
+TransactionTestUser.logical_database
+#=> 2
+
+## Multiple DataType types in single transaction
+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')
+end
+[
+  result.is_a?(MultiResult),
+  @user.scores.member?('multi_test'),
+  @user.profile['multi'],
+  @user.tags.member?('multi_tag'),
+  @user.activity.members.include?('multi_action')
+]
+#=> [true, true, "yes", true, true]
+
+# Cleanup
+@user.destroy!

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'
@@ -39,7 +43,11 @@ require_relative '../../support/helpers/test_helpers'
 
 ## Safe dump includes correct updated timestamp
 @safe_dump[:updated]
-#=> @now
+#=:> Float
+
+## Safe dump includes correct updated timestamp
+@safe_dump[:updated].to_i
+#=> @now.to_i
 
 ## Safe dump includes correct secrets_created count
 @customer.secrets_created.increment

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

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'
@@ -71,7 +75,7 @@ Customer.all_customers.size
 
 ## Familia class clear
 Customer.all_customers.delete!
-#=> true
+#=> 1
 
 ## Familia class replace 1 of 4
 Customer.message.value = 'msg1'