From 5959d12c0a8afedce2264de9224961e4837635e3 Mon Sep 17 00:00:00 2001
From: Simone Carletti <weppos@weppos.net>
Date: Sat, 13 Mar 2010 17:48:46 +0100
Subject: [PATCH] whois.nic.tv now correctly raises PropertyNotSupported when a
 property is not supported.

---
 CHANGELOG.rdoc                          | 2 ++
 lib/whois/answer/parser/whois.nic.tv.rb | 6 +++---
 test/answer/parser/whois.nic.tv_test.rb | 6 ++----
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/CHANGELOG.rdoc b/CHANGELOG.rdoc
index 8c119f551..26763dba5 100644
--- 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
index d385fa61a..6bbcf4379 100644
--- 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
index 30a15b2f8..9f5285a7f 100644
--- 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