whois.nic.tv now correctly raises PropertyNotSupported when a propert…

…y is not supported.
weppos · Mar 13, 2010 · 5959d12 · 5959d12
1 parent 98668ec
commit 5959d12
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/CHANGELOG.rdoc b/CHANGELOG.rdoc
@@ -18,6 +18,8 @@
 
 * CHANGED: whois.nic.it now correctly raises PropertyNotSupported when a property is not supported.
 
+* CHANGED: whois.nic.tv now correctly raises PropertyNotSupported when a property is not supported.
+
 
 == Release 1.0.5
 

diff --git a/lib/whois/answer/parser/whois.nic.tv.rb b/lib/whois/answer/parser/whois.nic.tv.rb
@@ -39,9 +39,8 @@ class WhoisNicTv < Base
           node("Domain Name") { |raw| raw.downcase }
         end
 
-        property_supported :domain_id do
-          nil
-        end
+        property_not_supported :domain_id
+
 
 
         property_supported :referral_whois do
@@ -78,6 +77,7 @@ class WhoisNicTv < Base
           node("Expiration Date") { |raw| Time.parse(raw) }
         end
 
+
         property_supported :registrar do
           # Return nil because when the response contains more than one registrar section
           # the response can be messy. See, for instance, the Verisign response for google.com.

diff --git a/test/answer/parser/whois.nic.tv_test.rb b/test/answer/parser/whois.nic.tv_test.rb
@@ -50,10 +50,8 @@ def test_domain
   end
 
   def test_domain_id
-    assert_equal  nil,
-                  @klass.new(load_part('/registered.txt')).domain_id
-    assert_equal  nil,
-                  @klass.new(load_part('/available.txt')).domain_id
+    assert_raise(Whois::PropertyNotSupported) { @klass.new(load_part('/registered.txt')).domain_id }
+    assert_raise(Whois::PropertyNotSupported) { @klass.new(load_part('/available.txt')).domain_id }
   end