familia 2.0.0.pre18 → 2.0.0.pre19

data/try/edge_cases/string_coercion_try.rb

@@ -130,15 +130,15 @@ process_identifier(@customer)
 
 ## Cleanup after test, 1
 @metadata.delete!
-#=> true
+#=> 1
 
 ## Cleanup after test, 2
 @customer.delete!
-#=> true
+#=> 1
 
 ## Cleanup after test, 3
 @session.delete!
-#=> true
+#=> 1
 
 ## to_s handles identifier errors gracefully
 badboi = BadIdentifierTest.new
@@ -154,4 +154,4 @@ badboi.to_s # .include?('BadIdentifierTest')
 
 ## Delete customer2
 [@customer2.exists?, @customer2.delete!]
-#=> [false, false]
+#=> [false, 0]

data/try/features/expiration/expiration_try.rb

@@ -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/relationships/indexing_try.rb

@@ -62,7 +62,7 @@ end
 @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')
@@ -86,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
@@ -189,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
@@ -251,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'
@@ -277,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
@@ -420,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/relationships_api_changes_try.rb

@@ -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/integration/connection/fiber_context_preservation_try.rb

@@ -166,7 +166,7 @@ end
 ## Operation guards prevent pipeline fiber issues before they occur
 begin
   # Ensure we're in strict mode for this test
-  Familia.configure { |config| config.pipeline_mode = :strict }
+  Familia.configure { |config| config.pipelined_mode = :strict }
 
   Fiber[:familia_connection] = [Customer.create_dbclient, Familia.middleware_version]
   Fiber[:familia_connection_handler_class] = Familia::Connection::FiberConnectionHandler
@@ -185,7 +185,7 @@ end
 
 ## Method aliases work correctly
 # pipeline alias for pipelined
-result1 = Familia.pipeline do |conn|
+result1 = Familia.pipelined do |conn|
   conn.set('alias_test', 'alias_success')
   conn.get('alias_test')
 end
@@ -203,7 +203,7 @@ result1.results.last == 'alias_success' && result2.results.last == 'alias_succes
 customer = Customer.new(custid: 'alias_test')
 
 # pipeline alias
-result1 = customer.pipeline do |conn|
+result1 = customer.pipelined do |conn|
   conn.set('horreum:alias1', 'success1')
   conn.get('horreum:alias1')
 end

data/try/integration/connection/operation_mode_guards_try.rb

@@ -36,7 +36,7 @@ end
 ## FiberConnectionHandler blocks pipelines
 begin
   # Ensure we're in strict mode for this test
-  Familia.configure { |config| config.pipeline_mode = :strict }
+  Familia.configure { |config| config.pipelined_mode = :strict }
 
   # Simulate middleware connection
   Fiber[:familia_connection] = [Customer.create_dbclient, Familia.middleware_version]

data/try/integration/connection/pipeline_fallback_integration_try.rb

@@ -1,13 +1,13 @@
 #
 # Tests pipeline fallback modes when connection handlers don't support pipelines.
-# Validates that pipeline_mode configuration works correctly with cached connections
+# Validates that pipelined_mode configuration works correctly with cached connections
 # and that the fallback behavior matches transaction fallback patterns.
 #
 
 require_relative '../../support/helpers/test_helpers'
 
 # Store original values
-$original_pipeline_mode = Familia.pipeline_mode
+$original_pipelined_mode = Familia.pipelined_mode
 $original_transaction_mode = Familia.transaction_mode
 
 # Test model for pipeline fallback scenarios
@@ -17,7 +17,7 @@ class PipelineFallbackTest < Familia::Horreum
 end
 
 ## Test 1: Strict mode raises error with cached connection
-Familia.configure { |c| c.pipeline_mode = :strict }
+Familia.configure { |c| c.pipelined_mode = :strict }
 
 # Cache connection at class level (uses DefaultConnectionHandler which doesn't support pipelines)
 PipelineFallbackTest.instance_variable_set(:@dbclient, Familia.create_dbclient)
@@ -28,7 +28,7 @@ customer.pipelined { |c| c.set('key', 'value') }
 #=~> /Cannot start pipeline with.*CachedConnectionHandler/
 
 ## Test 2: Warn mode falls back successfully with cached connection
-Familia.configure { |c| c.pipeline_mode = :warn }
+Familia.configure { |c| c.pipelined_mode = :warn }
 
 # Cache connection at class level
 PipelineFallbackTest.instance_variable_set(:@dbclient, Familia.create_dbclient)
@@ -51,7 +51,7 @@ $warn_result.results[2]
 #=> 'value1'
 
 ## Test 3: Fresh connections still support real pipelines in strict mode
-Familia.configure { |c| c.pipeline_mode = :strict }
+Familia.configure { |c| c.pipelined_mode = :strict }
 
 # Clear cached class-level connection to force CreateConnectionHandler
 PipelineFallbackTest.remove_instance_variable(:@dbclient) if PipelineFallbackTest.instance_variable_defined?(:@dbclient)
@@ -70,7 +70,7 @@ $fresh_result.results.size
 #=> 3
 
 ## Test 4: MultiResult format is correct for fallback
-Familia.configure { |c| c.pipeline_mode = :permissive }
+Familia.configure { |c| c.pipelined_mode = :permissive }
 
 # Cache connection at class level
 PipelineFallbackTest.instance_variable_set(:@dbclient, Familia.create_dbclient)
@@ -92,7 +92,7 @@ $multi_result.results.class
 #=> Array
 
 ## Test 5: Permissive mode silently falls back
-Familia.configure { |c| c.pipeline_mode = :permissive }
+Familia.configure { |c| c.pipelined_mode = :permissive }
 
 # Cache connection at class level
 PipelineFallbackTest.instance_variable_set(:@dbclient, Familia.create_dbclient)
@@ -111,18 +111,18 @@ $permissive_result.results
 #=> ['OK', 1, '1']
 
 ## Test 6: Pipeline mode configuration validation
-Familia.configure { |c| c.pipeline_mode = :invalid }
+Familia.configure { |c| c.pipelined_mode = :invalid }
 #=:> ArgumentError
 #=~> /Pipeline mode must be :strict, :warn, or :permissive/
 
-## Test 7: Default pipeline_mode is :warn
-Familia.instance_variable_set(:@pipeline_mode, nil)
-Familia.pipeline_mode
+## Test 7: Default pipelined_mode is :warn
+Familia.instance_variable_set(:@pipelined_mode, nil)
+Familia.pipelined_mode
 #=> :warn
 
 ## Cleanup: Restore original values
 Familia.configure do |c|
-  c.pipeline_mode = $original_pipeline_mode
+  c.pipelined_mode = $original_pipelined_mode
   c.transaction_mode = $original_transaction_mode
 end
 PipelineFallbackTest.remove_instance_variable(:@dbclient) if PipelineFallbackTest.instance_variable_defined?(:@dbclient)

data/try/integration/create_method_try.rb

@@ -37,7 +37,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 +55,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 +74,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 +90,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 +115,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 +134,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 +148,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 +172,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 +198,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 +214,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/data_types/datatype_pipelines_try.rb

@@ -0,0 +1,104 @@
+# 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!