Merge pull request #3 from Kaligo/chore/move-validation-message

Move Validations::Message to inside HeavyKeeper namespace

Author: hieuk09

.rspec

@@ -1,3 +1,4 @@
---format documentation
+--format progress
+--warnings
 --color
 --require spec_helper

CHANGELOG.md

@@ -1,3 +1,8 @@
+## [0.1.6] - 2025-02-04
+
+- Move `Validations::Message` to inside HeavyKeeper namespace to avoid
+conflict
+
 ## [0.1.5] - 2024-06-13
 
 - Support latest version of xx-hash

Gemfile.redis.4

@@ -11,7 +11,7 @@ gem 'rake', '~> 13.0'
 gem 'redis', '~> 4.0'
 
 group :test do
-  gem 'mock_redis'
+  gem 'mock_redis', '< 0.47.0'
   gem 'rspec'
   gem 'pry-byebug'
 end

lib/heavy_keeper/top_k.rb

@@ -3,7 +3,7 @@
 require 'dry-schema'
 require 'securerandom'
 require 'xxhash'
-require_relative '../validations/message'
+require_relative 'validations/message'
 
 module HeavyKeeper
   class TopK # rubocop:disable Metrics/ClassLength
@@ -217,7 +217,7 @@ def validate(options)
       result = Validator.call(options)
 
       if result.failure?
-        error = ::Validations::Message.new.build(result.errors.to_h).join('. ')
+        error = Validations::Message.new.build(result.errors.to_h).join('. ')
         raise HeavyKeeper::Error, error
       end
 

lib/heavy_keeper/validations/message.rb

@@ -0,0 +1,37 @@
+module HeavyKeeper
+  module Validations
+    class Message
+      CLASSIFY_SEPARATOR = '_'.freeze
+      TITLEIZE_SEPARATOR = ' '.freeze
+
+      # @errors [Hash | Array] output of dry-validation
+      #         after validating params
+      # @parent [Nil | String] key name of a field that has `errors`
+      #         after validating params
+      # Output: array of string that can be used to feed into
+      # Errors::InvalidParamsError
+      def build(errors, parent = nil)
+        case errors
+        when Hash
+          errors.flat_map do |key, value|
+            child = [parent, key].compact.join(' ')
+            build(value, child)
+          end
+        when Array
+          errors.flat_map do |error|
+            "#{titleize(parent.to_s)} #{build(error)}"
+          end
+        else
+          errors
+        end
+      end
+
+      private
+
+      def titleize(string)
+        # NOTE: this is not a robust implementation of titleize
+        string.split(CLASSIFY_SEPARATOR).map(&:capitalize).join(TITLEIZE_SEPARATOR)
+      end
+    end
+  end
+end

lib/heavy_keeper/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HeavyKeeper
-  VERSION = '0.1.5'
+  VERSION = '0.1.6'
 end