Fixed up alert truncating only to work on strings. Added test case.

2 files changed, 20 insertions, 1 deletions

diff --git a/lib/mauve/alert.rb b/lib/mauve/alert.rb
index 9d04250..a2033a5 100644
--- a/lib/mauve/alert.rb
+++ b/lib/mauve/alert.rb
@@ -272,7 +272,7 @@ module Mauve
       attributes.each do |key, val|
         prop = self.class.properties[key]
         next unless prop.is_a?(DataMapper::Property::String)
-
+        next unless val.is_a?(String)
         #
         # Truncate 
         #
diff --git a/test/tc_mauve_alert.rb b/test/tc_mauve_alert.rb
index 61bd117..5c6f2bd 100644
--- a/test/tc_mauve_alert.rb
+++ b/test/tc_mauve_alert.rb
@@ -475,4 +475,23 @@ EOF
     assert(!alert.suppressed?,"Alert marked as suppressed when the suppression period was never set")
   end
 
+  def test_long_fields_get_truncated
+    Configuration.current = ConfigurationBuilder.parse(@test_config)
+    Server.instance.setup
+
+    alert = Alert.new(
+      :alert_id  => "x"*257,
+      :source    => "test",
+      :subject   => "test"
+    )
+
+    assert(alert.save, "Alert with overly long field failed to save")
+    # This should trigger a warning.
+    logger_pop
+
+    alert.reload
+    assert(256, alert.alert_id.length)
+
+  end
+
 end