diff options
author | azul <azul@riseup.net> | 2016-12-02 10:28:46 +0000 |
---|---|---|
committer | azul <azul@riseup.net> | 2016-12-02 10:28:46 +0000 |
commit | 244d8a4702f98619078c0e09e485d7e1944e5013 (patch) | |
tree | d2eaf93312ae2c308397071fec5035c16ba98a68 /test | |
parent | 4159b6dd0df8e370fe95ca0c173552dd311cac3a (diff) | |
parent | 6851b606bdc985565104d41a48f1ecc81d132227 (diff) |
Merge branch 'test/debug-failures' into 'master'
test: more meaningful error messages in case of failures
See merge request !8
Diffstat (limited to 'test')
-rw-r--r-- | test/remote/nicknym_source_test.rb | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/test/remote/nicknym_source_test.rb b/test/remote/nicknym_source_test.rb index e2896ca..7840e10 100644 --- a/test/remote/nicknym_source_test.rb +++ b/test/remote/nicknym_source_test.rb @@ -18,9 +18,7 @@ class RemoteNicknymSourceTest < CelluloidTest def test_successful_query response = source.query(email_with_key) skip if response.status == 404 - json = JSON.parse response.content - assert_equal email_with_key.to_s, json["address"] - refute_empty json["openpgp"] + assert_pgp_key_in response rescue HTTP::ConnectionError => e skip e.to_s end @@ -28,13 +26,21 @@ class RemoteNicknymSourceTest < CelluloidTest def test_not_found response = source.query(email_without_key) skip if response.status == 200 - assert response.status == 404 + assert_equal 404, response.status rescue HTTP::ConnectionError => e skip e.to_s end protected + def assert_pgp_key_in(response) + json = JSON.parse response.content + assert_equal email_with_key.to_s, json["address"] + refute_empty json["openpgp"] + rescue JSON::ParserError + skip "invalid json response: #{response.content}" + end + def source @source ||= Nickserver::Nicknym::Source.new end |