Be able to set headers on GET requests

lib/twingly/http.rb

@@ -58,8 +58,8 @@ def initialize(logger:, base_user_agent:)
         @max_url_size_bytes = DEFAULT_MAX_URL_SIZE_BYTES
       end
 
-      def get(url, params: {})
-        http_response_for(:get, url: url, params: params)
+      def get(url, params: {}, headers: {})
+        http_response_for(:get, url: url, params: params, headers: headers)
       end
 
       def post(url, body:, headers: {})
@@ -87,11 +87,11 @@ def http_response_for(method, *args)
       end
       # rubocop:enable all
 
-      def http_get_response(url:, params: {})
+      def http_get_response(url:, params:, headers:)
         binary_url = url.dup.force_encoding(Encoding::BINARY)
         http_client = create_http_client
 
-        headers = default_headers
+        headers = default_headers.merge(headers)
 
         http_client.get do |request|
           request.url(binary_url)

spec/lib/twingly/http_spec.rb

@@ -505,6 +505,31 @@ class CustomError < StandardError; end
       end
     end
 
+    describe "headers" do
+      let(:headers) do
+        {
+          "Content-Type" => "application/json",
+        }
+      end
+
+      let(:request_response) do
+        client.get(url, headers: headers)
+      end
+
+      before do
+        headers_in_body_lamba = lambda do |request|
+          { body: request.headers.to_json }
+        end
+
+        stub_request(:get, url)
+          .to_return(&headers_in_body_lamba)
+      end
+
+      it "does request with specified headers" do
+        expect(JSON.parse(response.fetch(:body))).to include(headers)
+      end
+    end
+
     describe "params" do
       let(:params) do
         {

spec/spec_helper.rb

@@ -4,6 +4,7 @@
 # from https://github.com/rspec/rspec-core/issues/1983#issuecomment-108748690
 $LOAD_PATH.delete_if { |p| File.expand_path(p) == File.expand_path("./lib") }
 
+require "json"
 require "rspec"
 require "toxiproxy"
 require "vcr"