#4, refactor: add Rack env builder

Author: requiemformemories

lib/hanami/lambda/env.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require "rack"
+
+module Hanami
+  module Lambda
+    class Env
+      def initialize(event:, headers:, context:)
+        @event = event
+        @headers = headers
+        @content_type = extract_content_type
+        @http_headers = transform_http_headers
+        @context = context
+      end
+
+      def to_h
+        {
+          ::Rack::REQUEST_METHOD => @event["httpMethod"],
+          ::Rack::PATH_INFO => @event["path"] || "",
+          ::Rack::RACK_VERSION => ::Rack.release,
+          ::Rack::RACK_INPUT => StringIO.new(@event["body"] || ""),
+          ::Hanami::Lambda::LAMBDA_EVENT => @event,
+          ::Hanami::Lambda::LAMBDA_CONTEXT => @context
+        }.tap do |env|
+          env["CONTENT_TYPE"] = @content_type if @content_type
+        end.merge(@http_headers)
+      end
+
+      private
+
+      def extract_content_type
+        @headers.delete("Content-Type") ||
+          @headers.delete("content-type") ||
+          @headers.delete("CONTENT_TYPE")
+      end
+
+      def transform_http_headers
+        @headers.transform_keys { |k| "HTTP_#{k.upcase.tr('-', '_')}" }
+      end
+    end
+  end
+end

lib/hanami/lambda/rack.rb

@@ -41,19 +41,7 @@ def call(event:, context:)
       #
       # @since 0.1.0
       def build_env(event, headers, context)
-        {
-          ::Rack::REQUEST_METHOD => event["httpMethod"],
-          ::Rack::PATH_INFO => event["path"] || "",
-          ::Rack::RACK_VERSION => ::Rack.release,
-          ::Rack::RACK_INPUT => StringIO.new(event["body"] || ""),
-          ::Hanami::Lambda::LAMBDA_EVENT => event,
-          ::Hanami::Lambda::LAMBDA_CONTEXT => context
-        }.tap do |env|
-          content_type = headers.delete("Content-Type") ||
-                         headers.delete("content-type") ||
-                         headers.delete("CONTENT_TYPE")
-          env["CONTENT_TYPE"] = content_type if content_type
-        end.merge(headers.transform_keys { |k| "HTTP_#{k.upcase.tr('-', '_')}" })
+        Env.new(event: event, headers: headers, context: context).to_h
       end
     end
   end

sig/hanami/lambda/env.rbs

@@ -0,0 +1,29 @@
+module Hanami
+  module Lambda
+    class Env
+      @event: untyped
+
+      @headers: untyped
+
+      @content_type: untyped
+
+      @http_headers: untyped
+
+      @context: untyped
+
+      def initialize: (event: untyped, headers: untyped, context: untyped) -> void
+
+      # Build a hash of the Rack environment
+      #
+      # @return [Hash] the hash of Rack environment
+      #
+      def to_h: () -> ::Hash[untyped, untyped]
+
+      private
+
+      def extract_content_type: () -> untyped
+
+      def transform_http_headers: () -> untyped
+    end
+  end
+end

spec/hanami/lambda/env_spec.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+RSpec.describe Hanami::Lambda::Env do
+  subject(:env) { described_class.new(event: event, headers: headers, context: context) }
+
+  let(:event) do
+    {
+      "httpMethod" => "GET",
+      "path" => "/",
+      "body" => ""
+    }
+  end
+
+  let(:headers) do
+    {
+      "Content-Type" => "text/plain",
+      "X-Custom-Header" => "Custom Value"
+    }
+  end
+
+  let(:context) { double(:context) }
+
+  describe "#to_h" do
+    subject(:hash) { env.to_h }
+
+    it { is_expected.to include(::Hanami::Lambda::LAMBDA_EVENT) }
+    it { is_expected.to include(::Hanami::Lambda::LAMBDA_CONTEXT) }
+    it { is_expected.to include("CONTENT_TYPE" => "text/plain") }
+    it { is_expected.to include("HTTP_X_CUSTOM_HEADER" => "Custom Value") }
+  end
+end

spec/hanami/lambda/rack_spec.rb

@@ -33,13 +33,4 @@
     it { is_expected.to include(body: "Hello from Rack") }
     it { is_expected.to include(headers: {}) }
   end
-
-  describe "#env" do
-    subject(:env) { rack.build_env(event, headers, context) }
-
-    it { is_expected.to include(::Hanami::Lambda::LAMBDA_EVENT) }
-    it { is_expected.to include(::Hanami::Lambda::LAMBDA_CONTEXT) }
-    it { is_expected.to include("CONTENT_TYPE" => "text/plain") }
-    it { is_expected.to include("HTTP_X_CUSTOM_HEADER" => "Custom Value") }
-  end
 end