Revision: 306
Author: progrium
Date: Mon Jan 18 03:38:31 2010
Log: adding mechanize to list of pre-requisites and fixing a bug that causes silent failure instead of test fail
http://code.google.com/p/pubsubhubbub/source/detail?r=306

Modified:
 /trunk/testsuite/README
 /trunk/testsuite/hub_spec.rb

=======================================
--- /trunk/testsuite/README     Sun Jul 19 15:30:46 2009
+++ /trunk/testsuite/README     Mon Jan 18 03:38:31 2010
@@ -5,9 +5,10 @@
 directly from the current 0.1 PubSubHubbub Core working draft. However, not
 everything can be tested currently. This may change as the spec changes.

-This requires Ruby and RSpec, the later which can be installed with Ruby Gems -using "gem install rspec". RSpec is a great approach to testing and Ruby allows -for nice DSL-like testing. The interface is HTTP, so it's language independent. +This requires Ruby, RSpec and Mechanize, the later two can be installed with
+Ruby Gems using "gem install rspec mechanize". RSpec is a great approach to
+testing and Ruby allows for nice DSL-like testing. The interface is HTTP, so
+it's language independent.

 Using the Test Suite:

=======================================
--- /trunk/testsuite/hub_spec.rb        Sun Jul 19 16:21:34 2009
+++ /trunk/testsuite/hub_spec.rb        Mon Jan 18 03:38:31 2010
@@ -6,7 +6,11 @@
raise "Specify a hub URL by setting the HUB_URL environment variable." unless HUB_URL

 def wait_on(something)
-  Timeout::timeout(3) { break unless something.nil? while true }
+  begin
+    Timeout::timeout(3) { break unless something.nil? while true }
+  rescue Timeout::Error
+    nil
+  end
 end

 def as_optional

Reply via email to