Fix hostname coercion (#33)

Author: Fghurayri

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## [Unreleased]
 
+## [0.7.3] - 2025-09-12
+
+- Fix hostname coercion
+ 
 ## [0.7.1] - 2025-09-07
 
 - Add password rule

lib/mini_defender/rules/hostname.rb

@@ -11,7 +11,7 @@ def coerce(value)
     value.to_s
       .downcase
       .strip
-      .gsub(/.\z/, '')
+      .gsub(/\.\z/, '')
   end
 
   def passes?(attribute, value, validator)

lib/mini_defender/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MiniDefender
-  VERSION = "0.7.2"
+  VERSION = "0.7.3"
 end

test/rules/hostname_test.rb

@@ -2,7 +2,7 @@
 
 require 'test_helper'
 
-class PasswordTest < Minitest::Test
+class HostnameTest < Minitest::Test
   def setup
     @rule = MiniDefender::Rules::Hostname.new
   end
@@ -15,8 +15,13 @@ def test_pass_for_trailing_dot
     assert(@rule.passes?('hostname', 'www.example.com.', nil))
   end
 
-  def test_must_remove_trailing_dot
-    assert(@rule.coerce('www.example.com'))
+  def test_must_coerce_valid_host
+    host = 'www.example.com'
+    assert_equal(@rule.coerce(host), host)
+  end
+
+  def test_must_coerce_remove_trailing_dot
+    assert_equal(@rule.coerce('www.example.com.'), 'www.example.com')
   end
 
   def test_fail_for_non_ascii