diff options
author | azul <azul@riseup.net> | 2017-07-24 05:22:15 +0000 |
---|---|---|
committer | azul <azul@riseup.net> | 2017-07-24 05:22:15 +0000 |
commit | cdad3bedd7d90957d836d9d9c339e6b9dfc8b8d1 (patch) | |
tree | 2707fe4563d6be32cd6a9b2d1cb4c1f5f303273c /test/remote | |
parent | 38dd81116b85c103dbc5e9f08a8ffce26238921a (diff) | |
parent | 406234367544a4207141230683dddaccd98fb21a (diff) |
Merge branch 'fix/fd-leak' into 'master'
fix: filedescriptor leak from http_adapters
See merge request !13
Diffstat (limited to 'test/remote')
-rw-r--r-- | test/remote/hkp_source_test.rb | 11 | ||||
-rw-r--r-- | test/remote/nicknym_source_test.rb | 4 |
2 files changed, 12 insertions, 3 deletions
diff --git a/test/remote/hkp_source_test.rb b/test/remote/hkp_source_test.rb index 103b8ad..ff61513 100644 --- a/test/remote/hkp_source_test.rb +++ b/test/remote/hkp_source_test.rb @@ -1,8 +1,10 @@ require 'test_helper' require 'support/celluloid_test' +require 'support/http_adapter_helper' require 'nickserver/hkp/source' class RemoteHkpSourceTest < CelluloidTest + include HttpAdapterHelper def test_key_info uid = 'elijah@riseup.net' @@ -31,12 +33,17 @@ class RemoteHkpSourceTest < CelluloidTest protected - def assert_key_info_for_uid(uid, &block) - Nickserver::Hkp::Source.new.search uid do |status, keys| + def assert_key_info_for_uid(uid) + source.search uid do |status, keys| assert_equal 200, status yield keys end rescue HTTP::ConnectionError => e skip "could not talk to hkp server: #{e}" end + + def source + Nickserver::Hkp::Source.new adapter + end + end diff --git a/test/remote/nicknym_source_test.rb b/test/remote/nicknym_source_test.rb index 7840e10..4ca3033 100644 --- a/test/remote/nicknym_source_test.rb +++ b/test/remote/nicknym_source_test.rb @@ -1,5 +1,6 @@ require 'test_helper' require 'support/celluloid_test' +require 'support/http_adapter_helper' require 'nickserver/nicknym/source' require 'nickserver/email_address' @@ -7,6 +8,7 @@ require 'nickserver/email_address' # Please note the Readme.md file in this directory # class RemoteNicknymSourceTest < CelluloidTest + include HttpAdapterHelper def test_availablility_check source.available_for? 'mail.bitmask.net' @@ -42,7 +44,7 @@ class RemoteNicknymSourceTest < CelluloidTest end def source - @source ||= Nickserver::Nicknym::Source.new + @source ||= Nickserver::Nicknym::Source.new adapter end def email_with_key |