familia 2.0.0.pre2 → 2.0.0.pre4

data/try/core/utils_try.rb

@@ -7,104 +7,6 @@ Familia.debug = false
 
 # Test Familia utility methods and helpers
 
-##
-## ID Generation
-##
-
-## Familia.generate_id
-Familia.respond_to?(:generate_id)
-#=> true
-
-## Can generate a default base-36 ID
-id = Familia.generate_id
-[id.class, id.length > 10, id.match?(/^[a-z0-9]+$/)]
-#=> [String, true, true]
-
-## Generated IDs are unique
-[Familia.generate_id == Familia.generate_id]
-#=> [false]
-
-## Can generate an ID with a custom base (hex)
-hex_id = Familia.generate_id(16)
-[hex_id.class, hex_id.length == 64, hex_id.match?(/^[a-f0-9]+$/)]
-#=> [String, true, true]
-
-## Familia.generate_trace_id
-Familia.respond_to?(:generate_trace_id)
-#=> true
-
-## Can generate a default base-36 trace ID
-trace_id = Familia.generate_trace_id
-[trace_id.class, trace_id.length > 5, trace_id.length < 20]
-#=> [String, true, true]
-
-## Can generate a trace ID with a custom base (hex)
-hex_trace_id = Familia.generate_trace_id(16)
-[hex_trace_id.class, hex_trace_id.length == 16]
-#=> [String, true]
-
-## Familia.generate_hex_id
-Familia.respond_to?(:generate_hex_id)
-#=> true
-
-## Can generate a 256-bit hex ID
-hex_id = Familia.generate_hex_id
-[hex_id.class, hex_id.length == 64, hex_id.match?(/^[a-f0-9]+$/)]
-#=> [String, true, true]
-
-## Familia.generate_hex_trace_id
-Familia.respond_to?(:generate_hex_trace_id)
-#=> true
-
-## Can generate a 64-bit hex trace ID
-hex_trace_id = Familia.generate_hex_trace_id
-[hex_trace_id.class, hex_trace_id.length == 16, hex_trace_id.match?(/^[a-f0-9]+$/)]
-#=> [String, true, true]
-
-
-##
-## ID Shortening
-##
-
-## Familia.shorten_to_external_id
-Familia.respond_to?(:shorten_to_external_id)
-#=> true
-
-## Can shorten hex ID to external ID (128 bits)
-hex_id = Familia.generate_hex_id
-external_id = Familia.shorten_to_external_id(hex_id)
-[external_id.class, external_id.length < hex_id.length]
-#=> [String, true]
-
-## Can shorten hex ID to external ID with custom base (hex)
-hex_id = Familia.generate_hex_id
-hex_external_id = Familia.shorten_to_external_id(hex_id, base: 16)
-[hex_external_id.class, hex_external_id.length == 32]
-#=> [String, true]
-
-## Familia.shorten_to_trace_id
-Familia.respond_to?(:shorten_to_trace_id)
-#=> true
-
-## Can shorten hex ID to trace ID (64 bits)
-hex_id = Familia.generate_hex_id
-trace_id = Familia.shorten_to_trace_id(hex_id)
-[trace_id.class, trace_id.length < hex_id.length]
-#=> [String, true]
-
-## Can shorten hex ID to trace ID with custom base (hex)
-hex_id = Familia.generate_hex_id
-hex_trace_id = Familia.shorten_to_trace_id(hex_id, base: 16)
-[hex_trace_id.class, hex_trace_id.length == 16]
-#=> [String, true]
-
-## Shortened IDs are deterministic
-hex_id = Familia.generate_hex_id
-id1 = Familia.shorten_to_external_id(hex_id)
-id2 = Familia.shorten_to_external_id(hex_id)
-id1 == id2
-#=> true
-
 ##
 ## Key helpers
 ##

data/try/datatypes/boolean_try.rb

@@ -7,37 +7,36 @@ Familia.debug = false
 
 @hashkey = Familia::HashKey.new 'key'
 
-
 ## Boolean values are returned as strings, on assignment as string
-@hashkey["test"] = "true"
+@hashkey['test'] = 'true'
 #=> "true"
 
 ## Boolean values are returned as strings
-@hashkey["test"]
+@hashkey['test']
 #=> "true"
 
 ## Trying to store a boolean value to a hash key raises an exception
 begin
-  @hashkey["test"] = true
+  @hashkey['test'] = true
 rescue Familia::HighRiskFactor => e
   e.message
 end
 #=> "High risk factor for serialization bugs: true<TrueClass>"
 
 ## Boolean values are returned as strings
-@hashkey["test"]
+@hashkey['test']
 #=> "true"
 
 ## Trying to store a nil value to a hash key raises an exception
 begin
-  @hashkey["test"] = nil
+  @hashkey['test'] = nil
 rescue Familia::HighRiskFactor => e
   e.message
 end
 #=> "High risk factor for serialization bugs: <NilClass>"
 
 ## The exceptions prevented the hash from being updated
-@hashkey["test"]
+@hashkey['test']
 #=> "true"
 
 ## Clear the hash key

data/try/datatypes/datatype_base_try.rb

@@ -22,7 +22,7 @@ p [@a.name, @b.name]
 #=> true
 
 ## Limiter#qstamp
-@limiter1.counter.qstamp(10.minutes, '%H:%M', 1302468980)
+@limiter1.counter.qstamp(10.minutes, '%H:%M', 1_302_468_980)
 ##=> '20:50'
 
 ## Database Types can be stored to quantized stamp suffix
@@ -33,7 +33,7 @@ p [@a.name, @b.name]
 @limiter2 = Limiter.new :requests
 p [@limiter1.default_expiration, @limiter2.default_expiration]
 p [@limiter1.counter.parent.default_expiration, @limiter2.counter.parent.default_expiration]
-@limiter2.counter.qstamp(10.minutes, pattern: nil, time: 1302468980)
+@limiter2.counter.qstamp(10.minutes, pattern: nil, time: 1_302_468_980)
 #=> 1302468600
 
 ## Database Types can be stored to quantized numeric suffix. This

data/try/datatypes/hash_try.rb

@@ -51,5 +51,4 @@ require_relative '../helpers/test_helpers'
 @a.props.values_at 'fieldA', 'counter', 'fieldC'
 #=> ['1', '40', '3']
 
-
 @a.props.delete!

data/try/datatypes/list_try.rb

@@ -1,6 +1,5 @@
 # try/datatypes/list_try.rb
 
-
 require_relative '../../lib/familia'
 require_relative '../helpers/test_helpers'
 

data/try/datatypes/set_try.rb

@@ -1,6 +1,5 @@
 # try/datatypes/set_try.rb
 
-
 require_relative '../../lib/familia'
 require_relative '../helpers/test_helpers'
 
@@ -32,5 +31,4 @@ ret.class
 @a.tags.size
 #=> 3
 
-
 @a.tags.delete!

data/try/datatypes/sorted_set_try.rb

@@ -3,7 +3,6 @@
 require_relative '../../lib/familia'
 require_relative '../helpers/test_helpers'
 
-
 @a = Bone.new 'atoken'
 
 ## Familia::SortedSet#add
@@ -36,7 +35,7 @@ require_relative '../helpers/test_helpers'
 #=> ['metric1', 'metric2', 'metric3']
 
 ## Familia::SortedSet#rangebyscore with a limit
-@a.metrics.rangebyscore 1, 3, :limit => [0, 2]
+@a.metrics.rangebyscore 1, 3, limit: [0, 2]
 #=> ['metric1', 'metric2']
 
 ## Familia::SortedSet#increment

data/try/datatypes/string_try.rb

@@ -14,7 +14,7 @@ require_relative '../helpers/test_helpers'
 #=> 'GREAT!'
 
 ## Familia::String#value=
-@a.value.value = "DECENT!"
+@a.value.value = 'DECENT!'
 #=> 'DECENT!'
 
 ## Familia::String#to_s
@@ -62,5 +62,4 @@ require_relative '../helpers/test_helpers'
 @ret.value
 #=> '1050bytes'
 
-
 @ret.delete!

data/try/edge_cases/empty_identifiers_try.rb

@@ -1,48 +1,55 @@
-require_relative '../helpers/test_helpers'
+# try/edge_cases/empty_identifiers_try.rb
 
 # Test empty identifier edge cases
-group "Empty Identifier Edge Cases"
 
-setup do
-  @user_class = Class.new(Familia::Horreum) do
-    identifier :email
-    field :name
-  end
-end
+require_relative '../helpers/test_helpers'
 
-try "empty string identifier causes stack overflow" do
-  user = @user_class.new(email: "", name: "Test")
 
-  begin
-    user.exists?  # This should cause infinite loop
-    false
-  rescue SystemStackError
-    true  # Expected stack overflow
-  rescue => e
-    false # Unexpected error
+## empty string identifier handling
+begin
+  user_class = Class.new(Familia::Horreum) do
+    identifier_field :email
+    field :email
+    field :name
   end
+  user = user_class.new(email: '', name: 'Test')
+  user.exists? # Test actual behavior with empty identifier
+rescue SystemStackError
+  'stack_overflow'  # Stack overflow occurred
+rescue StandardError => e
+  e.class.name  # Other error occurred
 end
+#=> "Familia::NoIdentifier"
 
-try "nil identifier causes stack overflow" do
-  user = @user_class.new(email: nil, name: "Test")
-
-  begin
-    user.exists?
-    false
-  rescue SystemStackError, Familia::NoIdentifier
-    true  # Expected error
+## nil identifier handling
+begin
+  user_class = Class.new(Familia::Horreum) do
+    identifier_field :email
+    field :email
+    field :name
   end
+  user = user_class.new(email: nil, name: 'Test')
+  user.exists?
+rescue SystemStackError
+  'stack_overflow'
+rescue Familia::NoIdentifier => e
+  'no_identifier'
+rescue StandardError => e
+  e.class.name
 end
+#=> "no_identifier"
 
-try "validation workaround prevents stack overflow" do
-  user = @user_class.new(email: "", name: "Test")
-
-  # Workaround: validate before operations
-  if user.identifier.to_s.empty?
-    raise ArgumentError, "Empty identifier"
+## empty identifier validation check
+begin
+  user_class = Class.new(Familia::Horreum) do
+    identifier_field :email
+    field :email
+    field :name
   end
-
-  false  # Should not reach here
-rescue ArgumentError => e
-  e.message.include?("Empty identifier")
+  user = user_class.new(email: '', name: 'Test')
+  # Check if identifier is empty
+  user.identifier.to_s.empty?
+rescue StandardError => e
+  e.class.name
 end
+#=> true

data/try/edge_cases/hash_symbolization_try.rb

@@ -16,9 +16,9 @@ class SymbolizeTest < Familia::Horreum
   field :config
 end
 
-@test_hash = { "name" => "John", "age" => 30, "nested" => { "theme" => "dark" } }
+@test_hash = { 'name' => 'John', 'age' => 30, 'nested' => { 'theme' => 'dark' } }
 @test_obj = SymbolizeTest.new
-@test_obj.id = "symbolize_test_1"
+@test_obj.id = 'symbolize_test_1'
 @test_obj.config = @test_hash
 @test_obj.save
 
@@ -55,18 +55,18 @@ end
 #=> ["name", "age", "nested"]
 
 ## Nested hash in string result also has string keys
-@string_result["nested"].keys
+@string_result['nested'].keys
 #=> ["theme"]
 
 ## Values are preserved correctly in both cases
 @symbol_result[:name]
 #=> "John"
 
-@string_result["name"]
+@string_result['name']
 #=> "John"
 
 ## Arrays are handled correctly too
-@test_obj.config = [{ "item" => "value" }, "string", 123]
+@test_obj.config = [{ 'item' => 'value' }, 'string', 123]
 @test_obj.save
 @array_json = @test_obj.hget('config')
 #=> "[{\"item\":\"value\"},\"string\",123]"

data/try/edge_cases/json_serialization_try.rb

@@ -14,22 +14,21 @@ class JsonTest < Familia::Horreum
   field :simple      # This should store simple strings as-is
 end
 
-
 ## Test 1: Store a Hash - should serialize to JSON automatically
 test_obj = JsonTest.new
-test_obj.config = { theme: "dark", notifications: true, settings: { volume: 80 } }
+test_obj.config = { theme: 'dark', notifications: true, settings: { volume: 80 } }
 test_obj.config
 #=:> Hash
 
 ## Test 2: Store an Array - should serialize to JSON automatically
 test_obj = JsonTest.new
-test_obj.tags = ["ruby", "valkey", "json", "familia"]
+test_obj.tags = %w[ruby valkey json familia]
 test_obj.tags
 #=:> Array
 
 ## Test 3: Store a simple string - should remain as string
 test_obj = JsonTest.new
-test_obj.simple = "just a string"
+test_obj.simple = 'just a string'
 test_obj.simple
 #=:> String
 
@@ -40,25 +39,25 @@ test_obj.save
 
 ## Verify what's actually stored in Database (raw)
 test_obj = JsonTest.new
-test_obj.id = "json_test_1"
-test_obj.config = { theme: "dark", notifications: true, settings: { volume: 80 } }
-test_obj.simple = "just a string"
-test_obj.tags = ["ruby", "valkey", "json", "familia"]
+test_obj.id = 'json_test_1'
+test_obj.config = { theme: 'dark', notifications: true, settings: { volume: 80 } }
+test_obj.simple = 'just a string'
+test_obj.tags = %w[ruby valkey json familia]
 test_obj.save
 test_obj.hgetall
 #=> {"id"=>"json_test_1", "config"=>"{\"theme\":\"dark\",\"notifications\":true,\"settings\":{\"volume\":80}}", "tags"=>"[\"ruby\",\"valkey\",\"json\",\"familia\"]", "simple"=>"just a string"}
 
 ## Test 4: Hash should be deserialized back to Hash
 test_obj = JsonTest.new 'any_id_will_do'
-puts "Config after refresh:"
+puts 'Config after refresh:'
 puts test_obj.config
-puts "Config class: "
+puts 'Config class: '
 [test_obj.config.class, test_obj.config]
 ##=> [Hash, {:theme=>"dark", :notifications=>true, :settings=>{:volume=>80}}]
 
 ## Test 5: Array should be deserialized back to Array
 test_obj = JsonTest.new 'any_id_will_do'
-puts "Tags after refresh:"
+puts 'Tags after refresh:'
 puts test_obj.tags.inspect
 puts "Tags class: #{test_obj.tags.class}"
 test_obj.tags.inspect
@@ -67,7 +66,7 @@ test_obj.tags
 
 ## Test 6: Simple string should remain a string (this works correctly)
 test_obj = JsonTest.new 'any_id_will_do'
-puts "Simple after refresh:"
+puts 'Simple after refresh:'
 puts test_obj.simple.inspect
 puts "Simple class: #{test_obj.simple.class}"
 [test_obj.simple.class, test_obj.simple]
@@ -77,7 +76,7 @@ puts "Simple class: #{test_obj.simple.class}"
 test_obj = JsonTest.new 'any_id_will_do'
 puts "\n=== ASYMMETRY DEMONSTRATION ==="
 puts "Before save: config is #{test_obj.config.class}"
-test_obj.config = { example: "data" }
+test_obj.config = { example: 'data' }
 puts "After assignment: config is #{test_obj.config.class}"
 test_obj.save
 puts "After save: config is still #{test_obj.config.class}"

data/try/edge_cases/race_conditions_try.rb

@@ -1,60 +1,57 @@
-require_relative '../helpers/test_helpers'
-
 # Test connection race conditions
-group "Race Conditions Edge Cases"
 
-setup do
-  @user_class = Class.new(Familia::Horreum) do
-    identifier :email
-    field :counter
-  end
+require_relative '../helpers/test_helpers'
+
+## concurrent connection access test
+user_class = Class.new(Familia::Horreum) do
+  identifier_field :email
+  field :email
+  field :counter
 end
 
-try "concurrent connection access causes race condition" do
-  user = @user_class.new(email: "test@example.com", counter: 0)
-  user.save
-
-  threads = []
-  results = []
-
-  # Simulate high concurrency
-  10.times do
-    threads << Thread.new do
-      begin
-        user.incr(:counter)
-        results << "success"
-      rescue => e
-        results << "error: #{e.class.name}"
-      end
-    end
-  end
+user = user_class.new(email: 'test@example.com', counter: 0)
+user.save
 
-  threads.each(&:join)
+threads = []
+results = []
 
-  # May show race condition issues
-  errors = results.count { |r| r.start_with?("error") }
-  errors > 0  # Expects some race condition errors
-ensure
-  user&.delete!
+# Simulate high concurrency
+10.times do
+  threads << Thread.new do
+    user.incr(:counter)
+    results << 'success'
+  rescue StandardError => e
+    results << "error: #{e.class.name}"
+  end
 end
 
-try "connection pool stress test" do
-  users = []
-
-  # Create multiple users concurrently
-  threads = []
-  20.times do |i|
-    threads << Thread.new do
-      user = @user_class.new(email: "user#{i}@example.com")
-      user.save
-      users << user
-    end
+threads.each(&:join)
+user.delete!
+
+# Count successful operations
+successes = results.count { |r| r == 'success' }
+successes > 0 # Should have some successes
+#=!> StandardError
+
+## connection pool stress test
+## We're just checking whether it completes within a reasonable time frame.
+## If it does fail either bc of the duration or contention then it's a problem.
+success_count = 0
+threads = []
+mutex = Mutex.new
+
+# Test concurrent connections
+100.times do |i|
+  threads << Thread.new do
+    # Try to get a connection and perform an operation
+    dbclient = Familia.dbclient
+    dbclient.ping
+    success_count += 1
   end
+end
 
-  threads.each(&:join)
+threads.each(&:join)
 
-  # Check for connection issues
-  users.length > 0  # Some should succeed despite race conditions
-ensure
-  users.each(&:delete!) rescue nil
-end
+# Should have some successful connections
+success_count > 0
+#=%> 220