From 661ed3c406df3e87282814d177c851572776d712 Mon Sep 17 00:00:00 2001
From: Steve Kemp <steve@steve.org.uk>
Date: Thu, 15 Nov 2012 08:17:13 +0000
Subject:   Throw an exception if we attempt to redefine a macro.

---
 t/test-parser.rb | 36 +++++++++++++++++++++++++++++++++++-
 1 file changed, 35 insertions(+), 1 deletion(-)

(limited to 't')

diff --git a/t/test-parser.rb b/t/test-parser.rb
index c3e782f..f405433 100755
--- a/t/test-parser.rb
+++ b/t/test-parser.rb
@@ -57,7 +57,6 @@ class TestParser < Test::Unit::TestCase
     assert_nothing_raised do
       MonitorConfig.new("/dev/null" )
     end
-
   end
 
 
@@ -171,6 +170,41 @@ class TestParser < Test::Unit::TestCase
 
 
 
+  #
+  # Redefinining a macro should cause an error
+  #
+  def test_duplicate_macro
+
+
+    #
+    # Create the parser.
+    #
+    parser = MonitorConfig.new("/dev/null" )
+
+
+    #
+    #  With nothing loaded we should have zero macros - so the
+    # count of our macros hash should be zero
+    #
+    macros = parser.macros
+    assert( macros.empty? )
+    assert( macros.size() == 0 )
+
+
+    #
+    #  Define a macro twice.  The first will work, the second will fail
+    #
+    assert_nothing_raised do
+      parser.define_macro( "FOO is kvm1.vm.bytemark.co.uk and kvm2.vm.bytemark.co.uk." )
+    end
+
+    assert_raise ArgumentError do
+      parser.define_macro( "FOO is kvm1.vm.bytemark.co.uk and kvm2.vm.bytemark.co.uk." )
+    end
+  end
+
+
+
 
   #
   #  Test that we can define macros with only a single host.
-- 
cgit v1.2.1