webhook_agent_spec.rb 12KB

    require 'rails_helper' describe Agents::WebhookAgent do let(:agent) do _agent = Agents::WebhookAgent.new(:name => 'webhook', :options => { 'secret' => 'foobar', 'payload_path' => 'some_key' }) _agent.user = users(:bob) _agent.save! _agent end let(:payload) { {'people' => [{ 'name' => 'bob' }, { 'name' => 'jon' }] } } describe 'receive_web_request' do it 'should create event if secret matches' do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) expect(Event.last.payload).to eq(payload) end it 'should be able to create multiple events when given an array' do out = nil agent.options['payload_path'] = 'some_key.people' expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(2) expect(out).to eq(['Event Created', 201]) expect(Event.last.payload).to eq({ 'name' => 'jon' }) end it 'should not create event if secrets do not match' do out = nil expect { out = agent.receive_web_request({ 'secret' => 'bazbat', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(0) expect(out).to eq(['Not Authorized', 401]) end it 'should respond with customized response message if configured with `response` option' do agent.options['response'] = 'That Worked' out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['That Worked', 201]) # Empty string is a valid response agent.options['response'] = '' out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['', 201]) end it 'should respond with interpolated response message if configured with `response` option' do agent.options['response'] = '{{some_key.people[1].name}}' out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['jon', 201]) end it 'should respond with `Event Created` if the response option is nil or missing' do agent.options['response'] = nil out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['Event Created', 201]) agent.options.delete('response') out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['Event Created', 201]) end it 'should respond with customized response code if configured with `code` option' do agent.options['code'] = '200' out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['Event Created', 200]) end it 'should respond with `201` if the code option is empty, nil or missing' do agent.options['code'] = '' out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['Event Created', 201]) agent.options['code'] = nil out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['Event Created', 201]) agent.options.delete('code') out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") expect(out).to eq(['Event Created', 201]) end describe "receiving events" do context "default settings" do it "should not accept GET" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "get", "text/html") }.to change { Event.count }.by(0) expect(out).to eq(['Please use POST requests only', 401]) end it "should accept POST" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end end context "accepting get and post" do before { agent.options['verbs'] = 'get,post' } it "should accept GET" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "get", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end it "should accept POST" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end it "should not accept PUT" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "put", "text/html") }.to change { Event.count }.by(0) expect(out).to eq(['Please use GET/POST requests only', 401]) end end context "accepting only get" do before { agent.options['verbs'] = 'get' } it "should accept GET" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "get", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end it "should not accept POST" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(0) expect(out).to eq(['Please use GET requests only', 401]) end end context "accepting only post" do before { agent.options['verbs'] = 'post' } it "should not accept GET" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "get", "text/html") }.to change { Event.count }.by(0) expect(out).to eq(['Please use POST requests only', 401]) end it "should accept POST" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end end context "accepting only put" do before { agent.options['verbs'] = 'put' } it "should accept PUT" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "put", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end it "should not accept GET" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "get", "text/html") }.to change { Event.count }.by(0) expect(out).to eq(['Please use PUT requests only', 401]) end it "should not accept POST" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(0) expect(out).to eq(['Please use PUT requests only', 401]) end end context "flaky content with commas" do before { agent.options['verbs'] = ',, PUT,POST, gEt , ,' } it "should accept PUT" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "put", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end it "should accept GET" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "get", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end it "should accept POST" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.to change { Event.count }.by(1) expect(out).to eq(['Event Created', 201]) end it "should not accept DELETE" do out = nil expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "delete", "text/html") }.to change { Event.count }.by(0) expect(out).to eq(['Please use PUT/POST/GET requests only', 401]) end end context "with reCAPTCHA" do it "should not check a reCAPTCHA response unless recaptcha_secret is set" do checked = false out = nil stub_request(:any, /verify/).to_return { |request| checked = true { status: 200, body: '{"success":false}' } } expect { out= agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.not_to change { checked } expect(out).to eq(["Event Created", 201]) end it "should reject a request if recaptcha_secret is set but g-recaptcha-response is not given" do agent.options['recaptcha_secret'] = 'supersupersecret' checked = false out = nil stub_request(:any, /verify/).to_return { |request| checked = true { status: 200, body: '{"success":false}' } } expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload }, "post", "text/html") }.not_to change { checked } expect(out).to eq(["Not Authorized", 401]) end it "should reject a request if recaptcha_secret is set and g-recaptcha-response given is not verified" do agent.options['recaptcha_secret'] = 'supersupersecret' checked = false out = nil stub_request(:any, /verify/).to_return { |request| checked = true { status: 200, body: '{"success":false}' } } expect { out = agent.receive_web_request({ 'secret' => 'foobar', 'some_key' => payload, 'g-recaptcha-response' => 'somevalue' }, "post", "text/html") }.to change { checked } expect(out).to eq(["Not Authorized", 401]) end it "should accept a request if recaptcha_secret is set and g-recaptcha-response given is verified" do agent.options['payload_path'] = '.' agent.options['recaptcha_secret'] = 'supersupersecret' checked = false out = nil stub_request(:any, /verify/).to_return { |request| checked = true { status: 200, body: '{"success":true}' } } expect { out = agent.receive_web_request(payload.merge({ 'secret' => 'foobar', 'g-recaptcha-response' => 'somevalue' }), "post", "text/html") }.to change { checked } expect(out).to eq(["Event Created", 201]) expect(Event.last.payload).to eq(payload) end end end end end