From a96a8f552074e8a624f1420d2f58db4e90c3762c Mon Sep 17 00:00:00 2001 From: Zara Gebru Date: Tue, 12 Jul 2016 12:47:15 +0200 Subject: [Test] get email form hkp --- test/unit/request_handler_test.rb | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) (limited to 'test') diff --git a/test/unit/request_handler_test.rb b/test/unit/request_handler_test.rb index ef656f9..11cc59b 100644 --- a/test/unit/request_handler_test.rb +++ b/test/unit/request_handler_test.rb @@ -18,11 +18,20 @@ class Nickserver::RequestHandlerTest < Minitest::Test assert_response status: 500, content: "500 HTTP request must include a Host header.\n" end + def test_email_from_hkp + handle address: ['valid@email.tld'], headers: { "Host" => "http://nickserver.me" } + source = Minitest::Mock.new + source.expect :query, Nickserver::Response.new(200, "fake content"), [Nickserver::EmailAddress] + Nickserver::Hkp::Source.stub :new, source do + assert_response status: 200, content: "200 fake content" + end + end + protected - def handle(params = {}, headers = {}) + def handle(params = {}) + @headers = params.delete(:headers) || {} @params = Hash[ params.map{ |k,v| [k.to_s, v] } ] - @headers = headers end def assert_response(args) -- cgit v1.2.3 From 59263baee9fc04524c8a69af1f7293fa4315d3ff Mon Sep 17 00:00:00 2001 From: Zara Gebru Date: Wed, 13 Jul 2016 20:20:52 +0200 Subject: [FEATURE] fetch key by fingerprint - add fingerprint unit tests - add integration test - implement by_fingerprint --- test/integration/nickserver_test.rb | 14 ++++++-------- test/unit/request_handler_test.rb | 22 ++++++++++++++++++++++ 2 files changed, 28 insertions(+), 8 deletions(-) (limited to 'test') diff --git a/test/integration/nickserver_test.rb b/test/integration/nickserver_test.rb index 710c3a1..9aac6b4 100644 --- a/test/integration/nickserver_test.rb +++ b/test/integration/nickserver_test.rb @@ -26,7 +26,7 @@ class NickserverTest < Minitest::Test super end - def test_GET_served_via_SKS + def test_GET_key_by_email_address_served_via_SKS uid = 'cloudadmin@leap.se' key_id = 'E818C478D3141282F7590D29D041EB11B1647490' stub_sks_vindex_reponse(uid, body: file_content(:leap_vindex_result)) @@ -40,15 +40,13 @@ class NickserverTest < Minitest::Test end end - def test_POST_served_via_SKS - uid = 'cloudadmin@leap.se' - key_id = 'E818C478D3141282F7590D29D041EB11B1647490' - stub_sks_vindex_reponse(uid, body: file_content(:leap_vindex_result)) - stub_sks_get_reponse(key_id, body: file_content(:leap_public_key)) + def test_GET_key_by_fingerprint_served_via_SKS + fingerprint = 'E818C478D3141282F7590D29D041EB11B1647490' + stub_sks_get_reponse(fingerprint, body: file_content(:leap_public_key)) start do - params = {body: {"address" => uid}} - post(params) do |response| + params = {query: {"fingerprint" => fingerprint}} + get(params) do |response| assert_equal file_content(:leap_public_key), JSON.parse(response.to_s)["openpgp"] end end diff --git a/test/unit/request_handler_test.rb b/test/unit/request_handler_test.rb index 11cc59b..420e74b 100644 --- a/test/unit/request_handler_test.rb +++ b/test/unit/request_handler_test.rb @@ -27,6 +27,28 @@ class Nickserver::RequestHandlerTest < Minitest::Test end end + def test_fingerprint_to_short + handle fingerprint: ['44F2F455E28'] + assert_response status: 500, content: "500 Fingerprint invalid: 44F2F455E28\n" + end + + def test_fingerprint_is_not_hex + handle fingerprint: ['X36E738D69173C13Z709E44F2F455E2824D18DDX'] + assert_response status: 500, + content: "500 Fingerprint invalid: X36E738D69173C13Z709E44F2F455E2824D18DDX\n" + end + + def test_get_key_with_fingerprint_from_hkp + handle fingerprint: ['E36E738D69173C13D709E44F2F455E2824D18DDF'] + source = Minitest::Mock.new + source.expect :get_key_by_fingerprint, + Nickserver::Response.new(200, "fake fingerprint"), + ['E36E738D69173C13D709E44F2F455E2824D18DDF'] + Nickserver::Hkp::Source.stub :new, source do + assert_response status: 200, content: "200 fake fingerprint" + end + end + protected def handle(params = {}) -- cgit v1.2.3