active_validation 1.1.0 → 2.0.0

data/lib/active_validation/validators/base64_validator.rb

@@ -1,9 +1,24 @@
 class Base64Validator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless value =~ /^(?:[A-Za-z0-9+$]{4})*(?:[A-Za-z0-9+$]{2}==|[A-Za-z0-9+$]{3}=|[A-Za-z0-9+$]{4})$/
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.base64'))
     end
   end
 
+  private
+
+  def valid_format?(value)
+    value =~ /^(?:[A-Za-z0-9+$]{4})*(?:[A-Za-z0-9+$]{2}==|[A-Za-z0-9+$]{3}=|[A-Za-z0-9+$]{4})$/
+  end
+
+  def valid_length?(value)
+    value.present?
+  end
+
+  def valid?(value)
+    valid_length?(value) &&
+    valid_format?(value)
+  end
+
 end

data/lib/active_validation/validators/coordinate_validator.rb

@@ -7,7 +7,7 @@ class CoordinateValidator < ActiveModel::EachValidator
         "Unknown boundary: #{boundary.inspect}. Valid boundaries are: #{BOUNDARIES.map(&:inspect).join(', ')}"
     end
 
-    unless value.present? && valid_boundary?(value)
+    unless valid?(value, options)
       record.errors[attribute] << (options[:message] || I18n.t("active_validation.errors.messages.coordinate.#{boundary}"))
     end
   end
@@ -16,6 +16,10 @@ class CoordinateValidator < ActiveModel::EachValidator
 
   BOUNDARIES = [:coordinate, :latitude, :longitude]
 
+  def valid_length?(value)
+    value.present?
+  end
+
   def valid_latitude?(value)
     value >= -90 && value <= 90
   end
@@ -24,8 +28,8 @@ class CoordinateValidator < ActiveModel::EachValidator
     value >= -180 && value <= 180
   end
 
-  def valid_boundary?(value)
-    case options[:boundary]
+  def valid_boundary?(value, options)
+    case options.fetch(:boundary, :coordinate)
     when :latitude
       valid_latitude?(value)
     when :longitude
@@ -35,4 +39,9 @@ class CoordinateValidator < ActiveModel::EachValidator
     end
   end
 
+  def valid?(value, options)
+    valid_length?(value) &&
+    valid_boundary?(value, options)
+  end
+
 end

data/lib/active_validation/validators/credit_card_validator.rb

@@ -1,7 +1,7 @@
 class CreditCardValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless valid?(value, options)
+    unless valid?(value.to_s, options)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.credit_card'))
     end
   end
@@ -50,14 +50,12 @@ class CreditCardValidator < ActiveModel::EachValidator
   }
 
   def valid_format?(value, options)
-    if options[:strict]
-      value =~ /^[0-9]+$/
-    else
-      value =~ /^[0-9 -.]+$/
-    end
+    value =~ (options.fetch(:strict, false) ? /^[0-9]+$/ : /^[0-9 -.]+$/)
   end
 
   def valid_length?(value, options)
+    return(false) unless value.present?
+
     value_size_range = DEFAULT_LENGTHS.values.flatten.uniq.sort
     value_size       = value.size
 
@@ -112,18 +110,20 @@ class CreditCardValidator < ActiveModel::EachValidator
     when :visa
       DEFAULT_PREFIXES[:visa].any? { |prefix| value.start_with?(prefix) }
     else
-      responses = []
+      result = false
       DEFAULT_LENGTHS.each do |key, values|
         if values.include?(value_size)
-          responses << DEFAULT_PREFIXES[key].any? { |prefix| value.start_with?(prefix) }
+          result = DEFAULT_PREFIXES[key].any? { |prefix| value.start_with?(prefix) }
+          break if result
         end
+        break if result
       end
-      responses.include?(true)
+      return(result)
     end
   end
 
   def valid?(value, options)
-    striped_value = value.to_s.gsub(/\D/, '')
+    striped_value = value.gsub(/\D/, '')
 
     valid_format?(value, options) &&
     valid_length?(striped_value, options) &&

data/lib/active_validation/validators/currency_validator.rb

@@ -1,7 +1,7 @@
 class CurrencyValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless valid?(value, options)
+    unless valid?(value.to_s, options)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.currency'))
     end
   end
@@ -9,14 +9,15 @@ class CurrencyValidator < ActiveModel::EachValidator
   private
 
   def valid_format?(value, options)
-    if options[:strict]
-      value =~ /^\d+(\.\d{2})$/
-    else
-      value =~ /^\d*+(\.\d{1,2})$/
-    end
+    value =~ (options.fetch(:strict, false) ? /^\d+(\.\d{2})$/ : /^\d*+(\.\d{1,2})$/)
+  end
+
+  def valid_length?(value)
+    value.present?
   end
 
   def valid?(value, options)
+    valid_length?(value) &&
     valid_format?(value, options)
   end
 

data/lib/active_validation/validators/cusip_validator.rb

@@ -8,24 +8,26 @@ class CusipValidator < ActiveModel::EachValidator
 
   private
 
-  def valid_format?(value)
-    value =~ /^[0-9A-Z]{9}$/
-  end
-
   def valid_checksum?(value)
-    digits      = value.split('').map { |i| i.match(/[A-Z]/) ? (i.ord - 55) : i.to_i }
+    digits      = value.chars.map { |character| character.match(/[A-Z]/) ? (character.ord - 55) : character.to_i }
     even_values = digits.values_at(* digits.each_index.select { |i| i.even? })
     odd_values  = digits.values_at(* digits.each_index.select { |i| i.odd? })
     values      = odd_values.map { |i| i * 2 }.zip(even_values).flatten
-
-    values = values.inject(0) do |sum, i|
-      sum += (i / 10) + i % 10
-    end
+    values      = values.inject(0) { |sum, i| sum += (i / 10) + i % 10 }
 
     ((10 - values) % 10) % 10
   end
 
+  def valid_format?(value)
+    value =~ /^[0-9A-Z]{9}$/
+  end
+
+  def valid_length?(value)
+    value.present?
+  end
+
   def valid?(value)
+    valid_length?(value) &&
     valid_format?(value) &&
     valid_checksum?(value)
   end

data/lib/active_validation/validators/email_validator.rb

@@ -1,25 +1,29 @@
 class EmailValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless valid?(value, options)
+    unless valid?(value.to_s, options)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.email'))
     end
   end
 
   private
 
+  def valid_domain?(value, options)
+    options.empty? || options.any? { |domain| value.downcase.end_with?(".#{domain.downcase}") }
+  end
+
   def valid_format?(value)
     value =~ /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i
   end
 
-  def valid_domain?(value, options)
-    value_downcased = value.downcase
-    options.empty? || options.any? { |domain| value_downcased.end_with?(".#{domain.downcase}") }
+  def valid_length?(value)
+    value.present?
   end
 
-  def valid?(email, options)
-    valid_format?(email) &&
-    valid_domain?(email, [*(options[:domain])])
+  def valid?(value, options)
+    valid_length?(value) &&
+    valid_format?(value) &&
+    valid_domain?(value, [*(options[:domain])])
   end
 
 end

data/lib/active_validation/validators/equality_validator.rb

@@ -1,15 +1,22 @@
 class EqualityValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    to_value = options[:to]
-    operator = OPERATORS.fetch(options[:operator])
+    to = options.fetch(:to, nil)
+    if to.nil?
+      raise ArgumentError,
+        "ArgumentError: missing ':to' attribute for comparison."
+    end
 
-    if (options[:operator] == :less_than) || (options[:operator] == :greater_than)
-	    raise Exception if to_value.nil? || operator.nil?
-  	end
+    operator  = options.fetch(:operator, nil)
+    operators = OPERATORS.keys
+    unless operators.include?(operator)
+      raise ArgumentError,
+        "Unknown operator: #{operator.inspect}. Valid operators are: #{operators.map(&:inspect).join(', ')}"
+    end
+    operator  = OPERATORS.fetch(operator)
 
-    unless value.send(operator, record.send(to_value.to_sym))
-      record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.equality', attr: to_value, operator: operator))
+    unless value.send(operator.to_sym, record.send(to.to_sym))
+      record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.equality', attr: to, operator: operator))
     end
   end
 

data/lib/active_validation/validators/hex_validator.rb

@@ -1,9 +1,24 @@
 class HexValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless value =~ /^#?([A-Fa-f0-9]{6}|[A-Fa-f0-9]{3})$/
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.hex'))
     end
   end
 
+  private
+
+  def valid_format?(value)
+    value =~ /^#?([A-Fa-f0-9]{6}|[A-Fa-f0-9]{3})$/
+  end
+
+  def valid_length?(value)
+    value.present?
+  end
+
+  def valid?(value)
+    valid_length?(value) &&
+    valid_format?(value)
+  end
+
 end

data/lib/active_validation/validators/imei_validator.rb

@@ -12,24 +12,26 @@ class ImeiValidator < ActiveModel::EachValidator
     value =~ /\A[\d\.\:\-\s]+\z/i
   end
 
-  def valid_luhn?(value)
-    numbers = value.gsub(/\D/, '').reverse
-
-    sum, i = 0, 0
-
-    numbers.each_char do |ch|
-      n = ch.to_i
-      n *= 2 if i.odd?
-      n = 1 + (n - 10) if n >= 10
+  def valid_length?(value)
+    value.present?
+  end
 
-      sum += n
-      i   += 1
+  def valid_luhn?(value)
+    number = value.gsub(/\D/, '').reverse
+
+    total = 0
+    number.chars.each_with_index do |character, i|
+      result = character.to_i
+      result *= 2 if i.odd?
+      result = (1 + (result - 10)) if (result >= 10)
+      total  += result
     end
 
-    (sum % 10).zero?
+    (total % 10).zero?
   end
 
   def valid?(value)
+    valid_length?(value) &&
     valid_format?(value) &&
     valid_luhn?(value)
   end

data/lib/active_validation/validators/ip_validator.rb

@@ -1,9 +1,24 @@
 class IpValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless value =~ /^(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)$/
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.ip'))
     end
   end
 
+  private
+
+  def valid_format?(value)
+    value =~ /^(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)$/
+  end
+
+  def valid_length?(value)
+    value.present?
+  end
+
+  def valid?(value)
+    valid_length?(value) &&
+    valid_format?(value)
+  end
+
 end

data/lib/active_validation/validators/isbn_validator.rb

@@ -9,15 +9,19 @@ class IsbnValidator < ActiveModel::EachValidator
   private
 
   def valid_format?(value)
-    value = '' if value.nil?
-    value.gsub!(/-| /, '')
-    value.downcase!
+    value = '' if value.blank?
+    value = value.gsub(/-| /, '').downcase.chars
 
     [10, 13].include?(value.size) &&
-    value.chars.all? { |char| ['0','1','2','3','4','5','6','7','8','9','0','x'].include?(char) }
+    value.all? { |character| ['0', '1', '2' , '3', '4', '5', '6', '7', '8', '9', '0', 'x'].include?(character) }
+  end
+
+  def valid_length?(value)
+    value.present?
   end
 
   def valid?(value)
+    valid_length?(value) &&
     valid_format?(value)
   end
 

data/lib/active_validation/validators/isin_validator.rb

@@ -1,36 +1,38 @@
 class IsinValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless valid?(value)
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.isin'))
     end
   end
 
   private
 
-  def valid_format?(value)
-    value =~ /^((AF|AX|AL|DZ|AS|AD|AO|AI|AQ|AG|AR|AM|AW|AU|AT|AZ|BS|BH|BD|BB|BY|BE|BZ|BJ|BM|BT|BO|BQ|BA|BW|BV|BR|IO|BN|BG|BF|BI|KH|CM|CA|CV|KY|CF|TD|CL|CN|CX|CC|CO|KM|CG|CD|CK|CR|CI|HR|CU|CW|CY|CZ|DK|DJ|DM|DO|EC|EG|SV|GQ|ER|EE|ET|FK|FO|FJ|FI|FR|GF|PF|TF|GA|GM|GE|DE|GH|GI|GR|GL|GD|GP|GU|GT|GG|GN|GW|GY|HT|HM|VA|HN|HK|HU|IS|IN|ID|IR|IQ|IE|IM|IL|IT|JM|JP|JE|JO|KZ|KE|KI|KP|KR|KW|KG|LA|LV|LB|LS|LR|LY|LI|LT|LU|MO|MK|MG|MW|MY|MV|ML|MT|MH|MQ|MR|MU|YT|MX|FM|MD|MC|MN|ME|MS|MA|MZ|MM|NA|NR|NP|NL|NC|NZ|NI|NE|NG|NU|NF|MP|NO|OM|PK|PW|PS|PA|PG|PY|PE|PH|PN|PL|PT|PR|QA|RE|RO|RU|RW|BL|SH|KN|LC|MF|PM|VC|WS|SM|ST|SA|SN|RS|SC|SL|SG|SX|SK|SI|SB|SO|ZA|GS|SS|ES|LK|SD|SR|SJ|SZ|SE|CH|SY|TW|TJ|TZ|TH|TL|TG|TK|TO|TT|TN|TR|TM|TC|TV|UG|UA|AE|GB|US|UM|UY|UZ|VU|VE|VN|VG|VI|WF|EH|YE|ZM|ZW{2})([A-Z0-9]{9}))(\d{1})$/
-  end
-
   def valid_checksum?(value)
-    digits      = value.split('').map { |i| i.match(/[A-Z]/) ? (i.ord - 55) : i.to_i }
+    characters  = value.chars
+    digits      = characters.map { |character| character.match(/[A-Z]/) ? (character.ord - 55) : character.to_i }
     even_values = digits.values_at(* digits.each_index.select { |i| i.even? })
     odd_values  = digits.values_at(* digits.each_index.select { |i| i.odd? })
 
-    longest, shortest = if even_values.last == value.split('').map(&:to_i)
-      [even_values, odd_values]
-    else
-      [odd_values, even_values]
-    end
+    longest, shortest = (even_values.last == characters.map(&:to_i)) ? [even_values, odd_values] : [odd_values, even_values]
 
     longest = longest.map { |i| i * 2 }
-    values  = (longest.concat(shortest)).to_s.scan(/./).map(&:to_i)
-    sum     = values.inject(&:+)
+    values  = longest.concat(shortest).to_s.scan(/./).map(&:to_i)
+    total   = values.inject(&:+)
+
+    (10 - (total % 10)) % 10
+  end
+
+  def valid_format?(value)
+    value =~ /^((AF|AX|AL|DZ|AS|AD|AO|AI|AQ|AG|AR|AM|AW|AU|AT|AZ|BS|BH|BD|BB|BY|BE|BZ|BJ|BM|BT|BO|BQ|BA|BW|BV|BR|IO|BN|BG|BF|BI|KH|CM|CA|CV|KY|CF|TD|CL|CN|CX|CC|CO|KM|CG|CD|CK|CR|CI|HR|CU|CW|CY|CZ|DK|DJ|DM|DO|EC|EG|SV|GQ|ER|EE|ET|FK|FO|FJ|FI|FR|GF|PF|TF|GA|GM|GE|DE|GH|GI|GR|GL|GD|GP|GU|GT|GG|GN|GW|GY|HT|HM|VA|HN|HK|HU|IS|IN|ID|IR|IQ|IE|IM|IL|IT|JM|JP|JE|JO|KZ|KE|KI|KP|KR|KW|KG|LA|LV|LB|LS|LR|LY|LI|LT|LU|MO|MK|MG|MW|MY|MV|ML|MT|MH|MQ|MR|MU|YT|MX|FM|MD|MC|MN|ME|MS|MA|MZ|MM|NA|NR|NP|NL|NC|NZ|NI|NE|NG|NU|NF|MP|NO|OM|PK|PW|PS|PA|PG|PY|PE|PH|PN|PL|PT|PR|QA|RE|RO|RU|RW|BL|SH|KN|LC|MF|PM|VC|WS|SM|ST|SA|SN|RS|SC|SL|SG|SX|SK|SI|SB|SO|ZA|GS|SS|ES|LK|SD|SR|SJ|SZ|SE|CH|SY|TW|TJ|TZ|TH|TL|TG|TK|TO|TT|TN|TR|TM|TC|TV|UG|UA|AE|GB|US|UM|UY|UZ|VU|VE|VN|VG|VI|WF|EH|YE|ZM|ZW{2})([A-Z0-9]{9}))(\d{1})$/
+  end
 
-    (10 - (sum % 10)) % 10
+  def valid_length?(value)
+    value.present?
   end
 
   def valid?(value)
+    valid_length?(value) &&
     valid_format?(value) &&
     valid_checksum?(value)
   end

data/lib/active_validation/validators/mac_address_validator.rb

@@ -1,23 +1,37 @@
 class MacAddressValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless valid?(value)
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.mac_address'))
     end
   end
 
   private
 
+  DEFAULT_FORMATS = [
+    /^([\h]{2}:){5}[\h]{2}?$/i,
+    /^([\h]{2}[-|\.|\s]){5}[\h]{2}?$/i,
+    /^([\h]{6})[-|\.][\h]{6}?$/i,
+    /^([\h]{6}):[\h]{6}?$/i,
+    /^([\h]{4}[-|\.|\s]){2}[\h]{4}?$/i,
+    /^[\h]{12}?$/i
+  ]
+
   def valid_format?(value)
-    (value =~ /^([\h]{2}:){5}[\h]{2}?$/i) ||
-    (value =~ /^([\h]{2}[-|\.|\s]){5}[\h]{2}?$/i) ||
-    (value =~ /^([\h]{6})[-|\.][\h]{6}?$/i) ||
-    (value =~ /^([\h]{6}):[\h]{6}?$/i) ||
-    (value =~ /^([\h]{4}[-|\.|\s]){2}[\h]{4}?$/i) ||
-    (value =~ /^[\h]{12}?$/i)
+    result = false
+    DEFAULT_FORMATS.each do |pattern|
+      result = (value =~ pattern)
+      break if result
+    end
+    return(result)
+  end
+
+  def valid_length?(value)
+    value.present?
   end
 
   def valid?(value)
+    valid_length?(value) &&
     valid_format?(value)
   end
 

data/lib/active_validation/validators/name_validator.rb

@@ -1,9 +1,24 @@
 class NameValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless value =~ /\A([a-zA-Z'-]+\s+){1,4}[a-zA-Z'-]*\z/i
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.name'))
     end
   end
 
+  private
+
+  def valid_format?(value)
+    value =~ /\A([a-zA-Z'-]+\s+){1,4}[a-zA-Z'-]*\z/i
+  end
+
+  def valid_length?(value)
+    value.present?
+  end
+
+  def valid?(value)
+    valid_length?(value) &&
+    valid_format?(value)
+  end
+
 end

data/lib/active_validation/validators/password_validator.rb

@@ -1,7 +1,7 @@
 class PasswordValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless valid?(value, options)
+    unless valid?(value.to_s, options)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.password'))
     end
   end
@@ -9,14 +9,15 @@ class PasswordValidator < ActiveModel::EachValidator
   private
 
   def valid_format?(value, options)
-    if options[:strict]
-      value =~ /^(?=^.{1,255}$)((?=.*[A-Za-z0-9])(?=.*[A-Z])(?=.*[a-z]))^.*$/
-    else
-      value =~ /^[a-z0-9!@#$%^&*_-]{1,255}$/
-    end
+    value =~ (options.fetch(:strict, false) ? /^(?=^.{1,255}$)((?=.*[A-Za-z0-9])(?=.*[A-Z])(?=.*[a-z]))^.*$/ : /^[a-z0-9!@#$%^&*_-]{1,255}$/)
+  end
+
+  def valid_length?(value)
+    value.present?
   end
 
   def valid?(value, options)
+    valid_length?(value) &&
     valid_format?(value, options)
   end
 

data/lib/active_validation/validators/phone_validator.rb

@@ -1,9 +1,24 @@
 class PhoneValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless value =~ /^[0-9+\(\)#\.\s\/ext-]+$/i
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.phone'))
     end
   end
 
+  private
+
+  def valid_format?(value)
+    value =~ /^[0-9+\(\)#\.\s\/ext-]+$/i
+  end
+
+  def valid_length?(value)
+    value.present?
+  end
+
+  def valid?(value)
+    valid_length?(value) &&
+    valid_format?(value)
+  end
+
 end

data/lib/active_validation/validators/sedol_validator.rb

@@ -1,30 +1,35 @@
 class SedolValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless valid?(value)
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.sedol'))
     end
   end
 
   private
 
-  def valid_format?(value)
-    value =~ /^([A-Z0-9]{6})(\d{1})$/
-  end
-
   def valid_checksum?(value)
-    digits  = value.split('').map { |i| i.match(/[A-Z]/) ? (i.ord - 55) : i.to_i }
+    digits  = value.chars.map { |digit| digit.match(/[A-Z]/) ? (digit.ord - 55) : digit.to_i }
     weights = [1, 3, 1, 7, 3, 9, 1]
-    sum     = 0
 
-    digits.each_with_index do |i, idx|
-      sum += weights[idx] * i
+    total = 0
+    digits.each_with_index do |digit, i|
+      total += (weights[i] * digit)
     end
 
-    (10 - sum % 10) % 10
+    (10 - total % 10) % 10
+  end
+
+  def valid_format?(value)
+    value =~ /^([A-Z0-9]{6})(\d{1})$/
+  end
+
+  def valid_length?(value)
+    value.present?
   end
 
   def valid?(value)
+    valid_length?(value) &&
     valid_format?(value) &&
     valid_checksum?(value)
   end

data/lib/active_validation/validators/slug_validator.rb

@@ -1,9 +1,24 @@
 class SlugValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless value =~ /^[a-z0-9-]+$/
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.slug'))
     end
   end
 
+  private
+
+  def valid_format?(value)
+    value =~ /^[a-z0-9_-]+$/
+  end
+
+  def valid_length?(value)
+    value.present?
+  end
+
+  def valid?(value)
+    valid_length?(value) &&
+    valid_format?(value)
+  end
+
 end

data/lib/active_validation/validators/ssn_validator.rb

@@ -1,9 +1,24 @@
 class SsnValidator < ActiveModel::EachValidator
 
   def validate_each(record, attribute, value)
-    unless value =~ /^\A([\d]{3}\-[\d]{2}\-[\d]{4}|[\d]{9})\Z$/
+    unless valid?(value.to_s)
       record.errors[attribute] << (options[:message] || I18n.t('active_validation.errors.messages.ssn'))
     end
   end
 
+  private
+
+  def valid_format?(value)
+    value =~ /^\A([\d]{3}\-[\d]{2}\-[\d]{4}|[\d]{9})\Z$/
+  end
+
+  def valid_length?(value)
+    value.present?
+  end
+
+  def valid?(value)
+    valid_length?(value) &&
+    valid_format?(value)
+  end
+
 end