Minimal DNS now also extends to resolution of items in the source

lists too, as well as the hostname/IP being queried.

2 files changed, 9 insertions, 2 deletions

diff --git a/test/tc_mauve_source_list.rb b/test/tc_mauve_source_list.rb
index 6241a2f..138286a 100644
--- a/test/tc_mauve_source_list.rb
+++ b/test/tc_mauve_source_list.rb
@@ -91,7 +91,11 @@ class TcMauveSourceList < Mauve::UnitTest
 
   def test_ip_crossmatch_fail_when_minimal_dns_is_available
     Configuration.current.minimal_dns_lookups = true
+    dns_lookup_count_before_test = MauveResolv.count
 
+    #
+    # These should all fail, since no extra cossmatching is happening.
+    #
     sl = SourceList.new("test")
     assert_nothing_raised { sl += "test-1.example.com" }
     assert( !sl.includes?("www.example.com"), "www.example.com not found in #{sl.list}" )
@@ -99,6 +103,8 @@ class TcMauveSourceList < Mauve::UnitTest
     sl = SourceList.new("test")
     assert_nothing_raised { sl += "2001::/3" }
     assert( !sl.includes?("www2.example.com"), "www2.example.com not found in #{sl.list}" )
+
+    assert_equal(0, MauveResolv.count - dns_lookup_count_before_test, "Some DNS lookups took place, even when minimal_dns_lookups was set")
   end
 
   def test_remote_source_list
diff --git a/test/th_mauve_resolv.rb b/test/th_mauve_resolv.rb
index 10bdacc..3451690 100644
--- a/test/th_mauve_resolv.rb
+++ b/test/th_mauve_resolv.rb
@@ -22,10 +22,11 @@ module Mauve
          "www2.example.com"   => %w(1.2.3.5 2001:2::2)
         }
         if lookup.has_key?(host)
-          self.count += lookup[host].length if $debug
+          self.count += lookup[host].length 
           lookup[host]
         else
-          get_ips_for_without_testing(host)
+          self.count += 1
+          []
         end
       end